Einlesen einer Datei: Geschwindigkeitsoptimierung

Konstantin

Thx!

Ich werd's mir mal anschauen!

CU
Konstantin

volkard

teste doch erstmal ein lockeres
rtrn += buf;
rtrn += "\r\n";
statt des
rtrn = rtrn + buf + "\r\n";

wie viel bring das?

HumeSikkins

Hallo,
und falls du den VC 6 benutzt solltest du sicherstellen, dass du vor der Benutzung der neuen fstreams deine STL gefixed hast.

Konstantin

@volkard: Meinst du, der Unterschied macht so viel aus? Na ja, ich werds mal testen.

@HumeSikkins: Ich benutzte natürlich Borland.

Ich glaube der Beitrag komplette datei einlesen hilft mir da schon ganz gut weiter... Das hört sich ganz gut an!

Thx für die Antworten!

Konstantin

[ Dieser Beitrag wurde am 18.06.2003 um 08:30 Uhr von Konstantin editiert. ]

Original erstellt von Konstantin:
@volkard: Meinst du, der Unterschied macht so viel aus? Na ja, ich werds mal testen.
[ Dieser Beitrag wurde am 18.06.2003 um 08:30 Uhr von [qb]Konstantin editiert. ][/QB]

Bin zwar nicht Volkard, werde es aber trotzdem versuchen.

rtrn = rtrn + buf + "\r\n";
/*Hier werden min 2 Temporäre Variablen erstellt und wieder zerstört*/

rtrn += buf;
rtrn += "\r\n";
/*Hier wird keine Temporäre Variable benötigt*/

Darum ist +=, -=, %=, /= u.s.w. immer a = a + x vorzuziehen.

Wenn etwas falsche ist sagt es nur, bin für Wissen immer offen.

volkard

Original erstellt von <C Newbie>:
Wenn etwas falsche ist sagt es nur, bin für Wissen immer offen.

das 'e' da. (SCNR)

phlox81

Original erstellt von <C Newbie>:
**Bin zwar nicht Volkard, werde es aber trotzdem versuchen.
rtrn = rtrn + buf + "\r\n";
/*Hier werden min 2 Temporäre Variablen erstellt und wieder zerstört*/

rtrn += buf;
rtrn += "\r\n";
/*Hier wird keine Temporäre Variable benötigt*/
Darum ist +=, -=, %=, /= u.s.w. immer a = a + x vorzuziehen.

Wenn etwas falsche ist sagt es nur, bin für Wissen immer offen.**

Genau das sagt ja volkard...

Devil

phlox81

Original erstellt von HumeSikkins:
Hallo,
und falls du den VC 6 benutzt solltest du sicherstellen, dass du vor der Benutzung der neuen fstreams deine STL gefixed hast.

Hm, könntest du mir darüber nen link geben ?
Was meinst du ?

Devil

http://fara.cs.uni-potsdam.de/~kaufmann/?page=Programming&ProgKNr=1#Code

@devil81 Volkard hat gesagt was er ändern könnte, ich wollte mit dem Post sagen
warum das schneller ist, auch wenn nicht danach gefragt wurde.

Konstantin

Original erstellt von <C Newbie>:
**Bin zwar nicht Volkard, werde es aber trotzdem versuchen.
rtrn = rtrn + buf + "\r\n";
/*Hier werden min 2 Temporäre Variablen erstellt und wieder zerstört*/

rtrn += buf;
rtrn += "\r\n";
/*Hier wird keine Temporäre Variable benötigt*/
Darum ist +=, -=, %=, /= u.s.w. immer a = a + x vorzuziehen.

Wenn etwas falsche ist sagt es nur, bin für Wissen immer offen.**

Ok, sowas dachte ich mir schon. Ich glaube aber, dass am meißten Zeit beim einlesen an sich verloren geht.

CU & Thx
Konstantin

Shade Of Mine

Original erstellt von Konstantin:
Ich glaube aber, dass am meißten Zeit beim einlesen an sich verloren geht.

glaube ich eher nicht, denn getline geht halt nich schneller - es sei denn du nimmst OS interne Funktionen und mapst die ganze Datei in den speicher...

dein momentanes problem ist, dass du zuviele temporaere objekte erstellst, in denen immer die ganze bisher gelesene datei hin und her kopiert werden muss.

eleminiere diese temporaeren objekte und du wirst merklich schneller sein.

Konstantin

Nimmt so etwas wirklich soooo viel Zeit in Anspruch??? Das hätt ich ja nicht gedacht! Das gibt mir ja echt zu denken...

Na gut, dann werde ich mal ein bisschen herumexpereimetieren, wie es am schnellsten geht.

Danke an alle

Konstantin

Das Problem liegt bei dir nicht beim einlesen der Datei.
Bei mir dauert es ca. 0.05 sec. um eine 300 kb Datei in einen string einzulesen.
Wie verarbeitest du den string weiter ?
Vielleicht ist da etwas nicht optimal.

Shade Of Mine

Original erstellt von Konstantin:
**Nimmt so etwas wirklich soooo viel Zeit in Anspruch??? Das hätt ich ja nicht gedacht! Das gibt mir ja echt zu denken...
**

beispiel hier:
rtrn = rtrn + buf + "\r\n";

in rtrn stehen sagen wir 100KB
dann werden

rtrn = rtrn + buf + "\r\n";
            ^

diese 100KB hier einmal kopiert und den inhalt von buf kommt auch noch rein.

rtrn = rtrn + buf + "\r\n";
                  ^

hier wird das ganze dann nochmal gemacht - 100KB kopiert und "\r\n" drangehaengt...

das ganze wird dann rtrn zugewiesen, also wird nochmal alles kopiert.

oder zB hier:
rtrn = rtrn.substr(0, rtrn.length() - 2);

es wird ein substr gebildet - in dem quasi die ganzen 300KB kopiert werden, und dieser wird dann wieder in rtrn kopiert - und dieser wird by value zurueck gegeben, also nochmal kopiert.

wie du siehst wird bei dir _sehr_ viel kopiert. vorallem wenn der string lang wird, zB 300KB oder mehr - denn es wird dann ja immer alles kopiert (exponentiell)

dagegen ist volkards version ideal - da wird nur das noetigste kopiert und nie der ganze string (es wird immer nur hinten angehaengt)

Konstantin

@ Shade Of Mine:
Ok, das leuchtet mir ein. Ich habe das vorher nie so gesehen. Ich dachte immer der Operator += ist nur für schreibfaule ;-). Das der wirklich was anderes macht, hätte ich nicht gedacht.

@ Stefan:
Bisher lasse ich den eigentlich nur in einem Edit-Feld anzeigen.

CU
Konstantin

volkard

Original erstellt von Shade Of Mine:
wie du siehst wird bei dir _sehr_ viel kopiert. vorallem wenn der string lang wird, zB 300KB oder mehr - denn es wird dann ja immer alles kopiert (exponentiell)

warnung: beindruckendes fachwort erwürfelt. kunde muss neu bewertet werden.

Shade Of Mine

Original erstellt von volkard:
warnung: beindruckendes fachwort erwürfelt. kunde muss neu bewertet werden.

hab ich was falsches geschrieben?

wenn ja, dann bitte aufklären und mich nicht so verwirren...

exponentiell

Konstantin

Hi Community!

Der folgende Code hat keine nennenswerte Optimierung gebracht:

string read_file(const string &path)
{
    string buf;
    string rtrn = "";
    ifstream ifs;

    ifs.open(path.c_str(), ios::in);
    if(ifs.fail())
    {
        return "";
    }
    MessageBox(NULL, TEXT ("Anfang!"), "Überschrift", NULL);
    while(!ifs.eof())
    {
        getline(ifs, buf);
        rtrn += buf;
        if(!ifs.eof())
            rtrn += "
";
    }
    MessageBox(NULL, TEXT ("Ende!"), "Überschrift", NULL);

    return rtrn;
}

Dann hab ich das mal so ausprobiert:

string read_file(const string &path)
{
    ifstream ifs;

    ifs.open(path.c_str(), ios::in);
    if(ifs.fail())
    {
        return "";
    }

    istreambuf_iterator<char> beg(ifs), end;
    string rtrn(beg, end);

    return rtrn;
}

Da bekam ich folgende Fehlermeldung:

Error file.cpp 25: Could not find a match for 'std::basic_string<char,std::char_traits<char>,std::allocator<char> >::basic_string(std::istreambuf_iterator<char,std::char_traits<char> >,std::istreambuf_iterator<char,std::char_traits<char> >)' in function read_file(const std::basic_string<char,std::char_traits<char>,std::allocator<char> > &)

Fehlt mir da vielleicht eine Headerdatei?

CU Konstantin