Sudoku bissl mehr als Grundstruktur steht

**combie** · 27.10.2011, 14:16

In semtlichen Foren in denen ich bis jetzt war wurde ich Grundlegend mit meiner Frage ignoriert.

Das kann ich mir lebhaft vorstellen.

Auch ich finde den Code ekelig.
1. diese Konstanten
2. diese fürchterlichen Variablenbezeichner
3. die dauernden Wiederholungen

Vor readonly fehlt jeweils ein Leerzeichen
Auch heißt es readonly="readonly" wenn mich nicht alles täuscht

**AmicaNoctis** · 27.10.2011, 14:17

Hallo,

da der Code mir zu unübersichtlich ist, um selbst nachzusehen, was er macht, gib uns doch bitte mal ein paar Infos, worum es geht: Ist es ein Sudoku-Generator oder ein Sudoku-Solver?

Was ich schonmal nicht gut finde, ist, dass du alles in eine riesige Datei gepackt hast. Du solltest besser die Logik von der Ausgabe trennen. Ein objektorientierter MVC-Entwurf wäre das sinnvollste.

Gruß,

Amica

**darkii** · 27.10.2011, 14:27

Also es ist ein simpler Sudoku Solver der mir aus einer Textdatei eine Array hollt die dann in eine Tabelle übergeben wird.

Dass soll das ganze auf Knopf druck die User eingabe übernehmen und prüfen ob das sudoku richtig ist aber nur dadurch das in keiner Reihe/Spalte etwas doppelt vorkommen darf. Sollte dann etwas falsch sein soll er mir die Reihe/Spalte in ein hell rot machen und da wo der Fehler steckt Möglichst dunkel rot.

Und wegen der unübersichtlichkeit sry bin erst seit 2 Monaten am Php programmieren

**AmicaNoctis** · 27.10.2011, 14:34

Also eigentlich auch kein Solver, sondern nur ein Tool, um zu überprüfen, ob es richtig ist – ein Validator.

Du willst, dass das Tool dir den Fehler hervorhebt, aber es wird dir bei doppeltem Auftreten einer Zahl beide markieren müssen, da es nicht ohne weiteres ermitteln kann, welche von beiden jetzt falsch ist. Dafür würdest du wieder einen Solver benötigen, der aus der gegebenen Ausgangssituation alles automatisch vervollständigen kann. Nur auf diese Weise kann er auch Fehler finden. Wenn du dagegen alles eingibst, was du bisher hast (also ohne die vorgegebenen Zahlen von den selbst eingetragegen unterscheiden zu können), wird er dir immer nur sagen können, dass etwas falsch ist, aber nicht wo.

**darkii** · 27.10.2011, 14:41

Es wäre für mich in Ordnung wen er dann zwei falsch makieren würde natürlich wäre es andersherum besser

**AmicaNoctis** · 27.10.2011, 14:53

Du müsstest die fehlerhaften Stellen vorher ausfindig machen und beim Zusammenbauen der Tabelle an diesen Stellen z. B. ein bestimmtes class-Attribut hinzufügen. Ein weiteres Problem ist, dass du (soweit ich das sehe) die dritte Regel vergessen hast: Jede Zahl darf und muss genau ein Mal in jedem 3×3-Block vorkommen.

**darkii** · 27.10.2011, 14:58

Die hab ich bewusst weg gelassen da sie zu einer sudoku berechnung ja nicht nötig ist .

Fehler sind in dem Code bisslang noch keine eher muss ich eine problemlösung finden

Momentan mache ich ja leider nur fehlerhafte Spalten/ Zeilen ausfindig und welche Zahl in der Zeile fehlt.

Ps: Jedes Kästchen hatt bei mir die Atributte i1-i81 hast du das mit class attributen gemeint ?

**darkii** · 27.10.2011, 15:57

Ich hab da grad ne kleine Idee zu meinem Problem das der User es beeinflussen kann

for ($i=0 ; $i<= 80 ;$i++) {
if ($_SESSION['sudoku'][$i]== " " || "") {
unset ( $_SESSION['sudoku'][$i] );
}
}

ich will damit meine Standard Array also die gebenen Werte extrahieren und diese dann mit einer neuen Vergleichen (Eine neue also eine die meine Tabelle ausfüllt) und sollte bei den gegebenen Keys was anders sein solls die dann mit der SESSION austauschen so würde ich verhindern das ein User das ganze manipuliert nur ist die Frage kann ich das umsetzten ?

Sry ich drücke mich bei sowas sehr unverständlich aus

**AmicaNoctis** · 27.10.2011, 17:21

Zitat von darkii Beitrag anzeigen

for ($i=0 ; $i<= 80 ;$i++) {
if ($_SESSION['sudoku'][$i]== " " || "") {
unset ( $_SESSION['sudoku'][$i] );
}
}

Bitte Code immer in den entsprechenden Code-Tags posten (siehe Signatur). Der Teil hinter dem || ergibt immer false, meinst du vielleicht

PHP-Code:


$_SESSION['sudoku'][$i]== " " || $_SESSION['sudoku'][$i]== ""

Oder vielleicht sogar

PHP-Code:


trim($_SESSION['sudoku'][$i]) == ""

Was soll uns dieser Codeschnipsel sonst noch sagen? Das sieht aus, wie ein bisschen Session-Cleanup, aber wofür ist das im Kontext deines Problems gut?

Zitat von darkii Beitrag anzeigen

Sry ich drücke mich bei sowas sehr unverständlich aus

Das glaube ich irgendwie auch langsam

**darkii** · 28.10.2011, 08:19

Für den Kontext eher weniger aber fürs Verständniss und für die Verhinderung das User das Sudoku manipulieren.

Fürs Verständniss:

$Sudoku ist meine zufällige Array die ich aus einer Textdatei ziehe
$_Session["sudoku"] ist meiner Array die bei dem Klick auf den Ok Button wieder mitgenommen wird und dann in meine Tabelle eingetragen wird
$_POST ist quasi alles was in der Tabelle steht und vom User eingegeben wird sie wird dann in die Session geschrieben.

Nun würde ja meine $_POST die Session überschreiben und inteligente User (soll vorkommen) könnten Firebug/Dragonfly nutzen und die gegebenen Zahlen einfach ändern. Und deshalb mocht ich meine $-Session mit keys von leeren Einträgen aus der Array befreien um diese Werte dann mit der $_POST vergleichen um dadurch geänderte Werte wieder zu ändern

Klingt komplex ist aber so ^^ den nötigen Befehl dazu habe ich noch nicht gefunden ^^

Und zur Problemlösung an sich trägt es auch mal gar nichts bei

**ronronron3005** · 01.11.2011, 10:27

Zitat von darkii Beitrag anzeigen

...
In semtlichen Foren in denen ich bis jetzt war wurde ich Grundlegend mit meiner Frage ignoriert...

Ausser bei SelfPHP

**darkii** · 02.11.2011, 08:30

Aber da auch keine Lösung oder Hilfe ^^ Hier nochjemand etwas beizusteuern das mir helfen könnte ?

**combie** · 02.11.2011, 08:53

Klingt komplex ist aber so ^^

Ich halte den Ansatz für grundsätzlich falsch.

Schaue dir den Datenfluss an.

Es gibt 2 Quellen:
1. Halb ausgefüllte Sudokus in der Datei
2. Die User Eingaben

Den Index auf das gerade verwendete Sudoku lagerst du in Session. Es gibt für den User keine Möglichkeit dieses zu manipulieren.

Vom User kommen [COLOR="red"]NUR[/COLOR] die Daten, welche er eingibt. Daran kann er soviel manipulieren, wie er will. Darf er ja, soll er ja auch sogar.

**darkii** · 02.11.2011, 12:39

Das Problem ist blos immoment überschreibt die Eingabe die Daten die eingetragen werden ich muss eben nun schauen das ich das ganze teile...

Aber wichtiger wäre mir überhaupt das er die Fehler berechnen kann.

Sudoku bissl mehr als Grundstruktur steht