Wyświetlanie użytkowników z bazy Opcje

[LucKy_skiLL]

Mam taki skrypt na wyświetlywanie userów z bazy

[PHP] pobierz, plaintext 
<?php
echo "<div id='srodek'><div id='tlo'><center><b>Użytkownicy</b></center></div></div><br/><br/><br/><br/><br/>";
 
$wynik = mysql_query('SELECT admin FROM uzytkownik WHERE login ="'.$_SESSION['login'].'"');
$wynik = mysql_fetch_row($wynik); // Zamiana wyniku na tablice
 
 
if (($wynik[0] == 1) and (isset($_SESSION['login']))) { // dostęp dla zalogowanego użytkownika
 
	$result = mysql_query("SELECT Count(id) FROM uzytkownik");
	$row = mysql_fetch_row($result);
	$count_users = $row[0];
 
	if ($count_users > 0) {
		$result = mysql_query("SELECT * FROM uzytkownik");
		while ($row = mysql_fetch_assoc($result)) {
			if (($row['status']) == 1) {
				$row['status'] = 'Tak';
			} else {
				$row['status'] = 'Nie';
			}
 
			if (($row['admin']) == 1) {
				$row['admin'] = 'Tak';
			} else {
				$row['admin'] = 'Nie';
			}
			echo "<div id='srodek'><div id='tlo'><table width='98%' align='center' cellpadding='5' cellspacing='0'>
<tr><th><div align='center'>ID</div></th><th><div align='center'>Login</div></th><th><div align='center'>GG</div></th><th><div align='center'>E-Mail</div></th><th><div align='center'>Aktywny</div></th><th><div align='center'>Admin</div></th></tr>
<tr>
  <td><div align='center'><font size='2'>".$row['id']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['login']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['gg']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['email']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['status']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['admin']."</font></div></td>
</tr>\n
</table></div></div>";
		}
	}
} else {
	echo "<div id='srodek'><div id='tlo'><center>Musisz być zalogowany</center></div></div>";
}
 
?>
[PHP] pobierz, plaintext 

I wszystko jest fajnie, ale wyświetla tylko jednego użytkownika, a w bazie jest ich więcej.

(IMG:http://ss-host.pl/images/1vcv.png)

[LucKy_skiLL]

"Widzę tam na początku autoryzacje - po jakiego grzyba zapytanie do bazy zamiast po prostu sprawdzić czy istnieje w sesji."

Niby tak, ale może być zalogowany i nie być adminem, i wtedy miał by dostęp, a tego nie chcę.

[PHP] pobierz, plaintext 
<?php
echo "<div id='srodek'><div id='tlo'><center><b>Użytkownicy</b></center></div></div><br/><br/><br/><br/><br/>";
 
$wynik = mysql_query('SELECT admin FROM uzytkownik WHERE login ="'.$_SESSION['login'].'"');
$wynik = mysql_fetch_row($wynik); // Zamiana wyniku na tablice
 
 
if (($wynik[0] == 1) and (isset($_SESSION['login']))) { // dostęp dla zalogowanego użytkownika
 
	$result = mysql_query("SELECT Count(id) FROM uzytkownik");
	$row = mysql_fetch_row($result);
	$count_users = $row[0];
 
	if ($count_users > 0) {
		$result = mysql_query("SELECT * FROM uzytkownik");
 
		while ($row = mysql_fetch_assoc($result)) {
			if (($row['status']) == 1) {
				$row['status'] = 'Tak';
			} else {
				$row['status'] = 'Nie';
			}
 
			if (($row['admin']) == 1) {
				$row['admin'] = 'Tak';
			} else {
				$row['admin'] = 'Nie';
			}
			echo "<div id='srodek'><div id='tlo'><table width='98%' align='center' cellpadding='5' cellspacing='0'>
<tr><th><div align='center'>ID</div></th><th><div align='center'>Login</div></th><th><div align='center'>GG</div></th><th><div align='center'>E-Mail</div></th><th><div align='center'>Aktywny</div></th><th><div align='center'>Admin</div></th></tr>
<tr>
  <td><div align='center'><font size='2'>".$row['id']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['login']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['gg']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['email']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['status']."</font></div></td>
  <td><div align='center'><font size='2'>".$row['admin']."</font></div></td>
</tr>\n
</table></div></div>";
		}
	}
} else {
	echo "<div id='srodek'><div id='tlo'><center>Musisz być zalogowany</center></div></div>";
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował LucKy_skiLL 15.07.2012, 20:30:37

[markonix]

"Widzę tam na początku autoryzacje - po jakiego grzyba zapytanie do bazy zamiast po prostu sprawdzić czy istnieje w sesji."

Niby tak, ale może być zalogowany i nie być adminem, i wtedy miał by dostęp, a tego nie chcę.

Komentarz przy warunku mówi zupełnie o czym innym.
Sprawdzanie "czy admin" faktycznie ma sens ale ubrałbym to po pierwsze w funkcje, po drugie wystarczy to sprawdzić za pomocą count.

[PHP] pobierz, plaintext 
function isAdmin($username) {
 
	$result = mysql_fetch_row(mysql_query("SELECT COUNT(*) FROM FROM `uzytkownik` WHERE login = '$username' AND `admin` = 1"));
	return (boolean)$result[0];
 
}
[PHP] pobierz, plaintext 

Kolejny błąd to zliczanie użytkowników - skoro oglądasz wynik skryptu masz admina, a skoro masz admina to jesteś też użytkownikiem, skoro jesteś użytkownikiem to jest chociaż jeden rekord. Więc błędne koło i zapytanie zupełnie zbyteczne. Jeżeli już przyjmiemy jakieś zmiany w skrypcie to pobrałbym raz wszystkich userów, zliczył mysql_num_rows. Nie stosuje się go do liczenia bo pobiera wszystkie rekordy, ale i tak będą pobierane przecież do listy userów.

Nadal kod wygląda beznadziejne pod względem HTML/CSS.



Ten post edytował markonix 16.07.2012, 10:05:45