Witam

Mam taki kod:

[PHP] pobierz, plaintext 
<?php
 
	$row = $db->query_first("SELECT COUNT(*) AS itemCount FROM items WHERE isWaiting = '0'");
 
	if (!isset($_GET["page"]))
		$_GET["page"] = 0;
 
	$paginator = getPaginator($row["itemCount"],$_GET["page"],"glowna");
 
	$showPerPage = (int)SHOW_PER_PAGE;
	$showMe = $_GET["page"]*$showPerPage;
	$randAdPlace = rand(2,$showPerPage);
 
	$items = $db->query("SELECT *,ite.id,ite.addTime AS addTime, COUNT(comm.cid) AS commNo FROM items AS ite LEFT OUTER JOIN comments AS comm ON comm.picId=ite.id LEFT JOIN users ON (users.uid=ite.uId) WHERE ite.isWaiting = '0' GROUP BY ite.id ORDER BY ite.addTime DESC LIMIT " . $showMe . ", " . $showPerPage);
 
	$i = 0;
	while ($row = $db->fetch_array($items)) {
		$i++;
 
		$addClass = ($i < $showPerPage) ? ' morespace' : '';
 
		if ($i == $randAdPlace) {
?>
			<div class="contener3">
			<div class="advertItems">
			<?php
echo getGoogleAd('pub-xxxxxxxxxxxxxx','xxxxxxxxxxxx',336,280,'http://strona.pl/rek/kwadrat.html');
?>
			</div>
			</div><br>
<?php
		}
 
		$preLink = '';
		echo '<div class="contener">';
		require("includes/item.inc.php");
		echo '</div><br>';
	}
 
	if($i == 0) {
		echo @TEXT_PAGE_NULL;
	} else {
?>
	<div class="contener2">
	<div class="paginator">
<?php
		echo getGoogleAd('pub-xxxxxxxxxxxxxxx','xxxxxxxxxx',728,90,'http://strona.pl/rek/jd.html');
		echo $paginator;
?>
	</div>
	</div>
<?php
	}
?>
[PHP] pobierz, plaintext 

i mam pytanie czy te zapytania mysql powinny być jakoś kończone/zamykane - $db->close()?

nie. po wykonaniu wszystkich zapytan baza zamknie polaczenie

Zamknie na koniec wykonywania skryptu, a jeżeli skrypt będzie wykonywał się dłużej? Baza też ma ograniczoną liczbę połączeń, które może obsługiwać jednocześnie.

@autor
Jeżeli skończysz z zapytaniami, to zamknij sam połączenie.

Gdzie ustawić to zamykanie połączeń zaraz po "$items = $db->query" ? a może na końcu strony?

Jeżeli wiesz, że nie będziesz wykonywał kolejnego zapytania to przy ostatnim $db->query, aby wcześniej zamknąć połączenie (przed wygenerowaniem reszty strony)i zwolnić zasoby.

a czy nie powinno być zamykane przy każdym? Bo tak to otworzyły się 2 połączenia a zamknięte zostało 1

[PHP] pobierz, plaintext 
	$row = $db->query_first("SELECT COUNT(*) AS itemCount FROM items WHERE isWaiting = '0'");
	mysql_close();
 
	if (!isset($_GET["page"]))
		$_GET["page"] = 0;
 
	$paginator = getPaginator($row["itemCount"],$_GET["page"],"glowna");
 
	$showPerPage = (int)SHOW_PER_PAGE;
	$showMe = $_GET["page"]*$showPerPage;
	$randAdPlace = rand(2,$showPerPage);
 
	$items = $db->query("SELECT *,ite.id,ite.addTime AS addTime, COUNT(comm.cid) AS commNo FROM items AS ite LEFT OUTER JOIN comments AS comm ON comm.picId=ite.id LEFT JOIN users ON (users.uid=ite.uId) WHERE ite.isWaiting = '0' GROUP BY ite.id ORDER BY ite.addTime DESC LIMIT " . $showMe . ", " . $showPerPage);
	mysql_close();
[PHP] pobierz, plaintext 

Nie, to żadne dwa połączenia. Dwa zapytania ale jedno połączenie.

ok dzięki