Powtarzanie programu

Question

Witam

Mam stworzyć program gdzie podaje się pierwszą liczbę komputer losuje drugą i je mnoży. Program ma to powtarzać dopóki nie wciśnie się q. Nie wiem co ja robię źle ale program nie działa.
Oto mój program:
 

#include <iostream>
#include <cstdlib>
#include <conio.h>
#include <ctime>
using namespace std;

int main()
{
    int liczba1, wynik;
    char klawisz, koniec;
    srand( time( NULL ) );
    int liczba2=( std::rand() % 6 ) + 7;
    
    do
    {
        cout<<"Podaj liczbe do pomnozenie: ";
        cin>>liczba1;
        while (liczba1<0)
        {
            cout<<"Liczba nie moze byc ujemna"<<endl<<endl;
            cout<<"Podaj liczbe do pomnozenie: ";
            cin>>liczba1;
        }
    
        cout<<"Mnozenie "<<liczba1<<" * "<<liczba2<< std::endl;

        cout<<"Jezeli juz obliczyles nacisnij ENTER"<<endl<<endl;
        klawisz=getch();
        while (klawisz!=13)
        {
            cout<<"Wcisnij ENTER"<<endl<<endl;
            klawisz=getch();
        }
        while (klawisz==13)
        {
            wynik=liczba1*liczba2;
            cout<<"Wynik to: "<<wynik<<endl<<endl;
            cout<<"Jezeli chcesz zakonczyc nacisnij 'q' jezeli nie chcesz nacisnij inny klawisz"<<endl<<endl;
            koniec=getch();
        }    
    }
    while (koniec='q');
    {
        cout<<"Koniec";
    }
    
    return 0;

 

Comments

Wklej to w bloczek kodu 

//coś takiego

 

---

Propozycja  

#include <iostream>
#include <ctime>
using namespace std;
 
int main()
{
    int liczba1;
    char koniec;
    srand(time(NULL));
    int liczba2 = (rand() % 6) + 7;
     
    while(true)
    {
        cout << "Podaj liczbe do pomnozenie: ";
        cin >> liczba1;
        while (liczba1<=0)
        {
            cout << "Liczba nie moze byc ujemna lub rowna zero\n"
                 << "Podaj liczbe do pomnozenie: ";
             cin >> liczba1;
        }
     
        cout << "Mnozenie " << liczba1 << " * " << liczba2
             << "\nJezeli juz obliczyles nacisnij ENTER ";

        cin.ignore();
        cin.get();

        cout << "Wynik to: " << (liczba1*liczba2)
             << "\n\nJezeli chcesz zakonczyc nacisnij 'q' "
             << "jezeli nie chcesz nacisnij klawisz Enter ";

        cin.get(koniec);
        if (koniec=='q') break;
    }

    cout << "Koniec";
     
    return 0;
}

---

Dziękuje przyda się

Answer 1

Rozpoczynasz pętle do...while i jej nie zakańczasz tej pętli bo widzę kolejne nawiasy czyli wychodzi na to ze rozpoczynasz kolejną pętle while w której używasz tak jak ci napisał Author operatora przypisania a nie porównania.

Comments

Czyli jak by to poprawnie miało wyglądać bo nie za bardzo rozumiem?

---

operator przypisania to pojedynczy znak równa się (=)

operator porównania to podwójny znak równa się (==)

https://miroslawzelent.pl/kurs-c++/operatory-w-c++/

 

pętla do...while

https://cpp0x.pl/kursy/Kurs-C++/Poziom-2/Petla-do-while/18

---

A po za tym to usuń std:: sprzed funcji rand bo masz to zdefiniowane u góry linijka 5

---

@Marak123, Dziękuje bardzo

Answer 2

Używasz operatora przypisania a nie porównania. Pomijając to popełniasz wiele błędów - po co piszesz std:: jak wcześniej masz using namespace std;