Christian Seiler: mysql-tabellen mit php automatisch erstellen

Beitrag lesen

Hallo marcu,

$use_db = "use"$db;

Hier fehlt a) ein leerzeichen und b) der Verkettungsoperator:

$use_db = "use " . $db;

oder halt

$use_db = "use $db";

if (!mysql_query($use_db,$connect)) die("verbindungsfehler");        // Verbindungsaufbau

Ich würde lieber mysql_select_db nehmen...
if (!mysql_select_db($db, $connect)) die ("verbindungsfehler");

if (!mysql_query($use_db,$create_software)) die("create_software fehler"); // Erstellt Software-Tabelle

falsch. Richtig:

if (!mysql_query($create_software, $connect)) die(...);

if (!mysql_query($use_db,$connect,$create_ea)) die("create_ea fehler");    // Erstellt EA-Tabelle
if (!mysql_query($use_db,$connect,$create_hardware)) die("create_hardware fehler"); // Erstellt Hardware-Tabelle

s.o.

kann sein das so noch paar fehler drinn sind aber die dürften nich das erstellen behindern oder :(????

doch.

Ach ja, vielleicht ist *zusätzlich* zu den bekannten Fehlern noch register_globals aus, daher solltest Du auf Formularfelder über $_POST["variablenname"] anstelle von $variablenname zugreifen, also z.B.

if ($_SERVER["REQUEST_METHOD"] == "POST" && isset($_POST["submit"])) {

$connect = mysql_connect($_POST["hostname"], $_POST["user"], $_POST["pw"]) or die ("verbindungsfehler");

etc.pp.

Grüße,

Christian

--
Sollen sich alle schämen, die gedankenlos sich der Wunder der Wissenschaft und Technik bedienen und nicht mehr davon erfasst haben als eine Kuh von der Botanik der Pflanzen, die sie mit Wohlbehagen frisst.
                      -- Albert Einstein