[MySQL][PHP]Liczenie Użytkowników, Budowa? Opcje

[Matimor]

Witajcie Ponownie
Jak mogę zbudować licznik zarejestrowanych osób i ostatnio zarejestrowaną osobę, aby wyświetlało mi te dane na stronie głównej. Chcę to zrobić poprzez MySql ponieważ tam robiłem wpis tabeli.

Dane Tabeli:
Nazwa: user
Hasła: password
Loginy: login
ID: user_id

I na stronie głównej poprzez połączenie MySql wyświetlać mi ma:
Mamy [liczba] Zarejestrowanych Użytkowników
Ostatnio Zarejestrowany: [username]

[sadistic_son]

Masz tu prawie gotowe rozwiazanie:

[PHP] pobierz, plaintext 
$dbc = @mysql_connect(HOST, USER, PASSWORD) or die ('Nie moglem polaczyć się z MySQL-em: ' . mysql_error() );
	@mysql_select_db(DB_NAME) or die ('Nie udalo się wybrać bazy danych: ' . mysql_error() );
 
$query="SELECT nazwa_usera FROM tabela_uzytkownicy ORDER BY id asc";
 
$result = @mysql_query ($query);
$num = mysql_num_rows ($result);
 
if($num>0){
while($row = mysql_fetch_array($result, MYSQL_NUM){
$last_user=$row[0];
}
$ostatni=$last_user;
echo "Mamy $num Zarejestrowanych Użytkowników<br />
Ostatnio Zarejestrowany: $ostatni ";
}
else{
echo 'Nie ma zarejestrowanych userów';
}
 
[PHP] pobierz, plaintext 

Ten post edytował sadistic_son 23.08.2009, 09:11:07

[#luq]

sadistic_son bez sensu, po co pobierać wszystkich jak chce się tylko w tym miejscu mieć ostatniego oraz ilość wszystkich? Nie optymalne.

Matimor
Do pobrania ostatniego: ORDER BY, LIMIT
Do pobrania liczby wszystkich: COUNT

[Matimor]

Ok Dzięki ale mam jakiś taki błąd.

[PHP] pobierz, plaintext 
Parse error: syntax error, unexpected '{' in 
[PHP] pobierz, plaintext 

[sadistic_son]

sadistic_son bez sensu, po co pobierać wszystkich jak chce się tylko w tym miejscu mieć ostatniego oraz ilość wszystkich? Nie optymalne.

Matimor
Do pobrania ostatniego: ORDER BY, LIMIT
Do pobrania liczby wszystkich: COUNT

No racja, Twoj jest bardziej optymalny ale moj tez dziala.

[PHP] pobierz, plaintext 
Parse error: syntax error, unexpected '{' in 
[PHP] pobierz, plaintext 

Masz gdzies za duzo o jeden '{'. Wkodzie ktory ja napisalem wszystko sie zgadza.

[Matimor]

No racja, Twoj jest bardziej optymalny ale moj tez dziala.


Masz gdzies za duzo o jeden '{'. Wkodzie ktory ja napisalem wszystko sie zgadza.

Klamry mam ok więc o co chodzi

[Quantum]

Parse error: syntax error, unexpected '{' in

masz dalej napisane on line xx i w tej linii xx masz błąd, swoją drogą zaopatrz się w jakiś dobry edytor,
polecam na start notepad++, dużo łatwiej sprawdzić gdzie masz błąd

Ten post edytował sniffer32 23.08.2009, 09:52:27

[#luq]

No racja, Twoj jest bardziej optymalny ale moj tez dziala.

Jeśli oba sposoby działają, a podany przeze mnie, dodatkowo jest bardziej optymalny to niech autor pomyśli który zastosować

[thek]

#luq podał najoptymalniejszy chyba sposób
Robisz count w bazie oraz robisz ORDER BY id DESC LIMIT 1
Ja bym dodał jeszcze jedną rzecz... Wstaw w obu warunek WHERE active = 1, gdzie active oznacza, czy zarejestrowany użytkownik aktywował swoje konto. Bo na co mi wiadomość o liczbie userów, włącznie z tymi, którzy nawet nie byli na tyle inteligentni, by kliknąć w link aktywacyjny posłany do nich mailem? Istotni są tylko CI, którzy to zrobili, bo w ten sposób można wyeliminować część botów chcących zarejestrować konta by spamować w portalu.

[Matimor]

Odkopie lekko temat.

Nadal mam z tym problem pisze mi, że mam za dużo {

[PHP] pobierz, plaintext 
<?php
mysql_connect('***','***','***')
or die('Nieudane połączenie z bazą danych.');
mysql_select_db('***')
or die('Nie udało się wybrać bazy danych.');
 
$query = "SELECT username FROM users ORDER BY id asc";
 
$result = @mysql_query ($query);
$num = mysql_num_rows ($result);
 
if($num>0)
{
	while($row = mysql_fetch_array($result, MYSQL_NUM)
	{
	$last_user=$row[0];
	}
	$ostatni=$last_user;
}
?>
[PHP] pobierz, plaintext 

[Wicepsik]

W 14 linijce masz za mało nawiasów.

[thek]

Optymalizuj to chłopie i popraw byki

[SQL] pobierz, plaintext 
SELECT username FROM users ORDER BY id DESC
[SQL] pobierz, plaintext 

DESC ma być. ASC to sortowanie od najmniejszej do największej. A Ty masz mieć najnowszego usera, czyli z największym id, a więc ma być DESC. Na dodatek po co ślesz cała tabelę userów? Masz posłać tylko liczbę userów i najnowszego więc zrób porządne zapytanie

[SQL] pobierz, plaintext 
SELECT username, count(id) AS ile FROM users GROUP BY id DESC LIMIT 1
[SQL] pobierz, plaintext 

To da Ci tylko jeden rekord wynikowy, który zawiera zarówno nazwę ostatnio zarejestrowanego użytkownika, jak i liczbę userów w bazie. Jeśli będziesz miał w bazie 500 userów to będziesz do skryptu posyłał 500 rekordów by sprawdzić wartość tylko jednego z nich? Szczerze wątpię

Ten post edytował thek 25.08.2009, 21:56:04

[Matimor]

Ok zrobiłem to tak, ale teraz mi nie pokazuje tego w index.php, użyłem require.

[PHP] pobierz, plaintext 
<?php
mysql_connect('***','***','***')
or die('Nieudane połączenie z bazą danych.');
mysql_select_db('***')
or die('Nie udało się wybrać bazy danych.');
 
$query = "SELECT username, count(id) AS ile FROM users GROUP BY id DESC LIMIT 1";
 
$result = @mysql_query ($query);
$num = @mysql_num_rows ($result);
 
if($num>0)
{
	while($row = mysql_fetch_array($result, MYSQL_NUM))
	{
	$last_user=$row[0];
	}
	$ostatni=$last_user;
}
?>
[PHP] pobierz, plaintext 

[Kszyhuu]

$zapytanie = mysql_query("SELECT login FROM user");
$ilosc_wierszy = mysql_num_rows($zapytanie);
$last= mysql_query("SELECT * FROM user ORDER BY user_id DESC LIMIT 1");


echo 'Mamy <b>'.$ilosc_wierszy.'</b> zarejestrowanych użytkowników.<br>
Ostatnio zarejestrowany:<b>' .$last['user'].'</b>.';

Ten post edytował Kszyhuu 26.08.2009, 13:44:03

[Matimor]

Dalej to samo...

Cały czas mam błąd w tym $last['user']

[PHP] pobierz, plaintext 
Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING 
[PHP] pobierz, plaintext 

[thek]

Po co uruchamiać pętle Moje zapytanie zwraca maksymalnie 1 (słownie jeden) rekord. Jeśli sprawdzisz czy zwróciło cokolwiek to już tylko wyciągasz dane.

[PHP] pobierz, plaintext 
$result = mysql_query( "SELECT username, count(id) AS ile FROM users GROUP BY id DESC LIMIT 1" );
if( $result )
    $answer = mysql_fetch_array( $result );
 
$ile = $answer['ile'];
$last = $answer['username'];
[PHP] pobierz, plaintext 

Czy to naprawdę takie trudne? I zobaczcie o ile sobie w porównaniu do innych skróciłem kod. Myślenie nie boli

@Kszyhuu:

[SQL] pobierz, plaintext 
mysql_query("SELECT login FROM user");
[SQL] pobierz, plaintext 

?
Czy jeśli baza miałaby 10000 userów to zwracałbyś 10000 loginów by je sobie potem w php funkcją num_rows policzyć? Od tego masz gotowe funkcje w bazie danych więc używaj ich. Po to masz właśnie count() w bazie.

@Matimor: masz błąd bo masz spację wewnątrz echo. nazwa => T_ENCAPSED_AND_WHITESPACE o tym mówi. Zaraz po b a przed kropką.

[Matimor]

thek Wielkie dzięki działa poprawnie.

Temat do wygaszenia.