Как мне создать проверку для этого кода?

Question

Я пытаюсь сделать так, чтобы код проверял, находится ли пользовательский ввод между (и включая) 10 и 100.

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

int main()
{
    int numlist[20];

    for(int i = 0; i < 20; i++)
    {
        cout << "Enter # " << i + 1 << " : ";

        // here is where I am going wrong... 

        if ((numlist[i] <= 100) && (numlist[i] >= 10))
        {
            cin >> numlist[i];
        }
    }
}

Answer 1

Разве вы не должны поставить оператор ввода cin >> numlist[i] перед тестом if ((numlist[i] <= 100) && (numlist[i] >= 10))?

Answer 2

Просто чтобы дать немного другой способ сделать это, вы можете рассмотреть vector вместо массива и прочитать данные с istream_iterator вместе со стандартным алгоритмом:

std::vector<int> numlist;

std::remove_copy_if(std::istream_iterator<int>(std::cin), 
                    std::istream_iterator<int>(),
                    std::back_inserter(numlist), 
                    [](int i)->bool { return i<10 || i > 100; });

Редактировать: Полагаю, поскольку я использую лямбду C ++ 11, я мог бы также использовать C ++ 11 copy_if, который выражает намерение чуть более прямо:

std::copy_if(std::istream_iterator<int>(std::cin), 
             std::istream_iterator<int>(),
             std::back_inserter(numlist), 
             [](int i)->bool { return i>=10 && i<=100; });

Что касается «умного», то это вовсе не намерение, а то, что требуется, - это простое прямое выражение исходного намерения: копировать (фильтровать) данные из стандартного ввода в контейнер. Требуется немного времени, чтобы привыкнуть к идее рассматривать файлы как контейнеры (особенно такие, как std::cin, который обычно является интерактивным), но в конечном итоге файл является последовательностью, а istream_iterator / ostream_iterator просто позволяет вам обращаться с ними как с другими последовательностями .

Answer 3

похоже, что вы хотите сделать что-то вроде этого:

int temp = 0;

for (int i = 0; i < 20; i++)
{
  cin >> temp;
  if ((temp <= 100) && (temp >= 10))
    numlist[i] = temp;
}

Answer 4

Как уже отмечали другие, вы не можете проверить значение, которое вы даже не прочитали (от пользователя).Чтобы ограничить ввод, вы должны проверять ввод после cin в цикле do while, если он не удовлетворяет ограничению.