[PHP]Probleem met data opslaan in db

[Leon-]

Ik ben gister begonnen aan een control panel voor een website...alleen ik loop nu al vast, heb een tijdje geen php meer gedaan, het zou dus daar aan kunnen liggen.

Ik heb het volgende script geschreven:

PHP:

<?
include('connect.inc.php');
$tabelname = "admin";

if(!$content) {
?>
<form method="post"> 
<table>
      <tr>
        <td>Test</tr>
        <td><input type="text" name="content"></td>
            </tr>
            <td></td>
  <td><input type="submit" name="submit" value="submit"> <input type="reset" name="reset" value="reset"></td>
</table>
</form>
 <?
}
elseif ($submit) {
            $opdracht = "INSERT INTO $tabelname VALUES ('0', '$content)";
            $result = mysql_query($opdracht);
            if ($result) {
                echo "Succes...";
            } else {
                echo "Something went wrong with the database. mysql reports: ".mysql_error();
            }
        }
?>

Dit is mijn db connectie:

PHP:

<?php

//database connectie
$dbhost = "localhost";
$user = "user";
$pass = "wachtwoord";
$dbnaam = "dbnaam";
$server = mysql_connect($dbhost,$user,$pass);
mysql_select_db($dbnaam);

?>

Wanneer ik gegevens in de database wil opslaan en ik klik op de button submit, doet hij niks...

Ik hoop dat iemand een oplossing heeft voor mijn probleem!

[djluc]

if(isset($_POST['submit'])) o.i.d. Verder zou ik niet de submitknop maar een ander veld gebruiken om te bepalen wat de actie is omdat de submitknop niet wordt verzonden als je op enter drukt in plaats van op de button te klikken.

REQUEST_METHOD is een alternatief wat ook netjes is.

[ Voor 14% gewijzigd door djluc op 14-08-2005 12:48 ]

[foske]

Ik zou zeggen, ga eens wat variable uitprinten. Als hij helemaal niets uitprint, komt hij blijkbaar niet voorbij je If statements.
print dus $content en $submit eens uit. Want misschien hebben deze helemaal geen waarde?

[GlowMouse]

Staat register_globals aan? Zonee, kijk dan eens in de FAQ onder de vraag "Mijn POST en GET forms werken niet (meer)".


edit: je table lay-out klopt ook niet. Dat heeft niet te maken met dit probleem, maar is wel handig om goed te hebben:

code:

<table> 
  <tr> 
    <td>Test</td>
    <td><input type="text" name="content"></td> 
  </tr>
  <tr>
    <td></td> 
    <td><input type="submit" name="submit" value="submit"> <input type="reset" name="reset" value="reset"></td>
  </tr>
</table>

Daarnaast moet je bij een form de action-property opgeven.

[ Voor 98% gewijzigd door GlowMouse op 14-08-2005 12:59 ]

[Tux]

Maak geen gebruik van $content, maar van $_POST['content']

PHP:

//niet
if(!$content) { 
//maar
if ($_SERVER['REQUEST_METHOD'] == 'GET') {

PHP:

//niet
elseif ($submit) { 
//maar
elseif ($_SERVER['REQUEST_METHOD'] == 'POST') {

Verder zit er nog een fout in je MySQL query:

code:

INSERT INTO $tabelname VALUES ('0', '$content)

Je vergeet de ' na $content.
Je voert geen functie uit zoals addslashes, zodat er een SQL-injection lek in zit.
IMHO is het niet netjes om in een string value direct var's aan te spreken, ik gebruik altijd zoiets:

PHP:

$opdracht = "DIT IS EEN " . $vage . " QUERY";

[Verwijderd]

Leer jezelf aan om onder andere gebruik te maken van $_SERVER['REQUEST_METHOD'], in combinatie met bijvoorbeeld de waarde van een hidden field. Dus dan krijg je een situatie als;

PHP:

if($_SERVER['REQUEST_METHOD'] != 'POST' || ($_SERVER['REQUEST_METHOD'] == 'POST' && !isset($_POST['check']))) {

// hier form + hidden field met de naam CHECK

} else
if(isset($_SERVER['REQUEST_METHOD']) == 'POST' && isset($_POST['check'])) {

// verwerk data

}

Op deze wijze kun je namelijk ook, afhankelijk van bijvoorbeeld de waarde van de check-variabele, jouw pagina in meerdere delen opdelen. In het voorbeeld zijn twee `delen` te zien, afhankelijk van dus de handelingen van de gebruiker. Succes.

[ Voor 10% gewijzigd door Verwijderd op 14-08-2005 13:30 ]

[NMe]

Dit staat inderdaad gewoon letterlijk in de FAQ: P&W FAQ - PHP:Mijn POST en GET forms werken niet (meer). Verder is het niet de bedoeling dat wij hier jouw code gaan debuggen, dat mag je zelf doen. Om die twee redenen doe ik dit topic op slot.

[Creepy]

Hmmja.. kom op zeg Het gebruik van $_POST e.d. staat notabene in de FAQ Dat had je zelf ook wel kunnen vinden. Daarbij gaan we er ook vanuit dat je in eerste instantie zelf gaat debuggen en dat je ook aangeeft wat je hebt geprobeerd om je eigen probleem op te lossen. Dit laatste mis ik in de startpost.

Ik denk dat je er met de gegeven hints wel uit gaat komen. Mocht het echt niet lukken kijk dan even naar P&W FAQ - De "quickstart" zodat je weet wat we van een topicstart verwachten en open en nieuw topic

offtopic:
* Creepy aait -NMe-. Doe eens niet zo sneaky ervoor kruipen

[ Voor 7% gewijzigd door Creepy op 14-08-2005 13:37 ]