Mp3 numre

Prøv med:
$query = mysql_query("INSERT INTO mp3numre (kunstner, titel) VALUES ('$k','$t')") or die (mysql_error());

$query = "INSERT INTO mp3numre (kunstner, titel) VALUES ('$k','$t')";
mysql_query($query);

ahv: Hvorfor vil du først lægge query'en i en variabel og først derefter proppe den i mysql_query??

Ok hvad fejler koden her????

du mangler METHOD i din form linie - bør være: <form action="<? echo $php_self ?>" method="post">

Hvis du kører med en phpversion der er ver. 4.2.+ skal du tilgå variablerne med $_POST (se evt. www.wagners.dk/php - artiklen om $_GET og $_POST)

dvs din kode skal være:
<?
if(isset($_POST['add_mp3'])) {
mysql_connect("localhost","root");
mysql_select_db("mp3");
$query = mysql_query("INSERT INTO mp3numre (kunstner, titel)". "VALUES ('".$_POST['k']."','".$_POST['t']."')");
echo "dit nummer er tilføjet. tilføj flere eller vis dine numre <a href=\"mp3er.php\">her.</a>";
}
?>

Et forslag fra min side :)

<html>
<html>
<head>
  <title>Tilføj MP3-nummer</title>
</head>

<body>
<h3 align="center">Tilføj MP3-nummer</h3>
<br /><br />
<form action="<?php echo $_SERVER['PHP_SELF'] ?>" method="post">
Kunstner:<br />
<input type="text" name="k" />
<br /><br />
Titel:<br />
<input type="text" name="t" /><br />
<input type="submit" name="add_mp3" value=" - OK - " />
</form>
<?php
if ($_POST['add_mp3'])
{
    mysql_connect("localhost","root") or die(mysql_error());
    mysql_select_db("mp3") or die(mysql_error());
    $query = "INSERT INTO mp3numre (kunstner, titel) VALUES ('$_POST[k]','$_POST[t]')";
    mysql_query($query) or die(mysql_error());
    echo "Dit nummer er tilføjet. Tilføj flere eller vis dine numre <a href=\"mp3er.php\">her.</a>";
}
?>
</body>
</html>

Jeg undskylder for at være perfektionist ;)

jon>> perfektionist ok ;o)

meeen det er vist god skik at holde php og sql for sig derfor er det perfekte:
VALUES ('".$_POST['k']."','".$_POST['t']."')");

men indrømmet - dit vil også virke, jeg f.eks havde glemt $_SERVER['PHP_SELF']

avminarm: Havde det selvfølgelig med i overvejelserne, men fandt at det ville virke uden at skabe alt for megen forvirring. Af samme grund har jeg også udeladt det meste XHTML-stads, selvom jeg måtte have lidt med ;)

Og når vi alligevel snakker god kodeskik: Det føromtalte kan diskuteres - det kan <? ?> vs. <?php ?> _ikke_ ;)

jon>> enig (du ved copy/paste ;-])

og xhtml - jeg ved ikke - tror du ikke det vil forvirrer en nybegynder og der er vel ikke rigtig nogen vigtig grund til at benytte det - endnu - jeg ved godt man lige så godt kan vænne sig til det, meeen.

hvilket xhtml har du udeladt? - det ser ret fuldendt ud ;o)

avminarm: Flere ting er udeladt, DTD bl.a. ;) Og ja, det er klart en vanesag. Jeg skriver aldrig <br> i kode længere - det bliver automatisk til <br /> :)

rocky2000:
Fungerede det efter hensigten?