PHP Was falsch?

06/01/2012 11:32 DasPrinzip.#1
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?

Quote:
Parse error: syntax error, unexpected '{' in
Aber ist doch eigentlich alles richtig verschachtelt?
06/01/2012 12:01 .Marcel'#2
du hast ein } zu viel über dem else zweig
06/01/2012 12:17 DasPrinzip.#3
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?
06/01/2012 12:43 Hatake~Kakashi#4
Quote:
Originally Posted by .Marcel' View Post
du hast ein } zu viel über dem else zweig
er hat eher ein ')' zu wenig.
<3 einrücken von code ist so toll.
06/01/2012 14:52 NotEnoughForYou#5
jap es fehlt die ) beim isset
06/01/2012 14:55 Ravenstorm#6
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.
06/01/2012 16:11 PseudoPsycho#7
Quote:
Originally Posted by Ravenstorm View Post
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!'; } 
06/01/2012 16:22 Ravenstorm#8
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.
06/01/2012 17:03 Fratyr#9
Quote:
Originally Posted by PseudoPsycho View Post
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.
06/01/2012 17:20 PseudoPsycho#10
Quote:
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.
06/01/2012 19:36 Fratyr#11
Quote:
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.
[Only registered and activated users can see links. Click Here To Register...]
[Only registered and activated users can see links. Click Here To Register...]

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?
06/01/2012 19:58 .LaTeX#12
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
06/01/2012 20:10 Fratyr#13
Quote:
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.
06/01/2012 23:01 DasPrinzip.#14
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