Hallo zusammen,

Ich habe eine Frage zu diesem Tutorial.

Wieso kann ich eine SELECT-Liste nicht einfach so anlegen? :

PHP-Code:

echo "<select name="irgendwas" size="1"\n";

//hier steht die DB-Abfrage

$result = mysql_query($sql) or die (mysql_error());
while($ausgabe=mysql_fetch_assoc($result))
{
echo " <option value="$ausgabe[id]">$ausgabe[Name]</option>\n";
}
echo " </select>\n"; 


[Lars]

Also zuerst sind deine echo()s mal falsch

Der Vorteil daran, das ganze in eine Funktion zu packen, ist die Arbeitsersparnis und bei mehrerem Vorkommen der <select>-Liste und natürlich auch die bessere Wiederverwendbarkeit.

Ups...habe nicht escaped. Wenn es mal schnell gehen soll...*g*
Also gibt es grundsätzlich keine Einwände dagegen, wenn man es so macht?

Grüße
flupsi

[Lars]

Nein, ich denke nicht. Die Funktion ist halt der bequemere und überischtlichere Weg

[Ben]

Natürlich geht das so auch. Das ist ja gar nicht das Ziel dieses Tutorials.
Das Ziel ist ja, eine Funktion zu schreiben, die man immer wieder verwenden kann.

Weiterhin lernt man in dem Tutorial - bzw. ich hoffe, dass man es lernt - wie man solche Probleme mittels eigener Funktionen umsetzt.
Weiterhin hat Deine Version ja keine Möglichkeit der Konfiguration der SELECT-Liste.

Grüße, Ben.

Meine Frage habe ich aus einem anderem Grund gestellt. Ich habe bereits beide Versionen benutzt. Wenn ein und dieselbe Select-Liste öfter gebraucht wird, hat Deine Version mittels einer Funktion ganz klare Vorteile. Braucht man die Liste nur einmal, macht die Funktion keinen Sinn.

Naja, ich dachte, es gäbe vielleicht noch einen anderen Grund für den Einsatz einer Funktion. Es hätte ja sein können, das Select-Listen einen Sonderstatus bei der PHP-Programmierung haben. Da habe ich wohl etwas falsch verstanden und mich dann in dieser Idee verrannt.
Ich habe schon seit einigen Tagen nach der Besonderheit von Select-Listen in der PHP-Programmierung gegoogelt (und natürlich nichts gefunden).

Vielleicht wollte ich das Forum auch mal wieder mit einer ausgesucht blöden Frage auflockern (jetzt schäm ich mich)...

Grüße
flupsi

[Ben]

Zitat von flupsi

Wenn ein und dieselbe Select-Liste öfter gebraucht wird, hat Deine Version mittels einer Funktion ganz klare Vorteile. Braucht man die Liste nur einmal, macht die Funktion keinen Sinn.

Und das ist genau falsch.

Die Funktion erstellt doch jede beliebige Liste. Du gibst Daten und Konfigurationseinstellungen als Parameter in die Funktion und erstellst somit jede beliebige Liste.
Du könntest die Funktion ja sogar noch erweitern, um der Liste beliebige CSS-Eigenschaften oder Klassen hinzufügen zu können.

Meiner Ansicht nach ist die Arbeit mit Funktionen in den meisten Fällen sinnvoller, weil man einfach ein strukturiertes System erstellt.
Ist nur eine persönliche Erfahrung, aber ich hoffe eben, dass ich diese dem Ein oder Anderen noch mitgeben kann .

Zitat von flupsi

Es hätte ja sein können, das Select-Listen einen Sonderstatus bei der PHP-Programmierung haben.

Um es nochmal klar zu sagen: Nein.
Auswahllisten sind HTML-Elemente und haben insofern nichts mit PHP zu tun. Sie dienen ausschließlich der Darstellung von Inhalten. PHP ist zu diesem Zeitpunkt, wie allseits bekannt, schon längst mit der Arbeit fertig.

Zitat von flupsi

jetzt schäm ich mich

Quatsch. Meiner Ansicht nach hast Du es genau richtig gemacht. Du hast Dich mit dem Tutorial befasst, hattest eine eigene Idee, bist dieser Idee nachgegangen, bist zu keinem eindeutigen Ergebnis gekommen und hast dann hier gefragt.

Finde ich absolut richtig!

Nichtsdestotrotz ist der Thread im falschen Forum.

Der Thread wurde verschoben, wegen ...
... Postings im falschen Forum. Bitte beim nächsten Mal darauf achten.,

verschoben

Grüße, Ben.

[WarrenFaith]

Zitat von Ben

Quatsch. Meiner Ansicht nach hast Du es genau richtig gemacht. Du hast Dich mit dem Tutorial befasst, hattest eine eigene Idee, bist dieser Idee nachgegangen, bist zu keinem eindeutigen Ergebnis gekommen und hast dann hier gefragt.

Finde ich absolut richtig!

Find ich auch. Hast alles richtig gemacht, hast deinen Kopf benutzt und dann nachgehackt. so wie es sein muss

Zitat:

Und das ist genau falsch.

Die Funktion erstellt doch jede beliebige Liste. Du gibst Daten und Konfigurationseinstellungen als Parameter in die Funktion und erstellst somit jede beliebige Liste.
Du könntest die Funktion ja sogar noch erweitern, um der Liste beliebige CSS-Eigenschaften oder Klassen hinzufügen zu können.

Meiner Ansicht nach ist die Arbeit mit Funktionen in den meisten Fällen sinnvoller, weil man einfach ein strukturiertes System erstellt.
Ist nur eine persönliche Erfahrung, aber ich hoffe eben, dass ich diese dem Ein oder Anderen noch mitgeben kann .

Ich habe es mir zur Angewohnheit gemacht, Funktionen in eine extra Datei auszulagern. Diese wird in meiner index.php per require_once eingebunden.
Wenn ich dann Deine Funktion zweimal dort einsetze, um halt zwei verschiedene Select-Listen zu generieren, muss ich eine Menge Variablen umbenennen.
Brauche ich die 2. Liste nur einmal, ist der Aufwand sehr groß.
Wieder ein Denkfehler von mir, oder nachvollziehbar?

Grüße
flupsi

[WarrenFaith]

Denkfehler, warum passt du die Funktion nicht so an, dass du beide Listen mit dieser einen Funktion erstellen kannst?
Hau doch mal deine beiden Funktionen rein, dann schaun wir da mal drüber und packen die zusammen.
Ansonsten solltest du dir nochmal Infos über die Lebensdauer einer Variablen anschauen. hier

[Ben]

Hi,
einfaches Beispiel:

PHP-Code:

<?php

    require_once('functions.inc.php');

    # Liste 1
    $sql = 'SELECT data FROM tabelle1';
    $res = mysql_query($sql) or exit(mysql_error());
    $data = mysql_fetch_assoc($res);
    array_pop($data);

    $config = array( /* irgendwelche Sachen */ );
   
    buid_selectlist($data, $config, true);

    
    # Liste 2
    $sql = 'SELECT data FROM tabelle2';
    $res = mysql_query($sql) or exit(mysql_error());
    $data = mysql_fetch_assoc($res);
    array_pop($data);

    $config = array( /* irgendwelche Sachen */ );
   
    buid_selectlist($data, $config, true);     

?>

Ein und die selbe Funktion, zwei unterschiedliche Listen werden ausgegeben.

Grüße, Ben.

So, seit 2Std spiele ich nun mit dieser Funktion herum...was mir meine Frau inzwischen mit scheelen Seitenblicken quittiert *gg*

Wenn mich nicht alles täuscht, ist da ein kleiner Schwachpunkt im Script, wenn man es für mehrere Tabellen nutzen will.

PHP-Code:

$selectHTML .= '<option value="' . $key['id'] . '" ' . $preselected . '>' . $key['name'] . '</option>';
            $selectHTML .= "\n"; 


Das script setzt voraus, dass alle Tabellen die Feldnamen "id" und "name" beinhalten.
Habe ich Tabellen mit a) "id" und "name" und beispielsweise b) "userid" und "username",
muss ich eine neue Funktion schreiben. Sehe ich das richtig?

Wenn ich schon Eure Geduld strapaziere will ich dann doch eine (wenn auch kleine) Verbesserung einfliessen lassen.
Ich habe die CSS-Klasse mit eingebaut.

PHP-Code:

$selectHTML = '<select name="' . $config['name'] . '" size="' . $config['size'] . '" '" class="' . $config['class'] . '">';
        $selectHTML .= "n"; 


Dann muss nur noch die Klasse definiert werden:

PHP-Code:

$config['class']       = 'meine_klasse_fuer_selectfelder'; 


Vielleicht kann es ja jemand gebrauchen.

Grüße
flupsi

[WarrenFaith]

Korrektur (falsch Zeichensetzung)

PHP-Code:

$selectHTML = '<select name="' . $config['name'] . '" size="' . $config['size'] . '" class="' . $config['class'] . '">'; 


Für dein 'Name' 'Username' Problem gibt es eine einfache Lösung:
Du kannst deine SQL-Abfragen so ändern:

PHP-Code:

$sql = 'SELECT User_ID AS Value, Username AS Name FROM tabelle1'; 


Dann kannst du folgende Zuweisung in der Funktion machen:

PHP-Code:

$selectHTML .= '<option value="' . $key['Value'] . '" ' . $preselected . '>' . $key['Name'] . '</option>'; 


Die Definitionsbeschreibung (Kommentare über der Funktion) sollten aber erklären, was für Indexe dein Array haben soll.

[Ben]

Hi,
nunjaaa .. das ist nicht unbedingt richtig, was Du schreibst. Natürlich ist diese Funktion nun extra so aufgebaut, dass man den Zusammenhang zwischen Daten und Schlüssel des Arrays innerhalb der Funktion sieht.

Aber die Frage ist eigentlich auch recht gut (ich werde diesen Thread im Tutorial verlinken!).

Man könnte die Funktion natürlich insofern erweitern, dass man in das Konfigurationsarray nur vorbestimmte Attribute übernehmen kann. Im Tutorial verlinke ich zu den möglichen Attributen einer Auswahlliste. Zzgl. dann z.B. noch eine CSS-Klasse.

So. Das Konfigurationsarray wird dann wie bekannt gefüllt, wobei nicht alle Attribute verwendet werden müssen.
Innerhalb der Funktion fragen wir dann ab, ob das Attribut gesetzt ist. Ist das der Fall, so wird es eingebaut, ansonsten logischerweise nicht.

Beispiel.

PHP-Code:

if(array_key_exists('attributname', $config)) {
    $attributname = " " . $config['attributname'];
}
else {
    $attributname = '';
}

$selectHTML .= '<option' . $attributname . ' value="' . $data['id'] . '"> ... '; 


Jetzt sagst Du ... "ja, aber jetzt kann ich doch trotzdem ein $data-Array haben, indem anstatt "id", "userid" steht.
Klar. Das kannt Du haben. Allerdings wüsste ich gerade nicht so ganz, wie man das umgehen könnte, außer, indem man aus der Datenbank z.B. wie folgt ausliest

SELECT
    userid AS id,
    username AS name
FROM
    usertabelle
 

Dann existieren im Array letztlich auch nur die Schlüssel "id" und "name".
Müsste man sich mal Gedanken drüber machen, wie man das noch dynamisch lösen könnte.

Ich habe irgendwas im Hinterkopf, aber .. hmm .. ich komm gerade nicht darauf.

Grüße, Ben.

Die Möglichkeit, die Daten bei der DB-Abfrage zu manipulieren, habe ich noch garnicht in Betracht gezogen. Es funktioniert aber einwandfrei. Danke für die Anregung.

Hi,
nunjaaa .. das ist nicht unbedingt richtig, was Du schreibst. Natürlich ist diese Funktion nun extra so aufgebaut, dass man den Zusammenhang zwischen Daten und Schlüssel des Arrays innerhalb der Funktion sieht.

Ich wollte die Funktion nicht kritisieren...für mich war es eine Schwäche, weil ich keine Lösung gesehen habe habe (die ja eigentlich ganz einfach ist). Das hat sich ja nun erledigt.
Alles in Allem habe ich die letzten paar Std wohl eine Menge gelernt, womit Dein Tutorial voll und ganz den Zweck erfült hat, wofür Du es vorgesehen hattest.
Vielen Dank!

Grüße
flupsi

[Ben]

Zitat von flupsi

Ich wollte die Funktion nicht kritisieren

Na, ich find es gut, wenn sich jemand Gedanken macht und kritisiert. Ich bin schließlich nicht unfehlbar. Wäre auch leicht unheimlich.

Zitat von flupsi

Alles in Allem habe ich die letzten paar Std wohl eine Menge gelernt, womit Dein Tutorial voll und ganz den Zweck erfült hat, wofür Du es vorgesehen hattest.
Vielen Dank!

Wow. Super. Freu ich mich.

Grüße, Ben.

[WarrenFaith]

Ein besseres Lob gibt es für ein Tutorialersteller nicht *ben auf die schulter klopf*