[MySQL][PHP]Kick Opcje

[GoodStuff]

Cześć, kolejny problem napisałem pokoje i próbuję zrobić opcje wyrzucenia gracza z pokoju. Wyświetlam w pętli graczy a obok dorobiłem button z opcją wyrzucenia problem polega na tym, że nie wiem jakie dokładnie zapytanie dać. Aktualnie kasuje po przez button pierwszą osobę z listy...


Kawałek kodu z pokoje.php

[PHP] pobierz, plaintext 
	$gracze = $db->query("SELECT * FROM pokoje_gracz Where id_pokoje=".$shout['id']."");
	echo'</td>
    <td class="tg-031e" colspan="3" rowspan="4">Gracze:<br />'; while($gracze_pokoje=mysqli_fetch_array($gracze)) { echo'-<a href="profile.php?id='.$gracze_pokoje['id_gracza'].'">'.$gracze_pokoje['nazwa_gracz'].'</a>';
	if($shout['id_wlasciciela'] == $user['id'])
{
echo '<form action="?" method="POST"><button type="submit" name="kick" class="kick"></button></form>';
}
	echo'<br />';} 
[PHP] pobierz, plaintext 

Button kick

[PHP] pobierz, plaintext 
 if(isset($_POST['kick']))
{
	$gracze = $db->query("SELECT * FROM pokoje_gracz Where id_pokoje=".$shout['id']."");
	$gracze1=mysqli_fetch_array($gracze);
	$id_pokoje = $shout['id'];
	$id_gracza = $gracze1['id_gracza'];
 
 
 
	$db->query("DELETE FROM pokoje_gracz Where id_gracza=$id_gracza and id_pokoje=$id_pokoje")
	or die(mysql_error());  
 
 
 
header('Location: '.$_SERVER['REQUEST_URI']);
	}
[PHP] pobierz, plaintext 

[Aqu]

W jaki sposób Twój kod miałby działać? Nie przekazujesz w tym formularzu, którego gracza chcesz wyrzucić. Potem pobierasz wszystkich graczy i bierzesz sobie jakiegoś: $id_gracza = $gracze1['id_gracza']; (IMG:style_emoticons/default/facepalmxd.gif)

Dodaj sobie w tym formularzu ukryte pole - input type hidden.

[GoodStuff]

Gdy to dam co wtedy?

[Aqu]

Przekaż w nim ID gracza, którego chcesz wyrzucić.

[GoodStuff]

Zrobiłem w ten sposób ale usuwa wszystkich ...

[PHP] pobierz, plaintext 
while($gracze_pokoje=mysqli_fetch_array($gracze)) { echo'-<a href="profile.php?id='.$gracze_pokoje['id_gracza'].'">'.$gracze_pokoje['nazwa_gracz'].'</a>';
	if(isset($_POST['kick']))
{
 
	$id_pokoje = $shout['id'];
	$id_gracza = $gracze_pokoje['id_gracza'];
 
 
 
	$db->query("DELETE FROM pokoje_gracz Where id_gracza=$id_gracza and id_pokoje=$id_pokoje")
	or die(mysql_error());  
 
 
 
header('Location: '.$_SERVER['REQUEST_URI']);
	}
	if($shout['id_wlasciciela'] == $user['id'])
{
echo '<form action="?" method="POST"><button type="submit" name="kick" class="kick"></button></form>';
}
	echo'<br />';} 
[PHP] pobierz, plaintext 

[Neymar11]

Wiesz co? Nie wiem czy to przez to ale chyba w zapytaniu apostrofy zgubiłeś (IMG:style_emoticons/default/smile.gif)

[Aqu]

Podstawia tam chyba liczby, więc apostrofy mu niepotrzebne.

I gdzie w tym kodzie jest hidden input? Gdzie go odbierasz i na podstawie tego usuwasz gracza?

[GoodStuff]

Wiesz co? Nie wiem czy to przez to ale chyba w zapytaniu apostrofy zgubiłeś (IMG:style_emoticons/default/smile.gif)

Nie nie tutaj apostrofy nie mają znaczenia:

Chodzi o petlę jak to wykonać po za nią..

[PHP] pobierz, plaintext 
while($gracze_pokoje=mysqli_fetch_array($gracze)) { echo'-<a href="profile.php?id='.$gracze_pokoje['id_gracza'].'">'.$gracze_pokoje['nazwa_gracz'].'</a>';
	if($_POST['send'] == 4){
 
	$id_pokoje = $shout['id'];
	$id_gracza = $gracze_pokoje['id_gracza'];
 
$result = $db->query("DELETE FROM pokoje_gracz Where id_gracza=$id_gracza and id_pokoje=$id_pokoje")
or die(mysql_error());  
 
echo "<br />";
echo $result;
echo "<br />";
echo $id_pokoje;
echo "<br />";
echo $id_gracza;
echo "<br />";
 
	}
	if($shout['id_wlasciciela'] == $user['id'])
{
echo '<form action="?" method="POST"> <input type="hidden" name="send" value="4" /><button type="submit" name="kick" class="kick"></button></form>';
}
	echo'<br />';} 
[PHP] pobierz, plaintext 

Wszystko gra tak jak powinno przy sprawdzeniu usuwa poprawnie ale jako button kick naciskając na jeden usuwa wszystkich

przykładowo jest 2 graczy

gracz a i gracz b

przy wywołaniu echo

[PHP] pobierz, plaintext 
gracz a
echo result = 1
echo id_pokoje = 19
echo id gracza = 8
 
gracz b
echo result = 1
echo id_pokoje = 19
echo id gracza = 9
 
[PHP] pobierz, plaintext 

Więc zapytanie jest jak najbardziej ok. Tylko ta pętla...

Podstawia tam chyba liczby, więc apostrofy mu niepotrzebne.

I gdzie w tym kodzie jest hidden input? Gdzie go odbierasz i na podstawie tego usuwasz gracza?

[PHP] pobierz, plaintext 
echo '<form action="?" method="POST"> <input type="hidden" name="send" value="4" /><button type="submit" name="kick" class="kick"></button></form>';
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if($_POST['send'] == 4){
 
	$id_pokoje = $shout['id'];
	$id_gracza = $gracze_pokoje['id_gracza'];
 
$result = $db->query("DELETE FROM pokoje_gracz Where id_gracza=$id_gracza and id_pokoje=$id_pokoje")
or die(mysql_error());  
 
echo "<br />";
echo $result;
echo "<br />";
echo $id_pokoje;
echo "<br />";
echo $id_gracza;
echo "<br />";
 
	}
[PHP] pobierz, plaintext 

[Aqu]

1. IF wyrzuć z pętli, przecież to nie ma sensu.
2. Nie sprawdzaj if($_POST['send'] == 4), chcesz stworzyć odzielny IF dla każdego gracza?

[PHP] pobierz, plaintext 
3. <input type="hidden" name="kickPlayerId" value="ID GRACZA">
4. if(isset($_POST)) {
// DELETE FROM tabela WHERE room=xx AND player_id=$_POST['kickPlayerId']
 
}
 
5. Zabezpiecz to przed sql injection.
 
[PHP] pobierz, plaintext 

Ten post edytował Aqu 23.02.2015, 22:16:25

[Neymar11]

Spróbuj taką strukturę kodu:

[PHP] pobierz, plaintext 
if($_POST['send'] == 4){
 
	$id_pokoje = $shout['id'];
	$id_gracza = $gracze_pokoje['id_gracza'];
 
$q = "DELETE FROM pokoje_gracz Where id_gracza='$id_gracza' and id_pokoje='$id_pokoje'";
$result = $db->query($q)
or die(mysql_error());
[PHP] pobierz, plaintext 

[GoodStuff]

Dzięki za pomoc. Aqu To rozwiązało wiele moich problemów (IMG:style_emoticons/default/smile.gif)

Ten post edytował GoodStuff 23.02.2015, 22:40:33