Problem

[KeFu2K12]

Witam, mam problem.

cout << "Ile liczb chcesz odjac? ";
cin >> a;
for (i=1; i<=a; i++)
{
cout << "Podaj "<<i<<". liczbe: ";
cin >> liczby;
suma -= liczby;
}
cout << "Wynik: " << suma << endl;
system("pause");

Problem polega na tym, że gdy chcę odjąć dwie liczby np. 2-2 to wynik wychodzi mi -4 a powinien 0. Jak to naprawić ?

Z góry serdecznie dziękuję

[OczkoSX]

Numeracja tablicy (indeks) zaczyna sie od 0. W twoim przypadku robisz to od 1 więc wykraczasz poza zakres pamięci. Zmień w pętli warunek na i=0; i < a; i++.

Widzę że nie dałeś całego kodu. Upewnij się ze masz zadeklarowaną zmienną suma.

[Xau]

Wydaje mi się, że ta tablica nie jest ci do niczego potrzebna, ale nie widzę całego kodu, więc nie jestem pewien.

 

Numeracja tablicy (indeks) zaczyna sie od 0. W twoim przypadku robisz to od 1 więc wykraczasz poza zakres pamięci.

Nie wiadomo jaki rozmiar ma tablica, więc nie wiadomo czy indeks wychodzi poza tablice. Jest to błąd z innej przyczyny: Indeks zawsze będzie większy od zera, więc pierwszy element będzie nieużywany.

Liczby odejmujesz od zmiennej suma, więc jeśli suma = 0, to 0 - 2 - 2 = -4. Jeśli chcesz, żeby po podaniu 2 i 2 wyszło 0, od pierwszej liczby musisz odejmować resztę liczb.

[Minecraft2RP]

Po co uzywasz tablicy?

[KeFu2K12]

#include <iostream>

#include <cstdlib>

#include <cmath>

 

using namespace std;

 

int wybor,i,a=0,suma=0;

float liczby[100],x,y;

 

int main()

{

for(;

{

cout << "------------------------------" << endl;

cout << "| MENU GLOWNE |" << endl;

cout << "------------------------------" << endl;

cout << "| 1. Dodawanie |" << endl;

cout << "| 2. Odejmowanie |" << endl;

cout << "| 3. Mnozenie |" << endl;

cout << "| 4. Dzielenie |" << endl;

cout << "| 5. Pole kola |" << endl;

cout << "| 6. Obwod kola |" << endl;

cout << "| 7. Pole trojkata |" << endl;

cout << "| 8. Obwod trojkata |" << endl;

cout << "| 9. Pole prostokata |" << endl;

cout << "| 10. Obwod prostokata |" << endl;

cout << "| 11. Pole kwadratu |" << endl;

cout << "| 12. Obwod kwadratu |" << endl;

cout << "| 13. Pole trapezu |" << endl;

cout << "| 14. Obwod trapezu |" << endl;

cout << "| 15. Pole rownolegloboku |" << endl;

cout << "| 16. Obwod rownolegloboku |" << endl;

cout << "| 17. Pole rombu |" << endl;

cout << "| 18. Obwod rombu |" << endl;

cout << "| 19. Wyjdz |" << endl;

cout << "------------------------------" << endl;

cout << endl;

cout << "Wybierz: ";

cin >> wybor;

cout << endl;

 

switch(wybor)

{

case 1:

{

cout << "Ile liczb chcesz dodac? ";

cin >> a;

for (i=1; i<=a; i++)

{

cout << "Podaj " << i << ". liczbe: ";

cin >> liczby;

suma = suma + liczby;

}

cout << "Wynik: " << suma << endl;

system("pause");

}break;

 

case 2:

{

cout << "Ile liczb chcesz odjac? ";

cin >> a;

for (i=1; i<=a; i++)

{

cout << "Podaj "<<i<<". liczbe: ";

cin >> liczby;

suma -= liczby;

}

cout << "Wynik: " << suma<< endl;

system("pause");

}break;

case 3:

{

cout << "Podaj 1 liczbe: ";

cin >> x;

cout << "Podaj 2 liczbe: ";

cin >> y;

suma = x*y;

cout << "Wynik: " << suma << endl;

system("pause");

}break;

case 4:

{

cout << "Podaj 1. liczbe: ";

cin >> x;

cout << "Podaj 2. liczbe: ";

cin >> y;

suma = x/y;

cout << "Wynik: " << suma << endl;

system("pause");

}break;

case 5:

{

 

}break;

 

case 6:

{

 

}break;

 

case 7:

{

 

}break;

 

case 8:

{

 

}break;

 

case 9:

{

 

}break;

 

case 10:

{

 

}break;

 

case 11:

{

 

}break;

 

case 12:

{

 

}break;

 

case 13:

{

 

}break;

 

case 14:

{

 

}break;

 

case 15:

{

 

}break;

 

case 16:

{

 

}break;

 

case 17:

{

 

}break;

 

case 18:

{

 

}break;

 

case 19:

{

exit(0);

}break;

 

default: cout << "Nie ma takiej opcji w menu!" << endl;

system("pause");

system("cls");

}

}

return 0;

}

 

[REPREDATOR]

np. przed petle mozesz dac takie cos :

cout << "Podaj 1 liczbe: ";
            cin >> suma;


//---------------------------------
for (i=1; i<=a-1; i++)
{
cout << "Podaj "<<i+1<<". liczbe: ";
//... reszta bez zmian
}

[KeFu2K12]

up

 

Dzięki Za dużo stresu i nawet o tak prostej linijce kodu nie pomyślałem