Уже это предложение цикла
for (n = 0; n < M && s[0] != '\n'; ++n)
имеет неопределенное поведение, так как символьный массив s
не иниицализирован, а потому объект s[0]
имеет неопределенное значение.
В данном фрагменте
age2 = 2017 - age;
_itoa(age, newage,10);
^^^
я думаю, вы собираетесь на самом деле преобразовать age2
в строку, а не age
. Иначе, какой смысл вычисления значения age2
?
Данные предложения
char *finally = NULL;
strncat(finally,name);
strncat(finally, newage);
strncat(finally, town);
strcpy(array[n], finally);
также имеют неопределенное поведение, так как указатель finally
имеет значение NULL, то есть не указывает на корректно выделенную память.
Нет никакой необходимости использовать этот указатель. Почему бы сразу же не писать в массив array[n]
? Также стандартная C функция strncat
требует задания трех аргументов, а не двух.
Кроме того, если было введено n
строк, то в этом цикле
for (i = 0; i < n-1; i++)
условие должны быть задано как
for (i = 0; i < n; i++)
^^^^^
Первый цикл for
можно записать следующим образом
size_t n = 0;
for ( ; n < M && fgets( s, N, stdin ) != NULL && s[0] != '\n'; ++n )
{
sscanf_s( s, "%s %d %s", name, N, &age, town, N );
age = 2017 - age;
_itoa(age, newage, 10);
strcpy( array[n], name );
strcat( array[n], newage );
strcat( array[n], town );
}
Только в этом случае все слова в строке будут записаны слитно. Если хотите, чтобы они были записаны раздельно, то нужно добавлять пробелы между словами. Например,
strcpy( array[n], name );
strcat( array[n], " " );
strcat( array[n], newage );
strcat( array[n], " " );
strcat( array[n], town );
man asprintf
, только потом не забудьтеfree()
вызвать.asprintf()
вначале считывает строку формата и аргументы и пытается выяснить сколько памяти она займёт, потом аллоциоует память и печатает в неё согласно формату. Если память не нужно аллоцировать, то всё сводится к простому вызовуsnprintf()
. То, что вам привели ниже в качестве примера, лишь первый шаг по упрощению и очистке программы.