Forum » Server-Side scripting » Link aanklikbaar maken werkt niet
NJMedia
Berichten: 1146
avatar
Offline Stuur privebericht
Beste CP'ers,

Ik heb wat gegevens opgeroepen uit de database en nu wil ik deze graag aanklikbaar maken. Alleen krijg ik een rare fout, namelijk:
Quote

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use



Dit is de code waar het om gaat:
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
<h3> Hits </h3>
<?php
$query 
mysql_query("SELECT * FROM radio WHERE cat = 1");
while(
$hits mysql_fetch_assoc($query)) 
                    {  
echo 
"<ul><li><a onclick=\"luister('" $query['titel'] . "')\">" $hits['titel'] . "</a></ul></li> ";
                    } 
?>


Ik hoop dat iemand me kan helpen.

Met vriendelijke groet
Nillas
15-12-2010 23:22
Dit topic is 204 keer bekeken door 42 verschillende leden
Reacties op: "Link aanklikbaar maken werkt niet"
1 | 2 Volgende pagina
Logo Designer
Berichten: 1366
avatar
Offline Stuur privébericht
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
<h3> Hits </h3> 
<?php 
$query 
mysql_query("SELECT * FROM radio WHERE cat = "1"); 
while(
$hits = mysql_fetch_assoc($query))  
                    {   
echo "
<ul><li><a onclick=\"luister('" $query['titel'] . "')\">" $hits['titel'] . "</a></ul></li> "
                    }  
?>
15-12-2010 23:29
Hardstyle
Berichten: 225
avatar
Offline Stuur privébericht
vergeet niet de backslashes \

href=\"#\"
15-12-2010 23:30
Logo Designer
Berichten: 1366
avatar
Offline Stuur privébericht
Nee jackie zie je wat TS gaf de code zie dit code stukje:
Code | Selecteer Alles
minimaliseren
1
$query mysql_query("SELECT * FROM radio WHERE cat = 1");


Moet dit zijn:

Code | Selecteer Alles
minimaliseren
1
$query mysql_query("SELECT * FROM radio WHERE cat = "1");


Waarom: zie 1" achteraan moet "1" zijn lijkt me.

Als dit fout is ik wou je graag helpen maar accu is leeg, dus doe m'n laptop nu uit en opladen maar.
15-12-2010 23:33
NJMedia
Berichten: 1146
avatar
Offline Stuur privébericht
@A.bijlsma:

Code | Selecteer Alles
minimaliseren
1
Parse errorsyntax errorunexpected T_CONSTANT_ENCAPSED_STRINGexpecting ',' or ';'

Dit is wat ik krijg als ik jouw code gebruik.

Nieuwe reactie samengevoegd met originele reactie op 15.12.10 23:39:10:
@A.Bijlsma: Nee want dan krijg ik weer een andere error dat het nummer niet goed is.
15-12-2010 23:35
webbasing.com
Berichten: 593
avatar
Offline Stuur privébericht
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
<h3> Hits </h3>
<?php
$query 
mysql_query("SELECT * FROM radio WHERE cat ='1'");
while(
$hits mysql_fetch_assoc($query)) 
{  
      echo 
"<ul><li><a onclick=\"luister('" $query['titel'] . "')\">" $hits['titel'] . "</a></ul></li> ";

?>
16-12-2010 01:37
NJMedia
Berichten: 1146
avatar
Offline Stuur privébericht
Helaas, ik krijg dezelfde error.
Maar het ligt ook niet aan de query denk ik. Want die query doet het bij de rest wel gewoon. Kijk anders zelf even:
Klik en dan rechts bij 'all stations'.

Nillas
16-12-2010 09:08
Berichten: 552
avatar
Offline Stuur privébericht
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
8
<h3> Hits </h3>
<?php
$query 
mysql_query("SELECT * FROM `radio` WHERE `cat`=1");
while(
$hits mysql_fetch_array($query)) 
    {  
        echo 
'<ul><li><a onclick="luister("'$hits['titel'] .'")">' $hits['titel'] . '</a></ul></li>';
    } 
?>


Beetje veranderd, de oplossing is dat je $query['titel'] doet, terwijl dat natuurlijk $hits moet zijn.
16-12-2010 10:06
Rapifia Europe
Berichten: 3714
avatar
Offline Stuur privébericht
Hier is de verbeterde versie van je.
Deze werkt volgens mij 100%.

@ Hierboven,
Dat gaat niet werken, aangezien je in een url twee keer " opent.
Tevens zijn die `` niet echt mooi in een query...

Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
<h3>Hits</h3>
<?php
$query 
mysql_query("SELECT * FROM radio WHERE cat = '1'");
while(
$hits mysql_fetch_array($query)){  
    echo 
'<ul><li><a onclick=\"luister("'.$hits['titel'].'")\">'.$hits['titel'].'</a></ul></li>';

?>
16-12-2010 10:32
Berichten: 552
avatar
Offline Stuur privébericht
Ik wil niet zeuren maar je doet het zelfde en die van jou gaat neit werken want je gebruikt backslashes met een enkele '?
16-12-2010 10:38
Rapifia Europe
Berichten: 3714
avatar
Offline Stuur privébericht
@Frank,

Ik had die van jou niet bewerkt, maar direct vanaf de TS.
En die werkt, netzelf getest.
Maar je hebt gelijk, hierbij een nog betere versie:
Code | Selecteer Alles
minimaliseren
1
2
3
4
5
6
7
<h3>Hits</h3>
<?php
$query 
mysql_query("SELECT titel FROM radio WHERE cat = '1'");
while(
$hits mysql_fetch_array($query)){  
    echo 
'<ul><li><a onclick=\"luister("'.$hits['titel'].'")\">'.$hits['titel'].'</a></ul></li>';

?>


Ik weet niet hoeveel kolommen er zijn in die tabel, maar als je bijv. een users tabel hebt met 50 kolommen, en je haalt alles op terwijl je er maar 1 nodig hebt = server verpesting.
16-12-2010 10:41
Reageer op: "Link aanklikbaar maken werkt niet"
1 | 2 Volgende pagina
Je kan niet reageren omdat je niet bent ingelogd. Inloggen of Aanmelden