[php] mala rada porzadana Opcje

[Qspy]

Witam,
Pisze skrypt php do crona/phpbb. Niemusicie znac tabel standardowej bazy phpbb, mam tu male zapytanie ktore nietyra. Jezeli wykonam samo zapytanie ze zmiennej $sql3 w bazie danych uzyskam 93500 okolo. Jednak niemoge tego przetworzyc tym skryptem aby ta uzyskana liczbe z $sql3 zapisac do byle jakiej zmiennej zeby potem ja sobie poprzez dzialajace $sql1 zapisac do bazy..

Co wiem ze jest dobrze? Zapytanie w $sql3, zapytanie w $sql1. Jedyna rzecz jaka jest zle to przetworzenie zapytania z $sql3 aby zostalo wykonane i zwrocilo do zwyklej zmiennej liczbe. Wtedy juz dzialajacym zapytaniem $sql1 ja sobie zapisze do bazy.

Dzieki z gory za pomoc.

[PHP] pobierz, plaintext 
<?php
$sql3 = "SELECT COUNT( user_id ) AS total
			   FROM phpbb_users
			   WHERE user_id <> -1";
$result = $db->sql_query($sql3);
$row = $db->sql_fetchrow($result);
 
 
$sql1 = "UPDATE phpbb_config SET config_value = $row WHERE config_name = 'ilosc_userow'";
$db->sql_query($sql1);
?>
[PHP] pobierz, plaintext 

Ten post edytował Qspy 5.06.2007, 16:55:03

[SongoQ]

$row = $db->sql_fetchrow($result); to jest przyczyna

[Qspy]

Jakas podpowiedz? mogles dodac zdanie w ktorym bys mnie bardziej nakierowal.

[NoiseMc]

[PHP] pobierz, plaintext 
<?php
$row = $db->sql_fetchrow($result);
?>
[PHP] pobierz, plaintext 

zwraca caly wiersz a Ty potrzebujesz pojedyncza wartosc czyli zamiast:

[PHP] pobierz, plaintext 
<?php
$sql1 = "UPDATE phpbb_config SET config_value = $row WHERE config_name = 'ilosc_userow'";
?>
[PHP] pobierz, plaintext 

powinienes miec:

[PHP] pobierz, plaintext 
<?php
$sql1 = "UPDATE phpbb_config SET config_value = $row['total'] WHERE config_name = 'ilosc_userow'";
?>
[PHP] pobierz, plaintext 

[Qspy]

W ten sposob byl syntax ale zrobilem:

$row = $db->sql_fetchrow($result);
$row1 = $row['total'];

a potem updatowalem $row1
i tyra. Dzieki wielkie, sporo przedemna

[NoiseMc]

Popraw sobie jeszcze nazwy zmiennych na bardziej czytelne

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT COUNT( user_id ) AS total FROM phpbb_users WHERE user_id <> -1";
$result = $db->sql_query($sql);
$row = $db->sql_fetchrow($result);
 
// To mowi wiecej niz $row1
$userCount = $row['ilosc'];
 
// Tu mozesz nadpisac $sql bo starej wartosci nie potrzebujesz
$sql = "UPDATE phpbb_config SET config_value = $userCount WHERE config_name = 'ilosc_userow'";
$db->sql_query($sql);
?>
[PHP] pobierz, plaintext 

Dobre nazewnictwo i formatowanie kodu to podstawa zeby sie potem nie zgubic.

[Qspy]

Racja, skrypt bedzie maly jednak dobrze prawisz lepiej sie tak uczyc pisac.

Zazwyczaj najpierw pisze tak aby dzialalo, a potem optymalizuje pod wzgledem szybkosci/czytelnosci.