Programm zur Log Auswertung

Phill4563

Hallo Werner,
vielen Dank für deine Hilfe.
Die Skip fkt. ist echt super

Das mit dem Zeit Struckt schau ich mir selbst nochmal an und würde mich nochmal melden

Einzig die Zeile

while( getline( log >> skip<'|'>, name1, '|' ) >> skip<'|'> >> skip<'|'> >> value >> skip<'\n'> )

verstehe ich nohc nicht so ganz, vielleicht koenntest du mir nochmal kurz erläutern was hier passiert

Zusätzlich sollten die Werte für "value" nur dann addiert werden wenn sich in der letzten Spalte ein 'H' befindet.

Grüße Philipp

Werner Salomon

Phill4563 schrieb:

Einzig die Zeile
while( getline( log >> skip<'|'>, name1, '|' ) >> skip<'|'> >> skip<'|'> >> value >> skip<'\n'> )
verstehe ich nohc nicht so ganz, vielleicht koenntest du mir nochmal kurz erläutern was hier passiert

Hallo Philipp

genau wie dieser Code

int a = 14;
    int b = 78;
    cout << "Die Summe von " << a << " und " << b << " ist " << (a+b) << endl;

das gleiche macht wie

int a = 14;
    int b = 78;
    cout << "Die Summe von ";
    cout << a;
    cout << " und ";
    cout << b;
    cout << " ist ";
    cout << (a+b);
    cout << endl;

macht dieser Code:

string name1;
    int value;
    // Format:       " <Zeit>    |   <name1>  |   <name2>   |<skillname> |     <value> ..Rest..EOL"
    while( getline( log >> skip<'|'>, name1, '|' ) >> skip<'|'> >> skip<'|'> >> value >> skip<'\n'> )
    {
        cout << name1 << " -> " << value << endl;
    }

das selbe wie:

string name1;
    int value;
    for(;;) // Beenden der Schleife s.u.
    {
        log >> skip<'|'>;           // skip <Zeit>
        getline( log, name1, '|' ); // read <name1>
        log >> skip<'|'>;           // skip <name1>
        log >> skip<'|'>;           // skip <skillname>
        log >> value;               // read <value>
        log >> skip<'\n'>;          // skip all until EOL (End Of Line)
        if( log.fail() )
            break;  // <== hier wird die Schleife ggf. beendet

        cout << name1 << " -> " << value << endl;
    }

.. das alles einen Ausdruck zu schreiben, sieht vielleicht erst mal kryptisch aus, aber das ist reine Gewöhnung.

Phill4563 schrieb:

Zusätzlich sollten die Werte für "value" nur dann addiert werden wenn sich in der letzten Spalte ein 'H' befindet.

kein Problem, lies es einfach mit ein:

string name1;
    int value;
    string dpsorheal; // was ist das? WoW?
    // Format:                " <Zeit>    |   <name1>  |   <name2>   |<skillname> |     <value>         |   <dpsorheal>  | ..Rest..EOL"
    while( getline( getline( log >> skip<'|'>, name1, '|' ) >> skip<'|'> >> skip<'|'> >> value >> skip<'|'>, dpsorheal, '|' ) >> skip<'\n'> )
    {
        if( dpsorheal == "H" )
            m[name1]+=value;
    }

Bem.: das letzte skip<'\n'> wäre hier nicht mehr notwendig, da kein '|' hinter dem zuletzt gelesenen '|' steht.

Gruß
Werner

Phill4563

Danke für deine Antwort
ne ist für swtor, nicht für wow

danke nochmals

Grüße Phill

Werner Salomon

PS.: Hups - ich habe gerade gesehen, dass die Spalte 'value' nicht immer einen Zahlenwert enthält, dann bricht das Lesen natürlich mit Fehler ab ... melde mich später dazu noch mal.

Phill4563 schrieb:

21:39:54.305|Respektlos|Respektlos|ExitCombat||EC|

Werner Salomon

Werner Salomon schrieb:

ich habe gerade gesehen, dass die Spalte 'value' nicht immer einen Zahlenwert enthält, ...

Hallo Philipp,

so ein Problem löst man z.B. dadurch, dass man nicht direkt eine Zahl (hier ein int) einliest, sondern ein Objekt einer selbst geschriebenen Klasse, welches erst mal prüft, ob das was da kommt eine Zahl sein könnte oder eben nicht. Ich nenne den Typ eines solchen Objekts mal ' opt ' für 'optional' und dann sähe das so aus:

struct opt // opt für optional; liest ein int, wenn eine Zahl kommt, sonst wird der Wert =0 gesetzt
{
    explicit opt( int& value )
        : value_( value )
    {}
    friend std::istream& operator>>( std::istream& in, opt o ) // Lesefunktion für den Wrapper 'opt'
    {
        char c;
        if( in >> c && in.putback( c ) ) // lese das nächste Zeichen 'c' und stelle es auch gleich zurück!
        {
            if( ('0' <= c && c <= '9') || c == '-' || c == '+' ) // könnte eine Zahl sein
                in >> o.value_; // dann lese die Zahl
            else
                o.value_ = 0; // sonst definiert auf 0 setzen
        }
        return in;
    }
private:
    int& value_;
};

anschließend muss man die Einlesesequenz nur um das ' opt ' erweitern - ergo:

string name1;
    int value;
    string dpsorheal;                                                            //<-- hier optional lesen -->
    // Format:                " <Zeit>    |   <name1>  |   <name2>   |<skillname> |       [<value>]          |   <dpsorheal>  | ..Rest..EOL"
    while( getline( getline( log >> skip<'|'>, name1, '|' ) >> skip<'|'> >> skip<'|'> >> opt(value) >> skip<'|'>, dpsorheal, '|' ) >> skip<'\n'> )
    { // usw.

Gruß
Werner

Skym0sh0

das opt Objekt der Einlesefunktion müsste doch eien Referenz sein, oder irre ich mich da Werner?

friend std::istream& operator>>( std::istream& in, opt & /*<-- Referenz*/ o )

Werner Salomon

Skym0sh0 schrieb:

das opt Objekt der Einlesefunktion müsste doch eien Referenz sein, oder irre ich mich da Werner?
friend std::istream& operator>>( std::istream& in, opt & /*<-- Referenz*/ o )

.. probier's mal aus! wenn Du nicht gerade einen MSVC6-Compiler verwendest, sollte der Code dann gar nicht übersetzt werden. Der Ausdruck 'opt(value)' in dem while-Statemant erzeugt ein temporäres Objekt!
wenn Referenz, dann

friend std::istream& operator>>( std::istream& in, const opt& /*<-- const Referenz*/ o )

Das opt-Objekt selbst wird ja gar nicht verändert!

.. hätte ich vielleicht gleich dazu schreiben sollen, weil diese Nachfrage kommt fast immer.

Gruß
Werner

Skym0sh0

Das nicht, aber wird es nicht kopiert? Das ist ja nur ein Call-By-Value. Und Strukturen, die eine Referenz als Member halten sind doch nicht kopierbarbar.

Wieso geht das dann?
Oder anders gefragt, warum ergibt das keinen Fehler, kompiliert aber mit Referenz nicht?

mit Referenz
Ohne Referenz

Phill4563

Vielen Dank für die Hilfe Werner.

Grüße
Phill

Werner Salomon

Skym0sh0 schrieb:

Das nicht, aber wird es nicht kopiert? Das ist ja nur ein Call-By-Value. Und Strukturen, die eine Referenz als Member halten sind doch nicht kopierbarbar.

doch kopierbar sind sie; nur nicht zuweisbar. Zumindest nicht ohne dass man einen eigenen Zuweisungsoperator schreibt.

Skym0sh0 schrieb:

.. kompiliert aber mit Referenz nicht?

weil der Standard es schlicht nicht zulässt. Ich finde die Stelle, wo das steht, eben nicht wieder. Aber eine non-const reference auf ein temporäres Objekt ist nicht zulässig. Und deshalb compiliert es mit Referenz nicht - so der Compiler die Regel beherzigt.

Gruß
Werner

Sone

Ich finde die Stelle, wo das steht, eben nicht wieder.

§8.5.3 schrieb:

4.
Given types “cv1 T1” and “cv2 T2,” “cv1 T1” is reference-related to “cv2 T2” if T1 is the same type as T2, or T1 is a base class of T2. “cv1 T1” is reference-compatible with “cv2 T2” if T1 is reference-related to T2 and cv1 is the same cv-qualification as, or greater cv-qualification than, cv2.

5.
A reference to type “cv1 T1” is initialized by an expression of type “cv2 T2” as follows:
If the reference is an lvalue reference and the initializer expression

is an lvalue (but is not a bit-field), and “cv1 T1” is reference-compatible with “cv2 T2,” or

has a class type (i.e., T2 is a class type), where T1 is not reference-related to T2, and can be implicitly converted to an lvalue of type “cv3 T3,” where “cv1 T1” is reference-compatible with “cv3 T3”106 (this conversion is selected by enumerating the applicable conversion functions (13.3.1.6) and choosing the best one through overload resolution (13.3)),

then the reference is bound to the initializer expression lvalue in the first case and to the lvalue result of the conversion in the second case (or, in either case, to the appropriate base class subobject of the object).

Eine non-const lvalue-Referenz muss also immer zu einem non-const lvalue gebunden werden.
Das waren die gewöhnlichen Semantiken für lvalue-Referenzen.

Erst im nächsten Abschnitt wird dann für const-lvalue -Referenzen erklärt..

Otherwise, the reference shall be an lvalue reference to a non-volatile const type (i.e., cv1 shall be const), or the reference shall be an rvalue reference.
If the initializer expression

is an xvalue, class prvalue, array prvalue or function lvalue and “cv1 T1” is reference compatible with “cv2 T2”, or

has a class type (i.e., T2 is a class type), where T1 is not reference-related to T2, and can be implicitly converted to an xvalue, class prvalue, or function lvalue of type “cv3 T3”, where “cv1 T1” is reference-compatible with “cv3 T3”

then the reference is bound to the value of the initializer expression in the first case and to the result of the conversion in the second case (or, in either case, to an appropriate base class subobject). [...]

Otherwise, a temporary of type “cv1 T1” is created and initialized from the initializer expression using the rules for a non-reference copy-initialization (8.5). The reference is then bound to the temporary.

Achtung: Die letzten beiden Sätze treffen nicht immer zu.
schreibt, dann trifft das auf die obige Erklärung zu;

std::string const& = std::string{};

Wenn man jedoch

std::string const& = "ABC";

Dann trifft es auf die letzte Erklärung zu.

Zumindest nicht ohne dass man einen eigenen Zuweisungsoperator schreibt.

Wie sollen sie dann zuweisbar sein? Es ist klar festgelegt, dass eine Referenz ihren Referenten nicht ändern kann:

A reference cannot be changed to refer to another object after initialization. Note that initialization of a reference is treated very differently from assignment to it.

Daher ist in Klassen immer ein Zeiger besser aufgehoben.