Samododawanie się losowej liczby do wyniku

Question

#include <iostream>
#include <stdio.h>

using namespace std;

int wyjscie, tunel;
int stopien;
int alarm(int w);
char odp;

int main()
{
pocz_alarmu:
cout<<"Czy wywolac alarm? t/n\n";
cin>>odp;
if (odp='t','T')

{
    cout<<"UWAGA ALARM\n"<<"************************************************\n"<<"Ktorego stopnia? ";
    cin>> stopien;
 //   alarm(stopien);

    switch (stopien)
        {
        case 1:
        wyjscie=5;
        tunel=7; //tutaj dodaje do 7 liczbe 6295968

            cout<<alarm(stopien);
        break;

        case 2:
            wyjscie=10;
            tunel=5; //tutaj dodaje do 5 liczbe 6295968
            cout<<alarm(stopien);
        break;

        default:
        cout<<"Nie ma takiego stopnia!";
        break;

        }}
else if(odp=='n','N')
{
cout<<"Przeciez wszystko jest w porzadku";
}

else
{

goto pocz_alarmu;
}
    return 0;
}

int alarm(int w)            //Funkcja podmienia numery wyjscia i tunelu w zaleznosci od stopnia alarmu
{
cout<<"Prosze kierowac sie wyjsciem nr."<<wyjscie<<" Tunelem nr."<<tunel;
}

Dlaczego dodaje mi te liczby przy kompilacji programu, i czy da się ogólnie zrobić coś żeby to wszystko ładnie wyglądało?

Answer 1

Hmmm... A to Ci w ogóle działa? Choćby funkcja 'alarm' powinna zwracać wartość (return).

A to:

cout<<alarm(stopień);

jakim cudem działa? Wywalić cout, to może pójdzie...

Poza tym, czemu 'goto'? Nie lepiej wewalić wszystko w np. 'while'? Ja tam nie wiem, ale użycie goto to jakby prośba o kłopoty i niewygody. Daaaawno temu, w Basicu na C64 to owszem, ale tu jakoś niezbyt pasuje.

Wybacz, że się czepiam, jakby co to zignoruj

Comments

Faktycznie, teraz działa.

Poza tym z racji tego że jestem początkujący, każda rada mile widziana :)

I mógłbyś napisać jak wstawił być w to w while?

---

Dobra, na szybkiego takie coś, powinno działać. Wiem, niejeden powie, że można inaczej, lepiej, wydajniej - normalne, jasne, że można. Ale jak na teraz zrobiłem tak, grunt, że działa

#include <iostream>
#include <string>

using namespace std;

void alarm(int, int);

int main()
{
	int wyjscie{}, tunel{}, stopien{};
	string odp;

	do
	{
		cout << "Czy wywolac alarm? t/n, q - wyjscie" << endl;
		cin >> odp;
		if (odp == "t" || odp == "T")
		{
			cout << "UWAGA ALARM\n" << "************************************************\n" << "Ktorego stopnia? ";
			cin >> stopien;
			
			switch (stopien)
			{
			case 1:
				wyjscie = 5;
				tunel = 7; 
				alarm(stopien, tunel);
				break;
			case 2:
				wyjscie = 10;
				tunel = 5; 
				alarm(stopien, tunel);
				break;

			default:
				cout << "Nie ma takiego stopnia!";
				break;
			}
		}
		else if (odp == "n" || odp == "N")
		{
			cout << "Przeciez wszystko jest w porzadku" << endl;
			break;
		}
	} while (odp != "q");
	
	cin.get();
	cin.get();
}

void alarm(int w, int t)            
{
	cout << "Prosze kierowac sie wyjsciem nr." << w << " Tunelem nr." << t << endl;
}

Takie drobne uwagi:

• jeśli to nie jest absolutnie konieczne, nie twórz zmiennych globalnych (przed rozpoczęciem 'main');
• do funkcji można przekazać różną ilość i typ parametrów;
• stringa użyłem z przyzwyczajenia, jakoś to lubię :-)
• jeśli funkcja ma coś zwracać, wywołanie powinno być mniej więcej takie:

int costam = funkcja();
.
.
.
int funkcja()
{
........tu treść funkcji.....
return jakastam_zmienna;
}

 

Answer 2

Dalej:

if (odp = 't', 'T')

to na pewno działa?

Może lepiej tak:

if (odp =='t'||odp=='T')

Pojedynczy znak = to przypisanie, nie porównanie...