:: Buchempfehlung ::

ttkman · #1 27.03.2003, 12:09:31

schon mal im vorraus sorry für das lange script... bin hier an nem projekt für siemens und absoluter php-newbie ... habe ein flexibles php-file geproggt, das je nach ausgewählter datenbank die eingabemaske anpasst ... dies steht in der else anweisung ... in der if-anweisung wird, sobald der button gedrückt wird ein datensatz in eine sql-datenbank eingefügt.

jetzt zum problem ... die auswahl der db geschieht durch ein script bei dem 2 variablen --> db_name und table übergeben werden --> da geht es auch schon los... die variablen werden in der else-anweisung geladen (überprüft mit echo), in der if-bedingung bleiben sie jedoch leer (auch mit echo überprüft) .. bitte checked mal das script und macht mir mal nen vorschlag wie ich das problem lösen kann .. hab keine zeit das ganze nochmal umzuschreiben....danke schonmal ...

ANBEI: SCRIPT1+SCRIPT2

SCRIPT 1:
<!doctype html public "-//W3C//DTD HTML 4.0 //EN">
<html>
<head>
<title>Datensatz zu hinzufuegen</title>

</head>
<body>
<?php

if($submit)

{
MYSQL_SELECT_DB($_SESSION["db_name"]);
$result = (mysql_query("SELECT * FROM $_SESSION['table']"));
$anz=mysql_num_fields($result);
$k=0;
$i=0;
$lname=mysql_field_name($result,$i);
do
{
$lname=mysql_field_name($result,$i);
$Inhalt[$k] = """.$Feld[$i].""";
$lname2[$k] = $lname;
if($k<$anz-1)
{
$text2=$text2.$Inhalt[$k].",";
}
else
{
$text2=$text2.$Inhalt[$k];
}
if($k<$anz-1)
{
$text=$text.$lname2[$k].",";
}
else
{
$text=$text.$lname2[$k];
}
$i++;
$k++;
}
while($i<$anz);
$ertl = mysql_query ("INSERT INTO $table ($text) VALUES ($text2)");
?>
<?php
}
else
{
$conn = MySQL_Connect("localhost","root","")or die ("ERROR - Konnte eine Verbindung zur
Datenbank herstellen");
if ($conn) echo " Verbindung zur Datenbank erfolgreich hergestellt";
MYSQL_SELECT_DB($db_name);
$result = (mysql_query("SELECT * FROM $_SESSION['table']"));
$anz=mysql_num_fields($result);
$i=0;
echo "<form action="$PHP_SELF"method="post">";
echo "<center><input type="Submit" name="submit" value="Eingabe"><BR><BR>";
do
{
$lname=mysql_field_name($result,$i);
echo "<center>$lname<center><input name="Feld[$i]" size=20 maxlength=5 value=><br>";
$i++;
}
while($i<$anz);
}
echo "</form>";
?>
</body>
</html>

SCRIP2:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Strict//EN" //übergabescript
"http://www.w3.org/TR/html4/strict.dtd">
<html>
<head>
<title>Bitte $db_name,$table angeben</title>
</head>

<form action="insert.php" method="post">"
<input type="text" size="17" name="db_name">
<input type="text" size="17" name="table">
<br><br>
<center>
<input type="submit" value="OK">
</center>
</form>

</html>

Progman · #2 27.03.2003, 17:00:54

http://tut.php-q.net/
Das sieht verdammt unübersichtlich aus. Lies mal dieses Tutorial.

**|Coding** · #3 29.03.2003, 19:47:36

@Progman:

das liegt daran, weil er nicht das [php] oder [code] tag verwendet hat.

bbastix · #4 29.03.2003, 20:04:07

Hi,

ich würde auch zunächst mal versuchen, das Skript ein wenig zu "entwirren"!

MySQL:
In der "if-Anweisung" fehlt der Verbindungsaufbau. Pack die MySQL-Geschichten doch erstmal in eine Klasse (dazu gibt es bestimmt einige freie im Netz, aber ansonsten ist sowas ja auch schnell zusammengeschrieben).

Übergebene Variablen:
Du liest $db_name einmal aus $_SESSION aus und einmal aus dem globalen Namensraum. Beides dürfte nicht funktionieren, da du weder eine Session laufen hast, noch die Variable irgendwo global gesetzt hast.

HTML-Tabellen und -Forms:
Die GUI würde ich in zumindest als "Pseudo-"Templates auslagern, sonst wirst du ja nicht mehr froh! Also lieber "oben" mit PHP-Code die Eingaben verarbeiten und Ausgabewerte in einen Variablen-Kontainer packen und dann "unten" oder in einem Include-File nur den HTML-Code ausgeben, ergänzt um die PHP-Variablen und ggf. auch -Schleifen und -Bedingungen.

Kodierungskonventionen:

Deine Variablen haben ja hübsch durcheinandere Bezeichnungen:

lname - db_name (mal mit Unterstrich, mal verbunden)
Inhalt, Feld (wobei ich garnicht weiß, woher die kommen) - und dann wieder Kleingeschriebenes
Und dann diese "text2" und "lname2" ... okay, Geschmackssache.

Aber lies mal ein bischen zu dem Thema, schau dich um, was so üblich ist und leg dich dann (zunächst ja im eigenen Interesse) auf bestimmte Regeln fest.

Kommentieren!!
Ich seh keinen einzigen Kommentar in dienem Skript. Besser solltest du jeden Schritt kommentieren, den du machst, sonst weißt du nach wenigen Tagen nicht mehr, was du gemacht hast bzw. du musst es erst wieder herausfinden, anstatt es einfach abzulesen!

...und dann bietet auch dieses Board die Möglichkeit Code entsprechend zu kennzeichnen, damit die Einrückungen nicht verloren gehen:

Anstatt:

Code:

if ($trallala) {

    if (!do_something($trallala)) {

        $error[] = "Don't like to do so";
    }
}

kommt dann halt sowas bei raus:

if ($trallala) {

if (!do_something($trallala)) {

$error[] = "Don't like to do so";
}
}

Ja, und zum konkreten Problem nochmal:
Wenn der gepostete Code alles ist, dann kann da einiges nicht gehen, ansonsten poste doch mal das ganze Skript (am besten nachdem du es etwas geordnet hast...)

Basti

**|Coding** · #5 29.03.2003, 20:14:40

Zitat:

Original geschrieben von bbastix
...ansonsten poste doch mal das ganze Skript...

ich verweise jetzt, rein vorsichtshalber, mal auf diese seite: click

:: Buchempfehlung ::

:: Anbieterverzeichnis ::

Globale Branchen

Informieren Sie sich über ausgewählte Unternehmen im Anbieterverzeichnis von SELFPHP

:: Newsletter ::

Abonnieren Sie hier den kostenlosen SELFPHP Newsletter!