• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Losowanie bez powtórzeń . Problem z kodem .

VPS Starter Arubacloud
0 głosów
207 wizyt
pytanie zadane 16 czerwca 2017 w C i C++ przez somix12 Nowicjusz (190 p.)

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;
}

 

2 odpowiedzi

+3 głosów
odpowiedź 16 czerwca 2017 przez Gynvael Coldwind Nałogowiec (26,850 p.)
edycja 16 czerwca 2017 przez Gynvael Coldwind

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ć.

+1 głos
odpowiedź 16 czerwca 2017 przez mokrowski Mędrzec (155,460 p.)

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

Podobne pytania

0 głosów
1 odpowiedź 141 wizyt
0 głosów
2 odpowiedzi 1,135 wizyt
pytanie zadane 29 lipca 2017 w C i C++ przez KubaW Nowicjusz (120 p.)
0 głosów
2 odpowiedzi 585 wizyt
pytanie zadane 29 października 2016 w C i C++ przez BlueWee Użytkownik (730 p.)

92,453 zapytań

141,262 odpowiedzi

319,088 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...