Underlig fejl i mysql query

$row[tab] er i dette tilfælde lig med nyheder, og id er lig med 9

$que = mysql_query("SELECT * FROM '". $row[tab] ."' WHERE id=". $_GET[id] ."") or die(mysql_error());

Er det eksperten der har fjernet et ' (før where) ?

lol :S men det virker stadig ikke :(

hele "brødkrumme"-funktionen ->

    function setkrum($id)
        {
            // hent fra databasen fra den id der er aktuel.
            $query = mysql_query("SELECT * FROM menu WHERE id=". $id ."");
            // hent data
            $row = mysql_fetch_array($query);
            if($row[tekst] != 0)
                {
                // gem det sidste hentede data før det der allerede er
              $this->krumme = "<a href=\"index.php?pid=". $row[id] ."\">". $row[tekst] ."</a>". $this->krumme;
                }
            elseif($row[tekst] == 0)
                {
                $que = mysql_query("SELECT * FROM '". $row[tab] ."' WHERE id=". $_GET[id] ."") or die(mysql_error());
                $ro = mysql_fetch_array($que);
                $id = $ro[id];
                $tekst = $ro[$row[tab_tekst]];
                $this->krumme = "<a href=\"index.php?pid=". $row[id] ."&id=". $id ."\">". $tekst ."</a>". $this->krumme;
                }
            // er der en ovenliggende side
            if ($row[over] != 0)
            {
                $this->krumme = " &raquo; ". $this->krumme;
                // kør funktiionen igen bare med den nye id
                $this->setkrum($row[over]);
            }
            else
            {
                // returner brødsporet hvis der ikke er flere ovenliggende id'er
                return $this->krumme;
          }
      }

Der skal vel ikke bruges ' omkring et tabel navn:

$que = mysql_query("SELECT * FROM ". $row[tab] ." WHERE id=". $_GET[id] ."") or die(mysql_error());

Prøv evt med
$que = mysql_query($sql="SELECT * FROM '". $row[tab] ." WHERE id=". $_GET[id] ."") or die(mysql_error() . '<br />SQL:' . $sql);

Hvilket output får du?

Altså.. det burde være:

$que = mysql_query("SELECT * FROM '". $row[tab] ."' WHERE id=". $_GET[id]);

Så har queryen ingen fejl med mindre at $row[tab] og $_GET[id] er mærkelige..

Ah, fennec siger noget. Det skal være backticks (`) eller intet.

fennic: Du har ret.. jeg sumper.. så bare at hans ' ikke var afsluttet :)

jeg får:

SQL:SELECT * FROM 'nyheder WHERE id=9

altså efter at jeg har fjernet ' og '

ok jeg tåger lidt nu...
uden ' siger den: SQL:SELECT * FROM WHERE id=9

og med '': SQL:SELECT * FROM 'nyheder' WHERE id=9

er i allesammen blevet stumme :S ?

Du skal stadig ikke have ' omkring et tabelnavn. Det skal altså være sådan (min post 14:55:58):

$que = mysql_query("SELECT * FROM ". $row[tab] ." WHERE id=". $_GET[id] ."") or die(mysql_error());

eller som dennismp skrev i 14:57:31 med `:

$que = mysql_query("SELECT * FROM `". $row[tab] ."` WHERE id=". $_GET[id] ."") or die(mysql_error());

... Der er forskel på ' og `

Og der er ingen grund til at smide en tom streng på bagefter så det hele bliver til:

$que = mysql_query("SELECT * FROM `". $row[tab] ."` WHERE id=". $_GET[id]) or die(mysql_error());

Det ser rigtigt ud for mig, men den siger bare "incorrect table name ''"

altså som om der ikke står noget tabelnavn :S
mit script ser nu sådan ud:

                unset($bet);
                if($_GET[id] > 0)
                    {
                    $bet = "WHERE id=". $_GET[id] ."";
                    }
                $que = mysql_query("SELECT * FROM `". $row[tab] ."` ". $bet ."") or die(mysql_error());

bromer jeg er ikke helt med på hvad du har lavet der?

det jeg har ændret er bare fordi, at der ikke altid er et id.. det skulle ikke ændre noget

i den sidste del af queryen smider du en tom streng på ved at skrive . "". Det ændrer jo ikke strengen så derfor er det overflødigt.

Hvis den siger incorrect table name, så det jo fordi der ikke findes en tabel der hedde det. Har du printet væriden af $row[tab] og verificeret at der er en tabel med det navn?

Det her er ret weird:

Hvis jeg skriver: SELECT * FROM `$row[tab]` ", skriver den at der ikke er noget tabelnavn.

men hvis jeg skriver: SELECT * FROM ` $row[tab] ` ", så skriver den: Table 'balmer1. nyheder ' doesn't exist

... balmer1 er fordi databasen hedder balmer1

Jag vil gætte på du skal have " i dit Array udtræk:

$row["tab"]
$row["id"]

er der ikke et mellemrum før/efter punktum ?

prøv balmer1.nyheder

hvad mener du?

Hvis jeg skriver: $que = mysql_query("SELECT * FROM `$row[tab] ` ". $bet ."") or die(mysql_error() . '<br />SQL:' . $sql);

så står der: Table 'balmer1. ' doesn't exist

Jeg synes virkelig at det her er weird..!

fennec hvad mener du?

Hvad hvis du skriver

$que = mysql_query(sprintf("SELECT * FROM `%s` %s,$row['tab'],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

$que = mysql_query(sprintf("SELECT * FROM `%s` %s",$row['tab'],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

sådan..

fjern mellemrummet:
$que = mysql_query("SELECT * FROM `$row[tab]` ". $bet ."") or die(mysql_error() . '<br />SQL:' . $sql);

bare parse error på linien..

$que = mysql_query(sprintf("SELECT * FROM `%s`%s",$row['tab'],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

sku :)

dennis -> det er her hvor den bare siger: Incorrect table name ''

altså den kan ikke se at variablen indeholder noget.. tror jeg :S

bromer den siger det samme som dennis' gjorde :(  Incorrect table name ''

'' er ikke ", men ' + '

wicez: Hvad siger den, hvis du prøver det jeg har skrevet?

^^^^^^^^^^^der står det^^^^^^^^^^^^^^^^^^^^^

:P

som jeg skrev kl. 15.24.12

okay.. så er fejlen jo nok at der ikke er noget i $row['tab']. Sørg for at den variabel bliver sat korrekt.

ja.. jeg spurgte imens du rent faktisk svarede :)

kk...

der er noget i $row['tab'] - det kan du jo også se i det jeg skrev 15.14.02

Okay.. hvad siger den så med:

$que = mysql_query(sprintf("SELECT * FROM `%s`%s",$row[tab],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

det samme :(

Incorrect table name ''

Okay.. hvad giver

printf("SELECT * FROM `%s` : %s",$row[tab],$bet));

Det bør bare printe en streng..

parse error

printf("SELECT * FROM `%s` : %s",$row[tab],$bet);

Prøv at gemme den i en temp variabel:

$temp = $row["tab"];
$que = mysql_query("SELECT * FROM `$temp` $bet ") or die(mysql_error());

hvis jeg fjerne et af de 2 sidste ), så får jeg SELECT * FROM `` :

så er det den med incorrect table name

det er meget mærkeligt.. har du mulighed for at vise os dit script i en større sammenhæng. Jeg vil gerne se, hvordan du sætter din $row.. tror der er nogle problemer med enten dit variabel scope eller at du overskriver den på et tidspunkt.

det virker altså ret mærkeligt... for hver gang den siger incorrect table name, så burde det jo virke.!

Her er det hele ->

function setkrum($id)
        {
            // hent fra databasen fra den id der er aktuel.
            $query = mysql_query("SELECT * FROM menu WHERE id=". $id ."");
            // hent data
            $row = mysql_fetch_array($query);
            if($row[tekst] != 0)
                {
                // gem det sidste hentede data før det der allerede er
              $this->krumme = "<a href=\"index.php?pid=". $row[id] ."\">". $row[tekst] ."</a>". $this->krumme;
                }
            elseif($row[tekst] == 0)
                {
                unset($bet);
                if($_GET[id] > 0)
                    {
                    $bet = "WHERE id=". $_GET[id] ."";
                    }
                printf("SELECT * FROM `%s` : %s",$row[tab],$bet);
                $ro = mysql_fetch_array($que);
                $id = $ro[id];
                $tekst = $ro[$row[tab_tekst]];
                $this->krumme = "<a href=\"index.php?pid=". $row[id] ."&id=". $id ."\">". $tekst ."</a>". $this->krumme;
                }
            // er der en ovenliggende side
            if ($row[over] != 0)
            {
                $this->krumme = " &raquo; ". $this->krumme;
                // kør funktiionen igen bare med den nye id
                $this->setkrum($row[over]);
            }
            else
            {
                // returner brødsporet hvis der ikke er flere ovenliggende id'er
                return $this->krumme;
          }
      }

jeg har én tabel med alle mine sider i, id og link til hvilken fil der skal inkluderes... så er det jo så meningen at $this-> krumme skal replaces med --krum-- i min template.htm

hvilke felter har du i din tabel menu?

CREATE TABLE `menu` (
  `id` int(11) NOT NULL default '0',
  `link` varchar(255) NOT NULL default '',
  `tekst` varchar(255) NOT NULL default '',
  `beskrivelse` text NOT NULL,
  `se` int(11) NOT NULL default '0',
  `setil` int(11) NOT NULL default '0',
  `hvor` int(11) NOT NULL default '0',
  `andet` int(11) NOT NULL default '0',
  `over` int(11) NOT NULL default '0',
  `tab` varchar(255) NOT NULL default '',
  `tab_tekst` varchar(255) NOT NULL default '',
  PRIMARY KEY  (`id`)
) TYPE=MyISAM;

Så skal denne virke... og det gør den

$que = mysql_query(sprintf("SELECT * FROM %s %s",$row["tab"],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
SQL:


det virker jo næsten sygt

Den her krøver vidst at vi lægger 120 point til :S

hvad får du ud når du skriver

var_export($row);

Prøv lige denne så vi får SQL-sætningen ud også:

$que = mysql_query($sql=sprintf("SELECT * FROM %s %s",$row["tab"],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);

Jeg får:

array ( 0 => '1', 'id' => '1', 1 => 'nyheder/index.php', 'link' => 'nyheder/index.php', 2 => 'Forsiden', 'tekst' => 'Forsiden', 3 => 'Her kan du bl.a. læse nyheder', 'beskrivelse' => 'Her kan du bl.a. læse nyheder', 4 => '0', 'se' => '0', 5 => '5', 'setil' => '5', 6 => '1', 'hvor' => '1', 7 => '0', 'andet' => '0', 8 => '0', 'over' => '0', 9 => '', 'tab' => '', 10 => '', 'tab_tekst' => '', )

fennec ved din får jeg:

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
SQL:SELECT * FROM

yep.. som du kan se indeholder 'tab' ikke noget.. Derfor virker din query ikke.

Er der en der kender en lidt nemmere måde at lave det her breadcrumbs system på?

Hjemmesiden skal være så dynamisk som muligt, og jeg ved ikke omd er findes bedre metoder?

mere specifikt ineholder den række i menu du vælger ikke noget i tab feltet. Derfor får du fejl i din query. Ellers er den query jeg skrev sidst (faktisk de sidste par gange) korrekt nok.

men idet at jeg siger:

if($row[tekst] != 0)
                {
                // gem det sidste hentede data før det der allerede er
              $this->krumme = "<a href=\"index.php?pid=". $row[id] ."\">". $row[tekst] ."</a>". $this->krumme;
                }
            elseif($row[tekst] == 0)
                {
                unset($bet);
                if($_GET[id] > 0)
                    {
                    $bet = "WHERE id=". $_GET[id] ."";
                    }
                $que = mysql_query($sql=sprintf("SELECT * FROM %s %s",$row["tab"],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);
                $ro = mysql_fetch_array($que);
                $id = $ro[id];
                $tekst = $ro[$row[tab_tekst]];
                $this->krumme = "<a href=\"index.php?pid=". $row[id] ."&id=". $id ."\">". $tekst ."</a>". $this->krumme;
                }
            // er der en ovenliggende side

så er $row[tekst] jo ikke lig med nul der hvor $row[tab] ikke er sat

ikke?

og hvad så.. du bruger jo ikke $row[tekst] til noget i din query

nej, men hvis $row[tekst] er lig nul, så skal den jo ikke køre querien overhovedet..

og. du kan se fra var_export at $row[tekst] er lig "forsiden".. men $row[tab] er stadig tom...

men når $row[tekst] er lig "forsiden", så skal querien jo slet ikke køres

ahh... men det bliver den jo alligevel.. så testen $row[tekst] == 0 bliver sand.. hvad med $row["tekst"] == 0

if($row["tekst"] == 0)
                {
                unset($bet);
                if($_GET[id] > 0)
                    {
                    $bet = "WHERE id=". $_GET[id] ."";
                    }
                $que = mysql_query($sql=sprintf("SELECT * FROM %s %s",$row["tab"],$bet)) or die(mysql_error() . '<br />SQL:' . $sql);
                $ro = mysql_fetch_array($que);
                $id = $ro[id];
                $tekst = $ro[$row[tab_tekst]];
                $this->krumme = "<a href=\"index.php?pid=". $row[id] ."&id=". $id ."\">". $tekst ."</a>". $this->krumme;
                }
            else
                {
            // gem det sidste hentede data før det der allerede er
              $this->krumme = "<a href=\"index.php?pid=". $row[id] ."\">". $row[tekst] ."</a>". $this->krumme;
                }

stadig fejl :(

okay... måske bliver det castet til 0.. virker dog underligt.. prøv at skrive

if (strlen($row['tekst']) == 0) istedet for if($row["tekst"] == 0)

Det kunne godt se ud til at virke :P

fedt... det tog kun 800000 indlæg :)

:P

jeg prøver lige selv at fikse lidt med det, der er stadig lidt weird stuff.. men tak for hjælpen :P

i stedet for: if (strlen($row['tekst']) == 0)

så: if (strlen($row['tekst']) == 1)

for der står jo 0 i feltet

men jeg vil smutte nu - endnu en gang tak for hjælpen :P