Formulare - wie kann man nicht alle Felder ausfüllen müssen?

MissJour

Hallo an alle!
Wir haben eine kleine Frage:
Wir haben mehrere Felder in einem Formular, die man ausfüllen kann. Jedoch wollen wir es so einstellen, dass man nicht ALLE Felder ausfüllen MUSS. D.h. wenn man lediglich Vorname und Geburtsdatum eingeben würde, oder eine andere Kombination oder ein einzelnes Feld.
Wie kriegen wir das hin?
Unser Skript:

PHP

<head>
<meta charset="utf-8">
  <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/css/bootstrap.min.css">
  <link rel="stylesheet" href="main.css">
</head>
<body>
<?php
$db = mysqli_connect("localhost","root","","familytreefinder")
    or die("Verbindung zu MySQL oder der Datenbank gescheitert.");
mysqli_set_charset($db, "utf8");
?>
<form>
<label>Vorname eingeben</label><br>
<input name="vorname" type="text"><br>
<label>Nachname eingeben</label><br>
<input name="nachname" type="text"><br>
<label>Geschlecht wählen</label><br>
<select name="geschlecht" class="form-control">
  <option>w</option>
  <option>m</option>
</select><br>
<label>Geburtsdatum angeben</label><br>
<input name="gebdatum" type="text"><br>
<button type="submit" name="suchen" class="btn btn-default">Suchen</button>
</form>


<?php


if (isset ($_GET['suchen']) || ($_GET['geschlecht']) || ($_GET['vorname']) || ($_GET['nachname']) || ($_GET['gebdatum']) )
{
    $geschlecht = trim($_GET['geschlecht']);
    $geschlecht = preg_replace("/m,w/", "", $geschlecht);
    $vorname = trim($_GET['vorname']);
    $vorname = preg_replace("/a-z/", "", $vorname);
    $nachname = trim($_GET['nachname']);
    $nachname = preg_replace("/a-z/", "", $nachname);
    $gebdatum = trim($_GET['gebdatum']);
    $gebdatum = preg_replace("/1-9/", "", $gebdatum);

    $sql = "SELECT * FROM person WHERE Geschlecht = '".$geschlecht."' AND Vorname = '".$vorname."' AND Nachname = '".$nachname."' AND Geburtsdatum = '".$gebdatum."'";
    $ergebnis = mysqli_query($db, $sql);

    echo "<table border = '1' style = 'border-spacing: 0;'>\n";
    echo "<tr><th>ID</th><th>Geschlecht</th><th>Titel</th><th>Vorname</th><th>Nachname</th><th>Geburtsdatum</th><th>Geburtsname</th><th>Geburtsort</th><th>Beruf</th><th>Sterbedatum</th><th>Sterbeort</th><th>Notitzen</th></tr>";
    while ($row = mysqli_fetch_assoc($ergebnis)) 
    {
        echo "<tr>";
        foreach ($row as $key => $value)
        {
            echo "<td> $value&nbsp;</td>\n";
        }
        echo "<tr>";
    }
    echo "</table>\n";
}




mysqli_close($db);




?>
</body>

Alles anzeigen

Danke im Voraus!

Arne Drews

Hallo,

Ihr solltet unabhängig davon das Formular per POST senden lassen

HTML

<form method="post">
    <label>Vorname eingeben</label><br>
    <input name="vorname" type="text"><br>
    <label>Nachname eingeben</label><br>
    <input name="nachname" type="text"><br>
    <label>Geschlecht wählen</label><br>
    <select name="geschlecht" class="form-control">
        <option>w</option>
        <option>m</option>
    </select><br>
    <label>Geburtsdatum angeben</label><br>
    <input name="gebdatum" type="text"><br>
    <button type="submit" name="suchen" class="btn btn-default">Suchen</button>
</form>

Alles anzeigen

Dann müsst ihr einfach nur die empfangenen Daten "filtern", ob diese leer sind, als Beispiel:

PHP

// wenn das formular abgesendet wurde
if ( isset($_POST['suchen']) ) {


    // relevante Felder
    $aFields = [
            'geschlecht' => trim($_POST['geschlecht']),
            'vorname' => trim($_POST['vorname']),
            'nachname' => trim($_POST['nachname']),
            'gebdatum' => trim($_POST['gebdatum'])
        ];


    // nur befüllte felder filtern
    $aFields_Filtered = array_filter( 'empty', $aFields );


    // where bedingungen zusammenfügen
    $sWhere = '';


    foreach ( $aFields_Filtered as $key=>$value )
        $sWhere .= $key . "='" . $value . "'";


    $sql = "SELECT * FROM person WHERE " . $sWhere;
    $ergebnis = mysqli_query($db, $sql);


    // rest des codes (unverändert)    
    echo "<table border = '1' style = 'border-spacing: 0;'>\n";
    echo "<tr><th>ID</th><th>Geschlecht</th><th>Titel</th><th>Vorname</th><th>Nachname</th><th>Geburtsdatum</th><th>Geburtsname</th><th>Geburtsort</th><th>Beruf</th><th>Sterbedatum</th><th>Sterbeort</th><th>Notitzen</th></tr>";


    while ($row = mysqli_fetch_assoc($ergebnis)) {


        echo "<tr>";


        foreach ($row as $key => $value) {


            echo "<td> $value&nbsp;</td>\n";


        }


        echo "<tr>";


    }


    echo "</table>\n";


}

Alles anzeigen

ungetestet, sollte aber in etwa hinhauen...

MissJour

Hallo,

erst mal vielen Dank für die schnelle und ausführliche Antwort.
Wir haben das alles jetzt so verbessert, allerdings kommen jetzt wieder ziemlich viele Fehlermeldungen:

[FONT=&quot]Undefined variable: POST in den jeweiligen Zeilen, die wir bei $aFields verwenden, obwohl sie oben im form angegeben ist

[/FONT][FONT=&quot]Warning[/FONT][FONT=&quot]: array_filter() expects parameter 1 to be array

[/FONT][FONT=&quot]Warning[/FONT][FONT=&quot]: Invalid argument supplied for foreach()[/FONT][FONT=&quot]

[/FONT][FONT=&quot]Warning[/FONT][FONT=&quot]: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result

Wir sind jetzt ein bisschen überfordert. Das ist ein Projekt in für die Uni und wir müssen zum ersten Mal wirklich eine Datenbank in eine Website miteinbauen, weshalb wir noch ziemliche Anfänger sind [/FONT]

Arne Drews

Hallo,

Der einzige Fehler ist der erste, der Rest sind Folgefehler.
Zeig bitte mal euer aktuelles Script, ich vermute, ihr habt $POST verwendet, anstatt $_POST

MissJour

Ah, ups

PHP

<head>
<meta charset="utf-8">
  <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/css/bootstrap.min.css">
  <link rel="stylesheet" href="main.css">
</head>
<body>
<?php
$db = mysqli_connect("localhost","root","","familytreefinder")
    or die("Verbindung zu MySQL oder der Datenbank gescheitert.");
mysqli_set_charset($db, "utf8");
?>
<form method=post>
<label>Vorname eingeben</label><br>
<input name="vorname" type="text"><br>
<label>Nachname eingeben</label><br>
<input name="nachname" type="text"><br>
<label>Geschlecht wählen</label><br>
<select name="geschlecht" class="form-control">
  <option>w</option>
  <option>m</option>
</select><br>
<label>Geburtsdatum angeben</label><br>
<input name="gebdatum" type="text"><br>
<button type="submit" name="suchen" class="btn btn-default">Suchen</button>
</form>




<?php




if (isset ($_POST['suchen']) )
{
    $aFields = [
    'geschlecht' => trim($POST['geschlecht']),
    'vorname' => trim($POST['vorname']),
    'nachname' => trim($POST['nachname']),
    'gebdatum' => trim($POST['gebdatum'])
    ];

    $aFields_Filtered = array_filter('empty',$aFields);

    $sWhere = '';

    foreach ($aFields_Filtered as $key=>$value)
        $sWhere .= $key . "='" . $value . "'";

    $sql = "SELECT * FROM person WHERE" . $sWhere;
    $ergebnis = mysqli_query($db, $sql);

    echo "<table border = '1' style = 'border-spacing: 0;'>\n";
    echo "<tr><th>ID</th><th>Geschlecht</th><th>Titel</th><th>Vorname</th><th>Nachname</th><th>Geburtsdatum</th><th>Geburtsname</th><th>Geburtsort</th><th>Beruf</th><th>Sterbedatum</th><th>Sterbeort</th><th>Notitzen</th></tr>";
    while ($row = mysqli_fetch_assoc($ergebnis)) 
    {
        echo "<tr>";
        foreach ($row as $key => $value)
        {
            echo "<td> $value&nbsp;</td>\n";
        }
        echo "<tr>";
    }
    echo "</table>\n";
}








mysqli_close($db);








?>
</body>

Alles anzeigen

Aber die unteren drei Fehlermeldungen bleiben trotzdem

Arne Drews

Ah ja, mein Fehler, die Parameter sind vertauscht.
Bitte die Zeile mal so abändern:

Code

[COLOR=#333333]$aFields_Filtered = array_filter([/COLOR][COLOR=#333333]$aFields, [/COLOR][COLOR=#333333]'empty');[/COLOR]

Die anderen beiden sollten dann aber wieder Folgefehler und damit auch behoben sein.

MissJour

[FONT=&quot]Warning[/FONT][FONT=&quot]: array_filter() expects parameter 2 to be a valid callback, function 'empty' not found or invalid function name

[/FONT][FONT=&quot]Warning[/FONT][FONT=&quot]: Invalid argument supplied for foreach()

[/FONT][FONT=&quot]Warning[/FONT][FONT=&quot]: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result

kommen wieder. Müssen wir da vielleicht noch etwas für den Filter definieren?[/FONT]

Arne Drews

Das würde bedeuten, dass er die PHP Funktion empty nicht kennt, was ich mir nicht vorstellen kann. Kannst Du bitte nochmal den aktuellen Stand posten? Braucht auch nur der erste PHP-Block bis $sql sein.

Achso, welche PHP-Version habt ihr?
findet ihr raus, indem ihr folgendes im PHP-Block einsetzt:

PHP

echo phpversion();

MissJour

Wir haben Version 5.6.14 (Danke für Info, wie man das herausfindet)

PHP

<head>
<meta charset="utf-8">
  <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.4/css/bootstrap.min.css">
  <link rel="stylesheet" href="main.css">
</head>
<body>
<?php
$db = mysqli_connect("localhost","root","","familytreefinder")
	or die("Verbindung zu MySQL oder der Datenbank gescheitert.");
mysqli_set_charset($db, "utf8");




echo phpversion();
?>
<form method=post>
<label>Vorname eingeben</label><br>
<input name="vorname" type="text"><br>
<label>Nachname eingeben</label><br>
<input name="nachname" type="text"><br>
<label>Geschlecht wählen</label><br>
<select name="geschlecht" class="form-control">
  <option>w</option>
  <option>m</option>
</select><br>
<label>Geburtsdatum angeben</label><br>
<input name="gebdatum" type="text"><br>
<button type="submit" name="suchen" class="btn btn-default">Suchen</button>
</form>




<?php




if (isset ($_POST['suchen']) )
{
	$aFields = [
	'geschlecht' => trim($_POST['geschlecht']),
	'vorname' => trim($_POST['vorname']),
	'nachname' => trim($_POST['nachname']),
	'gebdatum' => trim($_POST['gebdatum'])
	];

	$aFields_Filtered = array_filter($aFields, 'empty');

	$sWhere = '';

	foreach ($aFields_Filtered as $key=>$value)
		$sWhere .= $key . "='" . $value . "'";

Alles anzeigen

Arne Drews

Das Verhalten kannte ich noch gar nicht. Ersetz die Zeile mal gegen das hier:

PHP

$aFields_Filtered = [];
array_walk( $aFields, function ($v, $k) use ( &$aFields_Filtered ) { if (!empty($v)) $aFields_Filtered[$k] = $v; } );

MissJour

Dann liegt es zumindest nicht nur an uns

Es sind zwei Fehlermeldungen weniger geworden, aber es kommt noch immer:
[FONT=&quot]Warning[/FONT][FONT=&quot]: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result[/FONT]

Arne Drews

Das deutet darauf hin, dass die Abfrage zuvor kein Ergebnis liefert.
So wie es für mich ausssieht, habt ihr das Leerzeichen hinter dem WHERE vergessen, da muss eins hin, weil ansonsten die Query fehl schlägt:

PHP

$sql = "SELECT * FROM person WHERE " . $sWhere; // <-- hier ist das Leerzeichen wichtig!

Sollte der Fehler damit nicht behoben sein, bitte mal nach der Zeile $sql ausgeben lassen und die Ausgabe posten.

MissJour

Leerzeichen hat gefehlt, aber auch mit Leerzeichen kommt die gleiche Fehlermeldung.

Wenn wir

Code

echo $sql;

machen, kommt raus
[FONT=&quot]SELECT * FROM person WHERE geschlecht='w'vorname='gesuchter vorname'nachname='gesuchter nachname'
also wahrscheinlich wieder AND-Verknüpft, weshalb keine Person ausgegeben wird. [/FONT]

Arne Drews

Ups... Ja klar, da habe ich vergessen das AND dazwischen zu mogeln.

Ersetze mal das hier:

PHP

// where bedingungen zusammenfügen
    $sWhere = '';


    foreach ( $aFields_Filtered as $key=>$value )
        $sWhere .= $key . "='" . $value . "'";

gegen das hier:

PHP

$aWhere = [];


foreach ( $aFields_Filtered as $key=>$value )
    $aWhere[] = $key . "='" . $value . "'";


$sWhere = implode( ' AND ', $aWhere );

MissJour

Jetzt steht das AND in der Ausgabe dazwischen, aber die Fehlermeldung
[FONT=&quot]Warning[/FONT][FONT=&quot]: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result
kommt immer noch und es wird auch immer noch keine Person ausgegeben. Müsste so etwas nicht irgendwie mit einer OR-Verknüpfung gehen? Oder ist das Ergebnis dann nicht unbedingt eindeutig, weil auch Ergebnisse angezeigt werden, die nur teilweise übereinstimmen?[/FONT]

Arne Drews

Mit Or verknüpft kann ja jede Kombination gültig sein. Die ausgefüllten Felder müssen schon AND verknüpft sein.
Wie sieht denn $sql jetzt aus, poste das nochmal?

MissJour

[FONT=&quot]SELECT * FROM person WHERE geschlecht='w'ANDvorname='eingegebener vorname'ANDnachname='eingegebener nachname'

es ist nur die AND-Verknüpfung dazu gekommen. Schade, eine OR-Verknüpfung wäre so schön einfach gewesen.[/FONT]

Arne Drews

Dann habt ihr jetzt aber wieder die Leerzeichen vor und nach dem AND weggelassen, die sind bei mir in #14 drin.
Ohne Leerzeichen versagt die Query und liefert kein gültiges Resultat. Genau das meckert er an.

Bitte diese Zeile exakt übernehmen:

PHP

$sWhere = implode( ' AND ', $aWhere ); // <-- auf die Leerzeichen vor und nach dem AND achten!

Wenn dann immer noch was schief geht, wäre noch zu klären, ob die Spaltenbezeichnungen geschlecht, vorname, nachname und gebdatum überhaupt korrekt sind?

Zitat

Schade, eine OR-Verknüpfung wäre so schön einfach gewesen.

AND ist genauso einfach, wenn man es richtig macht und liefert zudem sogar noch die richtigen Ergebnisse...

MissJour

Es hat funktioniert!
Vielen Dank! Das hat uns wirklich sehr geholfen.
Ich glaube, auf die Leerzeichen werden wir in Zukunft verschärft achten müssen...
Danke für deine Geduld

Ja, jetzt, wo der Code stimmt schon

Tags