nach if funktion wird nichts mehr angezeigt

[Forumsgast]

Hey,

ich habe ein kleines Script gebastelt.
ich suche schon seit Tagen den Fehler aber finde keinen Fehler...
hier ist auf jeden fall mal das script. das echo drunter wird nicht ausgegeben nichteinmal das </html> aber alle anführungszeichen sind gesetzt und das ?> dürfte das doch eigendlich alle Fehler beenden...
Bin völlig ratlos weil das vor der if anweisung ja funktioniert.
wobei das if funktioniert glaube ich. der datensatz ist auch in der datenbank da.

if($adresse!="$adresseaus")
{
#eintag
$eintragen = mysqli_query($daba, "INSERT INTO tabellenname (adresse, wert2) VALUES ('$adresse', '0')") or die(mysql_error());

echo "
<br><br><br><center><h1>Du wurden erfolgreich eingetragen.<br>
Bitte lade die Seite neu.
</h1></center>
";
}
else
{
print "Hier werbung rein.";
};

echo "hier?";
?> 

gruß janto

[Forumsgast]

Hi,

was soll das Semikolon nach der letzten geschweiften Klammer?

Zrur Fehlersuche auch gerne am Anfang eines Scriptes gerne folgende Zeilen einfügen:

<?php
ERROR_REPORTING(E_ALL);
ini_set('display_errors',1);
...

Das hier ist ganz schlechter Stil:

<br><br><br><center><h1>

Bitte mit CSS und div's arbeiten.

[Forumsgast]

if( $adresse != $adresseaus ) // DoubleQuotes (") sind hier überflüssig und sinnfrei!
{
#eintag
// hier konkatinieren oder besser DataBinding
$eintragen = mysqli_query($daba, "INSERT INTO tabellenname (adresse, wert2) VALUES ('" . $adresse . "', '0')") or die(mysql_error());

echo "
<br><br><br><center><h1>Du wurden erfolgreich eingetragen.<br>
Bitte lade die Seite neu.
</h1></center>
";
}
else
{
print "Hier werbung rein.";
} // hier war das ; überflüssig!

echo "hier?"; 

[Forumsgast]

if($adresse != $adresseaus ) // Was ist $adresseaus und was ist $adresse?
{
#eintag
$eintragen = mysqli_query($daba, "INSERT INTO tabellenname (adresse, wert2) VALUES ('$adresse', '0')") or die(mysql_error()); //mysql_-Funktionen sind veraltet und sollten nicht mehr benutzt werden. Sollte $adresse nicht gefiltert sein, besteht hier SQL-injection Gefahr

echo "
<br><br><br><center><h1>Du wurden erfolgreich eingetragen.<br> // Grammatikfehler
Bitte lade die Seite neu.
</h1></center>
";
}
else
{
print "Hier werbung rein."; // wieso einmal print und einmal echo?
};

echo "hier?";
?> 

Allgemein:
Error-reporting an
Debbugen
Vollständigen Code zeigen

[Forumsgast]

- "$adresseaus" Variablen qouten ist unsinnig und schlechter Stil http://php-de.github.io/jumpto/code-...n-doppelquotes

- Die mysql_ Erweiterung ist veraltet, und wird bald nicht mehr funktionieren. Du solltest auf PDO ( http://php-de.github.io/jumpto/pdo/
) oder mysqli umsteigen

- Ev. bist du anfällig für SQL -Injections, wenn du Daten von aussen einfach so in der Querys verwendest. Prüf das mal.

- Du solltest deinen Code sauber einrücken, das ist um einiges leserlicher als deine Wurst.

- Achja.. und wenn etwas nicht funktioniert wie gedacht - debugge: http://php-de.github.io/jumpto/leitfaden/

LG

[Forumsgast]

Öhm...

Die mysql_ Erweiterung ist veraltet, und wird bald nicht mehr funktionieren.

$eintragen = mysqli_query( 

[Forumsgast]

Keine Ahnung was du hast.
Funktioniert doch alles ?!
Zwar nicht schön und nicht ganz korrekt aber generell klappt es.

Bitte kein PHP und HTML mischen!!

Code einrücken etc...

Du kannst auch die alternative Schreibweise für ifs, foreach etc benutzen. Finde ich übersichtlicher als diese Klammer {}

<?php

$adresse = "testadresse";
$adresseaus = "testadresse";

if($adresse != $adresseaus):
    #eintag
    $eintragen = mysqli_query($daba, "INSERT INTO tabellenname (adresse, wert2) VALUES ('$adresse', '0')") or die(mysql_error());

    echo "<br><br><br><center><h1>Du wurden erfolgreich eingetragen.<br>Bitte lade die Seite neu.</h1></center>";
else:
    print "Hier werbung rein."."<br>";
endif;

echo "hier?";
?>

[Forumsgast]

Öhm...

Ich hatte nur Augen für das:

 ... or die(mysql_error()); 

janto : Das ist auch noch falsch, bleib konsequent bei mysqli_

LG

[Forumsgast]

@Fab4guy:

Zwar nicht schön und nicht ganz korrekt aber generell klappt es.

M.E. suggerierst Du damit einen falschen Ansatz.
Der Threadersteller sollte möglichst von Anfang an einen guten und korrekten Stil lernen + du hast in deinem Beispielcode zwar die If-/Foreach-Strukturen übersichtlicher geschrieben (wobei geschmackssache), aber die "Fehler" mitgenommen!

[Forumsgast]

@Fab4guy:

M.E. suggerierst Du damit einen falschen Ansatz.
Der Threadersteller sollte möglichst von Anfang an einen guten und korrekten Stil lernen + du hast in deinem Beispielcode zwar die If-/Foreach-Strukturen übersichtlicher geschrieben (wobei geschmackssache), aber die "Fehler" mitgenommen!

Sage ja auch nicht das er es so machen soll. Wollte ihm nur damit sagen das es noch andere Möglichkeiten gibt. Natürlich ist es Geschmackssache aber gerade bei viel Code finde ich es übersichtlicher.

Ich wollte ja auch nicht seine Fehler korrigieren. Das kann er selber! Er hat ja nur behauptet, dass das letzte echo nicht ausgegeben wird.

Das wollte ich ihm damit nur zeigen..

[Forumsgast]

Wollte ihm nur damit sagen das es noch andere Möglichkeiten gibt. Natürlich ist es Geschmackssache aber gerade bei viel Code finde ich es übersichtlicher.

Richtig ist Geschmackssache und darf natürlich immer mit angeboten werden.
Allerdings gebe ich wolf29 recht, dann hätte ein triviales Beispiel gereicht und nicht der fehlerhafte Code mit einer anderen Variante des IF

[Forumsgast]

Beim nächsten Mal dann ..

[Forumsgast]

Das alles hatte ich schon gemacht. Also Fehleranalyse.
Fehler war der hier:

VALUES ('".$adresse."', '0')") or die(mysql_error());

die variable die ich anzeigen wollte war nicht mit diesen ". $variabeln ." versehen.
da liefert die fehleranalyse auch keine fehleranzeige.
gut danke.

[Forumsgast]

janto : ... bleib konsequent bei mysqli_