Проверка с помощью strlen в c

Ваш gets вызов небезопасен. Он может принять больше символов, чем вы выделили. Бум.

Предполагая, что вы выделили не менее 8 байтов для .idCard (т.е. 7 символов плюс терминатор), рассмотрите возможность использования этого:

  fgets(newRes.idCard, 7+1, stdin);

Это гарантирует, что вы не примете слишком много символов. Он будет обрезан по мере необходимости. Проверьте возвращаемое значение fgets на успешность.

(man fgets)

Скомпилируйте этот пример кода и изучите. Надеюсь, вы найдете здесь то, что ищете.

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

#define MAX_ID_LEN (7)

int main() {
    char idCard[MAX_ID_LEN+1];
    printf("ID: ");
    if (fgets(idCard, MAX_ID_LEN+1, stdin)) {
        char *cLast = idCard + strlen(idCard)-1;
        if (*cLast == '\n')
            *cLast = 0;  // strip newline *if* present
        printf("Success!  idCard='%s'\n", idCard);
    } else {
        printf("Failure.\n");
    }
    return 0;
}

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

В качестве альтернативы, это, вероятно, меньше отвлекает вас: (просто используйте fscanf)

#include <stdio.h>

int main() {
    char idCard[7+1];
    printf("ID: ");
    fscanf(stdin, "%7s", idCard);
    printf("idCard='%s'\n", idCard);
    return 0;
}

Чтобы правильно сбросить stdin, вы должны сделать следующее:

int c;
while ((c = getchar()) != '\n' && c != EOF);

И вместо gets вы должны использовать fgets, потому что там вы можете ограничить количество символов к размеру вашего входного буфера.

gets не будет знать длину, и, таким образом, если пользователь введет больше, чем предусмотрено буфером, это может привести к переполнению буфера.

Почему функция fflush(stdin) не работает?

Слишком поздно проверять длину после чтения данных в слишком маленький буфер. Сначала прочитайте данные в большой буфер хранения.

fgets() — отличная замена устаревшему gets(). Не забудьте убрать потенциальное окончание \n.

Также избегайте магических чисел, таких как «7». Используйте что-то вроде sizeof(newRes.idCard).

В хорошо спроектированном коде fflush(stdin) не требуется, и его следует избегать, поскольку он не переносим.

int inputflag = 0;  // Not clear is OP needs this value after the while loop.
do {
  printf("Passenger ID no: ");
  char buf[sizeof(newRes.idCard) * 2]; ' twice as big as needed.
  if (fgets(buf, sizeof buf, stdin) == NULL) {
    inputflag = 1;
    break;
  }
  size_t Len = strlen(buf);
  if (Len && buf[Len-1] == '\n') buf[--Len] = '\0';
  if(Len >= sizeof(newRes.idCard)) {
    printf("invalid input \n");
    inputflag = 1;
  }
  else {
    strcpy(newRes.idCard, buf);
  }
} while (inputflag == 0);