Parse error når jeg forsøger at lave et forum

$sql = "INSERT INTO indlaeg (navn, email, besked)
VALUES ($navn,$email,$besked)";

Jeg ville nok lave det sådan her:
Se om der virker:


<html>
<title>Gæstebog</title>
<body bgcolor="#FFFFFF">

<form name="form1" method="POST" action="opdater.php">

Læg venligst en besked:
<BR>
Navn:
<input type="text" name="navn">
<BR>
Email:
<input type="text" name="email">
<BR>
<textarea name="besked" rows="8"></textarea>
<input type="submit" name="submit" value="Indsend">

</form>

<HR>

Tak til de personer. som allerede har lagt en besked!

<HR>

<?php

$conn = mysql_pconnect('localhost','root','')
    or die("Can't Connect To Server: ".mysql_error());

$db = mysql_select_db('gaestebog')
    or die("Can't Connect To Database: ".mysql_error());

$result=mysql_query("SELECT * FROM indlaeg");

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

?>

Navn: <? echo $myrow[1] ?>
<BR>
Email: <? echo $myrow[2] ?>
<BR>
Besked: <? echo $myrow[3] ?>
<BR>

<HR>

<?
}
?>


</BODY>
</HTML>

og her er koden til opdateringsfilen som skal hente fra formularen:

<?php

$navn = $_POST['navn'];
$email = $_POST['email'];
$besked = $_POST['besked'];

$conn = mysql_pconnect('localhost','root','')
    or die("Can't Connect To Server: ".mysql_error());

$db = mysql_select_db('gaestebog')
    or die("Can't Connect To Database: ".mysql_error());

$sql = "INSERT INTO indlaeg VALUES ('".$navn."',"'".$email."','".$besked"'";

$results = mysql_query($sql);

mail("per@mail.dk", "Nyt inlæg i gæstebogen
fra ".$navn,$besked,"From: ".$email);

?>

<html>
<title>Opdater</title>
<body bgcolor="FFFFFF">

Tak for det. <a href="laes.php">Klik for at vende tilbage til gæstebogen.</a>

</body>
</html>

fjernede "" men nu får jeg denne fejl:
Notice: Undefined variable: navn in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 8

Notice: Undefined variable: email in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 8

Notice: Undefined variable: besked in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 8

Notice: Undefined variable: navn in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13

Notice: Undefined variable: besked in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13

Notice: Undefined variable: email in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13
Tak for det. Klik for at vende tilbage til gæstebogen.

Hvis du vil så skriv det lige... men så kan jeg nok godt lave et lille eksempel det sikkert er bedre en IDG's, og også med kommentarer.. hvis du vil have det?

$sql = "INSERT INTO indlaeg VALUES ('".$navn."',"'".$email."','".$besked"')";

der manglede lige en ) ... men underligt at den ikke lige kan finde ud af de variabler

fjernede "" hvor?

$sql = "INSERT INTO indlaeg (navn, email, besked)
VALUES ($_GET['navn'],$_GET['email'],$_GET['besked'])";

Ahh, til hans indlæg? ikke mit... hehe... prøv at se om mit virker.. det som IDG har lavet ser ud til at være noget underligt noget... Man bruges f.eks ikke method="get" når man har en form.. Der bruger man method="POST"

mcqoat: Nej det er ikke.. de er jo sendt over en form.

jamen det må du da meget gerne, men jeg vil jo gerne lære noget og det irriterer mig at jeg ikke ved hvad der er galt...eftersom jeg har fulgt eksemplet straight....

mcqoat: Man kan da godt bruge GET i en FORM, de har jo forskellige fordele. Men her vil jeg give dig ret i at man burde bruge POST.

muddyw: se hvad jeg skrev anden gang. Det burde virke!

bromer >> du skrev: $sql = "INSERT INTO indlaeg (navn, email, besked)
VALUES ($navn,$email,$besked)";
altså uden ""    og så var det jeg fik en anden fejl :)

Yep.. og så postede jeg 22:12:45

$sql = "INSERT INTO indlaeg (navn, email, besked)
VALUES ($_GET['navn'],$_GET['email'],$_GET['besked'])";

ok, prøver lige

bromer >> giver denne fejl:
Parse error: parse error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 8

hvilken PHP version har du?

Hvis du deler det over flere linjer skal der . i starten af den næste linje

$sql = "INSERT INTO indlaeg (navn, email, besked) "
."VALUES ($_GET['navn'],$_GET['email'],$_GET['besked'])";


men der skal vist også ping omkring i SQL sætningens value del:

$sql = "INSERT INTO indlaeg (navn, email, besked) "
."VALUES ('".$_GET['navn']."','".$_GET['email']."','".$_GET['besked']."')";

Er i gang med at lave en bedre til dig som du nok forstår mere af

mcgoat >> dit script giver denne fejl:
Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13

jeg kører PHP4

Her er 3 filer:
connect.php, index.php og opdater.php:

Prøv at lave dem (copy paste) og kør din index.php.

connect.php:
<?
// Denne fil bliver brugt til at forbinde til database serveren samt vælge databasen som skal bruges


// Her forbindes til localhost (din maskine) med "root" som brugernavn og "" (ingenting) som password
$conn = mysql_pconnect('localhost','root','')
    or die("Can't Connect To Server: ".mysql_error()); // Hvis der kommer fejl så bliver det skrevet

// Her vælges den database som skal bruges
$db = mysql_select_db('gaestebog')
    or die("Can't Connect To Database: ".mysql_error());  // Hvis der kommer fejl så bliver det skrevet
?>


index.php:
<html>
<head>
<title>Min Gæstebog</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<form name="form1" action="opdater.php" method="post">
Navn: <input type="text" name="navn"><br><br>
email: <input type="text" name="email"><br><br>
Tekst: <textarea name="besked"></textarea><br><br>
<input type="submit" name="subm" value="Post Indlæg">
</form>
<br><br><br><hr><br>
Gamle Indlæg:
<br><br>

<?
require "connect.php"; // Her bliver filen connect.php inkluderet så der skabes forbindelse til databasen

$query = "SELECT * FROM indlaeg"; // Her vælges ALLE (*) indlæg i gæstebogen (Dette er en streng med en forespørgsel)

$result = mysql_query($query); // Send ovenstående forespørgsel til databasen

while($row = mysql_fetch_array($result)) // Hent de indlæg der nu skulle være
{
    echo "<br>";
    echo "Navn: ". $row['navn'];
    echo "<br>";
    echo "Email: ". $row['email'];
    echo "<br>";
    echo "Besked: ". row['besked'];
    echo "<br><hr>";
}
?>
</body>
</html>

og opdater.php:
<?php
require "connect.php"; // Her bliver filen connect.php inkluderet så der skabes forbindelse til databasen


// Da der bliver bruge method = "Post" så er disse variabler af typen POST og bruges på denne måde $_POST['navn']
// Der er 3 inputs på siden: name="navn", name="email" og name="besked"..
$navn = $_POST['navn'];
$email = $_POST['email'];
$besked = $_POST['besked'];

$sql = "INSERT INTO indlaeg VALUES ('".$navn."','".$email."','".$besked"')"; // Streng med forespørgsel

$results = mysql_query($sql); // Her bliver de variabler smidt ind i din database

mail("per@mail.dk", "Nyt inlæg i gæstebogen fra ".$navn,$besked,"From: ".$email); // Her får du sendt en mail(måske)
?>

<html>
<title>Opdater</title>
<body bgcolor="FFFFFF">

Tak for det. <a href="index.php">Klik for at vende tilbage til gæstebogen.</a>

</body>
</html>

eagleeye >> det hjalp med ping omkring i SQL sætningens value del:

$sql = "INSERT INTO indlaeg (navn, email, besked) "
."VALUES ('".$_GET['navn']."','".$_GET['email']."','".$_GET['besked']."')";

nu opdateres gæstebogen, men jeg får denne besked:

Notice: Undefined variable: navn in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13

Notice: Undefined variable: besked in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13

Notice: Undefined variable: email in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 13
Tak for det. Klik for at vende tilbage til gæstebogen.

mcgoat >> prøver lige din ting der :)

mail("per@mail.dk", "Nyt inlæg i gæstebogen fra ".$_GET['navn'],$_GET['besked'],"From: ".$_GET['email']);

<< var hvis du ville rette den fejl... men prøv mit eksempel og se om det virker.. har ikke testet det.. har bare skrevet det... hehe

jeg skal først oprette dem i phphmyadmin, ik...før det virker?

Nej, du har vel din database der ikke? den der hedder gaestebog?

Filerne skal bare ligge i den mappe hvor du havde de andre liggende og så køre index.php i din internet explorer...

f.eks: http://localhost/index.php

Vil gaette på at du bruger apache? og dine filer før lå i en mappe der hed htdocs?

giver denne fejl:
Parse error: parse error, unexpected '[', expecting ',' or ';' in g:\program files\apache group\apache\htdocs\gaestebog\index.php on line 30

ok er rettet :)    prøver igen

jeg får denne fejl:
Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 11

der bliver altså ved med at være problemer med linie 11:
$sql = "INSERT INTO indlaeg VALUES ('".$navn."','".$email."','".$besked"')";

hmmm... prøv med dette i din opdater.php:

<?php
require "connect.php"; // Her bliver filen connect.php inkluderet så der skabes forbindelse til databasen


// Da der bliver bruge method = "Post" så er disse variabler af typen POST og bruges på denne måde $_POST['navn']
// Der er 3 inputs på siden: name="navn", name="email" og name="besked"..
$navn = $_POST['navn'];
$email = $_POST['email'];
$besked = $_POST['besked'];

$sql = "INSERT INTO indlaeg VALUES ('$navn','$email','$besked')"; // Streng med forespørgsel

$results = mysql_query($sql); // Her bliver de variabler smidt ind i din database

mail("per@mail.dk", "Nyt inlæg i gæstebogen fra ".$navn,$besked,"From: ".$email); // Her får du sendt en mail(måske)
?>

<html>
<title>Opdater</title>
<body bgcolor="FFFFFF">

Tak for det. <a href="index.php">Klik for at vende tilbage til gæstebogen.</a>

</body>
</html>

der mangler et . i linje 11:

$sql = "INSERT INTO indlaeg VALUES ('".$navn."','".$email."','".$besked."')";

Du har en tabel i databasen "gaestebog" som hedder "indlaeg" ikke? som så indeholder "navn", "email", og "besked" ? og nok også et "id"?

Ahhh ja... godt set eagleeye :) ... prøv med det istedet for

der manglede bare et . bagefter $besked

hehe

ja jeg har :)

mcgoat det du har vist 30/09-2004 23:34:15 bør også virke.

punktummet er rettet, og jeg får ingen fejlmeldinger, men indlægget bliver ikke opdateret...dvs. man kan ikke se det i gæstebogen :(

vi er næsten ved at være der :) tror jeg

der bliver også sendt en mail videre om indlægget, men bogen bliver ikke opdateret!

Hvis du har mistanke om der sker en fejl kan du prøve at rette:

$results = mysql_query($sql); // Her bliver de variabler smidt ind i din database

Til dette:

$results = mysql_query($sql) or die(mysql_error()); // Her bliver de variabler smidt ind i din database

god ide.. for at se om der kommer en fejl der...

giver denne fejl: Column count doesn't match value count at row 1

Kan du se om det er fordi at de ikke bliver gemt i DB, eller fordi den ikke kan vise de indlæg som ligger i DB?

PRøv at tilføje dine kolonnenavne som du har haft, eks:

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('".$navn."','".$email."','".$besked"')";

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('$navn','$email','$besked')"; // Streng med forespørgsel

hvad hvis du prøver med det?

de bliver ikke gemt i DB, det gjorde de før

Er nok fordi der er et felt der f.eks hedder ID som jeg ikke har taget højde for da jeg altid gør alting manuelt og besværligt ved selv at indsaette et nyt id... hehe

Prøv en af disse:
$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('".$navn."','".$email."','".$besked."')";

eller

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('$navn','$email','$besked')"; // Streng med forespørgsel

Ja man kan vist skrive '' :

$sql = "INSERT INTO indlaeg VALUES ('','$navn','$email','$besked')"; // Streng med forespørgsel

giverdenne fejl: Parse error: parse error, unexpected T_CONSTANT_ENCAPSED_STRING in g:\program files\apache group\apache\htdocs\gaestebog\opdater.php on line 11 

:(

Brug det som mcgoat har vist :)  der var lidt copy/paste fejl i min den første.

har du prøvet begge disse?

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('".$navn."','".$email."','".$besked."')";

eller

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('$navn','$email','$besked')"; // Streng med forespørgsel

$sql = "INSERT INTO indlaeg (navn, email, besked) VALUES ('$navn','$email','$besked')"; // Streng med forespørgsel
virkede!!!!!
JAAAA, hold kæft drenge, hvor er i fede...nu skal jeg bare bruge noget tid på at nærlæse koden for at se hvad der egentlig sker i PHP :)

I skal have nogle point...smid et svar :)

og tak for jeres hjælp!!

Jamen ok, her et mit svar så ;)

Det var skam så lidt.. du kan få min msn hvis der er noget: McGoatXIV@hotmail.com

Brugte du min kode?? hehe... bare nysgerrig? er lidt lettere.. især fordi man ikke hele tiden skal forbinde til databasen, men at det ligger i en seperat fil

og er også en fordel at bruge pconnect istedet for connect da man ikke behøver at skulle lukke forbindelsen i hver fil

ja jeg brugte den kode med de 3 scripts....jeg er som sagt ny i PHP og skal først til at se hvordan det hele hænger sammen :)
Go' nat...

nat nat