Losowanie bez powtórzeń . Problem z kodem .

Question

Witam , chce zrobic program ktory wylosuje 14 liczb z przedziału od 1 do 30 bez powtórzenia się każdej z nich . Na końcu ma wypisać wyniki. Problem w tym że kompilator nic nie wypisuje . Proszę o spojrzenie i poinformowanie mnie gdzie zrobiłem błąd.

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

int liczba;
int liczby[15];
int i;
int j;

bool sprawdzanie(int liczbai , int index , int tablice[])
{

for(j=0; j<index; j++)
{
    if(liczbai==tablice[j])
      {
    return false;
      }
return true;
}

}

int wypis()
{
   for(i=0; i<14; i++)
    {
        cout<<liczby[i]<<endl;
    }

}

int main()
{
   srand(time(NULL));
   liczba=rand()%30+1;
   liczba=liczby[0];
   i=1;
   while(i<14)
   {
  liczba=rand()%30+1;

  if(sprawdzanie(liczba , i , liczby)==true)
   {
   liczba=liczby[i];
   i=i++;
   }
   }
   wypis();
    return 0;
}

 

Answer 1

Masz kilka bardzo ciekawych błędów w kodzie :) Generalnie można by to napisać inaczej (np. używanie zmiennych globalnych jest trochę niebezpieczne, a raczej "błędogenne", no i sam algorytm ma w takim wydaniu złożoność O(n**2) lub gorzej, i tak dalej), ale domyślam się, że chodzi Ci o doprowadzenie Twojego kodu do stanu używalności, więc:

1. Linie 38 oraz 46: masz tam odwrotnie przypisanie. Powinno być liczby[tutaj index] = liczba;
2. W linii 47 masz bardzo zabawny błąd określany mianem "undefined behaviour", który w tym wypadku polega na tym, że dwukrotnie próbujesz zapisać wartość do zmiennej i. Tak naprawdę wystarczy w tej linii samo i++ (to od razu zapiszę zmienioną wartość do zmiennej i, więc nie jest potrzebne oddzielne przypisanie).
3. EDIT: Ach, i jeszcze return true; z linii 20 powinien być przestawiony za najbliższy } (tj. powinien zostać wyrzucony poza pętle for).

Po poprawieniu tych trzech linii powinno działać.

Answer 2

Błąd jest na poziomie koncepcji:

1. Zadeklaruj tablicę lub lepiej vector wypełnioną wartościami od 1 do 30
2. Przemieszaj wartości z użyciem shuffle z <algorithm>
3. Wypisz 14 pierwszych wartości. 

To co starałeś się napisać jest:

1. Niedeterministyczne (nie wiadomo jak długo będzie działało.
2. Używa przestarzałego rand() które ma bardzo wiele wad
3. Jest napisane z błędami
4. Używa zmiennych globalnych