mysqli bind problemer

man bruger ikke or die i mysql i, men mysqli->error

så prøv det efter din bind_result og se om den spytter noget ud...

altså sådan her :)?



$stmt->bind_result($forum_traad.dato,$forum_traad.brugernavn,$forum_traad.fk_forum_kategori,$forum_traad.indhold,$forum_traad.overskrift,$forum_traad.id,$forum_kommentare.id,$forum_kommentare.fk_forum_traad,$forum_kommentare.brugernavn);
    echo $mysqli->error;
    $stmt->execute();
    $stmt->error();

for så får jeg den her fejl

Fatal error: Call to a member function bind_result() on a non-object in E:\wamp\www\forum.kategori.vis.php on line 34

Det er noget pudseløjerligt noget du har fået bikset sammen der...

For det første anvender du slet ikke mulighederne i prepared statements, selvom du anvender teknologien.

Du kan ikke binde et resultat på en forespørgsel du ikke har eksekveret.

Metoden ::error() skal anvendes på dit $stmt objekt, når det er det objekt du forsøger at ændre/eksekvere.

Og som #1 siger; du kan ikke blande mysql og mysqli sammen, som du har gjort i dit eksempel.

skal det være

$stmt->error();


hva er det for nogen muligheder jeg ikke anvender :)?

du kan evt se her hvordan man bruger mysqli: http://www.tutorialhero.com/click-56189-php_5_mysqli_prepared_statements.php

du blander det lidt nemlig...

#4
Kort fortalt er formålet med prepared statements at du sikrer dine data bedre; det gør du ikke ved at konkatenere dine forespørgselsdata ind i din SQL streng.

Den forkerte måde at gøre det på:


Den rigtige måde at gøre det på:


På den måde udnytter du den sikkerhed der ligger i at anvende prepared statements.

har prøvet at læse lidt på dit link men synes ikke rigtig jeg lærte noget, er heller ikke den bedste til engelsk så måske derfor :( .

men bind_param er det ikke kun hvis jeg skal indsætte noget i databasen?

har ihvertfald lavet det sådan her ud


http://pastebin.com/xjaKmhCq


er ikke helt sikker på den error virker

Nej, bind_param() er til at binde dine variabler til din prepared statement. Om du henter, indsætter, opdaterer eller sletter så bruger du bind_param; ellers er du jo lige så sårbar over for SQL injection som før.

::error() virker som det skal - det er dig der bruger det forkert.
Prøv at bruge manualen lidt: http://dk2.php.net/manual/en/mysqli-stmt.error.php

Der er gode eksempler på hvordan man kan bruge de forskellige metoder.

nu har jeg prøvet med det her

http://pastebin.com/w2C0B6q2

men får bare stadig samme fejl, kan det være min query som er helt gal?

Har du prøver at:

Efter som det er ved din prepare at det fejler.

Dog mangler du sikkert at skrive hvilken tabel godkend og fk_forum_kategori kommer fra.

så nu virker det med det her


$stmt = $mysql->prepare("
Select
forum_traad.dato,
forum_traad.brugernavn,
forum_traad.fk_forum_kategori,
forum_traad.indhold,
forum_traad.overskrift,
forum_traad.id,
forum_kommentare.id,
forum_kommentare.fk_forum_traad,
forum_kommentare.brugernavn
From
forum_traad Inner Join
forum_kommentare On forum_kommentare.fk_forum_traad = forum_traad.id
WHERE
forum_kommentare.godkendt = 'ja'
AND
fk_forum_kategori = ?
") or die($mysql->error());

$stmt->bind_result($dato,$brugernavn,$fk_forum_kategori,$indhold,$overskrift,$id,$id,$fk_forum_traad,$brugernavn);
    $stmt->bind_param("i", $kategoriID);
    $stmt->execute();
    echo $mysql->error;


skal jeg i min bind_result så skrive forum_traad.dato osv.?

og hvordan udskriver jeg fra den ene og den anden tabel?

echo $forum_traad.dato;?

eller hvordan :D

Nej du har jo allerede en bind_result med $dato, $brugernavn... ,
dog har du $id to gange, det er ikke så smart, da du så ikke kan se hvilken værdi du arbejder med. 
Det er bestemt ikke en god ide at have et . i dine variabelnavne.

men forstår ikke helt i begge mine tabeller har jeg et felt som hedder "dato" hvordan finder jeg ud af om det er dato'en fra tabellen "forum_traade" eller om det er datoen fra "forum_kommentare" ?

Det er nu meget simpelt.
bind_result virker på denne måde: første felt du angiver i din sql bliver bundet til første variabel du angiver i bind_result, andet felt til 2. variabel osv.

Men du bliver nød til at kalde bind_result EFTER du har kaldt execute().

okay lyder nice :), men hvis jeg nu har 2 felter som begge hedder dato hvad gør jeg så?

Jeg er ikke helt sikker på hvad du mener.
Du kan jo lave to forskellige variabler, en til hver dato felt.

i min database har jeg 2 tabeller som hedder

forum_traad
forum_kommentare

i begge de tabeller har jeg et felt som hedder dato


hvis jeg gerne vil echo dato'en fra forum_kommentare tabellen hvordan gør jeg det uden at den tror jeg mener den dato fra forum_traad tabellen :)?

håber du forstår

Ved at definere det i din felt liste i din select
Select
t.dato,
t.brugernavn,
t.fk_forum_kategori,
t.indhold,
t.overskrift,
t.id,
k.id,
k.fk_forum_traad,
k.brugernavn,
k.dato
FROM
forum_traad t INNER JOIN
forum_kommentare k ON k.fk_forum_traad = t.id
WHERE
k.godkendt = 'ja'
AND
t.fk_forum_kategori = ?

din bind kunne så være:
$stmt->bind_result(
    $traad_dato,
    $traad_brugernavn,
    $traad_fk_forum_kategori,
    $traad_indhold,
    $traad_overskrift,
    $traad_id,
    $kommentare_id,
    $kommentare_fk_forum_traad,
    $kommentare_brugernavn,
    $kommentare_dato);

Så vil $kommentare_dato indeholde værdien fra feltet forum_kommentare.dato
Og $traad_dato vil indeholde værdien fra feltet forum_traad.dato

ahh så skal det så bare skrives i den rigtige række følge som du sagde :)

yes sir, ellers kan mysql jo ikke finde ud af det.
Læg også mærke til at jeg har brugt alias til dine tabelnavne, det gør det lidt mere overskueligt.

men nu henter den samme tråd 4 gange :S det forstår jeg ikke helt


http://pastebin.com/jK9W7uhh

Det er nok fordi der er 4 kommentarer til din tråd.
Prøv at udskrive de to datoer, eller prøv at tjekke i PHPMyAdmin de to tabeller for den tråd id.

altså i min database i tabellen "forum_traad" har jeg 2 tråde den ene har fk_forum_kategori: 5 og den anden har 6.

så den henter den rigtige tråd men den udskriver den bare 4 gange fordi der er 4 kommentare i den tråd.

har prøvet at echo

echo "Tråd dato: ".$traad_dato;
echo "<br>kommentar dato: ".$kommentare_dato;

og så skriver den bare begge dato'er 4 gange :p. ved ikke helt hvad der er galt

har det noget af gøre med

t.fk_forum_kategori = ?

og $stmt->bind_param("i", $kategoriID);

Det har ikke specielt noget med mysqli at gøre.
Det har noget at gøre med dit resultat af den "INNER JOIN" og hvordan du har struktureret din PHP kode.

Din "INNER JOIN" vil give dig 4 rækker, hvor data fra "traad" tabellen vil være ens for alle rækker, sådan virker "INNER JOIN".

skal jeg så bruge left join eller noget andet :)?

Nej, du skal nok snare lave 2 sql query's.

hvad er det helt præcist du vil have vist siden du laver denne sql-forespørgsel?

prøv at forklar det, så er det måske lidt nemmere at hjælpe dig :)

jeg vil gerne have vist alle de tråde som er oprettet inde i den kategori man nu har valgt.


og ud for hver tråd vil jeg gerne have vist, hvem der har oprettet den og Hvem der sidst har svaret i tråden :) håber i forstår mig da jeg er gået lidt i stå med det :/

umiddelbart ville jeg mene at du ville få løst dit problem ved at smide:

ORDER BY k.id DESC GROUP BY t.id

i enden af din sql-streng...

er det sådan her den skal være


$stmt = $mysql->prepare("
Select
t.dato,
t.brugernavn,
t.fk_forum_kategori,
t.indhold,
t.overskrift,
t.id,
k.id,
k.fk_forum_traad,
k.brugernavn,
k.dato
FROM
forum_traad t INNER JOIN
forum_kommentare k ON k.fk_forum_traad = t.id
WHERE
k.godkendt = 'ja'
AND
t.fk_forum_kategori = ? ORDER BY k.id DESC GROUP BY t.id");


for jeg får den her fejl nu

Fatal error: Call to a member function bind_param() on a non-object

men har ikke rørt ved

$stmt->bind_param("i", $kategoriID);

så det ret mærkeligt

Brug nu mysql::error til at finde dine fejl i sql sætningen.
Du har fået vist hvordan du skal gøre!

jamen når jeg skriver

or die($mysql->error());

efter min sql sætning så skriver den

Call to undefined method mysqli::error()

Har du prøvet at lave en vardump på $mysql ??
Og du er sikker på at du har forbindelse til din mysql database ?

Evt. prøv at fjerne de to () efter error.
or die($mysql->error);

ja det er jeg sikker på da den jo godt kan udskrive noget hvis jeg ikke laver det der join :) men jeg får det her af vardump, er vardump ikke det samme som print_r($mysql); :)?




object(mysqli)#1 (17) {
  ["affected_rows"]=>
  int(-1)
  ["client_info"]=>
  string(50) "mysqlnd 5.0.5-dev - 081106 - $Revision: 1.3.2.27 $"
  ["client_version"]=>
  int(50005)
  ["connect_errno"]=>
  int(0)
  ["connect_error"]=>
  string(0) ""
  ["errno"]=>
  int(1064)
  ["error"]=>
  string(160) "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 'GROUP BY t.id' at line 18"
  ["field_count"]=>
  int(0)
  ["host_info"]=>
  string(37) "MySQL host info: localhost via TCP/IP"
  ["info"]=>
  NULL
  ["insert_id"]=>
  int(0)
  ["server_info"]=>
  string(20) "5.1.36-community-log"
  ["server_version"]=>
  int(50136)
  ["sqlstate"]=>
  string(5) "42000"
  ["protocol_version"]=>
  int(10)
  ["thread_id"]=>
  int(17)
  ["warning_count"]=>
  int(0)
}

fjernede de 2 () så nu får jeg den rigtige fejl :)

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 'GROUP BY t.id'

Ang mysqli::error, så havde jeg lige lavet en fejl. Det er ikke en funktion men "bare" en vairabel i mysqli. ( I mysql er det en funktion man skal kalde)


Som du kan se ud fra både vardump og udprintet, så har du en fejl på k.id. Det eneste forslag jeg kan komme med er at prøve med k.dato i stedet. Men jeg er ikke sikker.

nu har jeg prøvet at rette k.id til k.dato men får stadig samme fejl :( .

Jeg har prøvet at læse lidt om det der group men forstår det ikke helt, tror du at du kan forklare hvad det gør :)?

jeg fandt ud af problemet :) jeg skulle sætte group by FØR min order by :D.

tusind tak for jeres hjælp smid svar :)

well, okay, det vidste jeg faktisk ikke :s

kuhl det spiller! :)

fandt lige en fejl, den order ikke rigtig, den tager id 1 selv om den jo skal tage id 4 (altså den sidste kommentar id) har prøvet med order by DESC og order by ASC men ingen virker :S

du får ihvertfald lige point :)

var jeg den eneste der skulle have points?

nej det var en fejl kan godt lave en ny tråd hvis i andre vil have point os

I am a freelancer and most of time i work for data entry that's why i use computer shortcut keys and i have enough knowledge about it. If someone need advise about windows shortcut keys, feel free to visit me at http://shortkeys.top