Попытка заполнить массив пользовательским вводом

Question

Попытка заполнить массив пользовательским вводом

0

Я столкнулся с проблемой. Я пытаюсь сохранить введенные пользователем значения в массив до тех пор, пока они не войдут в -1, используя цикл do-while. Вот пример моего кода:

int j=0;
    do{
        cin >> nodes[j];
        j++;
    }
    while (nodes[j] != -1);

Может кто-нибудь сказать мне, почему это не работает и какой лучший подход будет? Благодарю!

user3192573 03 апр. 2014, в 03:57

Источник

1

Не увеличивается ли t j` уже при сравнении nodes[j] != -1 ?
Arun 03 апр. 2014, в 01:48

Теги:

c++

6 ответов

Ещё вопросы

Не увеличивается ли t j` уже при сравнении nodes[j] != -1 ?

Arun · Answer 1 · 2014-04-02T23-00-00.000Z

Разве j не увеличивается, когда nodes[j] != -1 сравнения nodes[j] != -1?

Кроме того, я полагаю, что мы не хотим хранить значение дозорного -1 в массиве.

for( int j = 0; j != NodeMax; ++j) {
    int tmp = 0;
    cin >> tmp;
    if( tmp == -1 ) {
        break;
    }
    node[ j ] = tmp;
}

mah · Answer 2 · 2014-04-02T22-55-00.000Z

Это не работает, потому что у вас есть инкремент, завершающий индекс (j) после приема ввода и перед чтением ввода. Вместо этого вы можете сравнить узлы [j -1] == -1

Также важно, что вы можете перехватить ваш массив, если вы не ввели максимальное количество элементов.

Спасибо! У меня есть максимальное количество элементов для этого массива ранее в моей функции, так что не беспокойтесь! T
Вам нужно прервать цикл при достижении этого максимума, даже если -1 не было предоставлено ... Это легко сделать, завершив -1 || J == Макс

timrau · Answer 3 · 2014-04-02T23-57-00.000Z

Как отметил @Arun, j уже увеличивается, когда отмечены nodes[j] != -1.

Измените следующим образом:

do {
    cin >> nodes[j];
} while (nodes[j++] != -1);

yizzlez · Answer 4 · 2014-04-02T23-55-00.000Z

Утверждение while (nodes[j] != -1); никогда не будет истинным, потому что вы увеличиваете j и перемещаетесь по следующему индексу перед проверкой значения. Лучшим подходом было бы следующее:

do{
    cin >> nodes[j];
}
while (j < SIZE && nodes[j++] != -1);

Теперь j увеличивается в два раза за один цикл!

Tahlil · Answer 5 · 2014-04-03T00-43-00.000Z

Вы увеличиваете j до проверки узла [j]! = -1. Либо сделайте это

int j=-1;
    do{
        j++;
        cin >> nodes[j];

    }
    while (nodes[j] != -1);

или

int j=0;
        do{

            cin >> nodes[j];

        }
        while (nodes[j++] != -1);

или

int j=0;
while(cin>>nodes[j] && nodes[j]!=-1)
{
     j++;
}

A Lan · Answer 6 · 2014-04-03T00-28-00.000Z

nodes[j] не являются nodes[j] вы вводите, так как это было ++, попробуйте следующее: while(cin>>node[j++]);