Найдите ошибку Please

[Green Gin]

Задача: Даны строки S1 и S2. Сформировать строку S таким образом, чтобы в нее с начала вошли все четные символы строки S1, а потом все нечетные символы строки S2, работая со строками как с массивами(не используя string.h)

Написанный мною код:

#include <stdio.h>
#include <locale.h>

const int size = 100;
char S1[size], S2[size], S[size];

void calculation(void)
{
	int j=0;
	for(int i=0; i<=size; i++)
	{
		if(S1[i]!='/0' && i%2!=0) {S[j]=S1[i]; j++;}
	}
	for(int i=0; i<=size; i++)
	{
		if(S2[i]!='/0' && i%2==0) {S[j]=S2[i]; j++;}
	}
}

void main(void)
{
	setlocale(LC_ALL, "rus");
	puts("Введите строку S1:");
	gets_s(S1);
	puts("Введите строку S2:");
	gets_s(S2);
	calculation();
	printf("Строка S: %s\n", S);
}

Собственно где ошибка, помогите пожалуйста.

[SteAlzzer]

Собсно, косяк в том, что ты в циклах используешь переменную size, вместо реального размера реальной строки. Так у тебя получается, что конечный массив S набивается мусорными значениями S1 и S2.
Собственно, исправить можешь так:
Либо дописать функцию, которая будет возвращать размер строки:

int getSize(char* string)
{
	for(int i = 0;;i++)
		if(string[i] == '\0')
			return i;
};

void calculation(void)
{
	int j=0;
	for(int i=0; i<=getSize(S1); i++)
	{
		if(S1[i]!='/0' && i%2!=0) {S[j]=S1[i]; j++;}
	}
	for(int i=0; i<=getSize(S2); i++)
	{
		if(S2[i]!='/0' && i%2==0) {S[j]=S2[i]; j++;}
	}
};

Либо сделать элегантно:

void calculation(void)
{
	int j=0;
	for(int i=0; S1[i]; i++)
	{
		if(S1[i]!='/0' && i%2!=0) {S[j]=S1[i]; j++;}
	}
	for(int i=0; S2[i]; i++)
	{
		if(S2[i]!='/0' && i%2==0) {S[j]=S2[i]; j++;}
	}
};

[Green Gin]

Спасибо огромное супер только про второй способ не понятно, не могли бы Вы подробнее про это пояснить?

[Syuf]

Либо сделать нормально:

void calculation()
{
    int pos = 0;
    for(int i = 0; s1[i]; i++)
        if(i%2 == 0)
            s[pos++] = s1[i];
    for(int i = 0; s2[i]; i++)
        if(i%2)
            s[pos++] = s2[i];
    s[pos] = '\0';
}

[Green Gin]

спасибо использовал смешанный вариант...

[SteAlzzer]

Сообщение от Green Gin

Спасибо огромное супер только про второй способ не понятно, не могли бы Вы подробнее про это пояснить?

цикл будет выполняться, пока значение s[i] не будет равно нулю. А как вы знаете, конец строки обозначается нулем.

[Green Gin]

Интересно, а как решить эту задачу с помощью string.h? То есть не рассматривая S1 и S2 как массивы. Знаю что нужно воспользоваться strcat(), но как передать символ S1[i] (например), вторым аргументом функции?

Ну помогите кто-нибудь (см. пост выше)... Хотя бы, направьте в нужном направлении

[Abstraction]

Да никак. То есть, можно читать строки в переменные типа std::string, а в функции вместо проверки на нуль-терминатор опираться на функцию std::string::size(). Теоретически, можно припазать ещё и стандартные алгоритмы, но это в любом случае не <string>.

[Green Gin]

странно... Препод сказал написать эту же программу, но рассматривая S1, S2 и S как строки, пользуясь функциями string.h ... можно конечно strlen() использовать для определения размера строки в цикле но как произвести конкатенацию символа в строку???

[Abstraction]

Зачем "конкатенацию"? Заказать строку длиной в среднее арифметическое длин входных и менять элементы через [] не вариант?