[php] session geeft niks weer

[hydex]

Ik ben nog een noob als het om php gaat, maar volgens mij is deze code goed. Ik heb alle tutorials goed doorgelezen, maar de waarde uit voeg toe word gewoon niet geprint.

PHP:

<?php


$dbLink = mysql_connect("localhost","140802","*******");

    mysql_select_db('140802');
    $strSqlCode = "SELECT * FROM producten;";
    
    $oResultaat = mysql_query($strSqlCode);


print ("<table width=950 border=0>");
print ("<tr>");
print ("<td>");
print ("<b>");
print "product_id";
print ("</td>");
print ("</b>");

print ("<td>");
print ("<b>");
print "bestel";
print ("</td>");
print ("</b>");




While ($row = mysql_fetch_array($oResultaat))
{

print ("<tr>");
print ("<td>");
print $row['product_id'];
print ("</td>");

session_start(); 

print ("<td>");
print ("<a href='?voegtoe=$row[product_id]'>[img]'images/cart.jpg'[/img]");
session_register("voegtoe");
print $voegtoe;

print ("</td>");
}
print ("</tr>");
print ("</table>");



?>

[PromWarMachine]

Erg vreemde code, om te beginnen. Je laat de HTML door PHP genereren? Dat is niet echt logisch. Je sessie gaat niet goed, omdat session_start() direct na <?php hoort te staan.

Voor meer informatie zou ik http://nl.php.net even bezoeken en kijken wat er te lezen staat over sessies.

Je sessievariabele wordt sowieso niet goed gevuld, want die "voegtoe" is op die manier helemaal niet bekend in je programma. Dit moet zijn:

PHP:

$_SESSION['voegtoe'] = $_GET['voegtoe'];

Session_register kan ook hoor, maar dan moet je wel gebruik maken van $_GET['voegtoe'] in ieder geval.

Op die site die ik net noemde, moet je ook maar lezen over globals

[ Voor 47% gewijzigd door PromWarMachine op 24-05-2004 12:14 ]

[NMe]

Bovendien: zet error_reporting(E_ALL); boven je code, dan was je er al achter wat er mis was voordat je dit topic opende.

PromWarMachine schreef op 24 mei 2004 @ 12:11:
Erg vreemde code, om te beginnen. Je laat de HTML door PHP genereren? Dat is niet echt logisch. Je sessie gaat niet goed, omdat session_start() direct na <?php hoort te staan.

Niet helemaal waar, je moet het alleen aanroepen voordat je iets output naar de browser.

[ Voor 61% gewijzigd door NMe op 24-05-2004 12:13 ]

[Skaah]

PHP:

While ($row = mysql_fetch_array($oResultaat))
{

print ("<tr>");
print ("<td>");
print $row['product_id'];
print ("</td>");

session_start();

Je sessie wordt zo voor iedere rij gestart.

offtopic:
Mijn advies met sessies + php: verspilde moeite. Schrijf je eigen sessie-class met een database, dat werkt altijd

Erg vreemde code, om te beginnen. Je laat de HTML door PHP genereren?

Tsja, je zou ook telkens kunnen inspringen, maar ja, of dat veel mooier is? Templates zijn het mooist, maar die zijn niet altijd nodig / mogelijk / handig / bekend.

[ Voor 33% gewijzigd door Skaah op 24-05-2004 13:08 ]

[NMe]

Skaah schreef op 24 mei 2004 @ 13:03:

offtopic:
Mijn advies met sessies + php: verspilde moeite. Schrijf je eigen sessie-class met een database, dat werkt altijd

Ik heb nooit problemen met de sessies van PHP zelf... Je moet ze gewoon goed gebruiken.

[Skaah]

offtopic:
Ik heb er niets dan ellende mee, verschillen tussen servers, tekstbestandjes die niet gemaakt kunnen worden, sessies die niet starten zonder waarschuwing, etc. etc. Ik heb zelf een sessieclass geschreven met een koekje en een databaserij, en d'r gelijk inloggen en rechten aangekoppeld (role based). En dat werkt áltijd.

[ Voor 5% gewijzigd door Skaah op 24-05-2004 13:12 ]

[NMe]

Om toch even on-topic te blijven:

PHP:

<?php 
session_start(); 

$dbLink = mysql_connect("localhost","140802","*******"); 

mysql_select_db('140802'); 
$strSqlCode = "SELECT * FROM producten;"; 
     
$oResultaat = mysql_query($strSqlCode); 

$voegtoe = $_SESSION["voegtoe"];

?>
<table width="950" border="0">
  <tr> 
    <td>
      <b>product_id</b>
    </td>
    <td>
      <b>bestel</b>
    </td>
  </tr>
<?php
While ($row = mysql_fetch_array($oResultaat)) 
{ 
?>
  <tr> 
    <td><?php echo $row["product_id"]; ?></td>
    <td><a href="?voegtoe=<?php echo $row["product_id"]; ?>">[img]"images/cart.jpg"[/img]<?php echo $_SESSION["voegtoe"]; ?></a></td>
  </tr>
<?php
}
?>
</table>

Het is mij een raadsel wat de sessievar voegtoe moet zijn of doen, maar dit is in ieder geval goede PHP en HTML code, in tegenstelling tot dat van TS.

[ Voor 51% gewijzigd door NMe op 24-05-2004 13:32 ]

[MatHack]

session_register en $_SESSION moet je niet tegelijk gebruiken. Zie ook P&W FAQ - PHP

Overigens vraag ik moet ook af waar de TS $voegtoe vandaan haalt, is dat een $_GET-variabele soms (die wordt meegegeven als querystring)?

[ Voor 42% gewijzigd door MatHack op 24-05-2004 13:32 ]

[mulder]

NMe84 schreef op 24 mei 2004 @ 13:26:

PHP:

While ($row = mysql_fetch_array($oResultaat)) 
{ 
?>
  <tr> 
    <td><?php echo $row["product_id"]; ?></td>
    <td><a href="?voegtoe=<?php echo $row["product_id"]; ?>">[img]"images/cart.jpg"[/img]
<?php echo $_SESSION["voegtoe"]; ?></a></td>
  </tr>
<?php
}
?>
</table>

Het is mij een raadsel wat de sessievar voegtoe moet zijn of doen, maar dit is in ieder geval goede PHP en HTML code, in tegenstelling tot dat van TS.

Ik zou die html in dat loopje lekker echo-en ipv php en html doorelkaar heen te gooien

[ Voor 77% gewijzigd door mulder op 24-05-2004 13:34 ]

[NMe]

MatHack schreef op 24 mei 2004 @ 13:31:
session_register en $_SESSION moet je niet tegelijk gebruiken. Zie ook P&W FAQ - PHP

I know, was een (te) snelle copy paste.

MatHack schreef op 24 mei 2004 @ 13:31:
Overigens vraag ik moet ook af waar de TS $voegtoe vandaan haalt, is dat een $_GET-variabele soms (die worden meegegeven als querystring)?

Ik vermoed dat TS $voegtoe als sessievar bedoelt, omdat hij ook session_register("voegtoe") deed. Daar heb ik mijn vrije vertaling van zijn script dan ook op aangepast.

Don Facundo schreef op 24 mei 2004 @ 13:32:
Ik zou die html in dat loopje lekker echo-en ipv php en html doorelkaar heen te gooien

Dat is persoonlijke voorkeur. Ik vind dit wel lekker werken, scheelt een hoop escapewerk.

[ Voor 19% gewijzigd door NMe op 24-05-2004 13:34 ]

[hydex]

ben er al uit thx voor al jullie reacties.. heb nog wel een vraagje. Het idee achter dit script is dat mensen klikken op een winkelwagen die het id van het product meegeeft in de sessie.. het probleem is nu dat het nu het 1 of het ander is in deze variabele.. Hoe kan ik nu het script zo maken dat elke keer dat er geklikt word er een waarde aan de sessie word toegevoegd zonder dat de ander verloren gaat?

De code is nu werkend:

PHP:

<?php
error_reporting(E_ALL); 
session_start();

$dbLink = mysql_connect("localhost","140802","");

    mysql_select_db('140802');
    $strSqlCode = "SELECT * FROM producten;";
    
    $oResultaat = mysql_query($strSqlCode);


print ("<table width=950 border=0>");
print ("<tr>");
print ("<td>");
print ("<b>");
print "product_id";
print ("</td>");
print ("</b>");

print ("<td>");
print ("<b>");
print "bestel";
print ("</td>");
print ("</b>");




While ($row = mysql_fetch_array($oResultaat))
{

print ("<tr>");
print ("<td>");
print $row['product_id'];
print ("</td>");



print ("<td>");
print ("<a href='?bestel=$row[product_id]'>[img]'images/cart.jpg'[/img]");


print ("</td>");
} //while loop word afgesloten
print ("</tr>");
print ("</table>");
// begin winkelwagen
print ("<br>");
print ("<br>");
$naarwinkelwagen = $_GET['bestel'];
if ($naarwinkelwagen > 0){

print ("inhoud winkelwagen");


session_register ("naarwinkelwagen");
print ("<br>");
print $naarwinkelwagen;
}
else {
print ("uw winkelwagen is leeg");
// einde winkelwagen
}
?>

[ Voor 21% gewijzigd door hydex op 24-05-2004 13:37 ]

[NMe]

hydex schreef op 24 mei 2004 @ 13:36:
ben er al uit thx voor al jullie reacties.. heb nog wel een vraagje. Het idee achter dit script is dat mensen klikken op een winkelwagen die het id van het product meegeeft in de sessie.. het probleem is nu dat het nu het 1 of het ander is in deze variabele.. Hoe kan ik nu het script zo maken dat elke keer dat er geklikt word er een waarde aan de sessie word toegevoegd zonder dat de ander verloren gaat?

Dan moet je arrays gaan gebruiken.

offtopic:
Verder is het misschien handig als je je HTML in de goeie volgorde afsluit. En je sluit bepaalde tags zelfs helemaal niet af. Behalve dat dat niet fijn leest in de code zelf, is het heel erg onvoorspelbaar hoe een browser dat weergeeft. Hou je HTML dus netjes. Kijk bijvoorbeeld eens naar de code die ik hierboven plaatste, en werk van daar uit verder.

[hydex]

durf het bijna niet meer te vragen.. maar naar 3 uur klooien durf ik het toch wel aan..

hoe krijg ik deze waarden in godsnaam in een array die werkt en waardoor de ik meerdere waarden kan weergeven in een session?

Ik vind wel veel informatie over sessions en over arrays maar niet hoe deze zijn te combineren.

[NMe]

Zo: $_SESSION["variabelenaam"]["index"]. Verder werkt het net als een gewone array.

[hydex]

en hoe genereer je dan de index ?

[NMe]

PHP:

$blaat = array(1, 2, 3, 4, 5);
$_SESSION["blaat"] = $blaat;
print_r($_SESSION);

Probeer maar es...

[Verwijderd]

nog even een korte reactie op het echoen. Ik heb hier thuis een 350 MHz bakkie staan en bij het schrijven van 197 rijen met redelijk wat data erin (schermvullend op 1024*768) duurde dat 15 secondes. Op mijn AMD 2500+ ontwikkel machine duurde het ong. een tiende van seconde.

Nadat ik alle echo's eruit gemikt had (deed er ong. 1 per cell) en alles aan elkaar concatte (gewoon nog op de .= manier) duurde het nog maar 1,5 second.

Waarom ik dit hier deel:
In je script staat zelfs een print voor een <TD> tag, zou gewoon uit PerformanceByDesign oogpunt, iets minder echo's gebruiken.

[NMe]

Verwijderd schreef op 24 mei 2004 @ 19:47:
nog even een korte reactie op het echoen. Ik heb hier thuis een 350 MHz bakkie staan en bij het schrijven van 197 rijen met redelijk wat data erin (schermvullend op 1024*768) duurde dat 15 secondes. Op mijn AMD 2500+ ontwikkel machine duurde het ong. een tiende van seconde.

Nadat ik alle echo's eruit gemikt had (deed er ong. 1 per cell) en alles aan elkaar concatte (gewoon nog op de .= manier) duurde het nog maar 1,5 second.

Waarom ik dit hier deel:
In je script staat zelfs een print voor een <TD> tag, zou gewoon uit PerformanceByDesign oogpunt, iets minder echo's gebruiken.

Waarom niet, zoals ik al voorgedaan heb hierboven ergens, helemaal geen echos? Waarom zou je het door PHP laten oplossen als het gewoon zo kan?

[Verwijderd]

NMe84 schreef op 24 mei 2004 @ 19:51:
[...]

Waarom niet, zoals ik al voorgedaan heb hierboven ergens, helemaal geen echos? Waarom zou je het door PHP laten oplossen als het gewoon zo kan?

Uit performance issues. Mijn pagina bestond uit een stuk of 10 kolommen en dus 197 rijen. voor iedere cel (echo ("<td>".$row['adres']."</td>");) en in den beginne ook nog voor de <TR> en </TR> dus 2364 echos.
Een echo is een I/O functie. Zoals ik in mijn post al aangaf duurde dit dus 15 secondes. nadat ik er nog slechts 1 echo van gemaakt had was dit nog maar 10% ervan.

Jouw oplossing is al een stuk beter qua performance, dan het twee keer onder elkaar zetten van een echo("<br>"); om twee keer een enter neer te zetten, maar op langzamere machines is heel veel echoen een killing business.

p.s. paging kan natuurlijk ook nog veel leed verminderen bij lange lijsten, maar waarom een probleem proberen aan te pakken, wat er met een iets andere constructie in de eerste plaats nooit geweest zou zijn

[NMe]

Verwijderd schreef op 24 mei 2004 @ 20:48:
p.s. paging kan natuurlijk ook nog veel leed verminderen bij lange lijsten, maar waarom een probleem proberen aan te pakken, wat er met een iets andere constructie in de eerste plaats nooit geweest zou zijn

Paging is sowieso niet bedoeld om dat probleem op te lossen, maar over het algemeen om bandbreedte te besparen.

Verder hebben webservers echt wel een betere processor dan een PII 350Mhz-tje, dus ook dat zal geen probleem zijn. Anders kun je net zo goed rekening gaan houden met een 286 als server...

[Tito129]

Nu we het toch even over performance hebben. Ik ben bezig met een ek-pooltje online te krijgen. Heb nog nooit echt met php en mysql geklooit maar heb vandaag wat gemaakt. Dit script moet wel op een 486 webserver draaien. Maar heb gemerkt dat het redelijk zwaar is, helemaal door 2 queries in een while loop.
Jullie nog ideetjes..

PHP:

<?php
      error_reporting(E_ALL);
      include("site.php");
      include("menu.php");


  html_header('Tuinland :: Bier', $menu, '');


  $connect = mysql_connect("localhost","ericbl","******")or die("Bad connect string: ".mysql_error());
  mysql_select_db("ericbl",$connect)or die("Bad database change: ".mysql_error());

  $sql = "select land_id_thuis,land_id_uit from pool_uitslagen;";
  $query = mysql_query($sql,$connect) or die(mysql_error());

      ?>
  <table width="100" border="0">
  <tr align="left">
    <th align="center">
      Thuisland
    </th>
    <th align="center">
      Uitland
    </th>
  </tr>

     <?php
While ($row = mysql_fetch_array($query))
{
    $thuisland  = $row["land_id_thuis"];
    $uitland = $row["land_id_uit"];

   //bepalen naam thuisland
    $thuislandquery = "select pl.land from pool_landen pl where pl.land_id = $thuisland;";
    $thuislandexec = mysql_query($thuislandquery,$connect) or die(mysql_error());
    $thuis_result = mysql_fetch_array($thuislandexec);
    $thuis_naam = $thuis_result["land"];

    //bepalen naam uitland
    $uitlandquery = "select pl.land from pool_landen pl where pl.land_id = $uitland;";
    $uitlandexec = mysql_query($uitlandquery,$connect) or die(mysql_error());
    $uit_result = mysql_fetch_array($uitlandexec);
    $uit_naam = $uit_result["land"];
        

    ?>
    <tr align="left">
        <td><?php echo "$thuis_naam" ?></td>
        <td align="center">-</td>
        <td><?php echo "$uit_naam" ?></td>
    </tr>
    <?php
    }
    ?>


</table>

[ Voor 50% gewijzigd door Tito129 op 24-05-2004 22:37 ]

[NMe]

@Tito: open daar liever een nieuw topic voor, en gebruik dan [php] tags. Dit is onleesbaar. En <php> tags zonder vraagtekens bestaan niet.

Ok das een stuk beter. De reden voor die zware load is hierboven al eens genoemd. Probeer alles apart in een string te schrijven, en dan aan het einde in één keer te echo-en.

[ Voor 42% gewijzigd door NMe op 24-05-2004 22:39 ]

[Tito129]

NMe84 schreef op 24 mei 2004 @ 22:36:
@Tito: open daar liever een nieuw topic voor, en gebruik dan [php] tags. Dit is onleesbaar. En <php> tags zonder vraagtekens bestaan niet.

Ok das een stuk beter. De reden voor die zware load is hierboven al eens genoemd. Probeer alles apart in een string te schrijven, en dan aan het einde in één keer te echo-en.

Hmm jah, lijkt me zo en zo een goed idee. Misschien optimaliseren van de query's maar daar al een poosje over zitten denken, maar niks nuttigs uitgekomen.

[Verwijderd]

Tito schreef op 24 mei 2004 @ 22:33:
Nu we het toch even over performance hebben. Ik ben bezig met een ek-pooltje online te krijgen. Heb nog nooit echt met php en mysql geklooit maar heb vandaag wat gemaakt. Dit script moet wel op een 486 webserver draaien. Maar heb gemerkt dat het redelijk zwaar is, helemaal door 2 queries in een while loop.
Jullie nog ideetjes..

Het eerste wat mij opviel, is (vanwege performance oogpunt) wat er hierboven ook al staat en door mij is aangezwengeld, maak 1 variabele, concat je output en 1 echo.

Verder zou ik even een beetje zelfstudie doen naar het fenomeen "inner join"

dat zou je twee queries in je while loop tot het verleden moeten doen behoren.

GL

[Tito129]

Verwijderd schreef op 25 mei 2004 @ 17:38:
[...]


Het eerste wat mij opviel, is (vanwege performance oogpunt) wat er hierboven ook al staat en door mij is aangezwengeld, maak 1 variabele, concat je output en 1 echo.

Verder zou ik even een beetje zelfstudie doen naar het fenomeen "inner join"

dat zou je twee queries in je while loop tot het verleden moeten doen behoren.

GL

Mja daar heb ik idd al naar gekeken maar kwam er niet echt uit, aangezien ik 2 lijsten er uit moet krijgen.
met

code:

 select pl.land from pool_landen pl,pool_uitslagen pu 
where pl.land_id = pu.land_id_thuis;

krijg ik er wel 1 lijst met landen uit, maar moet ik voor de land_id_uit nog steeds een extra query met een while loop doen.

Mijn db structuur is:
pool_landen -> land , land_id;
Pool_uitslagen -> land_id_thuis ,land_id_uit,datum;

[Verwijderd]

pool_landen -> land , land_id;
Pool_uitslagen -> land_id_thuis ,land_id_uit,datum

code:

select landthuis.land, landuit.land, uitslag.*
from pool_uitslagen uitslag
inner join pool_landen landthuis on uitslag.land_id_thuis = landthuis.land_id
inner join pool_landen landuit on uitslag.land_id_uit = landuit.land_id

dit zou moeten werken op een sqlserver 2000, maar volgens mij ook wel op een mySql.

ow let ook ff op je tabelnamen. je kan instellen dat ze case sensitive zijn, maar moet default wel goed staan

[ Voor 3% gewijzigd door Verwijderd op 25-05-2004 20:47 ]

[Tito129]

Verwijderd schreef op 25 mei 2004 @ 20:37:
pool_landen -> land , land_id;
Pool_uitslagen -> land_id_thuis ,land_id_uit,datum

code:

select landthuis.land, landuit.land, uitslag.*
from pool_uitslagen uitslag
inner join pool_landen landthuis on uitslag.land_id_thuis = landthuis.land_id
inner join pool_landen landuit on uitslag.land_id_uit = landuit.land_id

dit zou moeten werken op een sqlserver 2000, maar volgens mij ook wel op een mySql.

ow let ook ff op je tabelnamen. je kan instellen dat ze case sensitive zijn, maar moet default wel goed staan

wow jij snapt het. Thx
met een group by wedstrijd_id, staat ook alles nog is netjes op volgorde.
direct me script ff ombouwen. Ik voel me best wel is

[Verwijderd]

Tito schreef op 25 mei 2004 @ 21:02:
[...]


wow jij snapt het. Thx
met een group by wedstrijd_id, staat ook alles nog is netjes op volgorde.
direct me script ff ombouwen. Ik voel me best wel is

ik mag hopen dat je order by bedoeld voor sorteren.....

"order by poule, wedstrijddatum" zou ook leuke dingen moeten opleveren voor de voorrondes.