[levil]

Pisze #program w #c++ i wyrzuca mnie przy

test = ile / 2;
ile_przerw = rand()%test + 2;
cout << ile_przerw;
prz *przerwy = new prz[ile_przerw];

jak wartość ile >19 do 19 działa, jakieś pomysły ?
#programowanie #cpp #pytanie

[Felonious_Gru]

@levil: Wrzuć ccałość kodu np. na pastebin. Wrzuć też błąd, jaki wyskakuje.

[Kaczus2B]

@levil: Obawiam się, że skoro Cię "wywala", to błąd musi byc gdzie indziej, a tu się tylko ujawnia.

[levil]

kod: http://pastebin.com/vdzqWyAc
nie dostaje kodu błędu wyrzuca tu:

_MRTIMP2_PURE_NPURE void __CLRCALL_PURE_OR_CDECL _Locinfo::_Locinfo_ctor(_Locinfo *pLocinfo, const char *locname)
 { // switch to a named locale
 const char *oldlocname = setlocale(LC_ALL, 0);

[KrzaQ2]

@levil: uƶywanie nagiego new to antyidiom. Polecam std::vector

[Kaczus2B]

@levil: instancja = new inst[ile];

for (int i = 0; i <= ile; i++)//losowe wartosci dla zadan
wychodzisz poza tablicę o jeden element za daleko!

[KrzaQ2]

@levil: debugger w rękę i sprawdzasz. Nieczytelne nazewnictwo i formatowanie, magiczne numerki, łamanie DRY.

Aha no i z mniejszych problemów: uƶywasz rand() zamiast ``: https://channel9.msdn.com/Events/GoingNative/2013/rand-Considered-Harmful

[C.....m]

@KrzaQ2: alokuje pamięć ręcznie i tak, największym jego zmartwieniem to rand() w gównoaplikacji bez wymagań bezpieczeństwa zamiast przerzucenie się na wektor/unique_ptr/array/cokolwiek takiego?

Miejcie ludzie priorytety... ;) Tak samo zamiast iterować ręcznie po for to:
a) range-for typu for(typ& wartość : kontener) { ... }
b) iteratory typu for(auto it = kontener.begin(); it != kontener.end(); ++it) { ... }
Za długie funkcje, masakra jak się to czyta. Zmienne opisane niezbyt jasno. Ogółem

[KrzaQ2]

@CoJaCzytam: http://www.wykop.pl/wpis/16182671/#comment-55411955

[C.....m]

@KrzaQ2: Wybacz, widziałem że ktoś to napisał ale myślałem że to nie Ty. Zmęczenie robi swoje.