Hi.

I made config with entity class names list. All working fine except checking:

 

String[] mob = Config.entityList.split(",");
    			
String[] mobNames = new String[mob.length];
    			
for(int i = 0; i < mob.length; i++)
{
    mobNames[i] = mob[i];
}

for (int i = 0; i < mobNames.length; i++)
{
    if (event.entity.class.getSimpleName() == mobNames[i])
    {
        //
    }
    else
    {
        System.out.println(event.entity.class.getSimpleName() + " --- "+ mobNames[i]);
    }
}

 

I getting it:

"EntityZombie --- EntityZombie"

 

Wt..?

You can't use == to compare strings in java. Use equals().

 

As to design - you should consider using class comprasions (isAssignableFrom() and EntityList.class from mc).

Ernio is right, just use the Class#isAssignableFrom method.

 

The only reason to compare strings is if one of the objects isn't actually a class reference. i.e. I was read class names from the config file and had to validate against a class reference (and then because it's user-entered, I wanted to be able to check for typos, so there's some Levenshtein distance code in there too; inefficient as hell, but it only runs when there's no match for the supplied string).