Czy dobrze napisałem parametry przy funkcji rand(), chodzi o to by losowane liczby były z zakresu 100 - 999 włącznie
int liczba = rand() % 100 + 900;
C++ 
- Rejestracja:około 21 lat
- Ostatnio:prawie 3 lata
- Lokalizacja:Space: the final frontier
- Postów:26433
0
Ty tak poważnie? Losujesz coś i dodajesz do tego 900. Jak chcesz uzyskać coś mniejszego od 900?
0
Shalom napisał(a):
Ty tak poważnie? Losujesz coś i dodajesz do tego 900. Jak chcesz uzyskać coś mniejszego od 900?
liczby ujemne?
- Rejestracja:ponad 10 lat
- Ostatnio:ponad 9 lat
- Postów:420
0
Poprawiam:
int liczba = rand() % 900 + 100
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
Nie używaj rand jeśli nie musisz: http://channel9.msdn.com/Events/GoingNative/2013/rand-Considered-Harmful
Masz <random> z std::uniform_int_distribution:
mt19937 gen{random_device{}()};
uniform_int_distribution<> dis(100,999);
int val = dis(gen);
edytowany 1x, ostatnio: kq
- Rejestracja:ponad 10 lat
- Ostatnio:ponad 9 lat
- Postów:420
0
A jeszcze chciałem napisać taki program co losuje liczby od 100 do 999 włącznie i wypisze wszystkie 899 losowania bez powtórzeń, ale nie wiem czemu jak wypisze pierwszą liczbę to dalej nie wypisuje kod:
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
bool Prawda(int liczba, int tab[], int ile);
int main()
{
int tablica[999];
srand(time(NULL));
for(int i = 0; i < 999;)
{
int liczba = rand() % 900 + 100;
if(Prawda(liczba,tablica,i) == false)
{
tablica[i] = liczba;
cout << liczba << endl;
i++;
}
}
system("pause");
}
bool Prawda(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
tab[i] == liczba;
return true;
}
return false;
}
for(int i = 0; i < 999;) chcesz wylosować 999 liczb z przedziału 900 liczb? ;)
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
A jeszcze chciałem napisać taki program co losuje liczby od 100 do 999 włącznie i wypisze wszystkie 899 losowania bez powtórzeń,
Wrzuć 900 liczb kolejno do kontenera, np std::vector, potem std::shuffle i gotowe.
edytowany 2x, ostatnio: kq
Zobacz pozostały 1 komentarz
Po pierwsze, sam napisałeś, że nie działa zgodnie z Twoimi założeniami. Po drugie używasz
srand i rand. Po trzecie nazywasz funkcje w kompletnie nieczytelny sposób (co zwraca Prawda(x,y,z)?). Po czwarte wymyślasz koło na nowo zamiast użyć gotowych algorytmów (polecam obejrzenie http://channel9.msdn.com/Events/GoingNative/2013/Cpp-Seasoning ). Po piąte: używasz NULL zamiast nullptr oraz system("pause") do zatrzymania konsoli. No ok, może ciut się czepiam, ale współpracownika bym... zrugał za taki kod.
nie znam na tyle jęz angielskiego aby oglądać filmy po angielsku. Prawda(x,y,z) zwraca wartość true albo false zależy czy liczba się powtarza czy nie. Jeśli się nie powtarza to powinna wyświetlić się następna liczba wylosowana.
No to należy popracować nad angielskim, w tym fachu to podstawa :) . Zauważyłeś już kompletny brak związku między nazwą funkcji a jej działaniem?
Nie
std::random_shuffle a std::shuffle.
Fakt, dzięki, poprawione.
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
bool Prawda(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
tab[i] == liczba;
return true;
}
return false;
}
Nie sprawdzasz warunku, tylko wykonujesz porównanie, po czym po prostu zwracasz true. Poprawione:
bool Wystepuje(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
if(tab[i] == liczba)
return true;
}
return false;
}
Ale można to skrócić do: (pisane z pamięci, ale powinno być ok)
bool Wystepuje(int liczba, int tab[], int ile)
{
return any_of(tab, tab+ile, [=](int val) { return val == liczba; });
}
ewentualnie:
bool Wystepuje(int liczba, int tab[], int ile)
{
return any_of(tab, tab+ile, bind(equal_to<int>, liczba, placeholders::_1));
}
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
1
nowicjusz35 napisał(a):
ok dzięki, a czy istnieje taka możliwość aby wylosowane liczby z tablicy wyświetlać od najmniejszej do największej ?
Tak:
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
}
cout << endl;
edytowany 1x, ostatnio: kq
Zobacz pozostałe 7 komentarzy
@nowicjusz35 W przypadku gdy wylosowałeś wszystkie liczby z zakresu - po prostu wyświetl ich wartości w kolejności, tak jak pokazałem w poście (nie potrzebujesz sortować tablicy zawierającej liczby z zakresu 100-999, jeśli wiesz, że wszystkie występują). Jeśli nie - posortuj i wyświetl posortowany kontener.
ale oni wtedy wyświetlą się od najmniejszej do największej ? bo chyba będą pomieszane
Zauważ, że nie wyświetlasz zawartości tablicy, tylko liczby!
@kq - zwracam honor. Dopiero jak spojrzałem na kod to zobaczyłem o co chodzi. Gdy nowicjusz pisał co chce zrobić, myślałem, że chodzi tylko o wypisanie wyników bez powtórzeń. Ale dalej nie rozumiem, po co losować 899 liczb bez powtórzeń, tylko po to, żeby je wypisać w kolejności rosnącej, skoro moża to załatwic tą pętlą.
Myślę, że należy tutaj wziąć pod uwagę jego nick ;)
- Rejestracja:ponad 10 lat
- Ostatnio:ponad 9 lat
- Postów:420
0
Już mam program, który losuje liczby od 100 do 999 do tablicy bez powtórzeń za pomocą funkcji rand.
Tera chciałbym te wylosowane liczby z tej tablicy wyświetlić od najmniejszej do największej jak się da.
Losujesz 900 liczb czy mniej?
po prostu chcę zobaczyć czy takie coś jest możliwe, nigdy jeszcze takiego czegoś nie robiłem.
losuję 900 liczb równo
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
1
losuję 900 liczb równo
W takim razie postaraj się chwilkę logicznie pomyśleć. Masz 900 liczb, żadna się nie powtarza. Zakres ich wartości to [100,999], czyli jest ich 900. Inaczej mówiąc, masz gwarancję, że każda liczba z tego zakresu występuje dokładnie jeden raz. Wobec tego nie potrzebujesz przeprowadzać operacji na wylosowanej tablicy, po prostu wyświetl wartości pomiędzy 100 a 999, czyli:
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
}
cout << endl;
coś za bardzo nie działa, po pętli for to wstawić ?
Tam, gdzie chcesz wypisać wszystkie wylosowane liczby.
- Rejestracja:ponad 10 lat
- Ostatnio:ponad 9 lat
- Postów:420
0
coś nie chcę wypisać, robi się nieskończona pętla:
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
bool Wystepuje(int liczba, int tab[], int ile);
int main()
{
int tablica[999];
srand(time(NULL));
for(int i = 0; i < 999;)
{
int liczba = rand() % 900 + 100;
if(Wystepuje(liczba,tablica,i) == false)
{
tablica[i] = liczba;
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
i++;
}
}
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
}
cout << endl;
}
system("pause");
}
bool Wystepuje(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
if(tab[i] == liczba)
return true;
}
return false;
}
- Rejestracja:ponad 13 lat
- Ostatnio:około 4 lata
- Postów:107
1
/*
Tu sobie sortujesz to co masz
*/
// Tu wypisujesz liczby rosnąco od 100 do 999
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
}
cout << endl;
W wyniku masz:
100, 101, 102, 103, 104, **... **997, 998, 999,
edytowany 1x, ostatnio: Hell4Ge
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
Losujesz dopóki nie wylosujesz 999 różnych liczb z zakresu 900 liczb. Nic dziwnego, że pętla nigdy się nie kończy.
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
bool Wystepuje(int liczba, int tab[], int ile);
int main()
{
int tablica[900];
srand(time(nullptr));
// jest 900, nie 999, stad petla nieskonczona
// do tego w petli tylko losujesz, wyswietl po wylosowaniu
for(int i = 0; i < 900;)
{
int liczba = rand() % 900 + 100;
if(Wystepuje(liczba,tablica,i) == false)
{
tablica[i] = liczba;
}
}
// wylosowales, wyswietl wyniki:
for(int i = 100; i < 1000; ++i){
cout << i << ", ";
}
cout << endl;
// ugh
system("pause");
}
bool Wystepuje(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
if(tab[i] == liczba)
return true;
}
return false;
}
PS: indentacja. Wcześniej miałeś dobrą, a teraz "na odwal się"
edytowany 2x, ostatnio: kq
co masz na myśli mówiąc o indentacji ?
wyrównanie kodu. http://i.imgur.com/QkQb7KA.png nowy fragment wygląda tragicznie. (pomijając, że w złym miejscu wkleiłeś)
- Rejestracja:ponad 10 lat
- Ostatnio:ponad 9 lat
- Postów:420
0
faktycznie ten mój kod coś się pogorszył. Już ustawiłem wszystko tak jak powinno być:
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
bool Wystepuje(int liczba, int tab[], int ile);
int main()
{
int tablica[999];
srand(time(NULL));
for(int i = 0; i < 999;)
{
int liczba = rand() % 900 + 100;
if(Wystepuje(liczba,tablica,i) == false)
{
tablica[i] = liczba;
i++;
}
for(int i = 100; i < 1000; ++i)
{
cout << i << ", ";
}
system("pause");
}
}
bool Wystepuje(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
if(tab[i] == liczba)
return true;
}
return false;
}
tylko, że ta pętla co podałeś wyświetla po prosty liczby od 100 do 999, a chciałbym żeby to z tablicy w takiej kolejności wyświetlało
0
ale po co, jak wiadomo, że w tablicy są właśnie takie liczby
tak po prostu chciałbym zobaczyć jak taki program się piszę bo nigdy nie tego nie robiłem.
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
Wciąż masz pętlę nieskończoną:
// i nigdy nie przekroczy wartości 900
for(int i = 0; i < 999;)
Znów wyświetlasz w złym miejscu (t.j. w pętli nieskończonej):
for(int i = 0; i < 999;)
{
int liczba = rand() % 900 + 100;
if(Wystepuje(liczba,tablica,i) == false)
{
tablica[i] = liczba;
i++;
}
for(int i = 100; i < 1000; ++i)
{
cout << i << ", ";
}
system("pause");
}
Z sortowaniem i wyświetlaniem zawartości tablicy (co nie ma sensu w tym przypadku):
#include <algorithm>
#include <iostream>
#include <cstdlib>
#include <ctime>
using namespace std;
bool Wystepuje(int liczba, int tab[], int ile);
int main()
{
int tablica[900];
srand(time(NULL));
for(int i = 0; i < 900;)
{
int liczba = rand() % 900 + 100;
if(Wystepuje(liczba,tablica,i) == false)
{
tablica[i] = liczba;
i++;
}
}
// sortujesz zawartość tablicy
sort(begin(tablica), end(tablica));
// jak jesteś fanem średniowiecza i używasz kompilatora z tego okresu:
// sort(tablica, tablica+900);
// wyświetlasz zawartość tablicy
for(int i = 0; i < 900; ++i){
cout << tablica[i] << ", ";
}
cout << endl;
system("pause");
}
bool Wystepuje(int liczba, int tab[], int ile)
{
for(int i = 0; i < ile; i++)
{
if(tab[i] == liczba)
return true;
}
return false;
}
edytowany 2x, ostatnio: kq
ale mam przecież i++; a potem i < 999 to czemu jest nie skończona ?
ok dzięki
bo
i++ jest wykonywane wyłącznie jeśli dodasz nowy element do tablicy. A dodać nowego elementu nie możesz ponad 900, ponieważ po dodaniu 900 elementów masz 100% szans na powtórzenie się liczby.
- Rejestracja:prawie 12 lat
- Ostatnio:około 22 godziny
- Lokalizacja:Szczecin
2
Przy okazji, zamiast wymyślać koło na nowo, możesz też użyć algorytmów z biblioteki standardowej oraz pomyślunku. Cały Twój kod można uprościć do:
vector<int> data(900);
iota(begin(data), end(data), 100);
// zawartość data to teraz [100, 101, 102... 997, 998, 999]
shuffle(begin(data), end(data), mt19937{random_device{}()});
// zawartość data to teraz liczby z zakresu <100, 999> bez powtórzeń, w
// kolejności losowej
sort(begin(data), end(data));
// zawartość data to znów [100, 101, 102... 997, 998, 999]
ok dzięki
Shalom