Uwagi do kodu

[tomaszdrab]

No description provided.

[tomaszdrab]

projekt_sudoku/grafika.c

Line 33 in db13170

for(int i=0; i < 2000 && rzad <=8; i++)

Zastanawia mnie znaczenie tej liczby. Dlaczego akurat 2000?

[tomaszdrab]

projekt_sudoku/grafika.c

Lines 84 to 88 in db13170

	const gchar *name;
	if(i==0) name = "latwy";
	else if(i==1) name = "sredni";
	else if(i==2) name = "trudny";
	else if(i==3) name = "niestandardowy";

A może lepiej byłoby przed pętlą zadeklarować tablicę

const char* names[] = {"latwy", "sredni", "trudny", "niestandarodwy"};

i zamiast name posługiwać się wewnątrz names[i]?

[tomaszdrab]

projekt_sudoku/grafika.c

Lines 160 to 163 in db13170

	if(name[0] == 'l') PokazPodpowiedzi(&p1,50);
	else if(name[0] == 's') PokazPodpowiedzi(&p1,40);
	else if(name[0] == 't') PokazPodpowiedzi(&p1,30);
	else if(name[0] == 'n') PokazPodpowiedzi(&p1,20);

I w tym miejscu podobnie. A nawet okazuje się, że ma to związek z tą poprzednią tablicą, więc może lepiej byłoby ją zadeklarować tak, żeby była widoczna dla obu funkcji i w niej nawet zapisać odwzorowanie: łatwy -> 50, średni -> 40 itd.?

[MateuszM02]

Tutaj miała być pętla nieskończona, ale początkowo program był źle napisany i się nie kończył, więc dałem limit 2000 kroków. Potem, jak zrobiłem działający algorytm, to zapomniałem to zmienić na pętlę while. Liczba 2000 nie ma jakiegoś konkretnego znaczenia, po prostu początkowo uznałem, że jeśli program nie wygeneruje planszy w 2000 kroków to nie da rady w ogóle. Teraz wiem, że wygenerowanie planszy zajmuje 100-200 kroków, rzadko kiedy około 500 (w każdym razie zawsze udaje się wygenerować planszę) więc nie ma potrzeby w taki sposób zabezpieczać program przed pętlą nieskończoną. Teraz tam powinien być while (rzad <= 8) zamiast tego fora. pt., 28 sty 2022 o 10:31 Tomasz Drab ***@***.***> napisał(a):

…

https://github.com/MateuszM02/projekt_sudoku/blob/db13170867e172f2d0cafe8f71aca930a60fd606/grafika.c#L33 Zastanawia mnie znaczenie tej liczby. Dlaczego akurat 2000? — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AW434IC73MUFFNXJJVRMW5LUYJPAJANCNFSM5NAH26XQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>