Ocena programu :)

[mario22455]

Proszę o ocenę tego programu , jak na początki mojego programowania.Nazywa się maszyna losująca.

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
srand( time( NULL ) );
bool ok;
int liczba = ( rand() % 1000 ) + 1;
int strzal;
int nie;
bool wprowadzono = false;
   do
{
 if(liczba != strzal && wprowadzono )
 {
  cout << "Nie trafiles, sprubuj ponownie!" << endl;
 }
 cout << "Twoj strzal: ";
 cin >> strzal;
 ok = cin.good();
 if( strzal == liczba && ok )
 {
  cout << "Gratuluje trafiles!";
 }
 wprowadzono = true;
 if( strzal > 1000 || strzal < 1 )
 {
  cout << "Podales liczbe wieksza od 1000 lub mniejsza od 1!" << endl;
  wprowadzono = false;
 }
}while(liczba != strzal);
system("pause");
}

[Lesteross]

Dobrze, tylko w tej pętli od razu dałbym ten strzał, a później wszystkie warunki sprawdzające, ale może być.

Może się czepiam, ale po co ci ta zmienna "nie"?

[mario22455]

Ta zmienna "nie" to pozostałość od tego jak jeszcze kombinowałem z działaniem programu i teraz nie jest do niczego używana

[JebakLeśny]

a można wiedzieć, ile czasu się uczysz?

[Krycho]

Troszke przekombinowales Masz zmodyfikowany

 

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;
int main()
{
srand( time( NULL ) );
int liczba = ( rand() % 1000 ) + 1;
int strzal = 0;
do
{
	cout << "Twoj strzal: ";

	while( !(cin >> strzal) )
	{
		cout << "Wpisz liczbe.\nTwoj strzal: ";
		cin.clear();
		cin.sync();
	}

	if( strzal == liczba )
	{
		cout << "Gratuluje trafiles!\n";
	}
	else if( strzal > 1000 || strzal < 1 )
	{
		cout << "Podales liczbe wieksza od 1000 lub mniejsza od 1!" << endl;
	}
	else if(strzal < liczba) cout << "Celuj wyzej.\n";
	else if(strzal > liczba) cout << "Celuj nizej.\n";

}while(liczba != strzal);
system("pause");
   return 0;
}

[999sdb]

LOL, nie łatwiej tak?:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
srand( time( NULL ) );
int liczba = ( rand() % 1000 ) + 1;
int in;

cout << "Witaj!\n" << "Program wylosowal liczbe z przedzialu od 1 do 1000, zgadnij jaka to liczba.\n" << "Powodzenia!\n" << "\nTwoja liczba: ";
cin >> in;

while ( in != liczba)
{
 in > 1000 || in < 1 ? cout << "\nWpisales nieprawidlowa wartosc!\n" : cout << "\nTo nie ta liczba, spruboj ponownie\n";
 cout << "Twoja liczba: ";
 cin >> in;
}

cout << "\nZgadza sie! Liczba to " << liczba;
}

 

A jeszcze lepiej by było tak:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
int main()
{
srand( time( NULL ) );
int liczba = ( rand() % 1000 ) + 1, in;
cout << "Witaj!\n" << "Program wylosowal liczbe z przedzialu od 1 do 1000, zgadnij jaka to liczba.\n" << "Powodzenia, masz 10 szans!\n";
cout << "\nTwoja liczba: ";
cin >> in;
for (;
{
 int i = 10;
 while(--i && in != liczba)
 {
	 in > liczba ? cout << "\nZa duzo! Masz jeszcze " << i << " szans. \n" : cout << "\nZa Malo! Masz jeszcze " << i << " szans. \n";
	 cout << "\nTwoja liczba: ";
	 cin >> in;
 }
 in == liczba ? cout << "\nZgadza sie! Liczba to " << liczba << "\n": cout << "\nNiestety, nie udalo Ci sie.\n";
 cout << "\nJescze raz? (0 - tak, 1 - nie)\n";
 cin >> in;
 if (in) break;
}
}

 

 

A najlepiej:

#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
void easy(int liczba)
{
system("cls");
cout << "Witaj!\n" << "Program wylosowal liczbe z przedzialu od 1 do 1000, zgadnij jaka to liczba.\n" << "Powodzenia, masz 10 szans!\n";
int in, i = 10;
cout<< "\nTwoja liczba: ";
cin >> in;
while(--i && in != liczba)
{
 in > liczba ? cout << "\nZa duzo! Masz jeszcze " << i << " szans. \n" : cout << "\nZa Malo! Masz jeszcze " << i << " szans. \n";
 cout << "\nTwoja liczba: ";
 cin >> in;
}
in == liczba ? cout << "\nZgadza sie! Liczba to " << liczba << "\n": cout << "\nNiestety, nie udalo Ci sie.\n";
cout << "Jescze raz? (1 - tak, 0 - nie)\n";
cin >> in;
if (in) easy(liczba);
}
void hard(int liczba)
{
system("cls");
cout << "Witaj!\n" << "Program wylosowal liczbe z przedzialu od 1 do 1000, zgadnij jaka to liczba.\n" << "Powodzenia, masz 10 szans!\n";
int in, i = 10;
cout << "\nTwoja liczba: ";
cin >> in;
while(--i && in != liczba)
{
 in > liczba ? cout << "\nZa duzo! Masz jeszcze " << i << " szans. \n" : cout << "\nZa Malo! Masz jeszcze " << i << " szans. \n";
 cout << "\nTwoja liczba: ";
 cin >> in;
}
in == liczba ? cout << "\nZgadza sie! Liczba to " << liczba << "\n": cout << "\nNiestety, nie udalo Ci sie.\n";
cout << "Jescze raz? (1 - tak, 0 - nie)\n";
cin >> in;
if (in) easy(liczba);
}
int main()
{
srand( time( NULL ) );
int liczba = ( rand() % 1000 ) + 1, i;

cout << "Wybierz poziom trudnosci, easy = 1, hard = 2: ";
cin >> i;
if (i==1) easy(liczba);
else if (i==2) hard(liczba);
else cout << "Wpisales zla wartosc. Wynocha.";
}