c++ comparison between signed and unsigned integer expressions

[boobs]

Mam taki programik:

Kod: Zaznacz cały

#include <iostream>
#include <cstring>
#include <stdlib.h>
#include <cstdio> 
#include <ctime> 
#include <cmath>
using namespace std;

//Lotto

const unsigned ILOSC_LICZB = 6 ;
const int MAKSYMALNA_LICZBA = 49;



int main()
{

//deklaracja i wyzerowanie tablicy liczb
unsigned aLiczby[ILOSC_LICZB];
for (int i = 0; i < ILOSC_LICZB; ++i)
aLiczby[i] = 0;

//losowanie liczb

srand (static_cast<int>(time(NULL)));
for (int i = 0; i < ILOSC_LICZB ;)
{
    //wylosowaniee liczby
    aLiczby[i] = rand() % MAKSYMALNA_LICZBA + 1;
    
    
    // sprawdzenie czy sie ona nie powtarza
    bool bPowtarzaSie = false;
    for (int j = 0; j< i; ++j)
    {
        if (aLiczby[j] == aLiczby[i])
        {
            bPowtarzaSie = true;
            break;
        }
    }
    //jeżeli sie nie powtarza przechodzimy do nastepnej liczby
    if (!bPowtarzaSie) ++i;
}

// wyświetlanie wylosowanych liczb

cout << "wyniki losowania : " << endl;
for (int i=0; i < ILOSC_LICZB;i++)
cout << aLiczby[i] << " ";
// koniec



return 0;
}

Niby program się kompiluje lecz dostaje przy pętlach ,,for'' takie coś:

Kod: Zaznacz cały

lotto.cpp:21: warning: comparison between signed and unsigned integer expressions

[AdeBe]

No przecież to widać jak na dłoni: ILOSC_LICZB zadeklarowałeś jako unsigned int, ale jako licznika w pętlach używasz już zwykłego int.

Przy okazji jeszcze kilka uwag:
1. Odradzam stosowanie zmiennych globalnych. Do niczego nie są ci w tym programie potrzebne, a nauczysz się złych nawyków.
2. Bardzo dobrze, że stałe zadeklarowałeś jako const (a nie przez #define), jednak z reguły nazwy pisane wielkimi literami rezerwuje się właśnie dla dyrektyw preprocesora.

[boobs]

Witam.
Mój problem polega na tym iż programik losuje mi tylko raz losową liczbę, a powinien za każdym razem inną.

Kod: Zaznacz cały

#include <iostream>
#include <cstring>
#include <stdlib.h>
#include <cstdio> 
#include <ctime> 
#include <cmath> 
using namespace std;


int oewe()
{
    cout << "Wrum1 " << endl;
return 0;
}

int liczb()
{
    unsigned losowa;
    srand (static_cast<int>(time(NULL)));
    
    losowa = rand() % 100 + 2;
cout << losowa << endl;
return 0; 
}

int i = 0;

int main()
{
for (int xxx;xxx < 5;++xxx)
{
oewe();
liczb();

}


return 0;
}

[db]

Kod: Zaznacz cały

srand (static_cast<int>(time(NULL)))

Ustawiasz parę razy `seed' na tą samą wartość. Zobacz co zwraca time().

Kod: Zaznacz cały

 srand ((int) time(NULL));
 int n1 = rand() % 46 + 1;
 int n2 = rand() % 46 + 1;
 int n3 = rand() % 46 + 1;
 printf("%d %d %d\n", n1,n2,n3);

 srand ((int) time(NULL));
 n1 = rand() % 46 + 1;
 srand ((int) time(NULL));
 n2 = rand() % 46 + 1;
 srand ((int) time(NULL));
 n3 = rand() % 46 + 1;
 printf("%d %d %d\n", n1, n2, n3);

Kod: Zaznacz cały

26 38 7
26 26 26