Bug? Comparing an element of csStringArray with const char* doesn't work?

[mark]

I'm using a csStringArray and want to compare the strings it contains:

Code:

csStringArray datenArray;
datenArray.Push(csString("netcmd"));
if(datenArray[0]=="netcmd") {...}

But: The comparision inside the if-statement returns false!
I checked datenArray[0] with csPrintf and it's right.
The comparision only works after I initialized another csString with datenArray[0]:

Code:

csStringArray datenArray;
datenArray.Push(csString("netcmd"));
csString xxx = datenArray[0];
if(xxx=="netcmd") {...}

this works! Why?

I know that csStringArray::operator[] returns a csString& and csString has operator==(const char *Str) const so this must be a bug - or what?

[jorrit]

I'm using a csStringArray and want to compare the strings it contains:

Code:

csStringArray datenArray;
datenArray.Push(csString("netcmd"));
if(datenArray[0]=="netcmd") {...}

But: The comparision inside the if-statement returns false!
I checked datenArray[0] with csPrintf and it's right.
The comparision only works after I initialized another csString with datenArray[0]:

Code:

csStringArray datenArray;
datenArray.Push(csString("netcmd"));
csString xxx = datenArray[0];
if(xxx=="netcmd") {...}

this works! Why?

I know that csStringArray::operator[] returns a csString& and csString has operator==(const char *Str) const so this must be a bug - or what?

csStringArray::operator[] returns const char* and NOT csString&. So you can't use the == operator because that will simply compare pointers. You have to either assign to csString like you are doing or else use the strcmp() function.

Greetings,

[mark]

nice to know
csArray<csString> is what I need in this case. Thanks for the answer.

[jorrit]

nice to know
csArray<csString> is what I need in this case. Thanks for the answer.

Why? Note that csArray<csString> will be less efficient in array handling than csStringArray. I would recommend csStringArray.

Greetings,

Here I have to disagree with Jorrit
Many times ease of use goes over efficiency, so I would say unless you find your string array to be any major performance or memory bottleneck, use csArray<csString> if you feel like doing it.

-M

[jorrit]

Perhaps but on the other hand csStringArray has a number of string-array specific utility functions which are not present in csArray<csString>. So in that sense using csStringArray is actually easier.

Greetings,