[LucjanPetarda]

Napisalem sudoku solver ktory uzywa rekurencji i backtrakingu. Macie moze jakies pomysly jak to moznaby poprawic kod? https://paste.ofcode.org/QevL2xvEbieP6BCsWsHFgA #programowanie

[heniek_8]

@LucjanPetarda:
Dodaj wczytywanie stanu początkowego z excela

[M.....f]

@LucjanPetarda: Nazewnictwo takie se i niepotrzebne komentarze, które można zastąpić funkcjami. Śmiesznie to wygląda "print_brd(board)". Gdybyś miał klasę "Board" to metoda "isValid" ma sens, natomiast u ciebie jest to funkcja i jej nazwa nic nie znaczy dla mnie.

[s.....u]

@LucjanPetarda: napisz tak żeby nie były potrzebne komentarze bo kod sam się wytłumaczy

[zwei]

for c in range(len(brd)):

@LucjanPetarda: nie rób tak, jak potrzebujesz indeksu, to używaj enumerate. Zresztą w jednym miejscu piszesz tak jakby board mógł zmienić rozmiar, a w drugim hardkodujesz range od 1 do 10. Zdecyduj się

Docstringi rób przy pomocy ''' a nie

[zwei]

@zwei: jeszcze co do nazewnictwa. find = findempty()

find to koszmarna nazwa dla tej zmiennej, find to czasownik, a to nie jest funkcja, tylko pozycja pustej komórki. Nie lepiej nazwać to np. emptycell?

Dalej, zamiast używać indeksów typu pos[1], to sobie po

[zwei]

@zwei: ogólnie widać że ty tych rzeczy potrafisz użyć tylko jeszcze ci to nie weszło w nawyk

[LucjanPetarda]

@zwei @sepuqqu @Malkof I o taka krytyke nic nie robilem. Dzieki (⌐ ͡■ ͜ʖ ͡■)