Problem mit Array und DB eintragung

noBody · 19.03.2007, 08:33

Hallo,

ich habe von dieser Community von einem Freund gehört. Er sagt mir ihr seit ziemlich schnell im Antwort und sehr Hilfsbereit.

So genug Honig ums Maul geschmiert, komme wir doch mal zu meinen Problem

Also so gebe ich meine Kategorien für mein Forum aus:

PHP-Code:

  <?php

$cat_data = mysql_query("SELECT cat_id, cat_name, cat_sort FROM board_categories ORDER BY cat_sort ASC") OR die(mysql_error());

while($c_o = mysql_fetch_assoc($cat_data ))

{

echo '

<table class="main_config">

    <tr>

        <td width="75%">'.print_out($c_o['cat_name']).'</td>    

        <td><center><input type="text" name="sort_number[]" size="5" style="text-align:center" class="design" value="'.print_out($c_o['cat_sort']).'"><input type="hidden" name="cat_id[]" value="'.print_out($c_o['cat_id']).'"></center><td>

        <td>[url="admin/delete.php?cat_id='.print_out($c_o['cat_id']).'"]L&ouml;schen[/url]</td>

    </tr>

</table>
';

}

?>

So, wenn der Admin die Sortierung geändert hat und diese Speichern will kommt folgendes Script:

PHP-Code:

  <?php

if(isset($_POST['sort_cat']))

{

    foreach($_POST['sort_number'] as $number)

    {

        foreach($_POST['cat_id'] as $id)

        {

            $new_cat = mysql_query("UPDATE board_categories SET cat_sort = '".mysql_real_escape_string($number)."' WHERE cat_id = '".mysql_real_escape_string($id)."'") OR die(mysql_error());

        }

    }

 
header ('Location: ../index.php?admin=admin&config=cat_config&save=ready');

}

?>

Doch er speichert bei allen die selbe Zahl in der DB.
Mir wird aber kein Fehler ausgeben.

Wenn ich mir das Query ausgeben lasse:

Code:

UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '4'

Das ist defenitiv zu viel, ich habe nämlich nur 4 cats.
Ich denke mal, dass ich in der Schleife ein Fehler gemacht habe, aber mir viel auch keine bessere Lösung ein.

Und die große Frage ist nun, warum er überall das gleiche rein schreibt?

KingCrunch · 19.03.2007, 09:09

Zitat:

Zitat von noBody

ich habe von dieser Community von einem Freund gehört. Er sagt mir ihr seit ziemlich schnell im Antwort und sehr Hilfsbereit.

Wow, das is neu

Danke für das Lob

Also ohne jetzt deine Scripte im Detail durchexerziert zu haben:
Ich sehe 2 geschachtelte foreach-Schleifen und du sagst, du hast 4 Kategorien. Kam dann auf den Trichter mal zu zählen wieviele Queries du da hast und kam auf 14 (und wahrscheinlich verzählt

weil ich denke es müssten 16 sein

). Warum das jetzt net sooo ungewöhnlich is? Du hast 4 Eingabefelder pro Spalte. Nun interierst du einmal über eine Spalte und dann innerhalb noch einmal über eine andere Spalte. Da beide Spalten jeweils 4 Einträge besitzen, kommen auch 4*4=16 (wenn man richtig zählt

) Queries raus :>

Und nu lass mich ma überblicken, wie es richtiger wäre

Edit: OK, nachgedacht. Das du nur einmal über alle 4 Zeilen iterieren solltest, sollte klar sein. Du kannst nun, wenn du den key in der foreach mitnimmst, beide Array parallel ansprechen:

PHP-Code:

      foreach($_POST['sort_number'] as $key => $number)

    {

        $new_cat = mysql_query("UPDATE board_categories SET cat_sort = '".mysql_real_escape_string($number)."' WHERE cat_id = '".mysql_real_escape_string($_POST['cat_id'][$key])."'") OR die(mysql_error());

    }

Noch einfacher wäre es allerdings, wenn du im html selbt schon anstatt

Code:

<input name="sort_number[]" ... >

direkt mit der id arbeitest

Code:

<input name="sort_number[<?php echo $c_o['cat_id']; ?>]" ...>

Dann kannst du nämlich auf das $_POST['cat_id'][$key] verzichten

PHP-Code:

      foreach($_POST['sort_number'] as $id => $number)

    {

        $new_cat = mysql_query("UPDATE board_categories SET cat_sort = '".mysql_real_escape_string($number)."' WHERE cat_id = '".mysql_real_escape_string($id)."'") OR die(mysql_error());

    }

Ich hoffe, da hat sich jetzt kein Fehler eingeschlichen

noBody · 19.03.2007, 14:34

Danke.
Ich habe total vergessen das $_POST ja auch ein Array ist.
Bitte verzeiht mir diesen dummen Fehler.
Danke

KingCrunch · 19.03.2007, 15:02

Ich hoffe, du meinst das anders. Du verwendest POST durchaus als Array, bloss der Inhalt ist halt ebenso ein Array. Wobei ... den betrachtest du auch als array (foreach ^^)... Naja, sagen wir es so: Hab gestern meine Scripte auch mal ausgemistet und bin auf allerhand skuriles gestoßen

noBody · 19.03.2007, 15:17

Zitat:

Zitat von KingCrunch

Ich hoffe, du meinst das anders. Du verwendest POST durchaus als Array, bloss der Inhalt ist halt ebenso ein Array. Wobei ... den betrachtest du auch als array (foreach ^^)... Naja, sagen wir es so: Hab gestern meine Scripte auch mal ausgemistet und bin auf allerhand skuriles gestoßen

Man verbessert sich von Tag zu Tag.
Wenn ich an meine ersten Script denke, wird mir ganz schlecht

tinchen · 19.03.2007, 17:05

Zitat:

ich habe von dieser Community von einem Freund gehört. Er sagt mir ihr seit ziemlich schnell im Antwort und sehr Hilfsbereit.

Den Tipp musst du von mir haben.

Sorry, ich konnte nicht widerstehen.

PHP-Code:

   $new_cat = mysql_query("UPDATE board_categories SET cat_sort = '".mysql_real_escape_string($number)."' WHERE cat_id = '".mysql_real_escape_string($id)."'") OR die(mysql_error());

Was ich da zum Beispiel noch nicht verstehe ist, warum du da mysql_real_escape_string
verwendest. Sollte $id nicht eine Zahl bzw. ein Integer sein und wäre dann
(int) $id nicht sinnvoller, als den Interger im query zu einem String zu konvertieren?

Grüße
Tine

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Array Problem	dsmcg	PHP Tipps 2006	3	07.12.2006 14:34
ARRAY: alle möglichen Kombinationen	Tentacle	PHP Tipps 2006	13	12.05.2006 00:18
Array Problem	accomm	PHP Tipps 2006	1	04.04.2006 16:35
Mysql / Array Problem	marcels	PHP Tipps 2006	10	23.03.2006 18:30
Problem mit array in Datenbank	pixelcut	PHP-Fortgeschrittene	3	28.02.2006 18:20
array - Problem	skaterboy	PHP-Fortgeschrittene	4	26.02.2006 12:36
Übersicht aus mehrdimensionalem Array erzeugen	Crypi	PHP-Fortgeschrittene	0	30.01.2006 13:44
Menü mit Unterpunkten	supertramp	Beitragsarchiv	7	18.10.2005 22:40
Problem mir einem Array		PHP Tipps 2005-2	5	11.08.2005 11:33
(schnellere) Funktion zum Zusammenfassen von CSS		PHP-Fortgeschrittene	21	08.08.2005 16:47
Array Problem [Ansatz ?]		PHP Tipps 2005	0	27.01.2005 18:24
[Erledigt] Mehrdimensionales Array in eindimensionales Array umwandeln		PHP-Fortgeschrittene	3	03.01.2005 22:31
Problem mit Checkboxvalues in Array		PHP Tipps 2004	1	17.09.2004 15:00
Array Problem	andrew22	PHP Tipps 2004	9	17.09.2004 10:41

19.03.2007, 08:33
noBody Neuer Benutzer Registriert seit: 19.03.2007 Beiträge: 24	Problem mit Array und DB eintragung Hallo, ich habe von dieser Community von einem Freund gehört. Er sagt mir ihr seit ziemlich schnell im Antwort und sehr Hilfsbereit. So genug Honig ums Maul geschmiert, komme wir doch mal zu meinen Problem Also so gebe ich meine Kategorien für mein Forum aus: PHP-Code: <?php $cat_data = mysql_query("SELECT cat_id, cat_name, cat_sort FROM board_categories ORDER BY cat_sort ASC") OR die(mysql_error()); while($c_o = mysql_fetch_assoc($cat_data )) { echo ' <table class="main_config"> <tr> <td width="75%">'.print_out($c_o['cat_name']).'</td> <td><center><input type="text" name="sort_number[]" size="5" style="text-align:center" class="design" value="'.print_out($c_o['cat_sort']).'"><input type="hidden" name="cat_id[]" value="'.print_out($c_o['cat_id']).'"></center><td> <td>[url="admin/delete.php?cat_id='.print_out($c_o['cat_id']).'"]Löschen[/url]</td> </tr> </table> '; } ?> So, wenn der Admin die Sortierung geändert hat und diese Speichern will kommt folgendes Script: PHP-Code: `<?php if(isset($_POST['sort_cat'])) { foreach($_POST['sort_number'] as $number) { foreach($_POST['cat_id'] as $id) { $new_cat = mysql_query("UPDATE board_categories SET cat_sort = '".mysql_real_escape_string($number)."' WHERE cat_id = '".mysql_real_escape_string($id)."'") OR die(mysql_error()); } } header ('Location: ../index.php?admin=admin&config=cat_config&save=ready'); } ?>` Doch er speichert bei allen die selbe Zahl in der DB. Mir wird aber kein Fehler ausgeben. Wenn ich mir das Query ausgeben lasse: Code: UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '4' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '1' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '2' WHERE cat_id = '4'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '1'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '2'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '3'UPDATE board_categories SET cat_sort = '3' WHERE cat_id = '4' Das ist defenitiv zu viel, ich habe nämlich nur 4 cats. Ich denke mal, dass ich in der Schleife ein Fehler gemacht habe, aber mir viel auch keine bessere Lösung ein. Und die große Frage ist nun, warum er überall das gleiche rein schreibt?


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

19.03.2007, 14:34
noBody Neuer Benutzer Registriert seit: 19.03.2007 Beiträge: 24	Danke. Ich habe total vergessen das $_POST ja auch ein Array ist. Bitte verzeiht mir diesen dummen Fehler. Danke

19.03.2007, 15:02
KingCrunch Erfahrener Benutzer Registriert seit: 13.08.2007 Beiträge: 1.976	Ich hoffe, du meinst das anders. Du verwendest POST durchaus als Array, bloss der Inhalt ist halt ebenso ein Array. Wobei ... den betrachtest du auch als array (foreach ^^)... Naja, sagen wir es so: Hab gestern meine Scripte auch mal ausgemistet und bin auf allerhand skuriles gestoßen __________________ Nicht jeder Fehler ist ein Bug.