C Программа - объединить 2 строки во время цикла

Question

Эта программа должна принимать две строки и объединять каждый элемент один за другим во время цикла. Например, если у меня есть
первой строка, как: hlowrd
и второй строки, как: Эль-ол
окончательная строка должна быть привет мир

Однако, когда вторая строка длиннее, чем первый будет отрезать остальные символы. Я считаю, что это связано с тем, что я структурировал цикл for в функции comb, но я не уверен, как подойти к его исправлению.

#include <stdio.h> 
#include <string.h> 

#define N 50 

int read_text(char *str, int n); 
void combine(char *s3, const char *s1, const char *s2); 

int main(void) 
{ 
    char string1[N+1]; 
    char string2[N+1]; 
    char string3[N+1]; 
    printf("Enter: \n"); 
    read_text(string1, N); 
    printf("Enter: \n"); 
    read_text(string2, N); 
    combine(string3, string1, string2); 
    printf("New string is: %s\n", string3); 
    return 0; 
} 

int read_text(char *str, int n) 
{ 
    int ch, i = 0; 

    while ((ch = getchar()) != '\n') { 
     if (i < n) { 
      *str++ = ch; 
      i++; 
     } 
    } 
    *str = '\0'; 
    return i; 
} 

void combine(char *s3, const char *s1, const char *s2) 
{ 
    const char *s; 
    s = s1; 

    for(s = s3; *s1 != '\0'; s++) { 
     strcpy(s3++, s1++); 
     strcpy(s3++, s2++); 
    } 
    *s3 = '\0'; 
} 

Answer 1

Вы в настоящее время останавливаетесь только в конце первой строки. Вы хотите остановиться в конце более короткой строки, а затем добавить остаток от более длинного.

Изменить условия в вашем for цикле быть

for (;*s1 != '\0' && *s2 != '\0';) {...} 

(или просто изменить его на while петли, так как вам не нужно переменную s так или иначе):

while (*s1 != '\0' && *s2 != '\0') {...} 

в остановитесь, когда на конце самой короткой струны и, после петли, добавьте

if (*s1 == '\0') 
    strcpy(s3, s2); 
else if (*s2 == '\0') 
    strcpy(s3, s1); 

, чтобы объединить оставшуюся часть длинной строки с выходом.

Answer 2

Вы должны зацикливаться на том, что оставшаяся строка источника не является пустой, но также вы не можете использовать strcpy, поскольку вы выполняете слияние char-by-char. Попробуйте следующее:

void 
combine(char *s3,const char *s1,const char *s2) 
{ 
    int c1; 
    int c2; 

    c1 = *s1; 
    c2 = *s2; 

    while ((c1 != 0) || (c2 != 0)) { 
     if (c1 != 0) { 
      *s3++ = c1; 
      c1 = *++s1; 
     } 

     if (c2 != 0) { 
      *s3++ = c2; 
      c2 = *++s2; 
     } 
    } 

    *s3 = 0; 
}