Возникли проблемы в простой программе C

Question

У меня проблема с простой программой на Си. Даже если я введу год между 1000 и 1999, он все равно будет показывать недействительный год. Пожалуйста, скажите мне, что происходит ??

#include <stdio.h>

main()
{
    int year;
c:
    printf("\n\nEnter a Year: ");
    scanf("%d", year);

    if ((year < 1000) || (year > 1999))
    {
        printf("\n\nInvalid Year");
        goto c;
    }
    convert(year);

}
convert(int year)
{
    printf("%d", year);
}

Answer 1

Вам необходимо передать адрес scanf, то есть:

scanf("%d", &year);

Обратите внимание на амперсанд.

Answer 2

в scanf, остальные параметры должны быть указателями
это должно быть

  scanf("%d", &year);

Answer 3

Адам Зальцман победил меня, используй scanf("%d", &year) и старайтесь избегать операторов goto. Замените его на цикл while следующим образом:

main() {
  int year;
  printf("\n\nEnter a Year: ");
  scanf("%d", &year);

  while((year < 1000) || (year > 1999)) {
    printf("\n\nInvalid Year");
    printf("\n\nEnter a Year: ");
    scanf("%d", &year);
  }
}

еще лучшей конструкцией было бы использование конструкции do {} while (), но я оставлю это вам в качестве упражнения:)