Почему несколько операторов fgets перезаписывают массивы символов?

следующий предлагаемый код:

1. чисто компилирует
2. выполняет желаемую функцию
3. Выходит, если имя собаки слишком длинное
4. удаляет новую строку в конце имени собаки
5. удаляет новую строку в конце породы собак
6. не проверяет правильность длины породы собаки
7. правильно очищает после использования динамической памяти
8. избегает использования «магических» чисел
9. правильно выделяет динамическую память и проверяет/обрабатывает любые ошибки
10. использует: «максимум 8 символов», чтобы оставить место для завершающей новой строки и завершающего символа NUL

и теперь предлагаемый код:

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

#define MAX_NAME_LEN  10
#define MAX_BREED_LEN 10

typedef struct Dog 
{
    char name[ MAX_NAME_LEN ];
    char breed[ MAX_BREED_LEN ];
} Dog;

Dog * makeDog() 
{
    Dog *dog = malloc( sizeof( Dog ) );
    if( !dog )
    {
        perror( "malloc for struct Dog failed" );
        exit( EXIT_FAILURE );
    }

    printf("Dog's name? max 8 characters ");
    if( !fgets(dog->name, MAX_NAME_LEN, stdin) )
    {
        perror( "fgets for dog name failed" );
        exit( EXIT_FAILURE );
    }

    if( dog->name[ strlen( dog->name ) -1 ] != '\n' )
    {
        puts( "dog name too long, aborting" );
        exit( EXIT_FAILURE );
    }

    // remove trailing newline
    dog->name[ strcspn( dog->name, "\n" ) ] = '\0';

    printf("Dog's breed? max 8 characters");
    if( !fgets(dog->breed, MAX_BREED_LEN, stdin) )
    {
        perror( "fgets for dog breed failed" );
        exit( EXIT_FAILURE );
    }

    // remove trailing newline
    dog->breed[ strcspn( dog->name, "\n" ) ] = '\0';

    return dog;
}

int main() {
    printf("\n");

    Dog *dog = makeDog();

    printf("\n");

    printf("Dog's name: %s\n", dog->name);
    printf("Dog's breed: %s\n", dog->breed);

    free( dog );
}

Массив символов name объявлен с 10 элементами

char name[10];

Если вы используете следующий вызов fgets

fgets(dog.name, 10, stdin);

после ввода 10 символов 'a' вызов fgets считывает только 9 символов из входного буфера и добавляет к массиву завершающий нулевой символ '\0'.

Таким образом, массив будет содержать строку "aaaaaaaaa". Это то же самое, что инициализировать массив следующим образом

char name[10[ = { 'a', 'a', 'a', 'a', 'a', 'a', 'a', 'a', 'a', '\0' };

После этого входной буфер будет содержать один символ 'a' и новый символ строки '\n'. Эти символы будут прочитаны при следующем вызове

fgets(dog.breed, 10, stdin);

В результате массив bread будет содержать строку "a\n".

Это то же самое, что инициализировать массив следующим образом

char bread[10[ = { 'a', '\n', '\0' };

Если вы хотите хранить в массивах строки с большим количеством символов, вы должны увеличить массивы.

Например, если вы хотите ввести в качестве имени массива строку из 10 символов 'a', вы должны объявить массив как имеющий 12 элементов. Почему 12? Потому что помимо 10 символов 'a' и завершающего нулевого символа функция fgets также попытается прочитать символ новой строки '\n' из входного буфера. В противном случае этот символ будет прочитан вторым вызовом fgets.

Чтобы удалить символ новой строки из массива, вы можете использовать следующий подход

#include <string.h>

//...

fgets( dog.name, 12, stdin );
dog.name[strcspn( dog.name, "\n" )] = '\0';