[MySQL][PHP]Get i nieistniejąca podstrona Opcje

[artekp999]

Napisałem skrypt czatu i za pomocą metody Get stworzyłem pokoje.
Link wygląda tak czat.php?pokoj=1 itd.
Pokoi mam np.7 ale jak wpisze np pokoj 8 to zsumowuje wszystkie posty ze wszystkich pokoi.
Dodajac post w danym pokoju zapisuje jego nr do bazy a pozniej odczytuje go w danym pokoju

Teraz mój problem
Jeśli wpiszę np. 8 pokój którego nie ma to zsumowuje wszystkie posty. Jak zablokować dalsze pokoje których nie ma t.j 8,9,10 itd

Kod pominąłem mało istotne rzeczy

[PHP] pobierz, plaintext 
<?php
include ("polacz.php");
 
  echo '<center><a href="?pokoj=1">Podstrona</a>';
    echo '<a href="?pokoj=2">Podstrona</a>';
      echo '<a href="?pokoj=3">Podstrona</a>';
        echo '<a href="?pokoj=4">Podstrona</a>';
          echo '<a href="?pokoj=5">Podstrona</a>';
            echo '<a href="?pokoj=6">Podstrona</a>';
              echo '<a href="?pokoj=7">Podstrona</a></center><p>';
 
    if ($_GET['pokoj'] == '1') 
  {
    $kategoria = 1;
  }
    if ($_GET['pokoj'] == '2') 
  {
    $kategoria = 2;
  }
    if ($_GET['pokoj'] == '3') 
  {
    $kategoria = 3;
  }
    if ($_GET['pokoj'] == '4') 
  {
    $kategoria = 4;
  }
    if ($_GET['pokoj'] == '5') 
  {
    $kategoria = 5;
  }
    if ($_GET['pokoj'] == '6') 
  {
    $kategoria = 6;
  }
    if ($_GET['pokoj'] == '7') 
  {
    $kategoria = 7;
  }
 
$tresc = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['tresc'])));
$nadawca = $_SESSION['nick'];
$idnadawca = $user[id];
$wyslano = date('G:i:s');
{
echo '<center>';
echo "$kategoria";
echo '<div class="title">
Następną wiadomość możesz wysłać za <span id="sekundy"></span>&nbsp;sekund.
</div>';
echo '<form action="" method="post">
<input type="hidden" name="send" value="1">
<input type="hidden" name="kategoria" value="'.$kategoria.'" />
<table><tr>
<td><input type="text" name="tresc" size="40" maxlength="40"/></td><td><input name="dodaj" type="submit" value="Wyślij" id="button"></form></td></tr></table>'; 
}
?>
 
		<script type="text/javascript">
		o=document.getElementById('sekundy');
		function odliczaj(o,sek){
		o.innerHTML=sek
		if(sek>0)setTimeout(function(){odliczaj(o,--sek)},1e3)
		var input = document.getElementById('button');
		input.disabled = 'disabled';
		if(sek==0)input.disabled = "";
		}
		odliczaj(document.getElementById('sekundy'),15)
		</script>
 
<?php
$zapytanie = "INSERT INTO `czat` (`nadawca`, `idnadawca`, `wyslano`, `kategoria`, `tresc`) VALUES ('".$nadawca."', '".$idnadawca."', '".$wyslano."', '".$kategoria."', '".$tresc."')";
$ostatnio = 'UPDATE `uzytkownicy` SET `ostatniododal`= "'.$ostatniododal.'" WHERE `id`="'.$user[id].'"';
$idzapytania = mysql_query($zapytanie) or die(mysql_error());
$idostatnio = mysql_query($ostatnio) or die(mysql_error());
if($_POST['send'])
{
if(isset($idzapytania))
{
if($idzapytania === TRUE)
{
echo '<font color="green"><b>Poprawnie wysłano wiadomość</b></font><br />';
 
}
else
{
echo '<font color="red"><b>Nie udało się wysłać wiadomości. Proszę spróbować za jakiś czas!</b></font><br />';
}
}
}
}
echo '</center>';
 
$query = mysql_query("select * from czat WHERE `kategoria`='$kategoria' order by id desc limit 0,50"); 
while($rekord = mysql_fetch_array($query))
{
$naz .= '<a href="news.php?id='.$rekord[2].'">'.$rekord[1].'</a>('.$rekord[3].') '.$rekord[5].'<br>';
}
echo "$naz";
include ("glowne/stopka.php");
?>
[PHP] pobierz, plaintext 

[by_ikar]

Masakra niech ten GET będzie w przedziale, powiedzmy 1 <= 8, a nie robisz x instrukcji warunkowych.. Ewentualnie zrób to w tablicy lub swotch, przecież jest tyle możliwości? a jak GET będzie poza przedziałem 1-8 wyświetl błąd albo coś.. Poczytaj o DRY, o tworzeniu swoich funkcji w php, bo trochę powielasz kilka razy to co robisz, a są przecież pętle

[gorden]

[PHP] pobierz, plaintext 
$liczba_pokoi = 7; //sumuj z bazy, lub recznie zapisuj
if($_GET['pokoj'] > $liczba_pokoi) echo 'Niepoprawny pokój!'; else {
 //warunki z getem. jesli dajesz jedna instrukcje, mozesz pominac nawiasy klamrowe, uzywaj elseif
}
[PHP] pobierz, plaintext 

Ten post edytował gorden 29.05.2011, 18:45:22

[by_ikar]

[PHP] pobierz, plaintext 
$liczba_pokoi = 7; //sumuj z bazy, lub recznie zapisuj
if($_GET['pokoj'] > $liczba_pokoi) echo 'Niepoprawny pokój!'; else {
 //warunki z getem. jesli dajesz jedna instrukcje, mozesz pominac nawiasy klamrowe, uzywaj elseif
}
[PHP] pobierz, plaintext 

Chyba:

[PHP] pobierz, plaintext 
$liczba_pokoi = 7; //sumuj z bazy, lub recznie zapisuj
if($_GET['pokoj'] <= $liczba_pokoi) 
{
  echo 'Niepoprawny pokój!';
}
else 
{
  //warunki z getem. jesli dajesz jedna instrukcje, mozesz pominac nawiasy klamrowe, uzywaj elseif
}
[PHP] pobierz, plaintext 

Jak już

ehh, bbcode w tagu php nie działa, więc chodzi o operator porównania, ustawiłeś "GET większy od liczba_pokoi".

Ten post edytował by_ikar 29.05.2011, 18:59:08

[nekomata]

@up dobrze było . Get większy od liczby pokoi (7) jest właściwy , ty użyłeś większe lub równe 7 czyli 7 pokój będzie błędnie wyświetlony.

[artekp999]

problem rozwiązany

gorden twój sposób okazał się własciwy