Import/Export überflüssiger code ?

[Dwza]

Also im Import habe ich folgende Zeilen gefunden....

Code: PHP  [Auswählen]

        function xtcImport($filename) {
   
                $this->seperator    = CSV_SEPERATOR;
                $this->TextSign     = CSV_TEXTSIGN;
       
                //BOF - Dokuman - 2010-02-11 - set default textsign
                if (trim(CSV_TEXTSIGN) == '') {
                        $this->TextSign = '"';
                }
                //EOF - Dokuman - 2010-02-11 - set default textsign
                if (CSV_SEPERATOR == '') {
                        $this->seperator = "\t";
                }
                if (CSV_SEPERATOR == '\t') {
                        $this->seperator = "\t";
                }
                $this->filename      = $filename;
                $this->ImportDir     = DIR_FS_CATALOG.'import/';
                $this->catDepth      = 6;
                ...
               

In der Zeile 3 wird der Seperator gesetzt. Ist ja schon mal ok...
in der Zeile 11 wird nochmal geprüft ob die Seperatorkonstante leer war, wenn ja dann wird ein Tab im Seperator gesetzt. bis dahin auch ok, wobei ich das anders schreiben würde. aber was ich dann nicht mehr verstehe ist Zeile 14. Es wird geprüft ob die Seperatorkonstante ein Tab ist, wenn ja dann wird der Seperator auf Tab gesetzt. Ist doch in Anbetracht auf die vorhergehenden Zeilen sinnlos oder sehe ich das falsch ?

allgemien würde ich es auch eher so schreiben....

Code: PHP  [Auswählen]

        function xtcImport($filename) {
   
                $this->seperator    = ( empty( trim( CSV_SEPERATOR ) ) ? "\t" : CSV_SEPERATOR );
                $this->TextSign     = ( empty( trim( CSV_TEXTSIGN ) ) ? "\t" : CSV_TEXTSIGN );

                $this->filename     = $filename;
                $this->ImportDir    = DIR_FS_CATALOG.'import/';
                $this->catDepth     = 6;
                ...
               

hat doch den selben effekt nicht wahr ?

Linkback: https://www.modified-shop.org/forum/index.php?topic=29742.0

[Matt]

a) CSV_TEXTSIGN auf nen Tab zu setzen ist nicht sinnvoll
b) es gibt durchaus wichtige Unterschiede zwischen Single- und Doublequotes.

[Dwza]

da hatte ich wohl einen schreibfehler im post, sorry, natürlich sollte CSV_TEXTSIGN kein \t sondern ein " sein. natürlich kenne ich auch den unterschied zwischen einfach und doppelquotes.
aber, du hast mich erwischt... trotz meiner kenntnisse zum thema einfacher und doppelter quotes ist es mir nicht aufgefallen und demnach macht die zuweisung dann sinn. dann hat sich das schon erledigt  

die korrekte kurzschreibform wäre dann das

Code: PHP  [Auswählen]

$this->seperator=(in_array(trim(CSV_SEPERATOR),array('','\t'))?"\t":CSV_SEPERATOR);
$this->TextSign=(empty(trim(CSV_TEXTSIGN))?'"':CSV_TEXTSIGN);

[gemano24]

Hallo Leute,

ich habe meine CSV-Datei vom Lieferanten und möchte diese im Modified-Shop importieren. Bis dahin alles super. 

Jetzt kommt das Beste In der CSV sind URLs für die Bilder angegeben. Laut Lieferant, alle anklicken und runterladen.    Alle Bilder einzeln runterladen und abspeichern.....mein Lieferant spinnt, habe ich Ihm auch so geschrieben. Da muss ich ja 

Gibt es da eine andere Lösung, hat da jemand von euch eine bessere Lösung. Aus das man den Lieferanten tauschen sollte. 

Vielen Dank im Voraus für euere Antworten.

Gruss
Marko
 

[Matt]

Alle Bild-URLs in eine Datei kopieren (eine URL pro Zeile) und mit

Code: [Auswählen]

wget -i bilder.txtrunterladen. Dann die CSV manuell anpassen, dass nur noch der Name des Bildes in der CSV steht.

[gemano24]

zu erst vielen Dnak für deine Hilfe.

Also der erste Schritt, alle URLs in eine Datei und zweite Schritt mit dem Code wget -i bilder.txt runterladen? diesen Schritt, verstehe ich leider nicht so Sorry, wo gebe ich diesen Code den ein?

Gruss
Marko

[Dwza]

wenn du windows-user bist kannst du dir die passende datei aus dem internet herunterladen.
einfach mal nach wget 1.14 googeln.
dann öffnest du die konsole und gehst in das verzeichnis in dem die datei liegt...
natürlich in der konsole

bsp:
wget.exe auf c:\ kopieren
url datei auch auf c:\

start > ausführen : cmd (dann enter drücken)
eingabe: cd\ (dann enter drücken)
wenn dann da c:\ steht dann gibts du die vorhin genannte zeile ein.
wget.exe -1 urldatei.txt

denke das sollte es dann gewesen sein.

user von linux o.ä. gehen da ggf anders vor.

[gemano24]

@Dwza,

ja, dass habe ich gesucht.  für die tolle erklärung.

 

Vielen Vielen DANK!!!!

Gruss
Marko

[gemano24]

@Dwza

Vielleicht kannst du mir in Sachen Excel helfen. Das mit WGET hat super geklappt, dank deiner Anleitung! Nochmals Danke. 

Jetzt muss ich noch die CSV-Datei anpassen! Also in der CSV habe ich jetzt unter p_image die URLs zu den Bildern stehen. Nun muss ich die URL löschen, damit mir nur noch der Bildname stehen bleibt.

Bspiel: Unter p_image steht http://www.domain.de/xyz123.jpg
Damit jetzt nur xyz123.jpg in p_image stehen bleibt muss ich alles andere löschen.

Das bedeutet für mich, müsste jetzt alle 5000 Einträge einzeln löschen! Es sei denn, es gibt einen Trick bei Excel? Ich würde gerne alles makieren und Excel sagen, jetzt lösche http://www.domain.de/ und lass xyz123.jpg stehen. Kannst du mir da weiter helfen?

Gruss
Marko

PS. Mache dass zum ersten mal und kenne mch damit nicht so aus. SORRY, dass ich soviel Nachfrage :

[web0null]

Bitte sehr
Gruß

[gemano24]

@web0null

genial, einfach super. 

DANKE......DANKE......DANKE......DIR.......SEHR. 
Damit habe ich sehr viel Zeit sparen können. 

  Gruss
Marko

[gemano24]

ohman Leute,

bin völlig am durchdrehen!

In meiner CSV-Datei von anderem Lieferanten steht in p_image leider nichts drin.

Ich mache und tue und google hin und herr um raus zu finden wie ich folgendes machen will. 

Nun, meine CSV-Datei hat in p_image nicht drin. Jetzt kopiere ich die Artikelnummern da hinein. Bis dahin alles okay. Nun möchte ich hinter der Artikelnummer .jpg hinzufügen. Also ein Bspiel:

123456.jpg
123457
123458

durch das Ersetzen-Tool in Excel habe ich das die ganze Zeit probiert, jedoch in keiner weise hinbekommen! Das selbe wäre auch super, das ich auch vor der Artikelnummer eine Buchstabenreihe (Bspiel: XYZ.123456) setzen könnte, ohne dies alles einzeln machen zu müssen!!

Kann mir von euch einer hierbei BITTE.....BITTE helfen! Ansonsten werde ich total noch      

Gruss
Marko

[Matt]

Hab nur OpenOffice da, sollte aber mit einem benutzerdefinierten Zellenformat (bild-@.jpg) gehen.

Spricht eigentlich was dagegen, für neue Probleme neue Threads aufzumachen?

[gemano24]

nein, so ein Problem besteht nicht! Kann ich eröffnen! 

Wie soll dass mit dem Zellenformat gehen, bzw. wie mache ich dass? 

[Matt]

Auf dem Mac ist es Cmd+1, auf Windows dann vermutlich Strg+1. Aber auch da gibt's eine Hilfe, in der kann man nach sowas suchen.