[HaBo]

tanj · 27.05.11, 22:36

Hallo,

mittels folgender Funktion lasse ich mir eine Zeichenkette einlesen:

Code:

void parseSetOfIntegers(const char* listAsString, int* list)
{
    int i = 0;
    const char* p = listAsString;

    if (*p == 123)
    {
        while (*p)
        {   p++;
            if (*p >=48 && *p <= 57)
            {
                list[i] = atoi(p);
                i++;
            }
        }
    }
}

Die Zeichenkette soll die Form {X,XX,X} haben, fuer X nur ganze Zahlen. Das funktioniert auch wunderbar fuer X<10. Fuer X>=10 tritt das Problem auf, dass zwar, z.B. fuer {123, 6, 8} die einstelligen Zahlen normal eingelesen werden. Fuer die 123 aber passiert folgendes: In list[0] steht 123, in list[1] 23 und in list[2] 3.

Ich kann mir denken, dass es daran liegt, dass atoi alle Zeichen in einen Integer umwandelt die ab Adresse p stehen, fuer alle Zeichen, fuer die das moeglich ist. Es beginnt also bei Adresse von 1, ist kein Problem, also weiter 2 und 3 ist auch kein Thema und dann steht da ein Komma in Ascii und es bricht ab. In list[0] wird also 123 geschrieben. Es wird anschliessend der Pointer um eins erhoeht und wir starten bei 2, dass gleiche Spiel von vorn.

Muss ich als Loesung die Stelligkeit jeder einzelnen Zahl in list[i] berechnen und dementsprechend dann den Pointer erhoehen, oder gibt es bessere Loesungen?

Gruss

tanj

lookshe · 28.05.11, 01:10

Schau dir mal strtok an. Damit kommst du quasi an die einzelnen Zahlen (nicht Ziffern) und kannst diese mit atoi weiterverarbeiten.
Allerdings sollte man mit diesem Befehl vorsichtig sein, unter bestimmten Umständen, macht der ganz schön Murks im Speicher und man bekommt beim allozieren immer die selbe Speicheradresse. Tritt nicht auf, sofern man die Tokens in ein und der selben Variable zwischenspeichert.

Anzeige

- Anzeige -

Thunderb0lt · 28.05.11, 01:40

Eine andere - und wie ich finde einfachere - Variante wäre sscanf.
Oder, wenn du in C++ bist, brauchst du dich mit dem ganzen C Kram nicht herumschlagen und nimmst direkt streams.

benediktibk · 28.05.11, 10:11

Oder (ich finde die oberen Lösungen aber schöner, nur der Vollständigkeit halber) du durchsuchst vor dem Parsen den String nach Kommata und merkst dir deren Positionen.

mfg benediktibk

lookshe · 28.05.11, 13:41

Das wäre aber auch nur ein neues strtok, denn das arbeitet im Prinzip so.

Wenn die Zeichenfolge immer in der Form {X,X,X} ist, dann wäre sscanf natürlich eine Alternative, hab ich nicht genau genug gelesen...

enkore · 28.05.11, 20:20

Du kannst auch einfach Boost.Tokenizer benutzen

mauralix · 29.05.11, 17:21

Wenn man sich noch keine Splitfunktion geschrieben hat kann man einfach mit strstr() in einer Loop das nächste "," suchen und dieses durch '\0' ersetzen und dann einfach atoi() anwenden.

Anzeige

- Anzeige -

27.05.11, 22:36	#1 (permalink)
tanj Registriert seit: 13.04.07 Likes: 0	c++ Pointerproblem Anzeige Hallo, mittels folgender Funktion lasse ich mir eine Zeichenkette einlesen: Code: void parseSetOfIntegers(const char* listAsString, int* list) { int i = 0; const char* p = listAsString; if (p == 123) { while (p) { p++; if (p >=48 && p <= 57) { list[i] = atoi(p); i++; } } } } Die Zeichenkette soll die Form {X,XX,X} haben, fuer X nur ganze Zahlen. Das funktioniert auch wunderbar fuer X<10. Fuer X>=10 tritt das Problem auf, dass zwar, z.B. fuer {123, 6, 8} die einstelligen Zahlen normal eingelesen werden. Fuer die 123 aber passiert folgendes: In list[0] steht 123, in list[1] 23 und in list[2] 3. Ich kann mir denken, dass es daran liegt, dass atoi alle Zeichen in einen Integer umwandelt die ab Adresse p stehen, fuer alle Zeichen, fuer die das moeglich ist. Es beginnt also bei Adresse von 1, ist kein Problem, also weiter 2 und 3 ist auch kein Thema und dann steht da ein Komma in Ascii und es bricht ab. In list[0] wird also 123 geschrieben. Es wird anschliessend der Pointer um eins erhoeht und wir starten bei 2, dass gleiche Spiel von vorn. Muss ich als Loesung die Stelligkeit jeder einzelnen Zahl in list[i] berechnen und dementsprechend dann den Pointer erhoehen, oder gibt es bessere Loesungen? Gruss tanj

28.05.11, 01:40	#3 (permalink)
Thunderb0lt Registriert seit: 06.06.09 Likes: 6	Eine andere - und wie ich finde einfachere - Variante wäre sscanf. Oder, wenn du in C++ bist, brauchst du dich mit dem ganzen C Kram nicht herumschlagen und nimmst direkt streams. __________________ A good programmer looks both ways before crossing a one-way street. Die Politik kann uns nur für so dumm verkaufen, wie wir sind.

28.05.11, 20:20	#6 (permalink)
enkore Senior Member Registriert seit: 13.07.08 Likes: 85	Du kannst auch einfach Boost.Tokenizer benutzen __________________ "It is the human race! The deterioration of the spirit of man. Man undermining himself, causing a self-willed, self-imposed, self-evident self-destruction." +++ BREAKING +++ Troll ertrinkt im Planschbecken +++

29.05.11, 17:21	#7 (permalink)
mauralix Registriert seit: 17.04.06 Likes: 3	Wenn man sich noch keine Splitfunktion geschrieben hat kann man einfach mit strstr() in einer Loop das nächste "," suchen und dieses durch '\0' ersetzen und dann einfach atoi() anwenden. __________________ http://chm0815.blogspot.com

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Ansicht
Linear-Darstellung Zur Hybrid-Darstellung wechseln Zur Baum-Darstellung wechseln

28.05.11, 01:10	#2 (permalink)
lookshe Senior Member Registriert seit: 10.03.07 Likes: 19	Schau dir mal strtok an. Damit kommst du quasi an die einzelnen Zahlen (nicht Ziffern) und kannst diese mit atoi weiterverarbeiten. Allerdings sollte man mit diesem Befehl vorsichtig sein, unter bestimmten Umständen, macht der ganz schön Murks im Speicher und man bekommt beim allozieren immer die selbe Speicheradresse. Tritt nicht auf, sofern man die Tokens in ein und der selben Variable zwischenspeichert.

28.05.11, 10:11	#4 (permalink)
benediktibk Registriert seit: 03.05.07 Likes: 50	Oder (ich finde die oberen Lösungen aber schöner, nur der Vollständigkeit halber) du durchsuchst vor dem Parsen den String nach Kommata und merkst dir deren Positionen. mfg benediktibk

28.05.11, 13:41	#5 (permalink)
lookshe Senior Member Registriert seit: 10.03.07 Likes: 19	Das wäre aber auch nur ein neues strtok, denn das arbeitet im Prinzip so. Wenn die Zeichenfolge immer in der Form {X,X,X} ist, dann wäre sscanf natürlich eine Alternative, hab ich nicht genau genug gelesen...

[HaBo]

c++ Pointerproblem