PHP Was falsch?

[DasPrinzip.]

PHP Code:

if(isset($_POST['submit'])
{ 
if($_POST['link'] != '') 
{ 
echo "Du hast den Link erfolgreich eingetragen!"; 
} 
} 
else 
{ 
echo "Da ist wohl was schief gelaufen!";
} 


Bei mir kommt immer ein Fehler?
Ist doch eigentlich alles richtig?

Parse error: syntax error, unexpected '{' in

Aber ist doch eigentlich alles richtig verschachtelt?

[.Marcel']

du hast ein } zu viel über dem else zweig

[DasPrinzip.]

Muss der "{" nach dem ersten if nicht wieder geschlossen werden?

Weil wenn ich ich den rausnehme sieht das ganze so aus:

{
{
}
{
}

Dann wird der erste ja nicht mehr geschlossen?

[Hatake~Kakashi]

Originally Posted by .Marcel'

du hast ein } zu viel über dem else zweig

er hat eher ein ')' zu wenig.
<3 einrücken von code ist so toll.

[NotEnoughForYou]

*** es fehlt die ) beim isset

[Ravenstorm]

PHP Code:

if(isset($_POST['submit']))
{ 
   if($_POST['link'] != '') 
   { 
      echo "Du hast den Link erfolgreich eingetragen!"; 
   } 
   else 
   { 
      echo "Da ist wohl was schief gelaufen!";
   }
} 


so sollte es richtig sein...

Dein isset hat sich nicht geschlossen
Und wenn du das willst, dass entweder "erfolgreich" oder "fail" wieder gegeben wird, solltest du deine den Anweisungsblock deiner ersten If-Abfrage nicht schon vorher schließen.

[PseudoPsycho]

Originally Posted by Ravenstorm

PHP Code:

if(isset($_POST['submit']))
{ 
   if($_POST['link'] != '') 
   { 
      echo "Du hast den Link erfolgreich eingetragen!"; 
   } 
   else 
   { 
      echo "Da ist wohl was schief gelaufen!";
   }
} 


so sollte es richtig sein...

Dein isset hat sich nicht geschlossen
Und wenn du das willst, dass entweder "erfolgreich" oder "fail" wieder gegeben wird, solltest du deine den Anweisungsblock deiner ersten If-Abfrage nicht schon vorher schließen.

Stimmt.
Und in wirklich gutem Stil ist der Code nun auch nicht gehalten...
Wozu verwendest du eine !=''-Abfrage, wenn es für sowas die empty()-Funktion gibt..?!

Besser so:

PHP Code:

if(isset($_POST['submit']) && !empty($_POST['link']) ){
    echo 'Du hast den Link erfolgreich eingetragen!'; }
else {
    echo 'Da ist wohl was schief gelaufen!'; } 


[Ravenstorm]

Ja, hab nur seinen Code in eine richtige Funktion gebracht, wollte ihn jetzt nicht mit einer neuen Funktion plagen.
Der Code Stil bin ich momentan zu stark von Java geprägt, hatte da scheinbar einen Brainfuck während ich geantwortet habe und hier noch weiter gecodet habe (Alles durcheinander gemischt)
Die Empty Funktion vereinfacht das natürlich ungemein.

[Fratyr]

Originally Posted by PseudoPsycho

Stimmt.
Und in wirklich gutem Stil ist der Code nun auch nicht gehalten...
Wozu verwendest du eine !=''-Abfrage, wenn es für sowas die empty()-Funktion gibt..?!

Besser so:

PHP Code:

if(isset($_POST['submit']) && !empty($_POST['link']) ){
    echo 'Du hast den Link erfolgreich eingetragen!'; }
else {
    echo 'Da ist wohl was schief gelaufen!'; } 


Wieso predigst du gute Code Formatierung während deine Klammerung alles andere als richtig ist? Wenn dus schon in 4 Zeilen quetschen willst dann lass die Klammern lieber gleich weg bevor du so ungünstig formatierst.

[PseudoPsycho]

alles andere als richtig ist?

lol. "Schon klar." Da sieht man mal wieder, wieviel Ahnung du hast...
Es ist SCHEIß-EGAL, wo genau die Klammern stehen (also wieviele Leerzeichen/Zeilenumbrüche davor/dahinter sind)...
Ich wollte sie aber nicht weglassen, um den Threadersteller nicht wohlmöglich in Verwirrung zu bringen, sobald dieser mehrere Befehle mit einer Abfrage ausführen will.

In meinem Beitrag ging es anbei auch nicht um Formatierung, sondern um Code-Technisches.
Also, empty() statt =="" und einfache, statt doppelte Lerrzeichen.
Naja, Intelligenz wird ja auch nicht in der Schule unterrichtet, also kein Wunder, dass sie dir fehlt.

[Fratyr]

lol. "Schon klar." Da sieht man mal wieder, wieviel Ahnung du hast...
Es ist SCHEIß-EGAL, wo genau die Klammern stehen

Ist es nicht, aber gut das du soviel Ahnung hast die mir leider fehlt.



Aber ich finds klasse das du dich besser mit PHP Coding Standards auskennst als PEAR selbst.
Schämst du dich nicht so ausfallend zu werden wo du doch so falsch liegst?

[.LaTeX]

Wo ist euer Problem? Sollte es hier nicht eher um Technik als um Schönheit gehen?
In dem php.net manual sind sowieso nur Richtlinien - wenn jemand meint, er muss seinen Code anders formatieren/gestalten, warum soll er das nicht machen? Vllt. wird es für andere dadurch weniger lesbar, aber mein Gott, akzeptiert mal die Ansichten anderer ohne direkt ausfallend zu werden. Ist ja ätzend.
Und nur weil statt:

PHP Code:

if($bla == 1) {
    echo 'omg its true';
} else {
    echo 'omg its not true';
} 


ein anderer vllt.:

PHP Code:

if($bla==1) 
{
echo "omg its true";
} else { echo "omg its not true"; } 


schreibt, ist es nicht automatisch falsch.

Gruß,
.LaTeX

[Fratyr]

schreibt, ist es nicht automatisch falsch.

Lustig das jemand so kommt in einem Forum wo jeder hier mit seinem W3 validator ankommt, der wirklich nur eine Richtlinie ist aber gleichermaßen die weit verbreitesten Coding Standards ignoriert.

[DasPrinzip.]

Macht euch mal nicht fertig wegen einer frage?
Meine frage ist geklärt und jeder Mensch macht mal Fehler?
Wenn dann sucht euch roboterfreunde die alles perfekt machen