4. PerformanceTuning für Routine

PerformanceTuning für Routine

  • S

    Ist es dann möglich nach dem "," zu suchen?

    Denn wenn ich das komma als Trennzeichen betrachte und mich danach orientiere, dann
    geht es doch nicht mit Pos oder?

    Wenn ich z.b. folgenden String habe

    Kommentar,34,Das ist ein Test,12:99,45,Noch ein test

    dann möchte ich alle einzelnen Werte in einer eigenen Variablen haben.
    und ich kenne ja nicht die genauen Bezeichnungen oder Werte nach denen ich suchen muss. Da bleibt mir ja nix anderes übrig als nach dem "," zu suchen.
    Man könnte den eingelesenen Teil ja vom restlichen String abschneiden, doch die Frage ist, ob die Pos-Funktion eine viel bessere Performance leistet als die Variante mit den Pointern!?

    0
  • B

    Schau dir doch mal in der Hilfe TStringList::CommaText an. Damit geht das ganz automatisch.

    0
  • A

    Hallo

    also nochmal meine Hinweise auf die AnsiString-Methoden Pos und SubString, nachzlesen auch in der BCB-Hilfe.

    /Edit : Okay seit dem BCB6 gibts ja noch bessere Möglichkeiten.

    bis bald
    akari

    0
  • V

    Bei solchen Sachen wende ich meisst StringReplace an und ersetze den Seperator durch einen Zeilenumbruch dann sind alle Einträge jeweils in einer eigenen Zeile.

    TStringList *strList = new TStringList();
strList->Text = StringReplace(auftragsarbeitsdatensatz, ",", "\n", ReplaceFlags()<< rfReplaceAll);

for (int anz=0; anz < strList->Count; anz++) {
    if (anz==0) //Kommentar
    if (anz==1) m_asTag = strList->Strings[anz]; //Tag
    if (anz==2) m_asAuftragsnummer = strList->Strings[anz]; //Auftragsnummer
    if (anz==3) m_asTaetigkeitsnummer = strList->Strings[anz]; //Taetigkeitsnummer
    if (anz==4) m_asAnfang = strList->Strings[anz]; //Anfang
    if (anz==5) m_asEnde = strList->Strings[anz]; //Ende
    if (anz==6) m_asPause = strList->Strings[anz]; //Pause
}

delete strList;
    0
  • B

    @akari
    CommaText gibt es auch schon im BCB5. Ab dem BCB6 ist dann noch DelimitedText sowie Delimiter hinzugekommen, die das Ganze noch flexibler machen.
    Es gibt aber noch viele andere Möglichkeiten. Sehr schön ist auch boost::tokenizer.

    0
  • S

    Hi Leute, hab jetzt mal die Variante von VergissEs genommen. Ist auf jeden Fall eleganter und kürzer!
    Doch zum Thema Performance hat sie auch nicht mehr beigetragen. Wenns hochkommt hab ich jetzt eine verbesserung von 1 Sekunde 😞

    Ich habe gehört, dass es eine Möglichkeit gibt, in der benutzen DB eine Importfunktion zu schreiben, die eine Datei einfach einliest und die Werte in die jeweiligen Tabellen schreibt und das alles in sehr sehr kurzer Zeit.

    Hat da jemand erfahrung damit. Und gebt mir mal eine Schätzung ab, wie lange ihr die Verarbeitungszeit von 150 Datensätzen sieht!?

    Gruß

    0
  • ?

    Hast Du schon mal daran gedacht, einen Profiler einzusetzen? (z.B. AQtime)

    0
  • B

    Der Code von VergissEs ist schon etwas bearbeitungbedürftig.

    TStringList *strList = new TStringList;
strList->CommaText = auftragsarbeitsdatensatz;
if( strList->Count >= 6 )
{
    m_asTag = strList->Strings[1]; //Tag
    m_asAuftragsnummer = strList->Strings[2]; //Auftragsnummer
    m_asTaetigkeitsnummer = strList->Strings[3]; //Taetigkeitsnummer
    m_asAnfang = strList->Strings[4]; //Anfang
    m_asEnde = strList->Strings[5]; //Ende
    m_asPause = strList->Strings[6]; //Pause
} else
{
// Fehlerbehandlung
}
delete strList;
    0
  • V

    Braunstein schrieb:

    Der Code von VergissEs ist schon etwas bearbeitungbedürftig.

    nur wenn man daraus eine CommaText Version basteln will!

    0
  • B

    Nicht nur. Die for-Schleife ist auch unnötig.

    0
  • V

    Dann müsste ja deine if abfrage auch unnötig sein oder überprüfst du nicht auf vorkommen der Zeilen? 😮

    0
  • B

    Was glaubst, was diese if-Abfrage macht? Sie müsste korrekterweis aber so lauten:

    if( strList->Count > 6 )
    0
  • V

    Wenn du nochmal meinen Satz durch liesst wirst du festellen das ich geschrieben habe, das du durch dein if Abfrage die Anzahl überprüfst.

    Aber irgendwie drehen wir uns im Kreis, den je nach Anwendungsfall würde ich deine Lösung nehmen bzw. meine wenn ich Mehrfachparameter (also unterschiedliche Count habe aber trozdem eine feste Ordnung) hätte.

    Deshalb finde ich deine Aussage mein Codebeispiel sei bearbeitungsbedürftig für unsinnvoll, den es kommt immer drauf an was man daraus machen will und wie flexibel (meine Version) oder starr (deine Version) man es braucht/haben will.

    Diese Entscheidung muss der Thread Ersteller treffen nicht DU und auch nicht ICH

    In diesem Sinne

    0
  • T

    Um die String-Kopiererei ein wenig in den Griff zu bekommen, könntest du dir mal das "range"-Konzept von boost anschauen (http://www.boost.org/libs/range/index.html). Ich habe vor ein paar Monaten eine ähnliche Parsingroutine damit geschrieben, weils mir auch um Geschwindigkeit ging.
    Bei Gelegenheit werd ich die hier posten.

    Ansonsten kann ich mir nicht vorstellen, daß du für 150 Datensätze 15 Sekunden verlierst. Das ist ja "nix".
    Wie siehts denn mit deiner DB-Routine aus? Ich denke, daß hier viel Zeit draufgeht.

    0
Anmelden zum Antworten