Mach mal aus:

das hier:

shift holt nur das erste Element, man braucht aber sowohl den Usernamen als auch das Passwort. Die Spaltennamen im SQL musst Du wahrscheinlich auch noch anpassen...

Und an das Ende von der sub auth fehlt noch ein return $return...

Es wäre übrigens schöner wenn Du subs und anderen Code nicht so mischen würdest...

ersetzen mit



.. und wie renee bereits sagte: Spaltenname anpassen. Und countusers() anpassen -> return'en die entsprechenden Werte.

*weiter kritisieren*

Informationen der Datenbank sollten in ne config-Datei
so musst du wenn sich mal was ändert nicht direkt in deinen Quellcode

PS ist das beabsichtigt das dein Datenbanknutzer und sein Passwort hier gepostet wurden?! (wäre dir mit ner config gar net passiert ;) )

[quote=bieber,20.07.2007, 12:49]*weiter kritisieren*

Informationen der Datenbank sollten in ne config-Datei
so musst du wenn sich mal was ändert nicht direkt in deinen Quellcode

PS ist das beabsichtigt das dein Datenbanknutzer und sein Passwort hier gepostet wurden?! (wäre dir mit ner config gar net passiert ;) )[/quote]
100% ack

ist nicht so wild, der sql server ist von aussen nicht zu erreichen und das pass wird geändert sobald ich nicht mehr auf eure hilfe angewiesen bin *g*

desweiteren hat der user auch quasi nur die lese/schreib berechtigung für seine db, also selbst wenn jemand lokal drauf kommt (unwarscheinlich) wird nich viel passieren können... und selbst wenn, ich hab im moment einen table mit 2 einträgen drin ;)

so zurück zu den problemen, ich bekomme wenn ich einen user hinzufügen möchte die folgende fehlermeldung:

Quote

DBI::db=HASH(0x81f1124)->disconnect invalidates 1 active statement handle (either destroy statement handles or call finish on them before disconnecting) at ./slashem.pl line 21, <STDIN> line 5.
1You enter the World of SlashEm... Currently there are 1 Users registered

Can't call method "main" without a package or object reference at ./slashem.pl line 90, <STDIN> line 5.
mindfucked:/home/mindfuck/scripts#

die im script angegebenen db-values sind korrekt, ich hab echt kaum ahnung von sql, aber die fehlermeldung ist ziemlich nichts-sagend bzw ich kann nicht auf den fehler schliessen, weiss auch nciht warum er sich über die nicht-geschlossene session aufregt, eigendlich sollte die geschlossen sein...

code liegt hier: http://mindfucked.org/slashem.txt
(will nich schon wieder alles hier reinspammen :))

edit: der rest funzt, also nologin und loginokay, nur useradden halt nich, denke isn sql statement error maybe?\n\n

<!--EDIT|FlorianL|1184929423-->

[quote=RPerl,20.07.2007, 12:47]

ersetzen mit

[/quote]
Ist schöner, ja... Muss man aber nicht, da es die letzte Anweisung in einem Block ist. Da kann man das Semikolon weglassen.

90%ACK zu der Config. Ne Config ist eigentlich überflüssig, wenn es nur um ein kleines Skript geht. Da kann man so etwas auch in einem Bereich am Anfang des Skript schreiben. Sobald es mehr als ein Skript/Modul wird, würde ich es auch auslagern.
Aber das ist wie mit sehr vielen Dingen: auch eine Sache der persönlichen Vorliebe ;)

Nimm das $dbh->disconnect; bei countusers raus!

Edit: Und füge noch ein $sth->finish; in auth nach dem fetchrow_array; ein...\n\n

<!--EDIT|renee|1184929669-->

fehler is nun weg, danke ;)

aber er speichert nicht in die db, ist diese zeile korrekt?



Ansonsten geht das schon, beim connect mit meiner valid u/p combo wird das game gestartet, bin also nurnoch nen klitzekleines stück vom ziel entfernt ;)