Problem strchr

Ich kann nicht, warum das folgende Bit von C-Code funktioniert nicht:Problem strchr

int obtainStringLength(char* str, char c1, char c2) { 
    char* firstOcurrence = strchr(str, c1); 
    char* endOcurrence = strchr(str, c2); 
    return 2+(endOcurrence - firstOcurrence)/sizeof(char*); 
}

Die Idee, wie viele Zeichen zwischen c1 sind zu finden ist und c2:

printf("%d\n", obtainStringLength("abc def ghi", 'a', 'i')); //should yield 11

Leider druckt das immer 1. Was ist das Problem? Sollte nicht strchr funktionieren wie C# string.IndexOf()?

Quelle

2010-12-12 devoured elysium

Sie haben einen sehr seltsamen Compiler :) 'immer Drucken 1' ???? – pmg

Division von sizeof(char*)? Das ist falsch - das Ergebnis der Subtraktion zweier Zeiger ist ein numerischer Wert (ptrdiff_t), der der Anzahl der Werte entspricht, nicht ein Zeiger oder eine Differenz von Adressen.

Es gibt auch einen Fehler bei der Berechnung der Länge. So sollte die letzte Zeile wie folgt aussehen:

return 1 + (endOcurrence - firstOcurrence);

Quelle

2010-12-12 00:51:09 Kos

Nicht nur das, sondern jedes Element im String ist 'sizeof (char)' Bytes, nicht 'sizeof (char *)'. –

Richtig, aber nicht wirklich relevant hier - der Zeiger-arithmetische Mechanismus abstrahiert hier die Größe von spitzen Elementen. – Kos

Hmm ich dachte Subtrahieren eines Zeigers zu einem anderen wäre immer noch ein Zeiger :( –

Nein, strchr() einen Zeiger (die Adresse) gibt das Zeichen gesucht wird, oder NULL, wenn das Zeichen nicht gefunden wurde.

Das ist sehr verschieden von IndexOf().

Quelle

2010-12-12 00:51:21

Das OP scheint das bereits zu kennen, nach dem Code zu urteilen – Kos

Ich habe nicht gesehen, wo er NULL behandelt.Außerdem fragte er speziell nach IndexOf(). –

~~Da jedes Zeichen genau sizeof (char) Byte belegt; nicht sizeof (char*) Bytes.~~

Und sizeof (char) ist per Definition 1, so können Sie es weglassen:

return 1 + (endOcurrence - firstOcurrence);

Quelle

2010-12-12 00:51:52 pmg

Eigentlich, auch wenn 'sizeof (char)' nicht 1 wäre, würde die Division falsch sein.Das liegt daran, dass Zeiger Subtraktion Array ergibt Index oder Anzahl der Elemente, nicht Anzahl der Bytes zwischen den beiden Zeigern – jpalecek

@jpalecek: richtig, Antwort zerkratzt :) – pmg

Ihre return-Anweisung mehrere Probleme hat, aufgrund nicht Pointer-Arithmetik zu verstehen.

Die Pointer-Subtraktion teilt sich bereits durch die Elementgröße, und char* war sowieso der falsche Typ.

Und Sie sollten hinzufügen, 1, nicht 2.

Quelle

2010-12-12 00:52:11

Antwort

Verwandte Themen