[php] Query werkt niet

vrijdag 13 december 2002 20:04

Acties:

0 Henk 'm!

Topicstarter

Ik ben een beginneling in PHP en heb een scriptje gemaakt die FTP servers checkt als ze online zijn. Maar nu wou ik een formuliertje maken die gegevens in de database zet, zodat ik dit niet steeds zelf met PHPmyAdmin moet doen. Maarjah, nu krijg ik steeds een fout. Misschien dat 1 van julie de oplossing weet.
In regel '65' zit de fout. Het ligt dus aan de SQL query.

PHP:

<html>
<head>
<? include("../cfg/solopher.php");?>
<title><? echo("$title"); ?></title>
</head>
<? echo("$body"); echo("$font");?>
<p><font size="4"><b>New Server</b></font></p>
<!-- <form method="<? $HTTP_POST_VARS[post]; ?>" action="<? echo("$PHP_SELF"); ?>"> -->
<form method="post" action="<? echo("$PHP_SELF"); ?>">
<table width="75%" border="0" cellspacing="0" cellpadding="0">
    <tr>
    <td width="14%">Server IP</td>
    <td width="26%">
    <input type="text" name="ip" size="35"></td>
    <td width="60%">Insert the IP of the Server here</td>
    </tr>
    <tr>
    <td>Server Port</td>
    <td><input type="text" name="port"  size="35"></td>
    <td>Insert the Port of the Server here</td>
    </tr>
    <tr>
    <td>Server Name</td>
    <td><input type="text" name="name"  size="35"></td>
    <td>Insert the Name of the Server here</td>
    </tr>
    <tr>
    <td>Server Size</td>
    <td><input type="text" name="size""  size="35"></td>
    <td>Insert the Size of the Server here in GB's</td>
    </tr>
    <tr>
    <td>Username</td>
    <td><input type="text" name="user"  size="35"></td>
    <td>Insert the Username here (if ther is no username leave it empty)</td>
    </tr>
    <tr>
    <td>Passport</td>
    <td><input type="text" name="pass"  size="35"></td>
    <td>Insert the Passport here (if ther is no passport leave it empty)</td>
    </tr>
    </table>
<INPUT TYPE="HIDDEN" NAME="stage" value="1">
    <input type="image" src="send.gif" name="sub">
</form>

<?
if(!isset($_POST["stage"])) {
print "";
}else{

 /* Opent Verbinding met Database */
  mysql_connect("localhost", "root", "") or die("Error: Can't Connect MySQL Server");
  mysql_select_db("solopher") or die("Error: Can't Select Database");

 /* Pakt waardes van de form en zet ze om in normale variabelen */
  $ip   = $_POST["ip"];
  $port = $_POST["port"];
  $name = $_POST["name"];
  $size = $_POST["size"];
  $user = $_POST["user"];
  $pass = $_POST["pass"];

 /* Voegt waardes aan database toe */
$sql = 'INSERT INTO status (id, ip, port, name, size, user, pass) VALUES ('', '$ip', '$port', '$name', '$size', '$user', '$pass')';

$result = mysql_query($sql) or die(exit);  
echo("Your Information has been added.");

}
?>
<? echo("$end"); ?>

vrijdag 13 december 2002 20:06

Acties:

0 Henk 'm!

RedHat

PHP:

1	$result = mysql_query($sql) or die(exit);

word

PHP:

1	$result = mysql_query($sql) or die(mysql_error());

dan geeft ie een foutmelding, die mag je mij geven

[ Voor 20% gewijzigd door RedHat op 13-12-2002 20:06 ]

vrijdag 13 december 2002 20:07

Acties:

0 Henk 'm!

SchizoDuckie

Kwaak

[quote]

PHP:

1	$sql = 'INSERT INTO status (id, ip, port, name, size, user, pass) VALUES ('', '$ip', '$port', '$name', '$size', '$user', '$pass')'

leer eerst even hoe het werkt met strings aan elkaar plakken.
zo werkt het wel waarschijnlijk:

PHP:

1	$sql = "INSERT INTO status (id, ip, port, name, size, user, pass) VALUES ('', '$ip', '$port', '$name', '$size', '$user', '$pass')"

[ Voor 28% gewijzigd door SchizoDuckie op 13-12-2002 20:08 ]

Stop uploading passwords to Github!

vrijdag 13 december 2002 20:08

Acties:

0 Henk 'm!

supakeen

Hier had je me voor gemaild en de source code die ik je terug stuurde was dit:

PHP:

1	$sql = "INSERT INTO status (id, ip, port, name, size, user, pass) VALUES ('', '$ip', '$port', '$name', '$size', '$user', '$pass')"

vrijdag 13 december 2002 20:09

Acties:

0 Henk 'm!

RedHat

klopt, met single quote gaat het volgens mij niet werken, ben ik niet 100% zeker van

en vergeet bij bovenstaande van zmn niet ff een ; erachter te zetten.

[ Voor 30% gewijzigd door RedHat op 13-12-2002 20:09 ]

vrijdag 13 december 2002 20:17

Acties:

0 Henk 'm!

Solopher

Topicstarter

Okeej, dan !! Het werkt eindelijk

maar nu snap ik eigenlijk nog steeds niet wat ik fout gedaan heb en zo leer ik er eigenlijk dus niet. En staat er de volgende keer weer zo'n n00b topic van mij

vrijdag 13 december 2002 20:29

Acties:

0 Henk 'm!

Glimi

Designer Drugs

solopher schreef op 13 December 2002 @ 20:17:
Okeej, dan !! Het werkt eindelijk maar nu snap ik eigenlijk nog steeds niet wat ik fout gedaan heb en zo leer ik er eigenlijk dus niet. En staat er de volgende keer weer zo'n n00b topic van mij

Nee, dat gaat er volgende keer niet staan

Weet je hoe dat kan? Met het geweldige 4 stappen plan!

Lees Welkom in P&W -> Quickstart (update 2/10/2002)
Lees, en dan met bijzondere attentie de stukjes over PHP en SQL, Welkom in P&W (FAQ-28/11/2002)
Koop een boek over PHP of zoek ladingen tutorials over PHP, zodat je lekker kunt oefenen en naslag hebt
Heb je een probleem, zoek eerst op http://www.php.net/manual/en en www.google.com, alvorens het hier te posten

Ge-ga-ran-deerd geen n00b/ik-heb-geen-idee-waarmee-ik-bezig-ben topic meer

Herrinder je dit wel: Als je inzet toont, dan komt het wel goed

vrijdag 13 december 2002 20:31

Acties:

0 Henk 'm!

Verwijderd

Gaat fout omdat de variabele definitie stopt bij de 2e single qoute. De definitie loopt namelijk van de 1e tot de 2e qoute. Vandaar dat je daar dus " voor gebruikt...

vrijdag 13 december 2002 22:03

Acties:

0 Henk 'm!

jan-marten

utp

met ' ben je behoorlijk beperkt qua tekenreeks. Met de " kun je veel meer dingen invoeren (dus ook die ' ).
Staat wel ergens in de manual van php.

vrijdag 13 december 2002 22:10

Acties:

0 Henk 'm!

Kaastosti

Vrolijkheid alom!

Je hoeft die hele rij niet eens op te geven. Als ik het zo bekijk gok ik dat je alle waarden die je mogelijkerwijs in die tabel in kan voeren ook werkelijk invoert. Dan heb je hieraan genoeg:

PHP:

1	$sql = "INSERT INTO status VALUES ('', '$ip', '$port', '$name', '$size', '$user', '$pass')"

Probleem is al opgelost lees ik, maar het scheelt je bij volgende pagina's misschien tikken

Een vergissing is menselijk, maar om er echt een puinhoop van te maken heb je een computer nodig.

Onderwerpen