Problem początkującego

Question

Cześć,

nazywam się Łukasz i jestem właśnie na początku swojej koderskiej drogi. Ostatnio wyznaczyłem sobie projekt, którego celem jest napisanie programu, który prosi o podanie liczby od 3 do 30, a następnie losuje liczby pseudolosowe do momentu, aż wylosuje tę samą liczbę, którą podał użytkownik, a wtedy informuje go o tym i wydaje dźwięk z głośniczka systemowego. Program zamiast działać losuje w nieskończoność i zawiesza się. Dlatego zdecydowałem się poprosić o pomoc. Zdaję sobie sprawę, że kod może mieć mnóstwo błędów i może wyglądać tragicznie, ale proszę o zrozumienie, to początki :) Kiedyś będzie lepiej. Więc jeśli ktoś wie to proszę powiedzieć co poprawić w tym kodzie, aby program działał jak należy. Dziękuję i pozdrawiam!

 

#include <iostream>
#include <windows.h>
#include <time.h>

using namespace std;

int liczba_usera, liczba_losowa;

int main()
{
    cout << "Witaj, podaj liczbe miedzy 3, a 30";
    cin >> liczba_usera;

    while (liczba_usera = liczba_losowa);
    {
        srand(time(NULL));
        liczba_losowa = rand()%3-31;
        cout<<"Wylosowano Twoja liczbe:"<<liczba_losowa<<"\a";
    }

    return 0;
}

 

Comments

użyj == to jest porównanie, bo = to przypisanie.

Tam w while :D

Answer 1

Te losowanie jest skopane

1 srand(time(NULL)); <- umieszczasz odrazu po  int main () nie gdzieś w środku

2 ta liczba losowa  powina wyglądać tak liczba_losowa =  (rand()+3) % 31, chyba w każdym normalnym języku operatory maja swoją wagę i operator % ma najwyższą wagę jak dobrze pamiętam, czyli w przypadku bez nawiasów wykona operacje 3 % 31 a potem ten wynik z rand ()

3  = to przypisanie, a  czy równa się to ==

4 W tym while powinien być if, ponieważ jeśli  warunek zostanie spełniony to dopiero pętla wystartuje ( w tym przypadku tak nie jest )

Answer 2

Urzyłeś operatora '=' zamiast '==' w pętli. W c++ operatorem porówania jest podwójny znak równania, natomiast pojedynczy jest operatorem przypisania.

Pozdrawiam,

Kacper

Answer 3

Kod poprawiłem, dziękuję za szybką i rzeczową pomoc. Jednak teraz pojawił się nowy problem. Program zamiast losować, aż wylosuje liczbę podaną przez użytkownika pokazuje na ekranie liczbę jaką wylosował (a tą liczbą zawsze jest 0) i wydaje dźwięk. Coś jest nadal źle...
#include <iostream>
#include <windows.h>
#include <time.h>

using namespace std;

int liczba_usera, liczba_losowa;

int main()

{
    cout << "Witaj, podaj liczbe miedzy 3, a 30";
    cin >> liczba_usera;
    srand(time(NULL));

    while (liczba_usera == liczba_losowa);
    {
         liczba_losowa ==  (rand()+3) % 31;
        cout<<"Wylosowano Twoja liczbe:"<<liczba_losowa<<"\a";
    }

    return 0;
}

 

Comments

Już lepiej lecz nadal jest źle.

#include <iostream>

#include <windows.h>

#include <time.h>



using namespace std;



int liczba_usera, liczba_losowa;



int main()

{

    srand(time(NULL)); // powinien być tu 

    cout << "Witaj, podaj liczbe miedzy 3, a 30";

    cin >> liczba_usera;

   

    do

   {

        liczba_losowa ==  (rand()+3) % 31;

        if ( liczba_losowa == liczba_usera ) {

               cout<<"Wylosowano Twoja liczbe:"<<liczba_losowa<<"\a";
             

        }

    }while (liczba_usera != liczba_losowa)



    return 0;

}

 

---

Dzięki wielkie za pomoc! Jednak program nadal nie działa jak należy, po podaniu liczby (int liczba_usera) nic nie robi..

---

A skąpilowałeś to ?

Dodałeś bibliotekę cstdlib ?

Wstawiłeś średniki w odpowiednie miejsce ?

Zmieniłeś linijkę  liczba_losowa == (rand()+3) % 31; tam powino być 1x =