badanka

<?php
include 'header.php';
echo '<center>';
if(isset($_GET['id'])){
$lid = safe($_GET['id']);
$result = mysql_query("SELECT * FROM lan WHERE lid='$lid'");

/*
 Jag antar det kommer bara en rad ur ovanst�ende query? I vilket fall som helst s� skriver du ju �ver
 v�rdena i dina variabler varje varv, s� bara den sista blir i kraft.

 Om query returnerade flere rader s� har du ingen sorteringsordning satt, s� de kommer d� i godtycklig
 ordning som kan variera mellan olika k�rningar...

 S� jag �ndrade (tillbaks?) till en if-sats
*/

if ($row = mysql_fetch_array($result))
  {
$rader = $row['rader'];
$langd = $row['langd'];
$bredd = $row['bredd'];
$namn = $row['namn'];
$nr = $row['nr'];
  }
  echo '<br /><img src="images/scen1.jpg">';
$j=1;
echo '<table border="0"><tr>';
while($j<=$rader)
  {
  $j++;
  echo '<td>';
  $i = 1;
while($i<=$langd)
  {
/*
 * Har du olika tabeller f�r de olika LANarna? Jag tycker det ser ut som om du h�mtade data fr�n en tabell som �r namngiven i datan i en annan tabell?
 * S� g�r man inte. Det blir nog tillbaks till ritbordet och omstrukturering av tabellstrukturen d�. Ist�llet f�r en tabell per LAN kan du l�gga in
 * en kolumn som t.ex. har v�rdet av lanid.
 *
 * Sen skall du skriva databasqyeryn med en JOIN, s� som drew p�pekade.
 *
 * Jag f�rst�r mig inte heller riktigt p� ett hurdant bokningssystem du f�rs�ker g�ra. Du verkar ha n�gon sorts struktur var det �r utm�rkt om en plats
 * �r ledig eller upptagen? Men du har inget tidsbegrepp, beh�ver du inte det? T.ex. mellan vilka tidpunkter den �r bokad? Skulle jag designa det d�r
 * s� skulle jag p� n�got s�tt i databasen beskriva hur LAN:en ser ut. Sedan skulle jag ha en skild struktur (en tabell) f�r bokningar, bokningen pekar
 * sedan p� en plats i en LAN. Finns det inte en bokning s� �r platsen ledig, dvs de lediga platserna finns inte alls i datbasen utan logiken i din kod
 * vet att om platsen inte �r bokad s� �r den ledig... Sen beh�ver du antagligen en struktur f�r under vilka tider man kan boka LAN-platser i de olika LANarna...
 *
 * Bokningssystem h�r nog inte till de enklaste sakerna att implementera... Du kunde kanske fundera p� att �va upp dig med n�got lite enklare f�rst?
*/
  $laninfo1 = mysql_query("SELECT * from $namn$nr WHERE rad=$j AND langd=$i AND bredd='1'") or die(mysql_error());
  while($row1 = mysql_fetch_array($laninfo1))
  {
  $status1 = $row1['status'];
  if($status1 == 0)
        {
   echo '<img src="images/ledig.jpg" alt="L" border="0" title="Ledig">';
        }else{
   echo '<img src="images/upptagen.jpg" alt="U" border="0" title="Upptagen"><br />';
   }
        }
  $laninfo2 = mysql_query("SELECT * from $namn$nr WHERE rad=$j AND langd=$i AND bredd='2'") or die(mysql_error());;
  while($row2 = mysql_fetch_array($laninfo2))
  {
  $status2 = $row2['status'];
if($status2 == 0)
{
   echo '<img src="images/ledig.jpg" alt="L" border="0" title="Ledig"><br />';
        }else{
   echo '<img src="images/upptagen.jpg" alt="U" border="0" title="Upptagen"><br />';
   }
        }
  $i++;
  }
 echo '</td>';
  }
  echo '</tr></table>';
if(isset($_GET['rad']) && ($_GET['langd']) && ($_GET['plats'])){

?>
<form action ="&spara" method="post">
Boka platsen? <input type="radio" name="boka" value="1">Ja <input type="radio" name="boka" value="0">
<?php
}
}
else
{
$result = mysql_query("SELECT * FROM lan");
echo '<table border="0">';
echo '<th>Namn p� LAN</th>';
echo '<th>Plats</th>';
while($row = mysql_fetch_array($result))
  {
  if($row['active'] == 1)
  {
  echo '<tr><td><a href="lanplatser.php?id=' . $row['lid'] . '">' . $row['namn'] . ' [' . $row['nr'] . ']</a></td><td>' . $row['plats'] . '</td></tr>';
}
  }
  echo "</table>";
  }
  echo '</center>';
include 'footer.php';
/*
F�r att g�ra livet l�ttare f�r dig sj�lv s� rekommenderar jag numeriska (integer) f�lt d�r det bara �r m�jligt. Betydligt enklare att kolla inputten och enklare att skriva queryna ocks�.

Och skall man vara riktigt noga (vilket man ju alltid b�r vara :-) ) s� borde du quota outputten ocks� som en anv�ndare kommer �t att mata in. Annars kan de g�ra en HTML/JS-inject som g�r att n�n kan kapa en anv�ndares bl�ddrare eller bara trasa s�nder sidan visuellt f�r anv�ndaren...
*/
?>