Unbekannter Fehler!

[Trobus]

Hallo liebes Forum!
Mein Problem ist mir zu diesem Zeitpunkt nicht bekannt daher ich den Fehler nicht finden kann aber trotzdem nichts auf dewr Website angezeigt wird!

<?php
$file = 'var.txt';
x = "EIN/AUS";
$text = "";

echo ("<h1>The Website!</h1>");
if (isset($_GET['loud'])) {
        if(file_get_contents($file) == 1) {
                shell_exec('/var/www/html/ledan.sh');
                file_put_contents($file, "0");
                $x = "AUS!";
                $text = "Quitschen wurde EINgeschalten!";
        } else {
                exec("gpio -g mode 2 in");
                file_put_contents($file, "1");
                $x = "EIN";
                $text = "Quitschen wurde AUSgeschalten!";
                }
echo ("$text");
echo ("<br>");
}
echo ("<a href="?loud">Quietschen $x</a>");
?>

[Trobus]

Sorry für die störung hab den Fehler gefunden! Ich schäme mich gerade das ich so lange gebraucht habe und zu kapieren das ein "$" fehlt! Nochmal entschuldigung!!

[lstegelitz]

Deswegen diese beiden Zeilen am Anfang eines jeden Scripts (während der Entwicklung).

error_reporting(-1);
ini_set('display_errors', 1); 

[Messier 1001]

Deswegen diese beiden Zeilen am Anfang eines jeden Scripts (während der Entwicklung).

error_reporting(-1);
ini_set('display_errors', 1); 

Nein! Das gehört bei der Entwicklung in die php.ini In irgendwelchen PHP Dateien hat das höchstens mal im Notfall was zu suchen wenn alles auf dem Dev geht aber nicht auf dem Live System.

[protestix]

if(file_get_contents($file) == 1) 

Hier wird immer true zurückgegeben, solange die Datei vorhanden ist, das ist aber nicht das was man will. Man will nachsehen ob der Wert 1 in der Datei steht, damit ist dieses Konstrukt schon mal falsch.

Siehe unter Return was bei der Funktion steht.

[uha]

Hier wird immer true zurückgegeben, solange die Datei vorhanden ist

Das wäre mir aber neu!! file_get_contents liefert den eingelesen Inhalt oder im Fehlerfall false.

[protestix]

Danke uha, gut aufgepasst. Wollte eigentlich darauf hinaus, das man aufpassen soll bei solchen Werten, da wenn dort gegen 0 verglichen wird, man schnell unerwartete Ergebnisse erhalten kann, da 0 wie false ausgewertet wird.

Ok, dies ist hier jetzt nicht der Fall, aber wenn man "ein" und "aus" als Wert einer Variablen hat, kann man diesen auch so abspeichern, das vermeidet Fehler und ist meiner Meinung nach besser zu lesen.

Das Error-reporting sollte ihm zudem einen Fehler in der letzten Zeile ausgeben die mit dem echo.

Es ist auch besser dem Parameter loud einen Wert zuzuweisen

href="?loud=$x" 

Dann kann man den Link für ein und aus verwenden.