PHP Script bricht beim Return ab

**Kropff** · 25.01.2012, 12:38

Zeig erst mal, wie du die csv-Datei ausliest und vor allem, wie die csv-Datei ausieht (ein Beispiel reicht)

Peter

**cami3004** · 25.01.2012, 12:44

PHP-Code:


@products_id|language_code|products_name|products_description|products_short_description|products_keywords|products_url
5|de|Name| Beschreibung | xxx | xxx |xxx.html

PHP-Code:


/*Datenbank Vorgaben*/

define("server", "Server");
define("user", "db-Benutzer");
define("password", "passwort");
define("database", "db...");


$dir_handle = opendir(dirname(__FILE__));

while (($file = readdir($dir_handle)) !== false) {
    ini_set("max_execution_time", "120");
    if ($file != '..' AND $file != '.' AND (strtolower(substr($file, -4)) == ".csv")) {
        $data = parse_csv($file);

        if (update_db($data) >= 1){
            echo "Daten aus der Datei <b>" . $file . "</b> wurden in der Datenbank aktualisiert.<br>";
            unlink($file);
        }
        else{
            echo "Die Daten der Datei <b>" . $file . "</b> konnten <u>nicht</u> in die Datenbank &uuml;bernommen werde!<br>";
        }
    }
    ini_restore("max_execution_time");
}


function get_field_name($table_name, $DB = database)
{
    $table_meta_name = a....( /*$s...n*/);
    for ($i = 0; $i < mysql_..._fiel..(mysql_list_fields($DB, $t...e_name)); $i++) {
        array_push($table_meta_name, mysql_f..._name(mysql_list_fields($DB, $table_name), $i));
    }
    return ($table_me..._name);
}


function parse_csv($file)
{
    $file_handle = fo...($file, "r");
    $data = array();
    while (($col = fget...($file_ha..le, 0, '|')) !== FALSE) {
        foreach ($col as $key => $value) {
            $col[$key] = trim($col[$key]);
            $col[$key] = str_replace(';">"', '|">', $col[$key]);

            //echo $col[$key]."<br>";
        }
        array_push($data, $col);
    }
    fclose($file_handle);
    return $data;
}

function dbupdate($table, $values, $conditions, $DB = database)
{
    $query = 'UPDATE `' . $table . '` SET ';
    $key_value = '';
    $condition = ' WHERE ';
    foreach ($values as $key => $value) {
        $key_value .= '`' . $key . '`=\'' . $value . '\', ';
    }

    foreach ($... as $... => $cva...e) {
        $condition .= '`' . $ckey . '`=\'' . $..alue . '\' AND ';
    }
    $key_value = substr($key_value, 0, -2);
    $condition = substr($condition, 0, -5);
    //echo $query.$key_value.$condition."<br><br>";
    mysql_query($query . $key_value . $condition);
    return TRUE;

}

function update_db($data)
{
    $err = true;
    $sql_handle = mysql_pconnect(server, user, password);
    mysql_select_db(database, $sql_handle);

    //--------------------------------------------------------------------
    $fields_in_csv = $data[0];
    $table_handle = mysql_list_tables(database, $sql_handle);
    $tables = array();
    while ($row = mysql_fetch_row($table_handle)) {
        array_push($tables, $row[0]);
    }

    $tables = array_flip($tables);
    $tables_alph = $tables;

    foreach ($tables_alph as $tkey => $tval) {

        $tables_alph[$tkey] = 0;
    }
    foreach ($tables as $key => $value) {
        $tables[$key] = get_field_name($key);
        foreach ($tables[$key] as $innerkey => $innervalue) {

            if (array_...($innervalue, $fields_in_csv) !== FALSE)
                $tables_...h[$key] = $tables_alph[$key] + 1;
        }
    }

    asort($tables_alph);
    $tables_alph = array_flip($tables_alph);

    $table_to_update = end($tables_alph);
    //-----------------------------------------------------------


    $fields_in_table = array();

    $fields_in_table = get_field_name($table_to_update);

    $umlaute = array("ä" => "&auml;", "Ä" => "&Auml;", "ü" => "&uuml;", "Ü" => "&Uuml;", "ö" => "&ouml;", "Ö" => "&Ouml;");

    $update = array();
    $condition = array();
    $condi_count = 0;

    for ($i = 1; $i < count($data); $i++) {
        for ($y = 0; $y < count($fields_in_csv); $y++) {
            if ($fields_in_csv[$y][0] == "@") {
                $condi_count = $condi_count + 1;
                $condition = array_merge($condition, array(substr($fields_in_csv[$y], 1) => $data[$i][$y]));
            }
            else {
                $update = array_merge($update, array($fields_in_csv[$y] => strtr($data[$i][$y], $umlaute)));
            }

        }
        if ($condi_count >= 1) {
            dbupdate($table_to_update, $update, $condition);
            $err = $err + 1;
        }
        else {
            return $err = "Keine Bedingungen gefunden!";
        }
    }
    return $err;
}

**ezkimo** · 25.01.2012, 13:03

Dein Problem werden wahrscheinlich die Zeilenumbrüche innerhalb des HTML Codes sein. Die gleichen Zeilenumbrüche werden nämlich wahrscheinlich auch am Ende jeder CSV Zeile stehen. Demnach geht das Script wahrscheinlich davon aus, dass die Zeile beim ersten Vorkommen eines Zeilenumbruchs beendet ist.

Alternativ könntest Du beim Auslesen Deiner CSv Datei mit fgetcsv auch ein Enclosure setzen, welches dieses Problem wahrscheinlich beheben würde. Vorausgesetzt Deine Spalteninhalte sind mit einem Zeichen (Enclosure) umschlossen. Wenn sie einfach so wie in Deinem Beispiel ohne jegliches umschließendes Zeichen in der Datei stehen, ist das ... Künstlerpech.

**cami3004** · 25.01.2012, 13:16

Sicherlich sind die Absätze auch in der CSV Datei drin. Diese wurden ja aus dem HTML Code direkt aus der Datenbank übernommen.
Dann sollten diese der Bequemlichkeit in einem externen Editor bearbeitet werden und dann wieder mittlels Skript eingefügt werden.

Ich kann die Daten ja aus der SQL Datenbank mit Myphp Admin neu auslesen und somit dann auch den Spaltentrenner hinterlegen.

Ich werde dann mal die fgetcsv Funktion ersetzten und die neue Ausprobieren, dafür muss ich aber erstmal die neue Funktion kennenlernen.

Ist mir noch unbekannt.
Aber vielen Dank schonmal für den Tipp.

Gruß
Carsten

**Kropff** · 25.01.2012, 20:19

Zitat von cami3004 Beitrag anzeigen

Diese wurden ja aus dem HTML Code direkt aus der Datenbank übernommen.

Wäre da XML nicht besser zu verarbeiten? Vorrausgesetzt, der HTML-Code ist valide.

Peter

**cami3004** · 26.01.2012, 08:18

Kann ich denn ganz einfach in dem Script den Datei Typ ändern?
Wahrscheinlich doch nicht.

Alternativ hatte ich mir gedacht, kann ich doch noch einen Separaten Spaltentrenner in dem Skript einfügen, oder nicht?

Gruß
Cami3004

**h3ll** · 26.01.2012, 09:26

Ich versteh nicht warum du da mit Spaltentrenner herumtust. fputcsv() und fgetcsv() erledigen die ganze Arbeit für dich. Du musst dich um sonst nichts kümmern.

Warum baust du eigentlich eine persistende Datenbankverbindung auf?

Bei dir fehlt das SQL-Escaping der Daten.

Warum ersetzt du Umlaute durch HTML-Entities?

**cami3004** · 26.01.2012, 11:01

PHP-Code:


Ich versteh nicht warum du da mit Spaltentrenner herumtust. fputcsv() und fgetcsv() erledigen die ganze Arbeit für dich. Du musst dich um sonst nichts kümmern.

Warum baust du eigentlich eine persistende Datenbankverbindung auf?

Bei dir fehlt das SQL-Escaping der Daten.

Warum ersetzt du Umlaute durch HTML-Entities?

1.) Ich habe doch die Funktion fgetscv genutzt.
Diese hat aber leider nicht die Return und Spaltenabsätze übernommen.

2.) Die persistende Datenverbindung habe ich genutzt, weil ich mit dem Script mehrere Dateien nacheinander ausführe und diese dann gelöscht werden, damit nicht falsche Dateien upgedatet werden.

3.) Brauche ich das Escaping nicht nur, wenn ich Daten zwischen "" und ' ' habe?

4.) Ich ersetzte die Umlaute in HTML Entries, weil ich sehr oft schon beim Import Probleme mit den Umlauten gehabt habe, seit der umsetzung in HTML Entries ist das Problem beseitigt.

-
Was wäre denn dein Vorschlag, damit ich auch die Absätze richtig übertragen bekomme?
Carsten

**h3ll** · 26.01.2012, 11:09

Zitat von cami3004 Beitrag anzeigen

Ich habe doch die Funktion fgetscv genutzt.
Diese hat aber leider nicht die Return und Spaltenabsätze übernommen.

Dann machst du was falsch. Bei mir funktionierts. Eventuell ist auch einfach nur die CSV-Datei kaputt?

Zitat von cami3004 Beitrag anzeigen

Die persistende Datenverbindung habe ich genutzt, weil ich mit dem Script mehrere Dateien nacheinander ausführe und diese dann gelöscht werden, damit nicht falsche Dateien upgedatet werden.

Das Eine hat mit dem Anderen nichts zu tun. Also warum verwendest du eine persistente Datenbankverbindung?

Zitat von cami3004 Beitrag anzeigen

Brauche ich das Escaping nicht nur, wenn ich Daten zwischen "" und ' ' habe?

Du brauchst es _immer_!

Zitat von cami3004 Beitrag anzeigen

Ich ersetzte die Umlaute in HTML Entries, weil ich sehr oft schon beim Import Probleme mit den Umlauten gehabt habe, seit der umsetzung in HTML Entries ist das Problem beseitigt.

Damit kehrst du das Problem unter den Teppich, aber löst es nicht. Du hast offenbar ein Zeichensatzproblem, um das du dich kümmern solltest.

Zitat von cami3004 Beitrag anzeigen

Was wäre denn dein Vorschlag, damit ich auch die Absätze richtig übertragen bekomme?

Korrekte CSV-Dateien erzeugen.

**cami3004** · 26.01.2012, 11:18

PHP-Code:


function parse_csv($file)
{
    $file_handle = fopen($file, "r");
    $data = array();
    while (($col = fgetcsv($file_handle, 0, '|')) !== FALSE) {
        foreach ($col as $key => $value) {
            $col[$key] = trim($col[$key]);
            $col[$key] = str_replace(';">"', '|">', $col[$key]);
            //echo $col[$key]."<br>";
        }
        array_push($data, $col);
    }
    fclose($file_handle);
    return $data;

Hier habe ich die fgetcsv Funktion genutzt.
Ich denke nicht das da ein Fehler drin ist.

Die CSV Datei exportiere ich direkt aus dem PHPMyAdmin.

**cami3004** · 26.01.2012, 11:20

Pm

**h3ll** · 26.01.2012, 11:23

Was soll dieser Quatsch mit str_replace? Die ganze foreach-Schleife ist nutzlos.

Und warum schreibst du array_push() statt einfach nur $data[] = ?

Statt mir irgendwas zu schicken, kannst du ja im Forum eine (kleine) Beispieldatei posten, wo der Fehler nachvollziehbar auftritt.

**cami3004** · 26.01.2012, 11:29

Das Beispiel habe ich doch oben in der CSV Datei aufgelistet.
Immer wenn ein Absatz mit einem Return drin ist, bricht das Skript diese Zeile ab.

Die replace Zeile ist wirklich überflüssig, stimmt. Danke

**h3ll** · 26.01.2012, 11:30

Zitat von cami3004 Beitrag anzeigen

Das Beispiel habe ich doch oben in der CSV Datei aufgelistet.
Immer wenn ein Absatz mit einem Return drin ist, bricht das Skript diese Zeile ab.

Die replace Zeile ist wirklich überflüssig, stimmt. Danke

Sorry, hab ich übersehen.

PHP Script bricht beim Return ab