Wątek: Błąd w kodzie \radio i post

Witam. Zrobiłem taki kodzik (zlepiony z kilku różnych)

Kod:

<? if($_POST['avatar'] == 3){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
if($_POST['avatar'] == 4){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
if($_POST['avatar'] == 5){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}}} ?>
<table width="0" border="0">
<form method="POST" onsubmit="return checkError(this)">
	<input type="hidden" name="sel" id="sel" value="">
  <tr>
    <td><img src="avatars/3.png" width="153" height="153" /></td>
    <td><input type="radio" name="avatar" value="3" onclick="document.getElementById('sel').value = 'true'"></td>
  </tr>
  <tr>
    <td><img src="avatars/4.png" width="152" height="152" /></td>
    <td><input type="radio" name="avatar" value="4" onclick="document.getElementById('sel').value = 'true'"></td>
  </tr>
  <tr>
    <td><img src="avatars/5.png" width="153" height="153" /></td>
    <td><input type="radio" name="avatar" value="5" onclick="document.getElementById('sel').value = 'true'"></td>
  </tr>
  
</table>
<p><input type="submit" value="Zapisz!" name="submit" class="mod_submit"></p></form>

Jego zadaniem jest to, że użytkownik ma wyświetlonych kilka avatarów i może sobie wybrać jakiś (zaznacza go dzięki opcji radio) i wciska zapisz. Wtedy pojawia się aktualizacja w bazie i zmienia rekord na ten który był zaznaczony radiem.

Ogólnie skrpyt działa tylko dla avatar = 3, reszta nie trybi. What's wrong ?

Kod php:

<?php
//nie lubie short tagow
 if($_POST['avatar'] == 3){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
if($_POST['avatar'] == 4){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
if($_POST['avatar'] == 5){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}}} ?>

A nie tak:

Kod php:

 <?php

 if($_POST['avatar'] == 3){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}
else if($_POST['avatar'] == 4){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}
else if($_POST['avatar'] == 5){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
} ?>

Mogę się mylić ale godzina jest późna i przeraża mnie ladygaga z linka cezara na sb

Nie trybilo bo nie zamykales klamry warunkowej a pozniej zamknales wszystkie 3 na raz. Klamry powinny byc zamykane po kazdym zapytaniu i przed kazdym nastepnym ifem w tym wypadku.

Przyklad Alsona zadziala ale nie rozwiazuje on problemu tylko go omija poprzez przepisanie kodu ponownie ^^

fajne podejście dla 3 avatarów, gorzej jak potrzebujesz ich więcej, spróbuj tak:

Kod php:

<?php
$avatars = array("avatar1.gif", "avatar2.gif", "avatar3.gif", "avatar4.gif"); // itd, wpisujesz nazwy plików
// jeżeli wybrany avarat znajduje sie w naszej tablicy to update
if (in_array($_POST['avatar'], $avatars)){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}
?>
<table width="0" border="0">
<form method="POST" onsubmit="return checkError(this)">
    <input type="hidden" name="sel" id="sel" value="">
<?php
// teraz dla każdego avatara w tablicy generujemy radiobutton
foreach ($avatars as $value) { ?>
  <tr>
    <td><img src="avatars/<?php echo $value; ?>" width="153" height="153" /></td>
    <td><input type="radio" name="avatar" value="<?php echo $value; ?>" onclick="document.getElementById('sel').value = 'true'"></td>
  </tr>
<?php } ?>
</table>
<p><input type="submit" value="Zapisz!" name="submit" class="mod_submit"></p></form>

do tego będziesz musiał zmienić wyświetlanie i być może typ pola, bo w swoim przykładzie przechowujesz avatar jako cyfrę, a w moim przykładzie jest nazwa pliku

Zamieszczone przez tribe

fajne podejście dla 3 avatarów, gorzej jak potrzebujesz ich więcej, spróbuj tak:

Kod php:

<?php
$avatars = array("avatar1.gif", "avatar2.gif", "avatar3.gif", "avatar4.gif"); // itd, wpisujesz nazwy plików
// jeżeli wybrany avarat znajduje sie w naszej tablicy to update
if (in_array($_POST['avatar'], $avatars)){
mysql_query("UPDATE `users` SET `avatar` = '{$_POST['avatar']}' WHERE `login`='$data->login'");
}
?>
<table width="0" border="0">
<form method="POST" onsubmit="return checkError(this)">
    <input type="hidden" name="sel" id="sel" value="">
<?php
// teraz dla każdego avatara w tablicy generujemy radiobutton
foreach ($avatars as $value) { ?>
  <tr>
    <td><img src="avatars/<?php echo $value; ?>" width="153" height="153" /></td>
    <td><input type="radio" name="avatar" value="<?php echo $value; ?>" onclick="document.getElementById('sel').value = 'true'"></td>
  </tr>
<?php } ?>
</table>
<p><input type="submit" value="Zapisz!" name="submit" class="mod_submit"></p></form>

do tego będziesz musiał zmienić wyświetlanie i być może typ pola, bo w swoim przykładzie przechowujesz avatar jako cyfrę, a w moim przykładzie jest nazwa pliku

Ok dzięki, nom mam cyfrę, ponieważ avatar wywołuję w tak sposób
<img src=avatars/<? echo $user->avatar; ?>.png">