Dlaczego nie wchodzi w ifa?

Question

Witajcie, napisałem taki oto program, miał on służyć do losowania bez powtórzeń, wydaje mi się, że jest dobry, nie rozumiem jednak, dlaczego mimo spełnienia warunku if się nie wykonuje, przez co pętla nigdy się nie kończy...

 

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



int main()
{
	srand(time(NULL));
	
	bool zakres[101];
	int wylosowane[11];
	int liczba;
	int wylosowanych = 0;

	for (int i = 0; i <= 100;i++) zakres[i] = false;
	
	
	
	do
	{
		liczba = rand() % 100 + 1;
		cout << "oto twoja liczba: "<<liczba << endl;
		cout << "czy taka liczba już była? : " << zakres[liczba];
		system("pause");
		if (zakres[liczba] = false)
		{
			cout << "udalo sie wejsc w for";
			wylosowane[wylosowanych] = liczba;
			wylosowanych++;
			zakres[liczba] = true;

		}

		

	} while (wylosowanych != 10);

		for (int i = 0; i < 10; i++)
	{
		cout << wylosowane[i];
		
		}
	
	
	return 0;
}

 

Answer 1

Masz kilka błędów, możesz mi na samym początku wyjaśnić dlaczego rozmiary tablicy robisz o rozmiarach 11, 101, ... ? Stoi za tym jakiś logiczny powód, czy to wynika z głupoty każącej ci numerować od 1, a nie od 0?

W tej linii, przypisujesz wartości w tablicy false, a nie porównujesz, porównuje się przy użyciu podwójnego znaku równości ==

if(zakres[liczba] = false)

 

Comments

Numeruje tak, dlatego, że liczby, które losuje mają być z przedziału od 1 do 100, a mój sposób polega na tym, żeby nie prosić programu o sprawdzanie całej tablicy, tylko udanie się do szufladki z danym numerem.

Chociaż teraz jak to piszę to w sumie mogłem tylko napisać pod koniec, żeby wypisywało o liczbę o jeden większą.

Tak czy siak dzięki za pomoc, podejrzewam, że błąd wziął się z tego, że teraz na lekcjach w szkole programujemy w pascalu, stąd przyzwyczajnie i pomieszanie.

---

Stoi za tym jakiś logiczny powód, czy to wynika z głupoty każącej ci numerować od 1, a nie od 0?

Nie brzmi to przyjaźnie. ;) Wypełnianie ma zrobione poprawnie, pełen zakres zostaje obsłużony, więc dlaczego akurat o to pytasz?

---

Kto pyta nie błądzi, kto nie pyta nigdy się nie dowie ;)

Nie mam nic do niego, może z czasem zacznie pytać grzeczniej :P

---

Bo wciąż nie wykorzystuje całej zarezerwowanej pamięci (jasne, uzupełnia całą tablicę, ale ostatecznie w wyniku uwzględnia tylko pierwsze liczby, czyli mamy marnotractwo pamięci)

---

gdybym miał sprawdzać całą tablicę od początku do końca z kolei marnotrawilibyśmy czas.

jakie rozwiązanie proponujesz?

---

Ja mówię tylko o niepotrzebnym rezerwowaniu tablicy na [11] i [101] elementów, chyba że gdzieś wykorzystujesz element [10] i [100] to wtedy zwracam honor (o ile to jeszcze nie kwesta dodania 1, bo wtedy to żadna oszczędność).

---

dodania 1 i zarezerwowania tablicy [10] i [100], wtedy już oszczędzamy tak jakbyś chciał :p