Hilfe bei änderung eines "kleines" scriptes

[vt1816]

Zitat von Black Snake

[...]
Oder reden wir jetzt aneinander vorbei?

Wenn diese Stück Code ein Modul ist/werden soll, dann muss die VARIABLE $agt_id an einer anderen Stelle im vorhergehenden Code definiert werden/worden sein.

[urvater]

Ich habe mal gelesen, daß man Code in einer Datei so schreiben soll, daß man diese Seiite immer aufrufen kann ohne einen Fehler zu verursachen. Ich weiß nicht wieviele leere if/else es noch bei dir gibt. Aber eine echte Prüfung, wann was gemacht werden soll sehe ich dort nicht.

Du brauchst eigentlich nichts weiter machen als deinen Code so zu schreiben, daß deine Funktion nur aufgerufen wird, wenn tatsächlich die Bedingungen erfüllt sind. Das ist es eigentlich, was ich von Anfang an versucht habe dir zu zeigen.

Die Optimierung mache ich später.

Sehe ich als ungünstige Einstellung.

[cortex]

Zitat von urvater

Sehe ich als ungünstige Einstellung.

man könnte auch sagen "kurzsichtig". manche dinge kann man hinterher nicht mehr einpflegen - wenn überhaupt, artet das häufig in frickelei aus. stichwort: sicherheit...

cx

[Black Snake]

ok sry dann hab ich euch falsch verstanden!
Was die leeren If Abfragen betrifft, nun da wusste ich keine andere Möglichkeit um dann das else zu benutzen.
Habe auch mit VT1816 gesprochen und werde den ganzen Code (sofern mir möglich) nochmals mit kleinen Beschreibungen versehen um die Funktionen verständlicher zu machen.

Was die Sicherheit betrifft, da muß ich leider sagen, dass ich nicht weis wo die Risiken und so liegen. Aber das Modul können dann später eh nur die User sehn und benutzen, die den Status Member haben. Und die Webseite eh in die Datenbank und als File eine Log schreibt wer wo wann gewessen ist.

Hauptsächlich geht es mir derzeit um funktionalität um auch PHP besser zu lernen und zu verstehn. Die Sicherheit könnte man dann auch noch zum schluss überprüfen und gegebenenfalls den Code überarbeiten auch wenn es dann heisen würde alles zu überschreiben (was zwar ärgerlich wäre aber wie sagt man auch so schön? "Lernen durch schmerzen"^^ )

[DokuLeseHemmung]

Zitat von Black Snake

Was die leeren If Abfragen betrifft, nun da wusste ich keine andere Möglichkeit um dann das else zu benutzen.

PHP-Code:

// dieses:
if ($_POST['GO']) 
{ 
} 
else 
{
  // bla
}


// ist das gleiche wie:
if (!$_POST['GO']) 
{
  // bla
} 


[Black Snake]

omg übersehn thx werde es entfernen^^

[urvater]

Zitat von Black Snake

Was die Sicherheit betrifft, da muß ich leider sagen, dass ich nicht weis wo die Risiken und so liegen. Aber das Modul können dann später eh nur die User sehn und benutzen, die den Status Member haben. Und die Webseite eh in die Datenbank und als File eine Log schreibt wer wo wann gewessen ist.

SQL Injection und Cross-Site Scripting(XSS) wären da so mal auf die schnelle genannt. Wer sagt dir, daß deine "Member" tatsächlich menschliche Wesen sind? Bots sind da um einiges effektiver um sich anzumelden und Schindluder zu treiben.

Zitat von Black Snake

Die Sicherheit könnte man dann auch noch zum schluss überprüfen und gegebenenfalls den Code überarbeiten auch wenn es dann heisen würde alles zu überschreiben (was zwar ärgerlich wäre aber wie sagt man auch so schön? "Lernen durch schmerzen"^^ )

Im schlimmsten Fall können dies teure Schmerzen werden.

Ich weiß nicht was für dich wichtiger oder schöner ist: Schnell irgend ein Script in 30 Min fertig haben um es dann später mit extrem viel Aufwand komplett überarbeiten zu müssen oder einfach mal 1 Std. zu investieren und ggf. nur noch eine Schönheits-OP durchführen zu müssen/wollen. Doch eins kannst du mir glauben. Schnell ist meisten sehr teuer und wenn es nur die investierte Zeit ist.

[Black Snake]

Das ganze ist ja wunderschön, das ihr mich warnt! Aber woher soll ich wissen was ich machen darf und was nicht? Oder welchen Code ich benutzen darf und welchen nicht?

[Black Snake]

Aber mir fällt grad was anderes auf!
Ist das nicht das Gleiche

PHP-Code:

$agt_id = $_POST['id'];
if ($agt_id == "Schiffsklasse auswählen")
    {} 


und das

PHP-Code:

if ($_POST['id'] == "Schiffsklasse auswählen")
    {} 


???

[vt1816]

Zitat von Black Snake

Aber mir fällt grad was anderes auf!
Ist das nicht das Gleiche
[...]

JA!