PHP Problem: Datei erstellen

Tobse

Ich nochmal mit meim upload-script:
Ich habe da ein formular, in das der user dateien von seinem computer hochlädt. Er sieht das formular auf server1(ohne php) und es wird dann an server2(mit php, logisch) gesendet. Dor soll dann ein neuer ordner im verzeichnis uploads erstellt werden, der den name der eingegebenen mailadresse trägt. In den ordner sollen dann die daten gespeicher werden und eine Textdatei mit den anderen eingaben angelegt werden. Über phpmailer wird dann eine bestätigungs e-mail an die eingegebene adresse geschikt und eine mail an mich, um mir mitzuteilen, dass eine neue bestellung einging.
Nun sind da folgende probleme: Der ornder wird nicht erstellt, das heisst, die datei wird nicht erstellt und die dateien(bilder) werden nicht hochgeladen. Und zu allem Unglück wird nur die mail an mich verschickt.
Hier der Code:

PHP

<?php
$text = "";
$error = "";
$mailfeld=$_POST['email'];
$handy=$_POST['mbl'];
$ext=$_POST['aussenscreen'];
$desingtitel=$_POST['desingtitel'];
$desingthema=$_POST['thema'];
$normalpc=$_POST['normpc'];
$normalweb=$_POST['normweb'];
$menupc=$_POST['menupc'];
$menuweb=$_POST['menuweb'];
$navipc=$_POST['navipc'];
$naviweb=$_POST['naviweb'];
$popuppc=$_POST['popuopc'];
$popupweb=$_POST['popupweb'];
$aktionenpc=$_POST['aktionenpc'];
$aktionenweb=$_POST['aktionenweb'];
$ringpc=$_POST['ringpc'];
$ringweb=$_POST['ringweb'];
$smspc=$_POST['smspc'];
$smsweb=$_POST['smsweb'];
$something=$_POST['somethingtotell'];
$upload_verzeichnis="uploads/$mailfeld";
if(isset($_POST['text']))
{
    $errors = array();


    if (empty($_POST['text']))
      $error = "Kein Text eingegeben";
    else
    {
       mkdir("uploads/$mailfeld", 0777);
       $fp = @fopen("uploads/$mailfeld/infos.txt", "w+")
           or die ("Fehler beim öffnen der Datei");


       fputs ($fp, "email: $mailfeld\nhandy:$handy\naussenbildschirm:$ext\ntitel:$desingtitel\nthema:$desingthema
                    normalzustand: $normweb\nmenü: $menuweb\n navi: $naviweb\n popup: $popupweb\n aktionen: $aktionenweb\n klingelton: $ringweb\n smston: $smsweb");
       fclose($fp);
    }
}
if (isset($_FILES['$normalpc']['name'])) {
    $dateiname = $_FILES['$normalpc']['standby'];
if (move_uploaded_file($_FILES['$normalpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname1"))
if (isset($_FILES['$menupc']['name'])) {
    $dateiname1 = $_FILES['$menupc']['dektop'];
if (move_uploaded_file($_FILES['$menupc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname2"))
if (isset($_FILES['$navipc']['name'])) {
    $dateiname2 = $_FILES['$navi']['naviagtion'];
if (move_uploaded_file($_FILES['$navipc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname3"))
if (isset($_FILES['$popuppc']['name'])) {
    $dateiname3 = $_FILES['$popuppc']['popup'];
if (move_uploaded_file($_FILES['$popuppc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname4"))
if (isset($_FILES['$aktionenpc']['activity'])) {
    $dateiname4 = $_FILES['$aktionenpc']['activity'];
if (move_uploaded_file($_FILES['$aktionenpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname5"))
if (isset($_FILES['$ringpc']['name'])) {
    $dateiname5 = $_FILES['$ringpc']['klingelton'];
if (move_uploaded_file($_FILES['$ringpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname5"))
if (isset($_FILES['$smspc']['name'])) {
    $dateiname6 = $_FILES['$smspc']['smston'];
if (move_uploaded_file($_FILES['$smspc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname6"))
  //Klasse einbinden
  require('phpmailer/class.phpmailer.php');


  //Instanz von PHPMailer bilden
  $mail = new PHPMailer();


  //Absenderadresse der Email setzen
  $mail->From = "$mailfeld";


  //Name des Abenders setzen
  $mail->FromName = "ThemeMania";


  //Empfängeradresse setzen
  $mail->AddAddress("kontakt@thememania.net");


  //Betreff der Email setzen
  $mail->Subject = "Neues Desing bestellt";


  //Text der EMail setzen
  $mail->Body = "
Es wurde ein neues desing bestellt:\n thememania.bplaced.net/uploads/$mailfeld/
";


  //EMail senden und überprüfen ob sie versandt wurde
  if(!$mail->Send())
  {
     //$mail->Send() liefert FALSE zurück: Es ist ein Fehler aufgetreten
     echo "Die Email konnte nicht gesendet werden";
     echo "Fehler: " . $mail->ErrorInfo;
  }
$mail->ClearAttachments(); //Attachments entfernen
  //Instanz von PHPMailer bilden
  $mail = new PHPMailer();


  //Absenderadresse der Email setzen
  $mail->From = "no-reply@thememania.net";


  //Name des Abenders setzen
  $mail->FromName = "ThemeMania";


  //Empfängeradresse setzen
  $mail->AddAddress("$mailfeld");


  //Betreff der Email setzen
  $mail->Subject = "Dein Desing ist in bearbeitung/Your order is done";


  //Text der EMail setzen
  $mail->Body = "
Dein Desing ist jetzt in bearbeitung.\n
Danke, das du ThemeMania benutzt.\n
Du erhälst eine weitere E-Mail, wenn das Desing fertig ist.\n
Your Order is done, so your Theme is now in work.\n
Thanks for using ThemeMania.\n
You´ll get another E-Mail, when your Theme is finished.
";
?>

Alles anzeigen

Und ich will KEINEN neuen code, ich hätte nur gerne, dass sich einer die mühe macht, mir das script zu reparieren, da es nur zusammengebastelt ist weil ich kein php kann.

Bandit

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(E_ALL); und ini_set("display_errors", true);
3. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
4. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
5. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
6. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
7. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
8. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Tobse

Wenn man kein php kann, kann man das versuchen. Meinst du, da kommt was raus?? Ich werds trotzdem machen.

Bandit

Zitat von Tobse

Meinst du, da kommt was raus??

Ob man PHP kann oder nicht ist dabei wurscht. Manchmal ist das ein guter Weg, Fehler zu finden, wenn man keinen Debugger hat.

Tobse

Und as problem ist:
Parse error: syntax error, unexpected $end in /users/thememania/www/bestellen.php on line 127
Na toll. Und wie brng ich dem jz bei, dass das script aufhört?

Bandit

Woher sollen wir jetzt wissen, was du da eingebaut hast. Sicher ist aber, dass du dabei einen Syntax-Error hervorrufst.

Tobse

Line 127 ist
?>

Bandit

Das Script hat doch vorher funktioniert, bis auf das Anlegen der Datei. Jetzt hast du bestimmt versucht, "Richtig debuggen" umzusetzen und dabei hast du was falsch gemacht. Aber was du falsch gemacht hast, können wir nicht wissen!!!

Tobse

Das script hat vorher so funtioniert,d as die mails versandt wurden. aber ich BRAUCH die Dateien, die der user hochgeladen hat. also hab ich

PHP

if(isset($_POST['text']))
{
    $errors = array();


    if (empty($_POST['text']))
      $error = "Kein Text eingegeben";
    else
    {
       mkdir("uploads/$mailfeld", 0777);
       $fp = @fopen("uploads/$mailfeld/infos.txt", "w+")
           or die ("Fehler beim öffnen der Datei");


       fputs ($fp, "email: $mailfeld\nhandy:$handy\naussenbildschirm:$ext\ntitel:$desingtitel\nthema:$desingthema
                    normalzustand: $normweb\nmenü: $menuweb\n navi: $naviweb\n popup: $popupweb\n aktionen: $aktionenweb\n klingelton: $ringweb\n smston: $smsweb");
       fclose($fp);
    }
}

Alles anzeigen

und

PHP

if (isset($_FILES['$normalpc']['name'])) {
    $dateiname = $_FILES['$normalpc']['standby'];
if (move_uploaded_file($_FILES['$normalpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname1"))
if (isset($_FILES['$menupc']['name'])) {
    $dateiname1 = $_FILES['$menupc']['dektop'];
if (move_uploaded_file($_FILES['$menupc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname2"))
if (isset($_FILES['$navipc']['name'])) {
    $dateiname2 = $_FILES['$navi']['naviagtion'];
if (move_uploaded_file($_FILES['$navipc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname3"))
if (isset($_FILES['$popuppc']['name'])) {
    $dateiname3 = $_FILES['$popuppc']['popup'];
if (move_uploaded_file($_FILES['$popuppc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname4"))
if (isset($_FILES['$aktionenpc']['activity'])) {
    $dateiname4 = $_FILES['$aktionenpc']['activity'];
if (move_uploaded_file($_FILES['$aktionenpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname5"))
if (isset($_FILES['$ringpc']['name'])) {
    $dateiname5 = $_FILES['$ringpc']['klingelton'];
if (move_uploaded_file($_FILES['$ringpc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname5"))
if (isset($_FILES['$smspc']['name'])) {
    $dateiname6 = $_FILES['$smspc']['smston'];
if (move_uploaded_file($_FILES['$smspc']['tmp_name'],
                             "$upload_verzeichnis/$dateiname6"))

Alles anzeigen

dazugemacht. sonst nix.

Bandit

Ich würde mal die { und } zählen, müssen ja beide gleich viel vorhanden sein. Das ist bei dir nicht der Fall. Arbeite doch bitte mal einige Grundlagentutorials durch!

Tobse

Das is mir auch schon aufgefalln. Dann heist es aber, dass er } nicht erwartet hat.
Das script sollte am 21. um 12:00 irgendwie laufn, hab schon werbung gemacht *schwiz*

Bandit

PHP ist überrascht, dass das Script schon beendet ist. Denn es sind jede Menge öffnende Klammern vorhanden, die nie geschlossen werden. Das mag keine Programmiersprache.

Tobse

Ok, also alle geschlossen. Nun weiss ich aber nicht was

Code

Undefined index:  popuopc in [B]/users/thememania/www/bestellen.php[/B] on line [B]17
[/B]

bedeutet. (das kommt für alle variablen von uploads)

Bandit

Das bedeutet, dass es diese Variablen nicht gibt. Vermutlich gibt es diese Felder in deinem Formular nicht.

Ich wiederhole mich eigentlich nur ungern:

Zitat von bandit600

Arbeite doch bitte mal einige Grundlagentutorials durch!

http://www.php.net/manual/de
http://www.php-faq.de
http://www.php-einfach.de
http://tut.php-q.net/index.html
http://dev.mysql.com/doc/refman/5.1/de/index.html
http://ffm.junetz.de/members/reeg/DSP/
http://www.phptutorials.de
http://www.dynamic-webpages.de/05.tutorials.php
http://de.selfhtml.org/
http://www.css4you.de/index.html

Afrael

Bandit, die wahrscheinlichste Ursache ist immer noch, dass das Formualr noch nicht abgeschickt wurde

Tobse

Wie kann das sein?? Das formular geht üer method="post" an die php. Und so weit ich weiss gibt es die felder auch. ich prüfe das formular nochmal.

Afrael

Beim ersten Aufruf des Scripts gibt es logischerweise keine Postdaten, da du das Formular noch nicht abgeschickt hast. Ich deklariere meine Variablen immer so:

PHP

$mail = isset($_POST['mail'])? $_POST['mail'] : "";
$zahl = isset($_POST['zahl'])? (int) $_POST['zahl'] : 0;

Tobse

ja und wiso sind dann die anderen variablen verfügbar???

Afrael

Zitat von Tobse

ja und wiso sind dann die anderen variablen verfügbar???

Zitat

(das kommt für alle variablen von uploads)

Was denn nu?

The User

Na ganz einfach:
Es sollte popuppc und nicht popuopc heißen.

Tags