[PHP] Username zoeken

[Verwijderd]

Probleempje...
Misschien is het heel simpel, maar ik kom er niet meer uit. Zo'n beetje alles gelezen wat er te lezen valt, maar helaas.

Ik ben bezig met een forumpje te bouwen in met PHP/MySQL. Nu wil ik bij het tonen van een topic, natuurlijk ook de username laten zien van degene die post.

Tot zover allemaal prima, voor mijn idee is de structuur zoals ik em nu heb ook in orde, maar die naam krijg ik maar niet in beeld.

Ik zal alleen de relevante tabbelen en code hier posten:

Tabellen

forum_reply:
- reply_id
- reply_body
- reply_user (hier staat dus het user-id in van de reply-er)

forum_user
- user_id
- user_name

PHP:

//Opzoeken van juiste replies
$qry_reply = "SELECT * FROM forum_reply where reply_topic = '$topicid'";
$query2 = mysql_query($qry_reply)
     or die("Fout bij uitvoeren query");

//loop om replies te echoen
while ($reply = mysql_fetch_array($query2)) {

//Zoek username bij userid
      $userid = $reply['reply_user'];
      $qry_user = "SELECT * FROM forum_user where user_name = '$userid'";
      $query3 = mysql_query($qry_user)
              or die("Fout bij uitvoeren query");

// cel voor username
      echo "\t<td><br />";

//echo username
      while ($user = mysql_fetch_array($query3)) {
              echo $user['user_name'];
      }

//echo de rest

}

Hij laat nu dus alles zien, behalve de username. Het lijkt dus alsof ie nooit die 2e while-loop ingaat. $userid wordt wel gevuld.

Ziet iemand wat ik verkeerd doe? Of kan iemand me een stukje op weghelpen?

[Helmet]

Dit kan toch met innerjoins? dat bespaart je een query

edit: eens ff voor je opzoeken
woei: Lees dit eens

[ Voor 56% gewijzigd door Helmet op 26-06-2003 11:38 ]

[OxiMoron]

waarom innerjoin doen? als het goed is heb je geen null velden.. dus een gewone select van 2 tabellen zou genoeg moeten zijn:

PHP:

<?
//Opzoeken van juiste replies met users
$qry_reply = "SELECT * FROM forum_reply r, forum_user u WHERE r.reply_user = u.user_id AND r.reply_topic = '$topicid'"; 
$query2 = mysql_query($qry_reply) 
     or die("Fout bij uitvoeren query"); 

//loop om replies te echoen 
while ($reply = mysql_fetch_array($query2)) { 

// cel voor username 
      echo "\t<td><br />"; 
//echo username 
      while ($user = mysql_fetch_array($query3)) { 
              echo $user['user_name']; 
      } 

//echo de rest 

}
?> 

dit scheeld je ook nog eens gigantisch veel query's.. aangezien je nu met 1 query all reply's met users krijg ipv 1 query voor alle users en dan nog een query voor elke reply

[Verwijderd]

OxiMoron schreef op 26 June 2003 @ 11:46:
waarom innerjoin doen? als het goed is heb je geen null velden.. dus een gewone select van 2 tabellen zou genoeg moeten zijn:

En wat denk jij dat een inner join doet? Inderdaad, dat wat jij zegt

[Janoz]

En dat je orginele query niet werkt komt waarschijnlijk omdat je niet goed oplet . id is immers niet name..

Maar gooi je code zo snel mogenlijk weg en gebruik die van OxiMoron. Je wil immers niet voor een thread met 100 reacties 101 queries uitvoeren terwijl het ook gewoon in 1 kan.

Een database is gemaakt om gegevens met hun relatie op te slaan en makkelijk toegankelijk te maken. Gebruik het dan ook en probeer die functionaliteit niet in je eigen code te implementeren, de DB zal hier namelijk in bijan alle gevallen sneller in zijn.

[Verwijderd]

Woei! bedankt

Stom van me dat ik daar nooit aan gedacht heb. De structuur was toch niet helemaal goed

Het werkt nu zoals OxiMoron zei (er zijn inderdaad geen NULL-velden)

[Janoz]

Verwijderd schreef op 26 juni 2003 @ 11:48:
[...]
En wat denk jij dat een inner join doet? Inderdaad, dat wat jij zegt

Een inner join doet meer.. Outer join doet hetzelfde als de query van OxiMoron

[Verwijderd]

Janoz schreef op 26 June 2003 @ 11:51:
[...]


Een inner join doet meer.. Outer join doet hetzelfde als de query van OxiMoron

Inderdaad, ik bedacht me net, zat aan outer te denken maar zit te lullen over inner Doh!

[Verwijderd]

Janoz schreef op 26 June 2003 @ 11:49:
En dat je orginele query niet werkt komt waarschijnlijk omdat je niet goed oplet . id is immers niet name..

Dat was em inderdaad Dat voor zoiets stoms uren kan staren) Nog ff geprobeerd met user_id = '$userid' en toen werke het wel.

Sorry voor de nutteloze vulling van de database

[OxiMoron]

Verwijderd schreef op 26 June 2003 @ 11:54:
[...]


Dat was em inderdaad Dat voor zoiets stoms uren kan staren) Nog ff geprobeerd met user_id = '$userid' en toen werke het wel.

Sorry voor de nutteloze vulling van de database

lol.. idd.. dat was het probleem van je originele code