код работает не корректно

[AcTiV]

переменная all неправильно выводит значение, она выводит показатели counter`a (счетчика), а должна выводить сумму всех переменных total`ов

то ли я дурак не могу сделать цикл для того что бы тоталы суммировались и ихнее значение присваивалось переменной all,
то ли дурак совсем

//программа которая считает сумму товара
//и подсчитывает сумму всей покупки
//c_vo - количество
//counter - счетчик
//total - цена * количество
//zena - цена товара
//all - общее значение всех total
#include <iostream>
#include <conio.h>  //для getch
#include <iomanip>
#include <Windows.h>
using namespace std;

int main()
{
    SetConsoleOutputCP(1251);
    setlocale(LC_ALL, "RUS");   //локализация

    //задаем начальные значения
    float counter, total, c_vo, zena, all;
    string name;    //имя товара

    total = 0;  //цена товара умноженая на количество товара
    counter = 0;    //счетчик
    all = 0;    //сумма всех посчитаных товаров

    cout << "Добро пожаловать!" << endl;

    //этап обработки даных

    while (zena != -1)  //цикл выполняется пока не будет введено stop или -1
    {
        counter += 1;
        cout << "Введите -1 для конца ввода: " << endl;
        cout << "Введите имя: ";
        cin >> name;
        cout << "Введите цену: ";
        cin >> zena;
        cout << "Введите количество: ";
        cin >> c_vo;
        total = zena * c_vo;    //цена умноженная на количество
        cout << "\n Тотал = " << total << endl;
        cout << "\n" << endl;

        if (total != 0)   //сумма всех расчетов не равна 0
        {
            all = total * counter;  //если не равна 0, то вычисляем ИТОГ всего
        }
    }

    cout << "ИТОГ = " << all << endl;    //вывод всей суммы


    getch();    //задержка
    return 0;
}

[Abstraction]

На момент выхода из цикла all содержит произведение total в последней итерации цикла на количество итераций.
Если Вас вместо этого интересует сумма total для всех итераций цикла, логичнее было бы прибавлять к all total на каждой итерации, аналогично тому, как к counter на каждой итерации прибавляется 1.

[EUGY]

Мне непонятно условие выхода из цикла : цена == -1
И прежде чем будет выход из цикла, это отрицательное значение поучаствует в общем подсчете. Что можно ожидать?

[Perchik71]

автор хотел организовать, что-то вроде консольной меню.
т.е. ввод информации до тех пор пока пользователь этого хочет.
но обычно используют

for(;;)

а если автор написал "выход"
делаем ретурн - если в функции или брек.
у автора... всё равно должен выполниться код снизу.
но помойму проблема даже не в этом.
//----
подчёт должен быть за циклом... это однозначно.
ибо это не красивый подход, да и нелогичный.
у каждой операции должен быть свой логичный подход...
например цикл, только набивает... а после цикла идёт подчёт.
так будет просто правильней и не запутаешься.

[AcTiV]

спасибо всем )

наконец-то я понял как все это сделать

all += total;