Forum PHP.pl

Witam natrafiłem na wydaje mi się dość banalny problem, ale niestety nie potrafie go rozwiązać.

Posiadam następujacy kod:

[PHP] pobierz, plaintext 
<?php
$nick = $_SESSION['nick'];
 
 
include 'include/conn.php';
 
/* zapytanie do konkretnej tabeli */
$wynik = mysql_query("SELECT * FROM typy WHERE nick='$nick' ORDER BY data ASC") or
    die('Błąd zapytania');
$temp = '1970-01-01';
 
if (mysql_num_rows($wynik) > 0) {
?>
 
<table cellpadding="2" border=0>
<?php
    while ($r = mysql_fetch_assoc($wynik)) {
        if ($r['data'] != $temp) {
?>
 
<tr>
<td>Gospodarz</td>
<td></td>
<td>Gość</td>
<td>Data<br>Status Meczu</td>
<td></td>
<td>Na Kogo Stawiamy?</td>
<td></td>
<td>Wynik</td>
<td>Sprawdzony?<br>Jeśli tak zaznacz</td>
</tr>
<br><br>
<tr>
 
<tr><td colspan="8" style="text-align:center;"><?php echo $r['data']; ?></td></tr>
<?php
            $temp = $r['data'];
        }
?>
 
<td><?php echo $r['gospodarz']; ?></td>
<td>VS</td>
<td><?php echo $r['gosc']; ?></td>
<td>
<?php
if ($r['wynik'] <= 40)
    echo "Tego meczu nie typujemy ponieważ są małe szanse na wytypowanie zwycięzcy !!!";
 
elseif ($r['wynik'] >= 41 & $r['wynik'] <= 65)
    echo "Nie typujemy lub stawiamy na <b><font color='#E31515' > " .
        $lepszy . " z remisem</b></font> za mala stawke !!!";
 
elseif ($r['wynik'] >= 66 & $r['wynik'] <= 100)
    echo "Typujemy zwyciestwo + remis.";
 
elseif ($r['wynik'] >= 101 & $r['wynik'] <= 150)
    echo "Typujemy Zwyciestwo.";
 
elseif ($r['wynik'] > 150)
    echo "Typujemy za wieksza stawke.";
 
echo "<br><br>";
?>
<form action="index.php?go=mojetypy" target="_self" method="post">
</td>
<td> na </td>
<td><font color='#E31515' ><?php echo $r['lepszy']; ?></font></td>
<td>Z Wynikiem Równym:</td>
<td><?php echo $r['wynik']; ?></td>
<td><input name="chk1" value="1" type="checkbox"></td>
<td><input type="submit" name="Submit" value="Update">
</tr>
</form>
<?php
$id=$r['id'];
$checkbox1 = $_POST['chk1'];
$value = $_POST['1'];
 
if (isset($_POST['Submit'])) {
 
include("include/conn.php");
 
$query="UPDATE typy SET zgodny='1' WHERE id='$id' AND nick='$nick' ";
	mysql_query($query) or die(mysql_error());
		mysql_close();
 
echo "Dane zostały zaktualizowane.";
}
 
?>
 
<?php
  }
?>	
</table>
<?php
}
?>
 
[PHP] pobierz, plaintext 

niestety aktualizuje mi wszystkie rekordy w bazie a nie konkretny przy którym zostal zaznaczony checkbox. Czy ktoś mógłby wskazać mi miejsce gdzie popełniam błąd ?

KOD JEST KOMPLETNIE NIECZYTELNY! Staraj się trzymać jakiś porządek w kodzie bo szybciej się zgubisz niż skończysz pisać. My jako osoby, które chcą pomóc także chciałby się skupić na tym co mają zrobić, aniżeli na sprzątaniu śmietnika. Poza tym sporo błędów nawet w HTML.

Twój formularz nie ma kompletnie sensu. Każdy formularz zawarty na stronie jest taki sam, czyli załóżmy masz z 20 rekordów do wyświetlenia to wyświetli Ci 20 TAKICH SAMYCH formularzy.
Aby zanalizować zaznaczone opcje, tzn wszystkie naraz musisz utworzyć jeden formularz. Oraz sprawdzić go raz (poza pętlą).
$_POST['1'] nie przyjmie wartości, a w przypadku wyświetlania błędów przez parser dostaniesz WARNING o braku indeksu '1'. Klucze w tablicy POST są takie jak parametry "name" przy inputach/textareach/selectach.
include("include/conn.php"); korzystasz z tego dwukrotnie co jest błędne w tym przypadku.

[PHP] pobierz, plaintext 
$query="UPDATE typy SET zgodny='1' WHERE id='$id' AND nick='$nick' ";
[PHP] pobierz, plaintext 

$id jest to pole id, które wyciągasz z bazy danych. Do tego wykonujesz to przy każdym elemencie wyciąganym z bazy, więc jeśli formularz zostanie wysłany, to każdy element który wyświetlasz zostanie także zaktualizowany.
[ ... ]

Proponuję zacząć od czegoś prostszego albo przynajmniej nauczyć się czegokolwiek (HTML/PHP) chociaż na poziomie podstawowym.

Ten post edytował ethann 9.04.2012, 19:13:33