[PHP] update w pętli Opcje

[max_mcee]

Witam

Rekordy wyświetlam w pętli z bazy danych:


I załóżmy, że chcemy zaktualizować dane w bazie, więc pewnie trzeba to zrobić w pętli. I tutaj mam problem:

[PHP] pobierz, plaintext 
if(isset($_POST['spr'])) {
        $ilosc_wierszy = mysql_num_rows($wynik);
        //$id = $dane['id'];
 
        for ($z=1; $z<= $ilosc_wierszy; $z++) {
 
            mysql_query("UPDATE user SET (1 = '$_POST[1]', 2 = '$_POST[2]', 3 = '$_POST[3]') WHERE id ='$z'") or die (mysql_error());
 
        }
    }
[PHP] pobierz, plaintext 

Jak zaktualizować te dane w bazie ?

[lobopol]

Bez nawiasu po set

[PHP] pobierz, plaintext 
UPDATE user SET 1 = '{$_POST[1]}', 2 = '{$_POST[2]}', 3 = '{$_POST[3]}' WHERE id ='$z'
[PHP] pobierz, plaintext 

[max_mcee]

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '1 = 'W', 2 = 'W', 3 = 'W' WHERE id ='1'' at line 1

Niestety nadal nie działa.

[tolomei]

Witaj.

[PHP] pobierz, plaintext 
    mysql_query("UPDATE `user` SET `1` = '".addslashes($_POST[1])."', `2` = '".addslashes($_POST[2])."', `3` = '".addslashes($_POST[3])."' WHERE id = ".$z.") or die (mysql_error());
[PHP] pobierz, plaintext 

Pozdrawiam.

[max_mcee]

Teraz działa.
Jednak aktualizuje tylko ostatni wiersz. Może pokaże cały kod.

[PHP] pobierz, plaintext 
<?php
 
    include("config.php");
    $dni = array(
    '',
    'Pon',
    'Wto',
    'Śro',
    'Czw',
    'Pią',
    'Sob',
    'Nie',
    );
 
    $wiersz1 = '';
    $wiersz2 = '';
 
    for($i=1;$i<=date('t');$i++)
    {
    $wiersz1 .= '<td>'.$dni[date('N', mktime(0, 0, 0, date('n'), $i, date('Y')))].'</td>';
    $wiersz2 .= "<td>$i</td>";
    }
 
    echo '<form method="POST" action="zapis.php">'; 
    echo '<table>';
    echo '<tr>';
    echo '<td>Imię</td>';
    echo $wiersz2;
    echo '</tr>';
    echo '<tr>';
    echo '<td>i nazwisko</td>';
    echo $wiersz1;
    echo '</tr>';
    $sql = "SELECT * FROM user";
    $wynik = mysql_query($sql);
 
    $tablica = array();
    while($dane = mysql_fetch_array($wynik)) {
 
        echo '<tr><td><input type="hidden" name="id" value="'.$dane['id'].'" />'.$dane['name']."</td>";
        for ($d=1; $d<=date('t'); $d++){
        echo '<td><input type="text" name="'.$d.'" value="'.$dane[$d].'"></td>';
 
        }
    }
    echo'</tr>';
    echo'</table>';
    echo'<input type="hidden" name="spr" value="1" />';
    echo'<input type="submit" value="ZAPISZ" />';
    echo'</form>';
 
 
 
    if(isset($_POST['spr'])) {
        $ilosc_wierszy = mysql_num_rows($wynik);
        //$id = $dane['id'];
 
       for ($z=1; $z<= $ilosc_wierszy; $z++) {
 
            mysql_query("UPDATE `user` SET `1` = '".$_POST[1]."', `2` = '".$_POST[2]."', `3` = '".$_POST[3]."' WHERE id ='".$z."'") or die (mysql_error());
 
        }
 
    }
 
    ?>
[PHP] pobierz, plaintext 

jak to naprawić ?

Ten post edytował max_mcee 1.04.2012, 15:46:57

[Niktoś]

Czym jest $_POST[1] i $_POST[2] ?- jeśli któreś id pola ma nazwę zaczynającą się od liczby to jest to błąd.

Ten post edytował Niktoś 1.04.2012, 16:01:06

[max_mcee]

Tak wygląda baza:
To błąd ?

[Niktoś]

Przecież ja nie mówiłem o bazie tylko,tylko o polach input do których odwołujesz się $_POST["cośtam"];
Ty się jakoś w ogóle dziwnie odwołujesz do tablicy $_POST jak do tablicy indeksowanej ,a jest ona przecież hiperglobalną asocjacyjną.

[max_mcee]

[HTML] pobierz, plaintext 
<tr><td><input type="hidden" name="id" value="1" />user1</td><td><input type="text" name="1" value="r"></td>
<td><input type="text" name="2" value="r"></td>
<td><input type="text" name="3" value="r"></td>
 
itd.
[HTML] pobierz, plaintext 

[Niktoś]

<input type="text" name="1" value="r"></td>
<input type="text" name="2" value="r"></td>
<input type="text" name="3" value="r">

To co jest pogrubione jest nie poprawne.Wiesz jak będzie wyglądała tablica POST?Przykładowo,to coś takiego:
1=1
2=2
3=3
4=4
itd.
I jak potem chcesz się odwoływać?Powiedz co jest kluczem ,a co wartością?

Ten post edytował Niktoś 1.04.2012, 16:51:43

[max_mcee]

Chciałem się odwoływać, do id użytkownika.

załóżmy
$_POST[1] = W
$_POST[2] = R

jakoś nie mogę zrozumieć Twojego toku myślenia.. Możesz to dokładnie opisać ?

Ten post edytował max_mcee 1.04.2012, 17:01:22

[Niktoś]

[HTML] pobierz, plaintext 
    <?php
 
      include("config.php");
 
      $dni = array(
 
      '',
 
      'Pon',
 
      'Wto',
 
      'Śro',
 
      'Czw',
 
      'Pią',
 
      'Sob',
 
      'Nie',
 
      );
 
      $wiersz1 = '';
 
      $wiersz2 = '';
 
      for($i=1;$i<=date('t');$i++)
 
      {
 
      $wiersz1 .= '<td>'.$dni[date('N', mktime(0, 0, 0, date('n'), $i, date('Y')))].'</td>';
 
      $wiersz2 .= "<td>$i</td>";
 
      }
 
      echo '<form method="POST" action="zapis.php">';
 
      echo '<table>';
 
      echo '<tr>';
 
      echo '<td>Imię</td>';
 
      echo $wiersz2;
 
      echo '</tr>';
 
      echo '<tr>';
 
      echo '<td>i nazwisko</td>';
 
      echo $wiersz1;
 
      echo '</tr>';
 
      $sql = "SELECT * FROM user";
 
      $wynik = mysql_query($sql);
 
      $tablica = array();
      while($dane = mysql_fetch_array($wynik)) {
 
      echo '<tr><td><input type="hidden" name="id" value="'.$dane['id'].'" />'.$dane['name']."</td>";
 
      for ($d=1; $d<=date('t'); $d++){
 
      echo '<td><input type="text" name="nazwa'.$d.'" value="'.$dane[$d].'"></td>';
 
      }
 
      }
 
      echo'</tr>';
 
      echo'</table>';
 
      echo'<input type="hidden" name="spr" value="1" />';
 
      echo'<input type="submit" value="ZAPISZ" />';
 
      echo'</form>';
 
      if(isset($_POST['spr'])) {
 
      $ilosc_wierszy = mysql_num_rows($wynik);
 
      //$id = $dane['id'];
 
      for ($z=1; $z<= $ilosc_wierszy; $z++) {
 
      mysql_query("UPDATE `user` SET `1` = '".$_POST['nazwa1']."', `2` = '".$_POST['nazwa2']."', `3` = '".$_POST['nazwa3']."' WHERE id ='".$z."'") or die (mysql_error());
 
      }
 
      }
 
      ?>
[HTML] pobierz, plaintext 

[mortus]

@Niktoś: Zapis:

[HTML] pobierz, plaintext 
<input type="text" name="1" value="r">
<input type="text" name="2" value="r">
<input type="text" name="3" value="r">
[HTML] pobierz, plaintext 

jest niepoprawny, ale tylko z punktu widzenia specyfikacji (X)HTML. W PHP kluczem jest zawsze indeks w tablicy $_POST, a wartością wartość elementu z tablicy $_POST o wspomnianym indeksie. To nie jest błąd pod względem obslugi po stronie PHP.
No i nadal nie będzie działać.

@max_mcee: Najpierw kod:

[PHP] pobierz, plaintext 
while($dane = mysql_fetch_array($wynik)) { 
  echo '<tr><td><input type="hidden" name="id" value="'.$dane['id'].'" />'.$dane['name']."</td>";
  for ($d=1; $d<=date('t'); $d++){
    echo '<td><input type="text" name="'.$d.'" value="'.$dane[$d].'"></td>';
  }
}
[PHP] pobierz, plaintext 

W pętli while generujesz wiele elementów o nazwie id (name="id"), a po zatwierdzeniu formularza wysyłany jest identyfikator tylko jednego, ostatniego z tych elementów. Dzieje się tak właśnie dlatego, że elementy mają taką samą nazwę. Podobnie jest w przypadku pętli for. W momencie, gdy chcesz edytować wiele wierszy w tabeli musisz posłużyć się nazwami "tablicowymi", tj. takimi, które po zatwierdzeniu formularza przesyłają w $_POST tablice. Wyglądałoby to tak:

[PHP] pobierz, plaintext 
while($dane = mysql_fetch_array($wynik)) { 
  echo '<tr><td><input type="hidden" name="ids[]" value="'.$dane['id'].'" />'.$dane['name']."</td>";
  for ($d=1; $d<=date('t'); $d++){
    echo '<td><input type="text" name="col'.$d.'_vals[]" value="'.$dane[$d].'"></td>';
  }
}
[PHP] pobierz, plaintext 

Przesłane w ten sposób dane odbierasz w taki sposób:

[PHP] pobierz, plaintext 
if(isset($POST['spr'])) {
  foreach($_POST['ids'] as $key => $value) {
    $id_rekordu = $_POST['ids'][$key];
    $kolumna_1 = $_POST['col1_vals'][$key];
    $kolumna_2 = $_POST['col2_vals'][$key];
    // itd.
    mysql_query("UPDATE `user` SET `1` = $kolumna_1, `2` = $kolumna_2 WHERE `id` = $id_rekordu");
  } 
}
[PHP] pobierz, plaintext 

Pozostaje jeszcze kwestia optymalności skryptu. Niestety wykonywanie UPDATE-ów w pętli nie jest optymalnym rozwiązaniem, lepiej posłużyć się tutaj zapytaniem MULTI-INSERT z opcją ON DUPLICATE KEY UPDATE. Ale o tym może później, jak już ogarniesz powyższy kod.

Ten post edytował mortus 1.04.2012, 17:27:34

[max_mcee]

Nadal jest problem przy takim rozwiązaniu, ponieważ jak wpisuje w pierwszym polu W to nic się nie zmienia, a jak wpisuje w drugie pole W to i w pierwsze się wpisuje automatycznie.

[mortus]

Nadal jest problem przy takim rozwiązaniu, ponieważ jak wpisuje w pierwszym polu W to nic się nie zmienia, a jak wpisuje w drugie pole W to i w pierwsze się wpisuje automatycznie.

Nie ma innego rozwiązania, a to które podałem jest jedynym słusznym. Musisz je tylko u siebie wprowadzić... czy to zrobiłeś, nie wiem, bo nie widzę. Mam tylko nadzieję, że w ogóle wyrzuciłeś pętle for po zmiennej $z, bo ta pętla w ogóle nie powinna się tam znajdować.

Nie uważam, żeby napisanie całości za Ciebie było dobrym podejściem, więc pokaż kod, sprawdzę, czy są błędy i gdzie, i podpowiem co trzeba naprawić.

Czy manual kłamie?
Tablica asocjacyjna charakteryzuje się kluczami do których się odwołujesz,nie odwołujesz się do indexów.
Znajdź mi jeden choćby przykład w manualu, gdzie odwołuje się do wartości poprzez indeksy jak $_POST[0],$_POST[1],$_POST[2].

Zdanie wyrwane z kontekstu o niczym nie świadczy. To, że $_POST jest tablicą asocjacyjną to ja wiem doskonale. Ale w tablicy asocjacyjnej mogą się znaleźć również elementy indeksowane liczbami, więc nie ma w ogóle o czym gadać. Przykłady w manualu PHP są dostosowane do specyfikacji (X)HTML. Skoro zatem name w elemencie HTML nie może zaczynać się cyfrą, to i tak jest, i będzie w każdym z przykładów, jaki znajdziesz, co nie oznacza, że pole o nazwie 1 nie zostanie przesłane, i co nie oznacza, że wartością pola o nazwie 1 nie będzie inna z góry określona wartość. Klucze i wartości się nie pomieszają tylko dlatego, że nazwa pola będzie cyfrą. Najpierw to przetestuj, a później zgłaszaj sprzeciwy... ja testowałem kiedyś przesyłanie danych przez pola formularza o niewłaściwych nazwach i... działa.

Dobra kończę offtop. Widzę, że kolega Niktoś usunął swoją odpowiedź, ech...

Ten post edytował mortus 1.04.2012, 17:54:32

[max_mcee]

[PHP] pobierz, plaintext 
<?php
 
      include("config.php");
 
      $dni = array(
 
      '',
 
      'Pon',
 
      'Wto',
 
      'Śro',
 
      'Czw',
 
      'Pią',
 
      'Sob',
 
      'Nie',
 
      );
 
      $wiersz1 = '';
 
      $wiersz2 = '';
 
      for($i=1;$i<=date('t');$i++)
 
      {
 
      $wiersz1 .= '<td>'.$dni[date('N', mktime(0, 0, 0, date('n'), $i, date('Y')))].'</td>';
 
      $wiersz2 .= "<td>$i</td>";
 
      }
 
      echo '<form method="POST" action="zapis.php">';
 
      echo '<table>';
 
      echo '<tr>';
 
      echo '<td>Imię</td>';
 
      echo $wiersz2;
 
      echo '</tr>';
 
      echo '<tr>';
 
      echo '<td>i nazwisko</td>';
 
      echo $wiersz1;
 
      echo '</tr>';
 
      $sql = "SELECT * FROM user";
 
      $wynik = mysql_query($sql);
 
      $tablica = array();
      while($dane = mysql_fetch_array($wynik)) {
 
      echo '<tr><td><input type="hidden" name="ids[]" value="'.$dane['id'].'" />'.$dane['name']."</td>";
 
      for ($d=1; $d<=date('t'); $d++){
 
      echo '<td><input type="text" name="col'.$d.'_vals[]" value="'.$dane[$d].'"></td>';
 
      }
 
      }
 
      echo'</tr>';
 
      echo'</table>';
 
      echo'<input type="hidden" name="spr" value="1" />';
 
      echo'<input type="submit" value="ZAPISZ" />';
 
      echo'</form>';
 
      if(isset($_POST['spr'])) {
 
 
  foreach($_POST['ids'] as $key => $value) {
    $id_rekordu = $_POST['ids'][$key];
    $kolumna_1 = $_POST['col1_vals'][$key];
    $kolumna_2 = $_POST['col2_vals'][$key];
    // itd.
    mysql_query("UPDATE `user` SET `1` = $kolumna_1, `2` = $kolumna_2 WHERE `id` = $id_rekordu") or die (mysql_error());;
  } 
 
 
      }
 
      ?>
[PHP] pobierz, plaintext 

To rozwiązanie nie zmienia zawartości bazy, nic się nie dzieje, przeanalizowałem całoś co mi wysłałeś i rozumiem mechanizm działania, jednak nie wiem czemu nie zapisuje danych do bazy.

Unknown column 'w' in 'field list'

Ten post edytował max_mcee 1.04.2012, 19:06:00

[mortus]

Niektóre z aktualizowanych wartości są łańcuchami znaków i trzeba je objąć apostrofami:

[PHP] pobierz, plaintext 
mysql_query("UPDATE `user` SET `1` = '{$kolumna_1}', `2` = '{$kolumna_2}' WHERE `id` = $id_rekordu") or die (mysql_error());
[PHP] pobierz, plaintext 

EDIT:
Oczywiście zauważ, że brakuje reszty kolumn. Kolumny do zapytania możesz dodawać dynamicznie (np. w pętli for) lub tak, jak zaprezentowałem w przykładzie. Kolumn jest dużo, więc lepszym rozwiązaniem będzie pętla.

Ten post edytował mortus 1.04.2012, 23:57:45

[max_mcee]

Nadal ten sam błąd:

[PHP] pobierz, plaintext 
Unknown column 'w' in 'field list'
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
mysql_query("UPDATE `user` SET `1` = {$kolumna_1}, `2` = {$kolumna_2} WHERE `id` = $id_rekordu") or die (mysql_error());
[PHP] pobierz, plaintext 

bo rozumiem, że to powinno działać, nawet jeśli są tylko 2 pola ?
Bo w zasadzie updatem aktualizujemy tylko te pola które chcemy. Dobrze rozumiem ?

[mortus]

Nadal brakuje apostrofów, przyjrzyj się uważnie mojej poprzedniej wypowiedzi. Jeśli wartość (to co po znaku =, a nie przed), która ma być umieszczona w komórce jest napisem/łańcuchem znaków, to trzeba ją objąć apostrofami, w przeciwnym przypadku zostanie ona potraktowana, jak kolumna.

Oczywiście w ramach testów możesz aktualizować tylko dwie kolumny.

[max_mcee]

Ok działa

Czyli twierdzisz, że zamiast UPDATE warto by użyć:
http://dev.mysql.com/doc/refman/5.0/en/insert-on-duplicate.html