Zapis danych z formularza za pomocą pętli [PHP][MySQL] Opcje

[Danielcom]

Witam, muszę napisać skrypt do zarządzania menu. Niby prosta sprawa, ale mam mały problem. Chodzi o zapis danych z formularza napisanego w html.

Formularz wygląda mniej więcej tak:
(IMG:http://www.danielcom.pl/skjdhfldvcxhb/tabelka.jpg)

Dane pobierane są w pętli (foreach) z MySql.

Chcę teraz zapisać pola, które pozwalają na zmianę (opublikowany, order ...) z tego formularza za jednym podejściem, i nie bardzo mam pomysł jak to zrobić.

Proszę o pomoc.

[elmozaur]

nie rozumie pytania??
jesli masz pola to maja one jakies nazwy.
prawdopodobnie (a nawet logiczne) jest ze te pola wysylane sa POSTem.
po stronie odbierajacej piszesz $nazwa=$_POST['nazwa_pola'];
takie zmienne podajesz do bazy jako update i tyle

[Danielcom]

tak, ale problem polega na tym, że zmiennych $_POST['idmenu'] jest kilka (tyle ile linijek w formularzu) i jak rozdzielić każdy wiersz od siebie
domyślam się, że będzie tutaj potrzebna jakaś tablica, ale nie jestem pewien.

A może napisze inaczej, chcę zebrać wszystkie dane z tego formularza i automatycznie wszystko zapisać. Formularz jest generowany dynamicznie, nie wiem ile będzie wierszy, wszystko zależy od złożoności tego menu.

Ten post edytował Danielcom 23.01.2010, 17:20:16

[evolucja]

jak generujesz formularz to jako name każdego z pól ustawiasz:

[PHP] pobierz, plaintext 
<? $pola[$idmenu][1, 2 lub 3] ?>
[PHP] pobierz, plaintext 

potem updatujesz to tak:

[PHP] pobierz, plaintext 
foreach ($pola as $menuid) {
$query = "UPDATE `tabela` SET `1` = $menuid[1], `2` = $menuid[2] `l4` = $menuid[2] WHERE `menuid`= $menuid LIMIT 1 ;"
mysql_query($query);
}
[PHP] pobierz, plaintext 

zapytanie pisane na szybko więc nie będzie pewnie działać (IMG:style_emoticons/default/tongue.gif) mam nadzieję, że dobrze zrozumiałem i będzie to działać...

Ten post edytował evolucja 23.01.2010, 17:30:13

[elmozaur]

w takim ukladzie kazda wysylana kolumna musi byc tablica.

po stronie odbiorczej robisz petle
$i=1;
foreach($arrayidmenu as $wiersz=>$wartosc)
{
$opub=$tabopublikowany[$i];
$ord=$taborder[$i];

i tak dalej


no i zapis do bazy: update tabela set opublikowany='$opub', order='$ord' where id='$wartosc'

$i++;
}

cos takiego powinno przeleciec po elementach tablicyidmenu a tym samym po tablicach pozostalych wartosci

[Danielcom]

ok, to rozwiązanie bardzo mi się podoba, mam tylko jeszcze jedno pytanie

skąd się bierze tablica $arrayidmenu ? a jeśli trzeba ją wygenerować to jak to zrobić z takiego kodu.

[PHP] pobierz, plaintext 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
?>
<form action="zapisz.php" method="post" />
<tr>
    <td width="1%">&nbsp;</td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="activ" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td width="5%"><input type="text" name="order" value="<?php echo $submenu['order']; ?>" size="2" /></td>
    <td width="2%"><input type="text" name="idmenu" value="<?php echo $submenu['idmenu']; ?>" size="2"  /></td>
                 <td> <input type="submit"  /></td>
 
</tr>
[PHP] pobierz, plaintext 

[evolucja]

Jeśli użyłbyś mojego sposobu to tak:

[PHP] pobierz, plaintext 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
?>
<form action="zapisz.php" method="post" />
<tr>
    <td width="1%"> </td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="<? echo $arrayidmenu["$idmenu"]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td width="5%"><input type="text" name="<? echo $arrayidmenu["$idmenu"]; ?>" value="<?php echo $submenu['order']; ?>" size="2" /></td>
    <td width="2%"><input type="text" name="<? echo $arrayidmenu["$idmenu"]; ?>" value="<?php echo $submenu['idmenu']; ?>" size="2"  /></td>
                 <td> <input type="submit"  /></td>
 
</tr>
[PHP] pobierz, plaintext 

Ten post edytował evolucja 23.01.2010, 18:15:48

[Danielcom]

zrobiłem tak jak napisałeś, ale mam problem, mianowicie w z pliku do pliku przesyłane są głupoty typu

"http://localhost/skl/testy/form/wyswietl.php?<br+%2F>%0D%0A<b>Notice<%2Fb>%3A++Undefined+variable%3A+idmenu+in+<b>C%3A\wamp\www\skl\modules\menu\index.php<%2Fb>+on+line+<b>55<%2Fb>"

ps chcę odebrać to takiem kodem:

[PHP] pobierz, plaintext 
<?php 
$i=1;
foreach($arrayidmenu as $wiersz=>$wartosc)
{
$opub=$a[$i];
 
//no i zapis do bazy: update tabela set opublikowany='$opub', order='$ord' where id='$wartosc'
echo $opub;
$i++;
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Danielcom 23.01.2010, 19:21:56

[evolucja]

Notice Undefined variable 'idmenu' in \wamp\www\skl\modules\menu\index.php on line 55

Mój błąd. Zamień wszystkie

[PHP] pobierz, plaintext 
$arrayidmenu["$idmenu"];
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
 $arrayidmenu["$submenu['idmenu']"]
[PHP] pobierz, plaintext 

[Danielcom]

teraz to już nie wiem czy ja coś źle napisałem, czy poprostu mam dzisiaj kiepski dzień (IMG:style_emoticons/default/sciana.gif)

teraz wyświetla

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\wamp\www\skl\modules\menu\index.php on line 23

to jest cały kod

[PHP] pobierz, plaintext 
<?php        
echo '<ul>';  
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = 0 ORDER BY `order` ASC' ) as $row){                        # menu
 
?>
<tr>
    <td width="10%"><?php     echo $row['title']; ?></td>
</tr>
<?php
// echo'<a href="index2.php?go='.$row['go'].'" /><div class="mainlev">'.$row['title'].'</div></a>';        # generowanie linków  - teraz nie potrzebne
#echo '<ul>';
?><div id="submenu">
 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
?>
<form action="../../testy/form/wyswietl.php" method="get" />
<tr>
    <td width="1%">&nbsp;</td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="<?php echo  $arrayidmenu["$submenu['idmenu']"]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td> <input type="submit"  /></td>
 
</tr>
<tr></tr>
<tr></tr>
<?php
 
//echo' <a href="index2.php?go='.$submenu['go'].'" /><div class="sublev">'.$submenu['title'].'</div></a> ';        # wycięte <li></li><div class="sublevel"></div>
}
?>
</div><?php
#echo '</ul>';
// echo '</li>';
# echo '';
}
echo '</ul>';
?></table><?php
}catch (PDOException $e) {
echo 'Jeśli widzisz ten komunikat powiadom administratora.';
loger($e->getMessage(),'menu');    # generowanie i zapis informacji o błędzie, dacie, godzinie, hoscie oraz przeglądarce (plik ./log/log.log)
}
?></div><?php
}
?>
 
<?php menu();?>
[PHP] pobierz, plaintext 

i drugi plik którym chciałbym odbierać wszystkie zmienne (narazie jedna, nie chce mi się pisać (IMG:style_emoticons/default/biggrin.gif) )

[PHP] pobierz, plaintext 
<?php 
$i=1;
foreach($arrayidmenu as $wiersz=>$wartosc)
{
$opub=$a[$i];
 
//no i zapis do bazy: update tabela set opublikowany='$opub', order='$ord' where id='$wartosc'
echo $opub;
$i++;
}
?>
[PHP] pobierz, plaintext 

[evolucja]

[PHP] pobierz, plaintext 
<?php        
echo '<ul>';  
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = 0 ORDER BY `order` ASC' ) as $row){                        # menu
 
?>
<tr>
    <td width="10%"><?php     echo $row['title']; ?></td>
</tr>
<?php
// echo'<a href="index2.php?go='.$row['go'].'" /><div class="mainlev">'.$row['title'].'</div></a>';        # generowanie linków  - teraz nie potrzebne
#echo '<ul>';
?><div id="submenu">
 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
$menuid = $submenu['idmenu'];
?>
<form action="../../testy/form/wyswietl.php" method="get" />
<tr>
    <td width="1%"> </td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="<?php echo  $arrayidmenu["$menuid"]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td> <input type="submit"  /></td>
 
</tr>
<tr></tr>
<tr></tr>
<?php
 
//echo' <a href="index2.php?go='.$submenu['go'].'" /><div class="sublev">'.$submenu['title'].'</div></a> ';        # wycięte <li></li><div class="sublevel"></div>
}
?>
</div><?php
#echo '</ul>';
// echo '</li>';
# echo '';
}
echo '</ul>';
?></table><?php
}catch (PDOException $e) {
echo 'Jeśli widzisz ten komunikat powiadom administratora.';
loger($e->getMessage(),'menu');    # generowanie i zapis informacji o błędzie, dacie, godzinie, hoscie oraz przeglądarce (plik ./log/log.log)
}
?></div><?php
}
?>
 
<?php menu();?>
[PHP] pobierz, plaintext 

Nie, nie.. to ja mam dziś pecha ze wszystkim (IMG:style_emoticons/default/biggrin.gif) Teraz powinno być bez parsera...

A tak po za tym to czekaj... czy ja dobrze widzę? Masz osobny formularz i button dla każdego wiersza?

Ten post edytował evolucja 23.01.2010, 20:46:28

[Danielcom]

tak, faktycznie wsadziłem tutaj stary kod (IMG:style_emoticons/default/smile.gif) ale ogólnie to jest to samo i po poprawieniu tego błędu z przyciskiem.
W sumie myślałem, żeby zrobić to już tak, ze do każdego wiersza jest oddzielny przycisk myślałem, ze będzie prościej (IMG:style_emoticons/default/smile.gif) ale jednak wolał bym zrobic tak jak to wałkujemy (IMG:style_emoticons/default/smile.gif)
teraz mam tak

[PHP] pobierz, plaintext 
<?php        
echo '<ul>';  
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = 0 ORDER BY `order` ASC' ) as $row){                        # menu
 
?>
<tr>
    <td width="10%"><?php     echo $row['title']; ?></td>
</tr>
<?php
// echo'<a href="index2.php?go='.$row['go'].'" /><div class="mainlev">'.$row['title'].'</div></a>';        # wycięte <li></li><div id="mainlevel"> </div> 
#echo '<ul>';
?><div id="submenu">
 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
?>
<form action="../../testy/form/wyswietl.php" method="get" />
<tr>
    <td width="1%">&nbsp;</td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="<?php echo  $arrayidmenu["$submenu['idmenu']"]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td> </td>
 
</tr>
<tr></tr>
<tr></tr>
<?php
 
//echo' <a href="index2.php?go='.$submenu['go'].'" /><div class="sublev">'.$submenu['title'].'</div></a> ';        # wycięte <li></li><div class="sublevel"></div>
}
?>
</div><?php
#echo '</ul>';
// echo '</li>';
# echo '';
}
echo '</ul>';
?></table><?php
}catch (PDOException $e) {
echo 'Jeśli widzisz ten komunikat powiadom administratora.';
loger($e->getMessage(),'menu');    # generowanie i zapis informacji o błędzie, dacie, godzinie, hoscie oraz przeglądarce (plik ./log/log.log)
}
?></div><?php
}
?>
 
<?php menu();?>
 
 
<input type="submit"  />
[PHP] pobierz, plaintext 

Ten post edytował Danielcom 23.01.2010, 21:05:54

[rytek]

Dlaczego wysyłasz formularz metodą GET? Przy tak dużym formularzu, url będzie bardzo długi.
Lepiej (i bezpieczniej) zrobić to POSTem.

I zamiast

[PHP] pobierz, plaintext 
<td width="1%"><input type="text" name="<?php echo  $arrayidmenu["$submenu['idmenu']"]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
[PHP] pobierz, plaintext 

Wrzuć:

[PHP] pobierz, plaintext 
<td width="1%"><input type="text" name="<?php echo  $arrayidmenu[$submenu['idmenu']]; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
[PHP] pobierz, plaintext 

Niepotrzebne cudzysłowy.

Ten post edytował rytek 23.01.2010, 21:15:19

[Danielcom]

Ogólnie cały system chodzi na postach, teraz zmieniłem żeby zobaczyć co jest przesyłane.
Usuniecie cudzysłowów pomogło, ale dalej jest problem z odczytaniem tych danych (kod pobrany z tego tematu).

Notice: Undefined variable: arrayidmenu in C:\wamp\www\skl\testy\form\wyswietl.php on line 3

Warning: Invalid argument supplied for foreach() in C:\wamp\www\skl\testy\form\wyswietl.php on line 3


Wiem, że jestem dzisiaj męczący, ale jakoś mi dzisiaj nie idzie (IMG:style_emoticons/default/smile.gif)

[evolucja]

Foreach się chyba tutaj nie sprawdzi...

[PHP] pobierz, plaintext 
$a = 0;
while(!empty($_POST[$arrayidmenu[$a]])) {
echo $_POST[$arrayidmenu[$a]];
$a++;
}
[PHP] pobierz, plaintext 

To powinno ci wypisać wartości wszystkich pól... Warunek- nie mogą być puste, czyli tam gdzie puste daj 0.
Ciężko mi się myśli ale powinno działać...

Więc jeśli zadziała, to już chyba będziesz wiedział co robić..

Ten post edytował evolucja 23.01.2010, 21:47:47

[Danielcom]

jak zadziała (IMG:style_emoticons/default/smile.gif)
dalej jest błąd
Notice: Undefined variable: arrayidmenu in C:\wamp\www\skl\testy\form\wyswietl.php on line 3
dodałem cudzysłowy , błedu nie ma, ale za to nic się teraz nie wyświetla.

[PHP] pobierz, plaintext 
<?php
$a = 0;
while(!empty($_POST['$arrayidmenu[$a]'])) {
echo $_POST['$arrayidmenu[$a]'];
$a++;
}
?>
[PHP] pobierz, plaintext 

A co do pętli to faktycznie while będzie szybsze. (IMG:style_emoticons/default/smile.gif)

Chyba jednak sobota nie jest od pracy (IMG:style_emoticons/default/smile.gif)
Przyznam, że mam z tym systemem sporo problemów, ale całe temu mnie dobija.

[evolucja]

Po pierwsze, zobacz w źródle formularza jak ci się te nazwy pól generują, bo może tam jest błąd...

A w wyświetl.php dodaj sobie na początku:

[PHP] pobierz, plaintext 
echo "<pre>";
print_r ($_POST);
echo "<pre>";
[PHP] pobierz, plaintext 

A tymczasem pozdrawiam gdyż muszę już iść. Do jutra (IMG:style_emoticons/default/biggrin.gif)

[Danielcom]

faktyczniecoś się źle generuje, zaraz nad tym posiedzę[xml][/xml]

[HTML] pobierz, plaintext 
<td width="1%"><input type="text" name="<br />
<b>Notice</b>:  Undefined variable: arrayidmenu in <b>C:\wamp\www\skl\modules\menu\index.php</b> on line <b>49</b><br />
" value="szybowiec_info2" size="2" /></td>
    <td width="1%"><input type="text" name="<br />
<b>Notice</b>:  Undefined variable: arrayidmenu in <b>C:\wamp\www\skl\modules\menu\index.php</b> on line <b>50</b><br />
" value="" size="2" /></td>
    <td width="1%"><input type="text" name="<br />
<b>Notice</b>:  Undefined variable: arrayidmenu in <b>C:\wamp\www\skl\modules\menu\index.php</b> on line <b>51</b><br />
" value="1" size="2" /></td>
	<td> </td>
[HTML] pobierz, plaintext 

[evolucja]

[PHP] pobierz, plaintext 
<?php
foreach($pdo -> query('SELECT * FROM sol_menu WHERE idsub = '.$row['idmenu'].' and idsub!=0    ORDER BY `order` ASC' ) as $submenu){        # sub menu
$idmenu = $submenu['idmenu'];
?>
<form action="../../testy/form/wyswietl.php" method="get" />
<tr>
    <td width="1%"> </td>
    <td width="15%"><?php echo $submenu['title']; ?></td>
    <td width="10%"><?php echo $submenu['go']; ?></td>
    <td width="1%"><?php echo $submenu['idmenu']; ?></td>
    <td width="1%"><input type="text" name="<?php echo  '$arrayidmenu[]['.$idmenu.']'; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
    <td> </td>
 
</tr>
[PHP] pobierz, plaintext 

może tym razem... mam nadzieję, że nie przekombinowałem (IMG:style_emoticons/default/biggrin.gif)
powinno zwracać name w postaci: '$arrayidmenu[][22]' '$arrayidmenu[][24]'

Ten post edytował evolucja 24.01.2010, 11:16:41

[Danielcom]

działa (IMG:style_emoticons/default/smile.gif)
trochę zmodyfikowałem ten skrypt.

[PHP] pobierz, plaintext 
    <td width="1%"><input type="text" name="<?php echo  '$activ['.$submenu['idmenu'].']'; ?>" value="<?php echo $submenu['activ']; ?>" size="2" /></td>
[PHP] pobierz, plaintext 

dzieki temu generuje się taka tablica

[PHP] pobierz, plaintext 
Array
(
    [$activ] => Array
        (
            [22] => 1
            [24] => 0
            [7] => 1
            [21] => 0
            [3] => 0
            [13] => 1
            [23] => 1
            [15] => 1
            [4] => 1
            [18] => 1
        )
 
    [$pozycja] => Array
        (
            [22] => 4
            [24] => 0
            [7] => 
            [21] => 1
            [3] => 0
            [13] => 1
            [23] => 666
            [15] => 1
            [4] => 1
            [18] => 3
        )
 
    [$uprawnienia] => Array
        (
            [22] => 
            [24] => 0
            [7] => 1
            [21] => 1
            [3] => 0
            [13] => 1
            [23] => 555
            [15] => 1
            [4] => 1
            [18] => 0
        )
 
)
[PHP] pobierz, plaintext 

Ten post edytował Danielcom 24.01.2010, 12:50:09

[evolucja]

Czyli już sobie poradzisz? (IMG:style_emoticons/default/smile.gif)

[Danielcom]

Pokombinuję jeszcze z zapisywaniem do bazy, bo ta tablica generuje się trochę inaczej niż sam bym tego chciał (IMG:style_emoticons/default/smile.gif)
Będę prawdopodobnie musiał przepisać funkcję któa zapisuje mi dane do MySQL