simpel php vraagje

[reef]

Ik heb een vraagje over het ophalen van een record d.m.v. een PHP query.

Op de volgende manier gaat het wel, maar hij lijkt me wat omslachtig omdat er toch maar 1 waarde in de database zit (hij haalt slechts de waarde van de naam op van het record met 1 bepaald uniek id):

<?php $huidigecat = mysql_query("SELECT naam FROM categorie WHERE cat_id='$cid'"); ?>

<?php 
while($row = mysql_fetch_array($huidigecat))
  {
  echo $row['naam'];
  }
?>

Waarschijnlijk is het heel makkelijk maar ik ben aan het overstappen van ColdFusion en ASP naar PHP en ik kan het niet uitstaan dat het mij niet lukt. Met ColdFusion was bijvoorbeeld deze variabele #huidigecat.naam# direct geldig om de naam te krijgen. Met PHP zou deze variabele dan $huidigecat.naam moeten heten, maar dat werkt dus niet. Weet iemand hoe je deze variabele in PHP schrijft, of zit ik toch altijd aan die while loop vast?

[eXtreme Service]

<?php 
while($row = mysql_fetch_array($huidigecat))
  {
  echo $row['naam'];
  }
?>

Weet niet echt wat je wil bereiken maar misschien hetvolgende ?

<?php 
$row = mysql_fetch_array($huidigecat);
echo $row['naam'];
?>

Dan zit je niet aan die while loop vast.

Is dit wat je wilt hebben ?

[Ramon Fincken]

als je een LIMIT 1 toevoegt ( maar ook zonder ) kun je je while door een if veranderen, of inderdaad
$row = mysql_fetch_array($huidigecat);
doen

<?php 
$sql_huidigecat = "SELECT naam FROM categorie WHERE cat_id='".intval($cid). "' LIMIT 1";
// TODO: gaan we de DIE nog aanpassen?
$result_huidigecat = mysql_query($sql_huidigecat) or DIE(); 

if(mysql_num_rows($result_huidigecat) > 0)
{
   $row_huidigecat = mysql_fetch_array($result_huidigecat);
   echo $row_huidigecat['naam'];
}
else
{
   echo 'Deze categorie bestaat niet meer';
  // TODO: redirect naar homepage?
}
?>

[reef]

Aha, okee bedankt. mysql_fetch_array heb je dus altijd nodig om iets uit een query te bekijken.

En dat laatste voorbeeld zal ook goed van pas gaan komen.

[t.bloo]

Je krijgt van de query een result handle terug. Misschien handiger om die ook zo te noemen dan. Je moet altijd een tweede functie aanroepen om iets met die handle te doen.

[D. ter Horst]

Of nog beter:

$query = mysql_query("SELECT naam FROM categorie WHERE cat_id=" . intval($cid));
$resultaat = msql_result($query, 0, 'naam');

echo $resultaat; 

[dreamhost_nl]

Ik mis toch wel wat :

$query = mysql_query('SELECT naam FROM categorie WHERE cat_id='.(int)$cid.' limit 0,1',$link);
//foutafhandling query komt hier
$val = mysql_result($query, 0, 0); // eerste waarde
echo $val; 

Vooral de link naar de DB zou ik niet vergeten. Als je een tweede link krijgt, zie je anders door de bomen het bos niet meer.

[D. ter Horst]

Ik mis toch wel wat :

$query = mysql_query('SELECT naam FROM categorie WHERE cat_id='.(int)$cid.' limit 0,1',$link);
//foutafhandling query komt hier
$val = mysql_result($query, 0, 0); // eerste waarde
echo $val; 

Vooral de link naar de DB zou ik niet vergeten. Als je een tweede link krijgt, zie je anders door de bomen het bos niet meer.

Het lijkt me toch niet de bedoeling dat we een compleet script gingen schrijven? Daarnaast heeft hij het over 1 record dat bijgewerkt moet worden, foutcontrole kan maar is geen vereiste hier. Daarnaast hoeft de link niet meegegeven te worden wanneer men maar 1 database gebruikt, en werkt 0,0 ipv 0, 'veldnaam' net zo goed. Ook het getal als (int) casten is een slechtere oplossing dan intval gebruiken. Enige punt wat discutabel is de foutafhandeling wat mij betreft.

[Ramon Fincken]

@daniel, ok werkt ook, maar vind ik persoonlijk lelijk, al was het maar dat je de query direct in een mysql_query gooit

@dreamhost: waarom eerste waarde?, ok we hebben nu een select naam ... maar stel we veranderen wat in de DB tabel structure ... dan wil je dit niet zo doen en al zeker niet als je meer wil selecten dan enkel een naam.

[D. ter Horst]

Ramon,

Is een kwestie van smaak. Technisch gezien maakt het totaal geen verschil of je het nou wel of niet in een variabele zet en meegeeft aan mysq_query. Query's worden in de meeste gevallen niet aangepast, en heeft het dus geen zin om dubbelop te doen.

Overigens, door het in een variable te zetten reserveer je geheugen. Ook al is dit in dit geval een minimale hoeveelheid, in een loop kan dit wel degelijk een verschil maken voor de performance. Dan praten we wel over micro-optimalisatie, maar waarom zou je iets 'fout' aanleren.

Ik zou variabelen gewoon gebruiken waar ze voor zijn, en dat is voor variabele waarden

[Ramon Fincken]

ja klopt helemaal, maar voor debugging bijvoorbeeld is het net effe lekkerder om een echo $sql; te doen voor je $result=mysql_query($sql); dan zitten klooien in die ene regel. En ook dat is zeker een kwestie van smaak

Vergeet niet dat TS een beginner is

[reef]

Wel leerzaam dit allemaal.

[D. ter Horst]

ja klopt helemaal, maar voor debugging bijvoorbeeld is het net effe lekkerder om een echo $sql; te doen voor je $result=mysql_query($sql); dan zitten klooien in die ene regel. En ook dat is zeker een kwestie van smaak

Vergeet niet dat TS een beginner is

Daar heb je een punt Al zou ik persoonlijk in een applicatie nooit direct mysql_query aanroepen voor een query als het aan mij lag, maar een Database Abstraction Layer implementeren.

We weten beiden wel waar we over lullen, en beide oplossingen zijn goed. Ik denk dat de TS het idee wel krijgt :P Mijn reactie was in eerste instantie ook niet op jou gericht, maar op Dreamhost_NL, welke wijzigingen in mijn code aangaf welke (op 1 na) niet correct waren.

[Ramon Fincken]

amen

[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?