[PHP][MySQL]Problem z kodem edycji Opcje

[kondziu9516]

Witam otóż mam taki oto poniższy kod który ma za zadania aktualizowanie danej treści w bazie danych.
Wszystko do formularzy pobiera poprawnie...
Otóż skrypt wyświetla zawartość tabeli w postaci formularza. Za pomocą checkbox-a wybieram który rekord ma być edytowny. Wpisuje nową treść, klikam na edytuj i niestety nie wiem czemu jakoś nic nie robi... Znajdzie może ktoś z was jakiś błąd?
Proszę bardzoo o pomoc

Z GÓRY DZIĘKUJĘ I DAJĘ PLUSY !

Struktura tabeli aktualności

id INT auto_increment podstawowy
data date
tresc TEXT
user varchar(50)

Kod PHP

[PHP] pobierz, plaintext 
<?PHP
   $link = mysql_connect("localhost", "user", "password");
          mysql_select_db("baza");
 
         $query = "SELECT id, data, tresc, user FROM aktualnosci ORDER BY data DESC";
         $result = mysql_query($query) or die(mysql_error());  
 
 
         echo("<form action=\"edycja.php\" method=\"post\">");   
 
         while(($row = mysql_fetch_row($result)) != false)
           {
              $id = $row[0];
              $data = $row[1];
              $tresc = $row[2];
              $user = $row[3];
 
            // mało ważne ustawienia tabeli i formularza 
             echo("<center>");
             echo("<table width=\"300\" align=\"justify\">");
             echo("<tr>");
             echo("<td  width=\"300\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
             // 1 pole które chce edytować
             echo("Data <br> <input type=\"text\" value=\"$data\" name=\"data[]\">");
 
            echo("</td>");
            echo("</tr>");
            echo("<td width=\"300\" class=\"ramka2 nsl2\">");
            echo("<div align=\"justify\">");
 
            // 2 pole które chce edytować
            echo("Tresc <br> <textarea cols=\"70\" rows=\"5\" name=\"tresc[]\">$tresc</textarea>");
 
 
            echo("</div>"); 
            echo("</td>");
            echo("</tr>");  
            echo("<td  width=\"200\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
            // 3 pole które chce edytować
             echo("Uzytkownik <br> <input type=\"text\" value=\"$user\" name=\"user[]\">");
 
            echo("</td>");   
            echo("<td  width=\"100\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
             // checkbox o wartości id
             echo("<input type=\"checkbox\" name=\"doedycji[]\" value=\"$id\">");
 
 
            echo("</td>");     
            echo("</tr>");
            echo("</table>");
            echo("</center>");
            echo("<br>");
            echo("<br>");    
 
        }
     echo("<input type=\"submit\" value=\"edytuj\">");   
     echo("</form>"); 
 
     if($doedycji==0)
       {
            echo("Brak rekordow w bazie.");
       }
     else
      {
           // tu powinno się wszystko odbyć ale niestety ....
           foreach ($_POST['doedycji'] as $key=>$k)
             { 
                  $data = $_POST["data"][$key];
                  $user = $_POST["user"][$key];
                  $tresc = $_POST["tresc"][$key];   
 
                  if(mysql_query("UPDATE aktualnosci SET DATA='$data', TRESC='$tresc', USER='$user' WHERE id='$k'"))
                    {
                         echo("Rekord(y)został(y) edytowany(e)");
                    }
                 else
                   {
                        echo("Error");    
                   }
             }   
       }         
 
      mysql_close($link);   
 
?>
[PHP] pobierz, plaintext 

Czyli nikt nie może też znaleźć błędu?
Szkoda...;/

Może jednak ktoś jeszcze spróbuje?
Albo naprowadzi mnie inaczej jak to zrobić?

Ten post edytował kondziu9516 30.03.2012, 20:47:37

[g0ther]

Kod, który przedstawiłeś mieści się w pliku edycja.php ?

[jaslanin]

1. Używasz zmiennej $doedycji, ale nigdzie jej nie zainicjowałeś, chyba powinno być tak:

[PHP] pobierz, plaintext 
if ($_POST['doedycji'] == 0) {
    echo("Brak rekordow w bazie.");
} else {
[PHP] pobierz, plaintext 

2. Jak coś aktualizujesz, to kod aktualizujący wykonuj przed kodem pobierającym dane bo inaczej nie zobaczysz zmiany. Czyli przenieś blok kodu:

[PHP] pobierz, plaintext 
if ($_POST['doedycji'] == 0) {
    echo("Brak rekordow w bazie.");
} else {
 
    echo 'asd';
    // tu powinno się wszystko odbyć ale niestety ....
    foreach ($_POST['doedycji'] as $key => $k) {
        $data = $_POST["data"][$key];
        $user = $_POST["user"][$key];
        $tresc = $_POST["tresc"][$key];
 
        if (mysql_query("UPDATE aktualnosci SET DATA='$data', TRESC='$tresc', USER='$user' WHERE id='$k'")) {
            echo("Rekord(y)został(y) edytowany(e)");
        } else {
            echo("Error");
        }
    }
}
[PHP] pobierz, plaintext 

nad:

[PHP] pobierz, plaintext 
$query = "SELECT id, data, tresc, user FROM aktualnosci ORDER BY data DESC";
[PHP] pobierz, plaintext 

3. masz w tym bloku kod foreach który iteruje po tablicy, zawsze sprawdzaj czy tablica po której chcesz iterować to rzeczywiście tablica

4. Następnym razem jak masz jakiś problem ustaw raportowanie błędów w PHP na E_ALL, to mi pozwoliło szybko znaleźć błąd w Twoim programie.

5. Zamiast robić cały czas echo '<html>' i escapeować nieporządane znaki lepiej pisać tak:

[PHP] pobierz, plaintext 
<?php 
$var = 'asd';
?>
 
<div id='asd'><?php echo $var ?></div>
[PHP] pobierz, plaintext 

Kod jak powinien mniej więcej wyglądać:

[PHP] pobierz, plaintext 
<?PHP
 
$link = mysql_connect("localhost", "root", "");
mysql_select_db("test");
 
if (isset($_POST['doedycji'])) {
    if ($_POST['doedycji'] == 0) {
        echo("Brak rekordow w bazie.");
    } else if (is_array($_POST['doedycji'])) {
        // tu powinno się wszystko odbyć ale niestety ....
        foreach ($_POST['doedycji'] as $key => $k) {
            $data = $_POST["data"][$key];
            $user = $_POST["user"][$key];
            $tresc = $_POST["tresc"][$key];
 
            if (mysql_query("UPDATE aktualnosci SET DATA='$data', TRESC='$tresc', USER='$user' WHERE id='$k'")) {
                echo("Rekord(y)został(y) edytowany(e)");
            } else {
                echo("Error");
            }
        }
    }
}
 
$query = "SELECT id, data, tresc, user FROM aktualnosci ORDER BY data DESC";
$result = mysql_query($query) or die(mysql_error());
 
 
echo("<form action=\"edycja.php\" method=\"post\">");
 
while (($row = mysql_fetch_row($result)) != false) {
    $id = $row[0];
    $data = $row[1];
    $tresc = $row[2];
    $user = $row[3];
 
    // mało ważne ustawienia tabeli i formularza
    echo("<center>");
    echo("<table width=\"300\" align=\"justify\">");
    echo("<tr>");
    echo("<td  width=\"300\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
    // 1 pole które chce edytować
    echo("Data <br> <input type=\"text\" value=\"$data\" name=\"data[]\">");
 
    echo("</td>");
    echo("</tr>");
    echo("<td width=\"300\" class=\"ramka2 nsl2\">");
    echo("<div align=\"justify\">");
 
    // 2 pole które chce edytować
    echo("Tresc <br> <textarea cols=\"70\" rows=\"5\" name=\"tresc[]\">$tresc</textarea>");
 
 
    echo("</div>");
    echo("</td>");
    echo("</tr>");
    echo("<td  width=\"200\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
    // 3 pole które chce edytować
    echo("Uzytkownik <br> <input type=\"text\" value=\"$user\" name=\"user[]\">");
 
    echo("</td>");
    echo("<td  width=\"100\" class=\"ramka2 nsl\" bgcolor=\"#EAEAEA\" align=\"left\">");
 
    // checkbox o wartości id
    echo("<input type=\"checkbox\" name=\"doedycji[]\" value=\"$id\">");
 
 
    echo("</td>");
    echo("</tr>");
    echo("</table>");
    echo("</center>");
    echo("<br>");
    echo("<br>");
}
echo("<input type=\"submit\" value=\"edytuj\">");
echo("</form>");
 
mysql_close($link);
?>
[PHP] pobierz, plaintext 

Ten post edytował jaslanin 31.03.2012, 17:28:31