Finding index of array with given parameter

Question

I'm working on a java program, but I'm stuck on this specific section. This function is suppose to take an array as input and returns the index of the first occurrence of target in the input array, or -1 if not found. The function is suppose to call my contains method.

contains()

public static boolean contains(int[] input, int target) {
        for(int i = 0; i < input.length; i++){
            if (target == input[i]){

                return true;
            }
        }
        return false;
    }

indexof()

public static int indexOf(int[] input, int target) {
        if(contains(input, target) == true){
            return i;
        }
        return -1;
    }

I am trying to return what the variable i was in the contains method, but I'm not sure how to make the variable i transfer from the contains method to the indexof method without making it a public int in the constructor. Thanks for your help.

----------EDIT---------- The contains method is suppose to search the given array for the target value. If the target value exists somewhere in the array, return true. if not, return false. If any alterations need to be made to the contains method, that may be done as well.

You need to use the contains() method? It seems kind of redundant. — Tyler
– Tyler, Commented Mar 7, 2014 at 3:39
Just return the index i.e i in contains method or if not found return -1. Why do you need index of? — Amar
– Amar, Commented Mar 7, 2014 at 3:40
I need the contains method to only return if the target exists or not, and the indexOf method to return what the index of the number is. — Shan Solo
– Shan Solo, Commented Mar 7, 2014 at 3:47
maybe you could do this as a list. see stackoverflow.com/questions/4962361/… — TimeGuard
– TimeGuard, Commented Mar 7, 2014 at 4:00

Tyler · Accepted Answer · 2014-03-07 04:12:30Z

1

The only thing that would make sense would be to perform a check in your indexOf() using contains(), to make sure the variable exists.

public static boolean contains(int[] input, int target) {
    for(int i = 0; i < input.length; i++){
        if (target == input[i]){
            return true;
        }
    }
    return false;
}

public static int indexOf(int[] input, int target) {

    //If the value isn't there, return -1
    if(contains(input, target) == false){
        return -1;
    }

    //Go find the value if we know it is in there
    for(int i = 0; i < input.length; i++){
        if (target == input[i]){
            return i;
        }
    } 
    //We should never get here
    return -1;   
}

answered Mar 7, 2014 at 4:12

Tyler

18.2k10 gold badges55 silver badges89 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Shan Solo Over a year ago

This is the only one that makes sense right now. The instructor didn't say how we had to use the contains method, so I'm guessing this would be fine. Thank you!

Tyler Over a year ago

You're welcome. And you're right, it doesn't make that much sense to loop through the array twice.

djhaskin987 · Accepted Answer · 2014-03-07 03:40:40Z

1

indexOf should look a lot like contains. In fact, they should kind of be switched from what you have:

public static int indexOf(int[] input, int target) {
        for(int i = 0; i < input.length; i++){
            if (target == input[i]){
                return i;
            }
        }
        return -1;
    }

public static boolean contains(int[] input, int target) {
        if(indexOf(input, target) >= 0){
            return true;
        }
        return false;
    }

answered Mar 7, 2014 at 3:40

djhaskin987

10.2k5 gold badges54 silver badges90 bronze badges

1 Comment

Shan Solo Over a year ago

I have to call the contains method to the indexOf method.

user3390929 · Accepted Answer · 2014-03-07 03:42:13Z

1

There is a good chance you have this backwards. Either your instructor has a typo in his homework, or you misread or misunderstood.

It makes more sense for the contains() method to use the indexOf() method. This way should be quite easy to figure out.

answered Mar 7, 2014 at 3:42

user3390929

685 bronze badges

1 Comment

Shan Solo Over a year ago

"Build this indexOf() function by calling contains, which you’ve already made above." I already emailed him, he didn't make a typo.

KNU · Accepted Answer · 2014-03-07 04:44:30Z

Another way to do is by setting a static variable equals to index of target. The code is shown below :

import java.util.*;
import java.lang.*;
import java.io.*;


class TargetSearch
{
private static int posOfTarget;
public static void main (String[] args) throws java.lang.Exception
{
    int [] arr = {1,2,3,4,5,6};
    int target = 5;
    if(contains(arr,target)){
        System.out.println(target+" found at pos " + getPosOfTarget()+1);
    }
    else
        System.out.println(target+" doesn't exist in arr ");
}

public static boolean contains(int[] input, int target) {
for(int i = 0; i < input.length; i++){
    if (target == input[i]){
        setPosofTarget(i);
        return true;
    }
}
return false;
}

public static int indexOf(int[] input, int target) {
    if(contains(input, target) == true){
        //pos is already set in contains()
        return (getPosOfTarget());
    }
    else
        return -1;   
}

//getter setter of pos
public static int getPosOfTarget(){
    return posOfTarget;
};
public static void setPosofTarget(int i){
    posOfTarget = i;
};

}

Collectives™ on Stack Overflow

Finding index of array with given parameter

4 Answers 4

2 Comments

1 Comment

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

1 Comment

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related