Onderwerp: simpel php vraagje

Oorspronkelijk geplaatst door reef

Nu we toch bezig zijn, nog even 1 belangrijk ding: mysql_close

Op 1 pagina doe ik wel eens een stuk of 5 verschillende queries. In ColdFusion hoefde ik nooit de connectie te sluiten. Maar met PHP moet dat wel lees ik op verschillende tutorials. Is het verstandig om helemaal onderaan elke PHP-pagina de connectie met mysql_close te sluiten? Want ik kan me voorstellen dat het 5x openen en sluiten per pagina het doel voorbijstreeft i.v.m. de overhead die dat kost?

Het is niet meer dan netjes, maar bij het beeindigen van het script wordt de connectie sowieso geclosed. Je doet over het algemeen 1 x een connect/open, n x een query en dan optioneel een close.
In bepaalde gevallen (slave dbservers, meerdere database servers, meerdere users) kun je gemakkelijk 5 connects hebben, maar dat zie je niet vaak.

ik heb er geen harde specs over, maar ik zelf sluit de connectie altijd.

@John, ja klopt al heb ik hier ergens gelezen dat dat ook een serverinstelling is en dus niet binnen 1 seconde automatisch gebeurt.

Zover ik weet wordt het direct bij het einde van het uitvoeren automatisch gesloten (tenzij dus gebruik wordt gemaakt van pconnect)

Boys, niet zo moeilijk doen, de uitleg staat haarfijn op php.net:

Using mysql_close() isn't usually necessary, as non-persistent open links are automatically closed at the end of the script's execution.

Je maakt persistent connecties met mysql_pconnect, mysql_connect doet dat niet, en hoef je het dus niet af te sluiten volgens de php documentatie.

Kijk Daniel, ik heb gezien dat het meer dan significant in de laadsnelheid scheelt als je het afsluit ( bij de gewone connect ).

Ik zoek nu die discussie hier even op.

Maar documentatie is verreweg van heilig

update: gevonden:
http://www.webhostingtalk.nl/technis...ch-closen.html (mysql automatisch closen)

Oorspronkelijk geplaatst door Ramon Fincken

ik heb er geen harde specs over, maar ik zelf sluit de connectie altijd.

@John, ja klopt al heb ik hier ergens gelezen dat dat ook een serverinstelling is en dus niet binnen 1 seconde automatisch gebeurt.

True, pconnect() is in bepaalde gevallen handig en performance verhogend, maar je hebt idd. te maken met serverinstellingen, timeouts e.d.
Maar een close() wordt automatisch gedaan dus echt nodig is het niet. Al ben ik wel zeer voorstander van 'Wat je opent, dat sluit je ook'. MySQL is hier redelijk tolerant in, maar doe dit niet met gewone filehandles

Nog een vraag voor een variant op mijn eerste topic:
Stel, ik loop over de results van een tabel met artikelen. Dan wil ik tijdens de loop per artikel ook een query doen om de eerste foto van het artikel op te halen uit een tussentabel (of eventueel alle foto's met een extra loop). Ik dacht, ik doe het als volgt maar de variabele $row['id'] wordt niet geaccepteerd in de fotoresult query.

Code:

<?php
$result = mysql_query("SELECT id,naam FROM artikel");
while($row = mysql_fetch_array($result))
  {
  echo $row['id'] . " " . $row['naam'];
  
  $fotoresult = mysql_query("SELECT foto FROM artikelfoto WHERE artikel_idfk='$row['id']'");  
  $fotorow = mysql_fetch_array($fotoresult);
  echo $fotorow['foto'];
  }
?>

Heb ik ergens een syntaxfout gemaakt of kan dit niet zo in PHP? Ik ben zoals gezegd ColdFusion gewend en daarbij werkt het probleemloos op deze manier.

aangenomen dat ..
artikel_idfk gelijk is aan een categorie ( noem het liever cat_id ) en je een enkele foto wil hebben...

$sql_foto = "SELECT foto FROM artikelfoto WHERE foto != '' AND artikel_idfk='".$row['id']."' LIMIT 1";
echo dit eens en gooi het in phpmyadmin, kijken wat ie je teruggeeft ?

Oorspronkelijk geplaatst door Ramon Fincken

aangenomen dat ..
artikel_idfk gelijk is aan een categorie ( noem het liever cat_id ) en je een enkele foto wil hebben...

$sql_foto = "SELECT foto FROM artikelfoto WHERE foto != '' AND artikel_idfk='".$row['id']."' LIMIT 1";
echo dit eens en gooi het in phpmyadmin, kijken wat ie je teruggeeft ?

In phpmyadmin krijg je dan een foutmelding
Maar in denk dat je dit wil hebben:
$fotoresult = mysql_query("SELECT foto FROM artikelfoto WHERE artikel_idfk='".$row['id']."'");

leuke "laatst gewijzigd"

Thanks, dat tussen extra ". ." zetten was het!

Joepie eerste pagina is vertaald van ColdFusion naar PHP. Nu de rest nog

Mijn dank is groot voor deze beginnerscursus mannen!

Ramon, thanks voor de link. Ik maak er uit op dat het niet echt uitmaakt voor de performance van MySQL zelf, wel voor het aantal connecties dat open kan blijven staan. Meer een kwestie van een juiste configuratie, maar ik moet toegeven, dat van die timeout wist ik niet

Ik ben stiekem wel voorstander van een topic met zulke puntjes, waar men kan discussieren over dit soort punten welke niet onder beginnersvragen vallen of basiskennis zijn, maar wel impact kunnen hebben op een PHP script. Op het tweakers.net forum is ook al zo'n topic, maar dan hebben we het over PHP, C, Perl, Python, noem het maar, alles door elkaar. Een topic voor PHP hierover op WHT ben ik wel voor in.

Oorspronkelijk geplaatst door D. ter Horst

...
Een topic voor PHP hierover op WHT ben ik wel voor in.

Ik ook!