[ConorCC]

Hey There!

I just realized ArrayFindString will returns the index of the first occurence of the specified string. As it says in the documentation.

If you have an array like:
- de_dust2_long
- de_nuke

And you are looking for a string like:
- de_dust2

The function will returns 0 instead of -1. I know de_dust2 is part of de_dust2_long, but wouldn't be nice a 3rd param of the original function to search exact string or substing?

PHP Code:

ArrayFindStringEx(&Array: Which, szItem[], bExact = 1) {

    if ( !bExact )
        return ArrayFindString(Which, szItem);

    new szTmpItem[32], iArraySize = ArraySize(Which);

    for ( new i; i < iArraySize; i ++ ) {

        ArrayGetString(Which, i, szTmpItem, charsmax(szTmpItem));

        if ( equal(szItem, szTmpItem) )
            return i;
    }

    return -1;
} 


[fysiks]

This has been reported here but in the context of this being a bug of the original function. I personally agree with this being a bug for this function as opposed to a feature request to be added.

[Shadows Adi]

Check this out, it does the same thing but it compares every item from array.
https://forums.alliedmods.net/showth...=329651&page=2

[HamletEagle]

Originally Posted by fysiks This has been reported here but in the context of this being a bug of the original function. I personally agree with this being a bug for this function as opposed to a feature request to be added.

I don't know if I would call it a bug, but it certainly is not intuitive. In the example provided by op, I would expect to get -1, not 0.
At this point, the functionality can not be changed(to keep backwards compatibility), but a new argument shoule be added to control how the string is matched.

[fysiks]

Originally Posted by HamletEagle At this point, the functionality can not be changed(to keep backwards compatibility)

I certainly understand the argument but you could also argue that 1.9.0 has never actually been "released" and therefore people cant necessarily expect things not to change .

[CrazY.]

And should include another argument for case sensitiveness. In its current state, there is no real use, you'd better just write a stock for searching strings.