polamarciniak / sudoku_solver

0 stars 0 forks source link

Sugestie #1

Open tomaszdrab opened 3 years ago

tomaszdrab commented 3 years ago

Issue na sugestie i pomysły dotyczące projektu

tomaszdrab commented 3 years ago

Czy trudnym byłoby sprawienie, żeby przy wprowadzaniu zagadki kursor podświetlał się w miejscu, gdzie wprowadzamy liczbę? Wydaje mi się, że trudno zorientować się, gdzie się jest, kiedy wpisze się parę spacji.

tomaszdrab commented 3 years ago

Może warto byłoby, np. przy użyciu pomocniczej tablicy, podświetlić w rozwiązaniu te liczby, które były wprowadzone przez użytkownika.

tomaszdrab commented 3 years ago

https://github.com/polamarciniak/sudoku_solver/blob/5e55f9d82726fdb214a83b62ea53c1d9ac8580fd/plansza.h#L5-L12 Pliki nagłówkowe raczej nie powinny zawierać (nieinlineowanych) definicji funkcji. Plikom n_sudoku.h i plansza.h chyba bliżej do n_sudoku.cpp i plansza.cpp, a plik head.h podzieliłby się wtedy na n_sudoku.h i plansza.h.

tomaszdrab commented 3 years ago

Czy w funkcji is_valid sprawdzana liczba val nie wykrywa samej siebie jako kolizji?