Forum PHP.pl

Właśnie zauważyłem bezsensowność mojego kodu. Nie mam pomysłu jak go poprawić:

[PHP] pobierz, plaintext 
<?php
function data($column, $no) {
  connection();
  $query = "SELECT * FROM ksiazki where active = 1";
  $result = mysql_query($query);
  $row = mysql_num_rows($result);
  mysql_data_seek($result, $no);
  $r = mysql_fetch_assoc($result);
  print_r($r[$column]);
  mysql_close();
}
?>
[PHP] pobierz, plaintext 

Podając nazwę kolumny i numer wiersza, zwraca wynik. Miała mi uprościć życie, bo podaje ją w pętli. Dopiero jak dałem ją do pętli, która pobiera całość (ok. 9 kolumn, 21 rekordów [9*21= 189 zapytań]) to zauważyłem, że znacznie spadła wydajność skryptu. Podejrzewm, że wrzucając zapytanie do osobnej funkcji, również bedzie się wykonywała tyle razy ile jest iteracji w pętli. Jest jakieś rozwiązanie tego problemu? (Nie chodzi mi o takie, jakie jest kilka topiców niżej w temacie "cache", pliki odpadają)

Nie wiem czy dobrze Ciebie zrozumiałem...


Za każdym razem łączysz się z bazą i zamykasz połączenie - bez sensu - strata wydajności.
Aby pobrać rekordy raz połącz się z bazą i dane wyciągaj w pętli

[PHP] pobierz, plaintext 
<?php
connection();
$query = "SELECT * FROM ksiazki where active = 1";
$result = mysql_query($query);
while($row = mysql_num_rows($result))
{
   echo $row[0]; //itd...
}
mysql_close();
?>
[PHP] pobierz, plaintext 

Łukasz

EDIT
a jeśli już musisz wykontywać coś takiego w pętli to połącz się z bazą i rozłączaj poza funkcją którą wykonujesz w pętli

Ten post edytował kossa 31.08.2006, 18:07:12

Zrobię, tak jak pisałeś. Żebyś lepiej mnie zrozumiał podam ci przykład do powyższej funkcji:

[PHP] pobierz, plaintext 
<?php
<form action="./<?=basename($_SERVER['PHP_SELF']);?>?site=aukcje" method="post">
<select name="book" onchange="submit(this);">
for($i = 0; $i < rows(); $i++){
echo "<option value="$i" >";
data('id', $i);
echo ". ";
data('authors', $i);
echo " - ";
data('title', $i);
echo "</option>n";
}
</select>
</form>
?>
[PHP] pobierz, plaintext 

Teraz funkcja zwracana np. kolumnę authors, o podanym numerze $i, ale co z tego jak za każdym razem, jest wykonywane zapytanie do MySQL'a, a przecież można wykonać (chyba) 1x zapytanie, a potem operować w pętli tylko na zwróconym przez zapytanie "zasobie". Tylko właśnie nie potrafie czegoś takiego wymyślić.

Ten post edytował Coolmax 31.08.2006, 22:22:01

Przyznam się, że wciąż nie rozumiem... albo napisałeś znowu niezbyt jasno albo ja już po całym dniu klepania w klawiaturę nie kojarzę zbyt dobrze.

Mając $i możesz tylko raz pobrać z bazy potrzebny rekord

[SQL] pobierz, plaintext 
SELECT kolumna_1,kolumna_n FROM tabela WHERE kolumna_x=$i
[SQL] pobierz, plaintext 

Nie rozumiem, co masz na myśli aby później coś w pętli wykonywać.
Mając zwrócony rekord możesz już z nim zrobić wszystko. Co chcesz zrobić - napisz jaśniej.


Łukasz

Mi chodzi tylko o to aby w funkcji data:

[PHP] pobierz, plaintext 
<?php
function data($column, $no) {
  connection();
  $query = "SELECT * FROM ksiazki where active = 1";
  $result = mysql_query($query);
  mysql_data_seek($result, $no);
  $r = mysql_fetch_assoc($result);
  print_r($r[$column]);
  mysql_close();
}
?>
[PHP] pobierz, plaintext 

używanej za każdym razem w pętli, nie trzeba było wysyłać zapytania za każdym razem, czyli w $reslut, są wszystkie rekordy, które w kolumnie active mają wartość 1. To zapytanie chciałem, żeby było wykonanywane tylko raz, bo w nim jest już wszystko to, czego potrzebuję, następnie funkcją data operować na tych danych.

Ten post edytował Coolmax 1.09.2006, 10:48:00

[PHP] pobierz, plaintext 
<?php
connection();
$query = "SELECT * FROM ksiazki where active = 1";
$result = mysql_query($query);
while($row = mysql_num_rows($result))
{
   echo $row[0]; //itd...
}
mysql_close();
?>
[PHP] pobierz, plaintext 

pisałem Tobie wcześniej tu co jest tu powyżej - wszystko raz wybierasz i w $result masz wszystko

Łukasz