Взял книгу "С++ для чайников"... и тем не менее проблемы с функциями

[BDA]

#include<stdio.h>
#include<iostream>

using namespace std;

int
sumSequence()
{
    int accumulator = 0, cislo = 0;
    while (true) {
        cin >> cislo;
        if (cislo < 0)
            return accumulator;
        else
            accumulator += cislo;
    }
}

int
main(int arg, char* pszArgs[])
{
    int accumulatedValue = 0, tmp, fl = 2;
    do {
        cout << "Vvedite posledovatelnost:" << endl;
        tmp = sumSequence();
        accumulatedValue += tmp;
        if (fl)
           --fl;
    } while (fl || tmp);
    cout << "accumulatedValue = " << accumulatedValue << endl;
    system("pause");
    return 0;
}

Вот с таким костылем работает, как нужно, вроде
Благодаря нему цикл в main срабатывает как минимум 2 раза.

[Igor95]

return accumulator;

Вы после первой итерации (если было введено число положительное), возвращаете значенее и выходите из ф-ции.

[Bugrimov]

Сообщение от Igor95

return accumulator;

Вы после первой итерации (если было введено число положительное), возвращаете значенее и выходите из ф-ции.

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

[BDA]

Bugrimov, Igor95 комментировал код ТСа, скорее всего. Там действительно была такая проблема.

[МатроскинЯ]

всё работает, но теперь (я всё таки чайник) возникают несколько вопросов, в частности: каким образом срабатывает команда if(f1)? какое там условие используется (просто я привык, что в данном операторе как правило сравнения или хотя бы выражения)? и я всё таки использовал оператор for, так как смысл while(true) не понимаю, что он значит? "выполнять пока истинно", а что именно должно быть истинным? в общем у меня получилась следующая программа (как ни странно работающая адекватно) с учётом исправлений:

#include<stdio.h>
#include<iostream>
using namespace std;
int sumSequence()
{
    int accumulator=0;
    for(;;)
    {
           int cislo=0;
           cout << "Vvedi cislo:";
           cin >> cislo;
           if (cislo<0)
           {
           return accumulator;
           }
          else
          {
          accumulator=accumulator+cislo;
          }          
           
    } 
}

 int main(int arg, char* pszArgs[])
 {
     int accumulatedValue=0, tmp=0, f1=2;
     do
     {
         cout << "Vvedite posledovatelnost:";
         tmp=sumSequence();
         accumulatedValue+=tmp;
         cout << "accumulatedValue=:" << accumulatedValue << "\n";
         if (f1)
         {
                --f1;
         }
     } while (f1||tmp);
     cout << "zvezdec!";
     system("pause");
     return 0;
 }

хотелось бы услышать пояснения к не совсем понятным мне командам))) а именно выделенным красным цветом) заранее спасибо)

[BDA]

while (true) такой же бесконечный цикл, как и for ( ; ; ). Обычно пишут while (условие), но поскольку нужен бесконечный цикл, то условие всегда истинно. if (fl) нужно читать, как if (fl != 0), т.е. любое число, отличное от нуля, трактуется как истина, а ноль - ложь. while (fl || tmp) - "выполняй, пока fl не равно 0 или tmp не равно 0". Благодаря такой проверке при вводе двух отрицательных чисел в самом начале работы программы все сработает верно. if нужен, чтобы fl стало равно 0 и больше не уменьшалось.

[ACE Valery]

if(f1) равноценно условию if(f1 != 0)
while(true) - это бесконечный цикл (мы же не знаем, сколько раз будет введено число).

[МатроскинЯ]

таким образом мы получаем, что последовательность: 0, -1; 0, -1 является ошибочным... обидно, думал, что проблема решена... следовательно, для абсолютной точности работы программы нужно либо какое-то дополнительное условие, либо изменять старые... спасибо за разъяснение)

[BDA]

Спасибо за хороший тест (0 -1 0 -1). Действительно, нужно немного переделать логику.

#include <stdlib.h>
#include <iostream>

using namespace std;

int
sumSequence(int &flag)
{
    int accumulator = 0, cislo = 0;
    while (true) {
        cin >> cislo;
        if (cislo < 0)
            return accumulator;
        else {
            flag = 1;
            accumulator += cislo;
        }
    }
}

int
main()
{
    int accumulatedValue = 0, flag1 = 2, flag2;
    do {
        cout << "Vvedite posledovatelnost:" << endl;
        flag2 = 0;
        accumulatedValue += sumSequence(flag2);
        if (flag1)
           --flag1;
    } while (flag1 || flag2);
    cout << "accumulatedValue = " << accumulatedValue << endl;
    system("pause");
    return 0;
}

[МатроскинЯ]

чувствую замучаю с этими тестами, но, пусть данная программа решает проблему 0, -1; 0, -1, тем не менее появляется другая проблема: это бесконечный цикл, который не находит варианта, когда данное условие ложно

} while (flag1 || flag2);

даже при количестве отрицательных подряд порядка 10 он из цикла не выходит... тем не менее программа работает в том плане, что складывает адекватно (проверял переставляя строку вывода в тело цикла)... и, честно говоря, не улавливаю логику введения 3 переменных flag... так же интересно для чего в команде cout нужна команда end1, или это аналог "\n"?