Forum PHP.pl > Wyświetlanie użytkowników z bazy

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Wyświetlanie użytkowników z bazy

LucKy_skiLL

13.07.2012, 13:42:45

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.

cv65

13.07.2012, 13:45:01

[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'].'"');
while ($wynik = mysql_fetch_assoc($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

Pętla while

LucKy_skiLL

13.07.2012, 13:49:05

Gdzie jej użyć ?

cv65

13.07.2012, 14:01:45

Dałem Ci gotowca

[PHP] pobierz, plaintext 
$wynik = mysql_fetch_row($wynik);
[PHP] pobierz, plaintext

zwraca tylko jeden wynik
zaś

[PHP] pobierz, plaintext 
while ($wynik = mysql_fetch_assoc($wynik))
{
  echo $wynik[1];
}
[PHP] pobierz, plaintext

To jest przykład, pętla zwróci wszystkie wartości

LucKy_skiLL

13.07.2012, 14:09:59

No, ale ja wyświetlam rekordy tak

<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>

markonix

13.07.2012, 14:31:05

Czemu ten skrypt wykonuje aż 3 zapytania do tabeli `uzytkownik`?

Widzę tam na początku autoryzacje - po jakiego grzyba zapytanie do bazy zamiast po prostu sprawdzić czy istnieje w sesji.
Po co te zapytanie "autoryzujące" jest w pętli?
O css ktoś tu chyba nie słyszał.

LucKy_skiLL

15.07.2012, 17:02:11

Odświeżam, jak zastosować te pętle ?

markonix

15.07.2012, 19:28:28

Zamiast odświeżania może byś zamieścił poprawiony kod zgodnie z sugestiami?

LucKy_skiLL

15.07.2012, 20:24:16

"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

markonix

16.07.2012, 10:05:01

Cytat(LucKy_skiLL @ 15.07.2012, 21:24:16 )

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.

LucKy_skiLL

16.07.2012, 12:47:40

Zrobiłem tak

[PHP] pobierz, plaintext 
$all_1 = mysql_query("SELECT * from uzytkownik ORDER BY id DESC, login DESC LIMIT 0, 50")or die(mysql_error()); while ($users_row = mysql_fetch_array($all_1)) 
{
	$numer++
	echo <<< HTML
	<tr>
		<td>{$users_row['login']}</td>
	</tr>
	HTML;
}
[PHP] pobierz, plaintext

I ładnie wyświetla mi dwóch użytkowników których mam, moje pytanie brzmi, jak wyświetlić ich w mojej tabelce, robiąc tak, wyświetla mi tylko jedną osobe.

[PHP] pobierz, plaintext 
<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'>".$users_row['id']."</font></div></td>
					<td><div align='center'><font size='2'>".$users_row['login']."</font></div></td>
					<td><div align='center'><font size='2'>".$users_row['gg']."</font></div></td>
					<td><div align='center'><font size='2'>".$users_row['email']."</font></div></td>
					<td><div align='center'><font size='2'>".$users_row['status']."</font></div></td>
					<td><div align='center'><font size='2'>".$users_row['admin']."</font></div></td>
				</tr>
			</table>
[PHP] pobierz, plaintext

markonix

16.07.2012, 14:20:12

Zależy co chcesz osiągnąć - jeżeli ma powtarzać wiersz nagłówkowy to w pętli powinny się zawrzeć oba wiersze <tr>, jeżeli tylko konkretne dane to wiersz drugi.
I popraw ten HTML i zastosuj CSS bo nie można na to patrzeć!

LucKy_skiLL

16.07.2012, 14:50:13

Tak jak pisałem wcześniej, mam taki kod na tabele która ładnie wyświetla to co chce, ale wyświetla tylko jednego użytkownika, drugiego nie, tak więc pytam, jak i co zastosować, aby osiągnąć ten efekt.

[PHP] pobierz, plaintext 
<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'>".$users_row['id']."</font></div></td>
        <td><div align='center'><font size='2'>".$users_row['login']."</font></div></td>
        <td><div align='center'><font size='2'>".$users_row['gg']."</font></div></td>
        <td><div align='center'><font size='2'>".$users_row['email']."</font></div></td>
        <td><div align='center'><font size='2'>".$users_row['status']."</font></div></td>
        <td><div align='center'><font size='2'>".$users_row['admin']."</font></div></td>
    </tr>
</table>
[PHP] pobierz, plaintext

markonix

17.07.2012, 11:14:04

Po pierwsze poprawić ten badziewiasty kod HTML co powtarzam po raz 3.

Co do samego wyświetlania użytkowników przecież napisałem co trzeba zrobić, potrafisz czytać?

LucKy_skiLL

17.07.2012, 12:42:55

Już sobie sam poradziłem.

markonix

17.07.2012, 14:21:52

Cytat(LucKy_skiLL @ 17.07.2012, 13:42:55 )

Już sobie sam poradziłem.

To jest taka niepisana tradycja, że umieszcza się zwykle rozwiązanie dla potomnych.

LucKy_skiLL

17.07.2012, 17:17:02

Tutaj znalazłem w sumie gotowca.

http://www.w3schools.com/php/php_mysql_select.asp

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.