Syntaks ved brug af last_insert_id

... en anden sql der udføre en insert ...

$lastid = last_insert_id();
mysql_query("UPDATE nyheder set billede = '$billede_name' WHERE id = '$lastid'") or die(mysql_error());

Det ser rigtigt ud. Men gør du det umiddelbart efter en INSERT?

tag også lige at

  mysql_query(....) or die(mysql_error());

så du kan se om der er fejl

Der findes følgende funktion som du måske kan bruge:

int mysql_insert_id ( [resource link_identifier])

"tilføjet noget til databasen" .... skal du tilføje skal du da ikke bruge en UPDATE ??

ljweb/tipsen: man behøver jo ikke bruge PHP-funktionen, når der findes
en udmærket i SQL.

Er last_insert_id() en mysql funktion!?

OK - jeg klapper bare i :-)

min fejl - jeg skal opdatere min database

/klgn

Man kan jo altid lære noget nyt, tipsen :)

klgn: Du skal altså bruge UPDATE forstår jeg. Men er der et felt med
pågældende id?

ja, jeg skal indsætte under en eksisterende id og det skal være den senest oprettede...

Så virker det også .... kommer der en fejl med die?

Og har du lavet en INSERT umiddelbart før?

jeg kan lige poste scriptet, hvis det er...

ja tak

Her it is
<?

  require("n-include.php");
  require("../../mysql-funktioner.php");

  aabn_forbindelse_og_vaelgdb();

  if ($d && $m && $y && $overskrift && $tekst && $forfatter) {

      $dato      = addslashes("$y-$m-$d");
      $overskrift = addslashes($overskrift);
      $tekst      = addslashes($tekst);
      $id        = addslashes($id);
      $forfatter  = addslashes($forfatter);

      if ($id) {
        mysql_query("UPDATE nyheder SET dato = '$dato', overskrift = '$overskrift', tekst = '$tekst', forfatter = '$forfatter' WHERE id = '$id'") or die(mysql_error());
      }

      else {
        mysql_query("INSERT INTO nyheder (dato, overskrift, tekst, forfatter) VALUES ('$dato', '$overskrift', '$tekst', '$forfatter')") or die(mysql_error());
      }

      }
     
  ?>
 
 

<HTML>
<HEAD>
<TITLE>Admin til nyheder part 2 (tilføj billede)</TITLE>
</HEAD>
<BODY>
<H1>Dette er teksten:</H1>

<?
  $result = mysql_query("SELECT id, overskrift, tekst, forfatter, billede, DATE_FORMAT(dato, '%d-%m-%Y') AS formateretdato FROM nyheder order by id desc limit 1") or die(mysql_error());
 
 

            print "<table cellspacing=\"0\" cellpadding=\"2\">\n";

            while ($row = mysql_fetch_array($result)) {

                $bgcolor = $a++ % 2 ? '#DEDEDE' : '#F0F0F0';

                $row[overskrift] = htmlentities(stripslashes($row[overskrift]));
                $row[tekst]      = htmlentities(stripslashes($row[tekst]));
                list($d, $m, $y) = explode('-', $row[formateretdato]);

                $new_date = month($m)." $y";

               

                print "<tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Dato: $d/$m-$y</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Overskrift: $overskrift</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Tekst: $tekst</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Forfatter: $forfatter</td></tr>";
        print "</tr>\n";

                $old_date = $new_date;

            }

          print "</table>\n";

        if(isset($billede))
    {
    $desti="../../billeder/";// stien hvor filen skal gemmes
    $desti.=$billede_name;
    if (copy($billede,$desti)) {
        print "Billede tilføjet";
      echo mysql_query("UPDATE nyheder set billede = '$billede_name' WHERE id = $lastid") or die(mysql_error());
    }

    else {
        print "Der skete en fejl, billede ikke tilføjet";
    }
    unlink($billede);
   
    }
?>
<H1>Tilføj billede til teksten</H1>

<FORM ENCTYPE="multipart/form-data" ACTION="add_picture.php" METHOD="post">
<INPUT TYPE="hidden" name="MAX_FILE_SIZE" value="102400">
<INPUT NAME="billede" TYPE="file">
<INPUT TYPE="submit" VALUE="Tilføj billede">
</FORM>

</BODY>
</HTML>

i stedet for $lastid skulle der stå: last_insert_id

Du kan ikke gøre det af to omgange. Det skal ske umiddelbart efter din
INSERT. Jeg læser ovenstående som at du først indsætter et nyhed,
og så i en ny form vælger et billede. Er det korrekt?

Vi kan sagtens fixe det, hvis det er det...

ja, det er korrekt forstået...

Nede i billedformen laver du så et

<INPUT TYPE="hidden" name="billedid" value="<?= last_insert_id()?>">

og bruger $billedid til din update.

Jeg får en fejlmeddelse:
Fatal error: Call to undefined function: last_insert_id() in c:\apache\htdocs\ntk\admin\nyheder\add_picture.php on line 89

Nå ja  mysql_insert_id()

Billedet bliver fint uploadet, men det bliver ikke tilføjet i databasen

værdien af billedid er også tom...

Står der en fornuftig værdi i formen (vis kilde) ?

næh, der står ingenting (den er tom)

det er underligt. Jeg har sat den til at gemme billedid lige efter, der er blevet indsat - og jeg kan også printe den rigtige værdi ud - men jeg kanh ikke opdatere under den id...

Hvad med at sige

$res = mysql_query("SELECT MAX(id) as mx FROM tabelnavn") or die( mysql_error() );
if ( $row = mysql_fetch_array($res) ) {
    $maxid = $row[mx];
} else {
    $maxid = 0;
}

Den skulle da give den største brugte id-værdi i databasen uafhængingt af om der er blevet indsat noget lige før eller ikke.

mvh JakobA

Jeps - det virker satme, JakobA

Post et svar - og der er points!!

/klgn

man skal ikke - aldrig - bruge max til at finde den sidste id.

Er der to der opdaterer samtidigt kommer der en fejl.

erikjacobsen > er det egentlig ikke det samme problem med last_insert_id() ?

Nej, med last_insert_id() / mysql_insert_id() registreres den
autonummererede id pr. connection. Dermed er den sikker.
Så min anbefaling er klart at finde ud af i ovenstående kode hvordan
man holder styr på den. Det kan ikke være så svært. Men ok, hvis
spørgeren er tilfreds - fx hvis han er den eneste der bruger
systemet - så er det jo godt nok.

øøh, jeg er selvfølgelig interesseret i den mest sikre løsning. Jeg konstaterede blot, at JakobAs løsning fungede, hvilket ikke var tilfældet med last_insert_id() eller mysql_insert_id

Hvis nogle har bud på, hvordan disse kan implementeres tilfredsstillende i koden er jeg lydhør

erikjacobsen > kan samtidige requests ikke bruge hinandens connection? Hmm... nu burde jeg jo nok oprette et spørgsmål selv, for det er noget jeg har tænkt lidt over... men hvis du lige vidste det, kunne det jo være rart at vide....

Du siger jo mysql_connect på hver side og hver af disse har deres styr på det her.

Ellers var der jo ikke noget der virkede.... Men pas på hvis du bruger tråde, der deles
om en connection

grunden til, at jeg tvivlede lidt var, at MySQL jo ikke håndterer transaktioner, og en sådan opførsel for last_insert_id() ville jo være noget der lignede en transaktion....

Men nok om det, jeg vælger at tro på dig *s*

klgn: Jo, du skal jo holde styr på hvor de oplysninger du har brug for kommer fra.
Hvorfor oploader man ikke billeder med det samme? Kan det ikke gøres samtidigt
skal du så blot finde ud af hvor den tomme værdi, du så før kommer fra.

he he, ja altså hvorfor den er tom...

Nej, det er ikke en transaktion, Enhver "INSERT" er atomisk, og den skal bare
aflevere værdien. Bliver det mere kompliceret hedder det transaktioner!

erikjacobsen-> ja, det ku jeg godt have sagt mig selv... Problemet er blot, at mit kendskab til disse funktioner er ret begræsent - for ikke at sige ikke-eksisterende.
Jeg har virkelig kigge på koden - og jeg kan ikke finde ud af, hvordan den værdi pludselig skal blive nullet - især ikke når jeg beder mysql_insert_id() ind lige efter insert into...

Jeg savner noget konkret...

Og det samme gør vi :) Det er svært at sige hvad der er galt når vi ikke kan se din kode ...

den er sgu da postet længere oppe

men den kommer da bare her igen...

  if ($d && $m && $y && $overskrift && $tekst && $forfatter) {

      $dato      = addslashes("$y-$m-$d");
      $overskrift = addslashes($overskrift);
      $tekst      = addslashes($tekst);
      $id        = addslashes($id);
      $forfatter  = addslashes($forfatter);

      if ($id) {
        mysql_query("UPDATE nyheder SET dato = '$dato', overskrift = '$overskrift', tekst = '$tekst', forfatter = '$forfatter' WHERE id = '$id'") or die(mysql_error());
      }

      else {
        mysql_query("INSERT INTO nyheder (dato, overskrift, tekst, forfatter) VALUES ('$dato', '$overskrift', '$tekst', '$forfatter')") or die(mysql_error());
     
        $maxid=mysql_last_insert();
      }

      }
     
  ?>
 
 

<HTML>
<HEAD>
<TITLE>Admin til nyheder part 2 (tilføj billede)</TITLE>
</HEAD>
<BODY>
<H1>Dette er teksten:</H1>

<?
 
  $result = mysql_query("SELECT id, overskrift, tekst, forfatter, billede, DATE_FORMAT(dato, '%d-%m-%Y') AS formateretdato FROM nyheder order by id desc limit 1") or die(mysql_error());
 
 

            print "<table cellspacing=\"0\" cellpadding=\"2\">\n";

            while ($row = mysql_fetch_array($result)) {

                $bgcolor = $a++ % 2 ? '#DEDEDE' : '#F0F0F0';

                $row[overskrift] = htmlentities(stripslashes($row[overskrift]));
                $row[tekst]      = htmlentities(stripslashes($row[tekst]));
                list($d, $m, $y) = explode('-', $row[formateretdato]);

                $new_date = month($m)." $y";

               

                print "<tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Dato: $d/$m-$y</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Overskrift: $overskrift</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Tekst: $tekst</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Forfatter: $forfatter</td></tr>";
       
        print "</tr>\n";

                $old_date = $new_date;

            }

          print "</table>\n";

        if(isset($billede))
    {
    $desti="../../billeder/";// stien hvor filen skal gemmes
    $desti.=$billede_name;
    if (copy($billede,$desti)) {
        print "Billede tilføjet";
      echo mysql_query("UPDATE nyheder set billede = '$billede_name' WHERE id = '$maxid'") or die(mysql_error());
   
    }

    else {
        print "Der skete en fejl, billede ikke tilføjet";
    }
    unlink($billede);
   
    }
?>
<H1>Tilføj billede til teksten</H1>

<FORM ENCTYPE="multipart/form-data" ACTION="add_picture.php" METHOD="post">
<INPUT TYPE="hidden" name="MAX_FILE_SIZE" value="102400">
<INPUT NAME="billede" TYPE="file">
<INPUT TYPE="submit" VALUE="Tilføj billede">
</FORM>
</BODY>
</HTML>

Du har jo ikke gjort som jeg skrev i

  Kommentar fra erikjacobsen 
  21/04-2002 12:18:35

well, hvis du læser mine svar til dit pågældende indlæg, så er det pårøvet uden held...

Det virker ikke

Og der er det jeg bliver "sur". Du viser os ikke hvad du har prøvet,
og udelukker dermed at vi kan finde fejlen for dig.

Alright - godt ord igen!!

Her er det, jeg har prøvet - uden held...

  if ($d && $m && $y && $overskrift && $tekst && $forfatter) {

      $dato      = addslashes("$y-$m-$d");
      $overskrift = addslashes($overskrift);
      $tekst      = addslashes($tekst);
      $id        = addslashes($id);
      $forfatter  = addslashes($forfatter);

      if ($id) {
        mysql_query("UPDATE nyheder SET dato = '$dato', overskrift = '$overskrift', tekst = '$tekst', forfatter = '$forfatter' WHERE id = '$id'") or die(mysql_error());
      }

      else {
        mysql_query("INSERT INTO nyheder (dato, overskrift, tekst, forfatter) VALUES ('$dato', '$overskrift', '$tekst', '$forfatter')") or die(mysql_error());
     
      }

      }
     
  ?>
 
 

<HTML>
<HEAD>
<TITLE>Admin til nyheder part 2 (tilføj billede)</TITLE>
</HEAD>
<BODY>
<H1>Dette er teksten:</H1>

<?

  $result = mysql_query("SELECT id, overskrift, tekst, forfatter, billede, DATE_FORMAT(dato, '%d-%m-%Y') AS formateretdato FROM nyheder order by id desc limit 1") or die(mysql_error());
 
 

            print "<table cellspacing=\"0\" cellpadding=\"2\">\n";

            while ($row = mysql_fetch_array($result)) {

                $bgcolor = $a++ % 2 ? '#DEDEDE' : '#F0F0F0';

                $row[overskrift] = htmlentities(stripslashes($row[overskrift]));
                $row[tekst]      = htmlentities(stripslashes($row[tekst]));
                list($d, $m, $y) = explode('-', $row[formateretdato]);

                $new_date = month($m)." $y";

               

                print "<tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Dato: $d/$m-$y</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Overskrift: $overskrift</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Tekst: $tekst</td></tr>";
        print "    <tr><td bgcolor=\"$bgcolor\">Forfatter: $forfatter</td></tr>";
       
        print "</tr>\n";

                $old_date = $new_date;

            }

          print "</table>\n";

        if(isset($billede))
    {
    $desti="../../billeder/";// stien hvor filen skal gemmes
    $desti.=$billede_name;
    if (copy($billede,$desti)) {
        print "Billede tilføjet";
      echo mysql_query("UPDATE nyheder set billede = '$billede_name' WHERE id = '$maxid'") or die(mysql_error());
   
    }

    else {
        print "Der skete en fejl, billede ikke tilføjet";
    }
    unlink($billede);
   
    }
?>
<H1>Tilføj billede til teksten</H1>

<FORM ENCTYPE="multipart/form-data" ACTION="add_picture.php" METHOD="post">
<INPUT TYPE="hidden" name="MAX_FILE_SIZE" value="102400">
<INPUT TYPE="hidden" name="maxid" value="<? mysql_insert_id()?>">
<INPUT NAME="billede" TYPE="file">
<INPUT TYPE="submit" VALUE="Tilføj billede">
</FORM>
</BODY>
</HTML>

jeg har også prøvet med last_insert_id()

og - jeg har indsat semikolonet - stadig uden held...

Tag et kursus i læsning :)

<?= mysql_insert_id()?>

okay okay - du vinder - det virker;)

Jeg synes ellers, jeg havde prøvet det - nu vil jeg lige teste, om det virker, når jeg updater...

/Kristian

jeps, det funger - men hvorfor det ligmedstegn??

vinder? måske jeg skulle til at spille lotto  ;)

skriv det som svar - og du får points...

Ok, <?= xxx ?> er en forkortelse for <? print xxx; ?>
Så skal man ikke skrive så meger

meget

nå, ja, svar

siger mange tak for hjælpen - points afsendt...

Kendte ikke den med = - men man skal jo også blive klogere hver dag:)