CODEWITHSUNDEEP
javaarraylist
Lior Iluz
Lior Iluz

Reputation: 26563

Java - get element position in array

I'm familiar with the ways I can get an element position in array, especially the ones showed here: Element position in array

But my problem is I can't figure out how to convert this code to fit my needs.

What I want to check is if a String has a match in an ArrayList and if so, what's the index of the String in the ArrayList.

The annoying part is I managed to verify the String is in the ArrayList (see first line of my code)

listPackages is the ArrayList

current_package is the String I want to find its position in listPackages.

Here's my code:

if (listPackages.contains(current_package)) {

        int position = -1;
        for(int j = 0; j < listPackages.size(); j++) {

            if(listPackages[j] == current_package) {
              position = j;
                  break;
              }
            }
    }

Would appreciate any help!

Thanks!

Upvotes: 21

Views: 82109

Answers (4)

Android Killer
Android Killer

Reputation: 18489

Hope this will help you.change your code like this:

if (listPackages.contains(current_package)){
int position=listPackages.indexOf(current_package);
}

Also if you will make position variable as global you can access its value outside this block of code. :)

Upvotes: 2

mcfinnigan
mcfinnigan

Reputation: 11638

just use the call listPackages.indexOf(current_package);

ArrayList.contains(Object o) calls indexOf(Object o) internally in ArrayList:

/**
 * Returns <tt>true</tt> if this list contains the specified element.
 * More formally, returns <tt>true</tt> if and only if this list contains
 * at least one element <tt>e</tt> such that
 * <tt>(o==null&nbsp;?&nbsp;e==null&nbsp;:&nbsp;o.equals(e))</tt>.
 *
 * @param o element whose presence in this list is to be tested
 * @return <tt>true</tt> if this list contains the specified element
 */
public boolean contains(Object o) {
return indexOf(o) >= 0;
}

Upvotes: 3

Jayendra
Jayendra

Reputation: 52779

use the indexof method to get the position - 

listPackages.indexOf(current_package)

http://download.oracle.com/javase/1.4.2/docs/api/java/util/ArrayList.html#indexOf(java.lang.Object)

Upvotes: 1

Jon Skeet
Jon Skeet

Reputation: 1500525

Use indexOf:

int index = listPackages.indexOf(current_package);

Note that you shouldn't generally use == to compare strings - that will compare references, i.e. whether the two values are references to the same object, rather than to equal strings. Instead, you should call equals(). That's probably what was going wrong with your existing code, but obviously using indexOf is a lot simpler.

Upvotes: 44

Related Questions