if Anweisung

Skared

Servus

Ich bin gerade dabei mir PHP anzueignen, nur komme ich bei einer sache nicht weiter.

PHP

<?php


echo "<a href='?action=1' class='side'>go 1</a><br>";
echo "<a href='?action=2' class='side'>go 2</a>";


if ($action == "1"){
    echo "Eins";
}




if ($action == "2"){
    echo "Zwei";
}


?>

Alles anzeigen

Es soll volgendes bewirken:

wenn man auf Go 1 klickt soll die dazugehörige if ausgefürt werden.
http://localhost/test.php?action=1
wenn man auf Go 2 klickt soll die dazugehörige if ausgefürt werden.
http://localhost/test.php?action=2

Bandit

Keine Ahnung, wo du das so gelesen, aber suche dir etwas aktuelleres, das ist total veraltet. Siehe http://www.php.net/manual/de/security.globals.php

Skared

thx für deinen Link aber das ist nicht das Thema was ich da versuche. es soll über einen link nur ein teil der test.php ausgefürt werden und dies soll nicht durch einen cookie bestimmt werden. es geht auch nicht um den inhalt der if anwendungen, sondern nur um die ausfürung der einzelnen anwendungen. dies soll in der url zu sehen sein.

Afrael

Ja, allerdings kannst du nicht über $action auf den Parameter zugreifen, verwende $_GET['action'].

Bandit

Ich habe dich schon richtig verstanden, aber du verstehst die Problematik nicht. Der Zugriff auf $action ist falsch und deshalb funktionieren die if-Abfragen nicht.

PHP

<?php
  echo "<a href='?action=1' class='side'>go 1</a><br>
        <a href='?action=2' class='side'>go 2</a><br><br>";
 
   if (isset($_GET['action']))
   {
      switch ($_GET['action'])
      {
         case '1': echo "eins"; break;
         case '2': echo "zwei"; break;
         default: echo "Weder 1 noch 2"; break;
      }
   }
   else
      echo "Kein Parameter vorhanden";
?>

Alles anzeigen

Skared

Zitat von Afrael

Ja, allerdings kannst du nicht über $action auf den Parameter zugreifen, verwende $_GET['action'].

dank dir so geht

Code

if ($_GET['action'] == "1"){
    echo "Eins";
}




if ($_GET['action'] == "2"){
    echo "Zwei";
}

Bandit

Dann schreibe mal an den Anfang des Scriptes die Zeilen

PHP

error_reporting(E_ALL);
ini_set("display_errors", true);

und schon meckert PHP beim ersten Aufruf ohne Parameter. Diese Version ist unsauber!!

Afrael

Bandit, du musst den Newbie nicht gleich an-bolden, nur weil er nicht weiß, wie man Variablen korrekt initialisiert, finde ich.

Bandit

Was ist so schlimm daran? Sofort richtig lernen, umso leichter wird es später.

Unregistriert

Zitat von bandit600

Was ist so schlimm daran?

Kann ich dir sagen, es macht dich erstmal unsympathisch
Aber wiederum auf der anderen Seite, haste recht.

Nun habe ich das nächste Prob. Und zwar wenn ich nun einen Eintrag machen will, dann speichert er nur die id und nicht die Variablen $name und $massage.

Code gaestebucheintrag.php

PHP

<?php
include "config.php";




//FORMULAR
if ($_GET['action'] == "formular"){
$ipadresse = getenv("REMOTE_ADDR");
echo "<br><center>Es müssen alle Felder mit einem * ausgefüllt werden!</center><br><br>
<form name='userform' method='post' action='?action=send'>
<table align='center' cellpadding='0' cellspacing='0' class='tbl'><tr>
	<tr>
		<td width='200'>Dein Name:*
		</td>
		<td><input type='text' name='name' maxlength='50' class='textbox' style='width: 200px;'>
		</td>
	</tr>
	<tr>
	<td width='200'>Deine Message:*
	</td>
	<td><textarea name='message' rows='5' class='textbox' style='width: 250px'></textarea>
	</td>
	</tr>
</table>";




echo "<table align='center' cellpadding='0' cellspacing='0' class='tbl'>
	<tr>
		<tr>
			<td align='center' colspan='2'><input type='submit' name='sendmessage' value='Eintragen' class='button'>
			</td>
		</tr>
	</tr>
</table>";
}


//SEND
if ($_GET['action'] == "send"){


// Verbindungs-Objekt samt Zugangsdaten festlegen
@$db = new mysqli($db_host, $db_user, $db_pass, $db_name); 


// Verbindung überprüfen
if (mysqli_connect_errno()) {
    printf("Verbindung fehlgeschlagen: %s\n", mysqli_connect_error());
    exit();
}


// Datensatz-Inhalte festlegen
$result = "INSERT INTO gaestebuch (
	name,
	message)
VALUES (
	'$name',
	'$message')";

	if ($db->query($result)) {
    // Meldung bei erfolg
    echo "Eintrag erfolgreich eingetragen!<br><a href='?action=formular'>OK und zurück zum Formular</a>";      
} else {
	 // Meldung bei fehlschlag
    echo "Eintrag konnte nicht eingetragen werden!<br><a href='?action=formular'>OK und nochmal versuchen</a>";
	}
}
?>

Alles anzeigen

Afrael

Ja, dann lass dir doch vor dem Eintragen $name und $message ausgeben.

PHP

var_dump($name);
var_dump($message);

Du wirst sehen, dass sie wieder nicht initalisiert sind. Du musst auf jede Variable, die durch ein Formular übergeben wird, mit $_GET['variablenname'] zugreifen, wenn das Formular method="get" hatte. Hatte es hingegen method="post", benutzt du $_POST['variablenname'].

Skared

habe es nun hinbekommen

PHP

<?php
include "config.php";
//SEND
if (isset($_POST['senden'])) {


// Verbindungs-Objekt samt Zugangsdaten festlegen
@$db = new mysqli($db_host, $db_user, $db_pass, $db_name); 


// Datensatz-Inhalte festlegen


$result = "INSERT INTO gaestebuch (
    name,
    massage)
VALUES (
    '".$_POST['name']."',
    '".$_POST['massage']."')";

    if ($db->query($result)) {
    // Meldung bei erfolg
    echo "Eintrag erfolgreich abgesendet!<br><a href='?action=formular'>OK und zurück zum Formular</a>";      
} else {
     // Meldung bei fehlschlag
    echo "Eintrag konnte nicht abgesendet werden!<br><a href='?action=formular'>OK und zurück zum Formular</a>";
    }
}


//FORMULAR
if ($_GET['action'] == "formular"){
echo "<br><center>Es müssen alle Felder mit einem * ausgefüllt werden!<br></center>
<form name='post' method='post' action='?action=send'><br><br>
<table align='center' cellpadding='0' cellspacing='0' class='tbl'><tr>


<tr><td width='200'>Name:</td><td><input type='text' name='name' value='$ipadresse' readonly maxlength='50' class='textbox' style='width: 200px;'></td></tr>


<tr><td width='200'>Message:</td><td><input type='text' name='massage' value='' maxlength='50' class='textbox' style='width: 200px;'></td></tr>


</table>";




echo "<table align='center' cellpadding='0' cellspacing='0' class='tbl'><tr>
<tr><td align='center' colspan='2'><br><br><br><input type='submit' name='senden' value='Senden' class='button'></td></tr></tr></table>
</form>\n";
}
?>

Alles anzeigen

Kann geclost werden

Bandit

Unbedingt ansehen: mysql_real_escape_string

Skared

Servus bandit,

Wenn ich das richtig verstanden habe, wäre es möglich, wenn ich so ein Kennwort geschützten Bereich erstelle, sich da einzuloggen ohne Logindaten. Was ich wiederum nicht verstehe, ist das es nur ein Gästebuch Eintrag ist und nicht geschützt ist. Worauf sollte ich den da achten?

Gruß Skared

Bandit

Nein, das siehst du falsch. Du musst die Doku schon richtig lesen:

Zitat

mysql_real_escape_string() ruft die Funktion mysql_real_escape_string der MySQL-Bibliothek auf, die folgende Zeichen mit einem Backslash ('\') versieht: \x00, \n, \r, \, ', " und \x1a.
Die Funktion muss immer (mit wenigen Ausnahmen) verwendet werden, um Daten abzusichern, bevor sie per Query an MySQL übermittelt werden.

reeperbahner11

Zitat von Skared

Servus bandit,

Wenn ich das richtig verstanden habe, wäre es möglich, wenn ich so ein Kennwort geschützten Bereich erstelle, sich da einzuloggen ohne Logindaten. Was ich wiederum nicht verstehe, ist das es nur ein Gästebuch Eintrag ist und nicht geschützt ist. Worauf sollte ich den da achten?

Gruß Skared

Steht doch auf der Seite wo Bandit als letztes gepostet hatte.

Zitat

Die Funktion muss immer (mit wenigen Ausnahmen) verwendet werden, um Daten abzusichern, bevor sie per Query an MySQL übermittelt werden.

edit: bandit war schneller;)

Skared

Hm dann schau ich mir das Später mal genauer an

Nun habe ich folgendes Problem. Ich lass mir die Einträge über eine "while" anzeigen und möchte per Klick auf Ansehen nur den einen anzeigen lassen. Er ruft die if ($_GET['action'] == "select") zwar auf aber "$data['name']" und "$data['massage']" bleiben leer. Ich nehme an, es liegt daran das die $data['id'] nicht übermittelt wird.

PHP

<?php
include "config.php";




echo "<table align='center' width='100%' class='tbl-border'>
<tr>
    <td width='50%' class='tbl1'>Name</td>
    <td width='50%' class='tbl1'>Optionen</td>
</tr></table>";
// Verbindungs-Objekt samt Zugangsdaten festlegen
@$db = new mysqli($db_host, $db_user, $db_pass, $db_name);


// Verbindung überprüfen
if (mysqli_connect_errno()) {
    printf("Verbindung fehlgeschlagen: %s\n", mysqli_connect_error());
    exit();
}
// Datensatz-Ausgabe festlegen
$sql_befehl = "SELECT * FROM gaestebuch";
if ($resultat = $db->query($sql_befehl)) {
    // Datensatzzeile übergeben
    while($data = $resultat->fetch_array()){

echo "<form name='post' method='post' action='?action=select'>
<table align='center' width='100%' class='tbl'>
<tr>
<td width='50%' class='tbl1'>".$data['name']."</td>
<td width='50%' class='tbl1'><input type='submit' name='id' value='Ansehen' class='button'></td>
</table>";


    } 
}


//Eintrag ansehen
if ($_GET['action'] == "select") {


$sql_befehl = "SELECT * FROM gaestebuch WHERE id='".$_GET['id']."'";
if ($resultat = $db->query($sql_befehl)) {
    // Datensatzzeile übergeben
    $data = $resultat->fetch_array();


echo "<table align='center' class='tbl1-border'>
    <tr>
        <td width='200'>Name:
        </td>
        <td><input type='text' name='id' value='".$data['name']."' maxlength='50' class='textbox' style='width: 250px;'>
        </td>
    </tr>
    <tr>
        <td width='200'>Eintrag:
        </td>
        <td><textarea class='textbox' rows='10' style='width: 400px'>".$data['massage']."</textarea>
        </td>
    </tr>
</table>";
    }
}
?>

Alles anzeigen

Was mache ich da falsch?

Bandit

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(E_ALL); und ini_set("display_errors", true);
3. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
4. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
5. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
6. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
7. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
8. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

http://www.php.net/manual/de
http://www.php-faq.de
http://www.php-einfach.de
http://tut.php-q.net/index.html
http://dev.mysql.com/doc/refman/5.1/de/index.html
http://ffm.junetz.de/members/reeg/DSP/
http://www.phptutorials.de
http://www.dynamic-webpages.de/05.tutorials.php
http://de.selfhtml.org/
http://www.css4you.de/index.html

Skared

ich habe error_reporting(E_ALL); und ini_set("display_errors", true); mal mit rein gemacht und das hat meinen Verdacht bestätigt, dass in der

if ($_GET['action'] == "select") {

was falsch ist.

Fehlermeldung:

Code

[B]Notice[/B]:  Undefined index:  action in [B]D:\xampp\htdocs\gaestebuch\ansehen.php[/B] on line [B]36[/B]

Und an was liegt das ?

Bandit

Siehe Posting #5, da steht doch, wie du auf Existenz von Variablen prüfen kannst!

Tags