Problem w Prostym Programie #Problem

Problem w Prostym Programie #Problem
Jan Gliszczyński
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 5 lat
  • Postów:1
0

Witam otóż napotkałem problem w swoim programie który pobiera liczby z klawiatury liczby tak długo aż suma wprowadzanych liczb przekroczy 100 lub zostanie wpisana dwa razy liczba 7. Oto on:

Kopiuj
#include <iostream>;

using namespace std;

short suma;
short liczba;
short liczby7;


main () 
{
	cout <<"Witaj Program pobiera liczby z klawiatury az ich suma osiagnie sto lub wcisniesz dwukrotnie liczbe siedem";
	while(suma<100||liczby7==2){
	
	cin >> liczba;
	suma=suma+liczba;
	
	
	if (liczba==7)  {
	liczby7++;
	
	
	}
}
	
	
	system("pause");
	return 0;
	
}

Program przed dodaniem drugiego członu (dwukrotnie wciśnięcie liczby siedem działał poprawnie z góry dziękuje za pomoc.

edytowany 4x, ostatnio: Jan Gliszczyński
kq
Wysłów się co ten program ma robić, bo z obecnego opisu nie idzie tego wywnioskować.
SnaaPP
  • Rejestracja:ponad 9 lat
  • Ostatnio:3 miesiące
  • Postów:20
1
Kopiuj
#include <iostream>

int main()
{
    int suma = 0;
    int liczby7 = 0;

    std::cout << "Witaj Program pobiera liczby z klawiatury az ich suma osiagnie "
                 "sto lub wcisniesz dwukrotnie liczbe siedem\n";

    while ((suma < 100) && (liczby7 < 2))
    {
        int liczba;
        std::cin >> liczba;
        suma += liczba;
        if (liczba == 7) ++liczby7;
    }
}

  1. Niepotrzebny średnik przy #include <iostream>
  2. Niezainicjalizowanie zmiennych suma i liczba7
  3. Zły warunek pętli - ma być &&
  4. Brak int'a przy main
  5. Staramy się nie deklarować zmiennych globalnych
  6. Beznadziejnie sformatowany kod. Polecam np. http://format.krzaq.cc/
edytowany 3x, ostatnio: SnaaPP

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.