[R.....y]

https://pastebin.com/HnR9W61U

ktoś wie czemu te warunki są niemożliwe do spełnienia?

#programowanie #programista15k #cpp

[MamCieNaHita]

@RedveKoronny:

te warunki

150 linijek kodu

[patrolez]

@RedveKoronny: kod jest ujowy, bo więcej jest w komentarzach niż w treści.
Niegodnie jest to mianować C++, a raczej C z klasami lub też C z - bo w sumie nic prócz tego tam nie ma z C++.

[R.....y]

@MamCieNaHita: no całość ma 150 warunków kodu z czego jakas połowa to puste linie i komentarze, te warunki sa dosyć proste. sprawdzam w którym miejscu tabeli jestem. następnie sprawdzam wszystkie poprzednie, jezeli liczba sie powtorzyła to losuj dalej i tak w kółko

[R.....y]

@patrolez: co to musi mieć, bez obiektów i klas żeby to był c++?

[R.....y]

moge wkleić bez komentarzy, ale powodzenia w połapaniu sie co jest czym potem :P

[patrolez]

co to musi mieć

@RedveKoronny: mieć klasy i obiekty? One właśnie powodują, że da się pisać kod bez komentarzy.

[lionbest]

@RedveKoronny: Czepiają się.
Ale jak chcesz wiedzieć to zamiast new int* to
std::vector>
oraz
#include

[R.....y]

@patrolez: cóż, jak naucze sie podejscia obiektowego to go może użyje, ale narazie mi sie spieprzył kod na wskaźnikach więc nie wiem czy używanie obiektowego podejścia będzie dobrym pomysłem

[patrolez]

@RedveKoronny: widzę, że większość twoich prototypów ma int** table jako pierwszy argument.
Jest to idealny kandydat do stworzenia jakiegoś rodzaju hermetyzacji.

[R.....y]

@lionbest: dzięki, a wytłumaczyłbyś co to robi?

[patrolez]

jak naucze sie podejscia obiektowego

@RedveKoronny: trzeba było dodać tag #naukaprogramowania to bym się aż tak nie przywalał :P
W każdym razie wiedz, że to jest za dużo kodu względem tak ogólnego pytania.

[R.....y]

@patrolez: nie znam wielu tagów na wykopie :P
możliwe, poprzednia wersja programu nie dość że działała, to była trochę mniejsza (albo porownywalna) ale była kompletnie bez funkcji, i potencjalnie była sporo wolniejsza bo nie miała ani jednego wskaźnika na których chce to teraz zrobić, też w ramach nauki

[R.....y]

@lionbest: jak dodam ten vector to będe musiał w programie coś dalej pozmieniac?

[patrolez]

@RedveKoronny: nie rób tego, jeżeli twoim celem jest nauka wskaźników.

Nowoczesny C++ to praktycznie unikanie gołych wskaźników gdzie się da.
std::vector to jedno z obejść i ułatwień - nie na czas nauki wskaźników.

[R.....y]

@patrolez: dlaczego? i czego się używa zamiast nich?

[patrolez]

@RedveKoronny: znaczy, żeby zrozumieć to czego się używa w nowoczesnym C++, to i tak wypadałoby znać arytmetykę wskaźników - że każdy typ ma swój własny rozmiar itp.

Innymi słowy twój kod na twoim etapie nauki jest super!

Nowoczesny C++ to przede wszystkim tzw. sprytne wskaźniki, których mechanizm opiera się ściśle o zasięg/cykl-życia (scope) oraz tzw. podejście RAII. C++ rozróżnia się też od C przez istnienie

[R.....y]

@patrolez: dzięki wielkie, zainteresuje się tym :)
a dałbyś radę mi powiedzieć co jest z moim kodem warunki CheckSquares, CheckRows i CheckColumn nie mogą zostać spełnione? bo z tego co widzę warunki są identyczne co w pierwszej wersji programu https://pastebin.com/kEAM3YbY która zwracała wypełnioną tabelę w mniej niż sekundę (raz sie udało w 0 cykli, zwykle potrzebuje ok. 2000 cykli)

[lionbest]

@RedveKoronny: W ogóle ten kod jest zły, brak tu backtrackingu czy w postaci rekurencji, czy innej.

[R.....y]

@lionbest: jak juz kolega napisał wyżej, zapomniałem tagu #naukaprogramowania, jestem narazie na etapie wskaźników

[lionbest]

@RedveKoronny: No dobra jak przekroczysz 1000 iteracji to cofasz się do początku, co jest bardzo losowe i oznacza to, że praktycznie musisz wylosować całą planszę prawidłową inaczej twój algorytm nie zadziała.

Jedyna różnica jest taka, że w CheckSquares ustawiasz nową tablicę, a nie zapisujesz random do starej.

Backtracking polega na tym, że wstawiając kolejne pole, jeżeli sprawdzisz wszystkie możliwości, to cofasz się do poprzedniego wstawienia i wybierasz następną możliwą opcję,