[PHP]Newbie vraag

[Verwijderd]

Doordat ik nog niet zolang met php werk, krijg ik het volgende niet voor elkaar. Ik heb een lijst met daar in alle regisseuren die in een database staan. Nu was mijn bedoeling om via die lijst alle dvd's weer te gegeven die in de database staan. In mijn list.php staat het volgende:

PHP:

$list_result = mysql_query("SELECT DISTINCT regisseur FROM dvd ORDER BY regisseur ASC");
$regisseur = mysql_fetch_array($list_result);
    
do{
echo "<a href=\"..\beta.php?info=1&content=$regisseur[regisseur]\">";
echo $regisseur["regisseur"];
echo "</a>";
echo "<br>";
} while ($regisseur = mysql_fetch_array($list_result));

Zoals jullie kunnen zien wordt hiermee mijn beta.php aangeroepen. Zoals de naam al zegt is dit een beta dus staat er nog niet echt veel informatie in over de dvd's zelf, maar dit staat er in:

PHP:

<?php
include ("config.php");

$info = $_GET['info'];

if ($info == '1'){

mysql_select_db("$db_naam");
$list_result = mysql_query("SELECT * FROM dvd WHERE regisseur LIKE '%content%' ORDER BY titel ASC");
$regisseur = mysql_fetch_array($list_result);

echo $regisseur["titel"];
}

?>

Nu het probleem, ik krijg op mijn beta.php helemaal niets te zien en dat terwijl ik aangeeft dat hij de titels van de dvd's moet weergeven.
Wat doe ik fout.

P.S. Modjes, ik kon geen juiste titel bedenken, dus mochten jullie het er niet mee eens zijn pas het dan graag aan....

[blizt]

Gaat je query wel goed?
Doe 'ns:

PHP:

if(!$list_result)
{
echo mysql_error();
}

else
{
//doe code
}

edit:

PHP:

$list_result = mysql_query("SELECT * FROM dvd WHERE regisseur LIKE '%content%' ORDER BY titel ASC");

%content%? moet dat niet %$_GET['content']% zijn oid?

[ Voor 45% gewijzigd door blizt op 04-05-2004 17:42 ]

[Verwijderd]

blizt schreef op 04 mei 2004 @ 17:39:
Gaat je query wel goed?
Doe 'ns:

PHP:

if(!$list_result)
{
echo mysql_error();
}

else
{
//doe code
}

Wanneer ik dat aanpas in me script krijg ik geen foutmelding. Dus mag er nu van uitgaan dat het wel goed zit met me query.

hehehe te vroeg, zal het nu eens in me beta aanpassen

Na het aanpassen van me beta.php krijg ik geen foutmelding, maar geeft hij ook mijn titels niet weer

[ Voor 22% gewijzigd door Verwijderd op 04-05-2004 17:44 ]

[Spooksel]

Je haalt $content zo te zien niet op uit de URL?

PHP:

$content = $_GET['content'];

En dan je sql:

PHP:

$sql = "SELECT * FROM dvd WHERE regisseur LIKE '". $content ."' ORDER BY titel ASC";
$list_result = mysql_query($sql);

[Verwijderd]

Thnx spooksel, het werkt. Ik was in de veronderstelling dat content ook gelijk met info werd opgehaald....

[blizt]

Dat zei ik dus ook al in m'n edit hè...

[Spooksel]

$_GET['var'] direct gebruiken voor zoiets heb ik al eens problemen mee gehad in het verleden, vandaar dat ik het eerst in een 'gewone' variabele zet.

[gorgi_19]

The_One
Kan je volgende keer een duidelijke titel kiezen? "Newbie vraag" zegt niet zo veel over de inhoud. Zie ook: *!*!* Over topictitels in P&W - Updated 25 feb 04 *!*!*

[blizt]

Spooksel schreef op 04 mei 2004 @ 18:00:
$_GET['var'] direct gebruiken voor zoiets heb ik al eens problemen mee gehad in het verleden, vandaar dat ik het eerst in een 'gewone' variabele zet.

Je moet natuurlijk ook wel iets van htmlentities, strip_tags oid doen hè
Maar je hoeft 't echt niet eerst in een var te douwen ... hoe kom je aan dat sprookje?

[Skaah]

Spooksel schreef op 04 mei 2004 @ 18:00:
$_GET['var'] direct gebruiken voor zoiets heb ik al eens problemen mee gehad in het verleden, vandaar dat ik het eerst in een 'gewone' variabele zet.

Ja, dit is een heel verschil!

PHP:

$info = $_GET['info']
mysql_query('SELECT * FROM tabel WHERE column="'.$info.'"');
// of
mysql_query('SELECT * FROM tabel WHERE column="'.$_GET['info'].'"');

Je bedoelt iets als

PHP:

$info = mysql_real_escape_string(stripslashes($_GET['info']));
mysql_query('SELECT * FROM tabel WHERE column="'.$info.'"');

Blizt: dat soort dingen doe je bij de output. Een database is om informatie op te slaan, niet om het op te maken.

[ Voor 14% gewijzigd door Skaah op 04-05-2004 21:01 . Reden: stripslashes toegevoegd. ]

[blizt]

Skaah: SQL-injection? Als jij gewoon alles toelaat als $info, .... Nja, dat weet je toch wel?

[NMe]

blizt schreef op 04 mei 2004 @ 20:31:
Skaah: SQL-injection? Als jij gewoon alles toelaat als $info, .... Nja, dat weet je toch wel?

Met mysql_real_escape_string ben je beveiligd tegen injection...

[Skaah]

blizt schreef op 04 mei 2004 @ 20:31:
Skaah: SQL-injection? Als jij gewoon alles toelaat als $info, .... Nja, dat weet je toch wel?

Ik bedoel dingen als htmlentities() e.d. Stripstags hoor je niet te gebruiken imho bij het opslaan, wat nu als iemand iets als

<piet> hoi
<jan>ook hoi

in je database wil stoppen? Htmlentities bij het tonen voorkomt XSS (Cross-Site-Scripting).

[ Voor 33% gewijzigd door Skaah op 04-05-2004 21:04 ]

[DanceTrend]

Het is al opgelost enzo volgens mij maar ik zeg het toch. In deze regel:

PHP:

$list_result = mysql_query("SELECT * FROM dvd WHERE regisseur LIKE '%content%' ORDER BY titel ASC");

staat %content% en dus NIET %$content%

[ Voor 7% gewijzigd door DanceTrend op 04-05-2004 21:05 ]

[NMe]

DanceTrend schreef op 04 mei 2004 @ 21:03:
Het is al opgelost enzo volgens mij maar ik zeg het toch. In deze regel:

code:

$list_result = mysql_query("SELECT * FROM dvd WHERE regisseur LIKE '%content%' ORDER BY titel ASC");

staat %content% en dus NIET %$content%

Waarom post je een oplossing die ten eerste al aan bod is geweest en ten tweede fout is i.v.m. de standaard instelling van superglobals?

[blizt]

@DanceTrend:

blizt schreef op 04 mei 2004 @ 17:39:
[...]
edit:

PHP:

$list_result = mysql_query("SELECT * FROM dvd WHERE regisseur LIKE '%content%' ORDER BY titel ASC");

%content%? moet dat niet %$_GET['content']% zijn oid?

Skaah schreef op 04 mei 2004 @ 21:00:
[...]

Ik bedoel dingen als htmlentities() e.d. Stripstags hoor je niet te gebruiken imho bij het opslaan, wat nu als iemand iets als
.

Wie heeft 't hier over opslaan? Met een select sla ik niets op hoor...

[ Voor 5% gewijzigd door blizt op 04-05-2004 21:06 . Reden: Typo :( ]

[DanceTrend]

NMe84 schreef op 04 mei 2004 @ 21:05:
[...]

Waarom post je een oplossing die ten eerste al aan bod is geweest en ten tweede fout is i.v.m. de standaard instelling van superglobals?

Omdat er niemand is die daar over begon volgens mij Maar das dus wel zo ik had er overheen gelezen D8 dat ik t hele topic goed had gelezen.

[Skaah]

DanceTrend schreef op 04 mei 2004 @ 21:03:
Het is al opgelost enzo volgens mij maar ik zeg het toch. In deze regel:

code:

$list_result = mysql_query("SELECT * FROM dvd WHERE 
regisseur LIKE '%content%' ORDER BY titel ASC");

staat %content% en dus NIET %$content%

Zo kies je alle rijen uit je tabel waarbij het veld 'regisseur' het woord 'content' bevat (dus: 'contents, '[e-mail]'). Of jij formuleert vreemd, of ik lees slecht. Ligt 'm vast aan mij.

PHP:

include ('config.php'); 

$content = mysql_real_escape_string(stripslashes($_GET['content']));

if (isset($_GET['info']) && $_GET['info'] == '1')
{ 
  mysql_select_db($db_naam);  // deze regel hoort eigenlijk in je config.php
  $list_result = mysql_query('SELECT * FROM dvd WHERE regisseur LIKE "%'.$content.'%" ORDER BY titel'); 
  while ($regisseur = mysql_fetch_array($list_result))
  {
    echo $regisseur['titel'].'<br />'; 
  }
}

blizt schreef op 04 mei 2004 @ 21:05:
@DanceTrend:

[...]


[...]

Wie heeft 't hier over opslaan? Met een select sla ik niets op hoor...

Waarom gebruik je dan striptags op een SELECT?

[ Voor 34% gewijzigd door Skaah op 04-05-2004 21:20 ]

[blizt]

Misschien een domme vraag Skaah, maar waarom moet $_GET['info'] volgens jou de waarde 1 bevatten?

[Skaah]

blizt schreef op 04 mei 2004 @ 21:11:
Misschien een domme vraag Skaah, maar waarom moet $_GET['info'] volgens jou de waarde 1 bevatten?

Verwijderd schreef op 04 mei 2004 @ 17:34:
[...]

PHP:

<?php
include ("config.php");

$info = $_GET['info'];

if ($info == '1'){
//[...]

[...]

[blizt]

Ah, niet gezien.... My bad.
Over die strip_tags: ook een select-query is toch zo te veranderen dat ie andere dingen doet dan jij wilt?

offtopic:
4e keer dat ik dit probeer te posten, hopen dat 't nu lukt

[Skaah]

blizt schreef op 04 mei 2004 @ 21:19:
Ah, niet gezien.... My bad.
Over die strip_tags: ook een select-query is toch zo te veranderen dat ie andere dingen doet dan jij wilt?

offtopic:
4e keer dat ik dit probeer te posten, hopen dat 't nu lukt

Jazeker, maar strip_tags is ervoor om htmltags zoals <b>, <script>, <body>, <br />, </em> en dergelijke uit een string te halen. Je verwart de functie misschien met add- en stripslashes()?

(zonder underscore, met underscore, gedoe man)

[blizt]

Argh, je hebt gelijk. Again: my bad.... Had niet goed nagedacht over wat ik gebruikte.
Toch goed dat er meer mensen op dit forum zijn, die je soms verbeteren