Searching an Array for a combination of letters and numbers

Question

I'm a first year programmer.

I've been trying to search an array which has stored four variables, with an input.

All of the examples I've found make use of int, and searches for a number within a list.

My program must search for a combination of letters and numbers. (Ex. COP 2800)

import java.util.Scanner;

public class courseInfo {
   public static int courseInfo(int[] list, int key) {

      Scanner input = new Scanner(System.in);

      // Input course name
      System.out.print("Enter course name: (Ex. COP 2800) ");
      double courseInput = input.nextDouble();

      for (int i = 0; i < list.length; i++) {
         if (key == list[i])
            return i;

         }
         return -1;
      }

      public static void main(String[] args) {
         int[] list = {COP 2800, PSY 1012, EVR 2001, COP 1000};
         System.out.println(linearSearch(list, courseInput));
   }
}

Please use layman's terms, I've only been in this class for three weeks.

If I remove COP, PSY, EVR, and COP from line 21, I return a different error;

courseInfo.java:22: error: cannot find symbol
         System.out.println(linearSearch(list, courseInput));
                                               ^
  symbol:   variable courseInput
  location: class courseInfo
1 error

The main method doesn't know the `courseInput` variable as it only exists in the `courseInfo` method. A second question is where the `linearSearch` method is defined. Also, your `int[]` will not compile. `COP 2800` is not an integer. Consider using a `String[]` and use `equals()` instead of `==` as `String`s are objects. — Jyr, Feb 21 '17 at 23:04
So many errors... but let's start with: where's your `linearSearch` method defined? — azurefrog, Feb 21 '17 at 23:04

score 0 · Answer 1 · answered Feb 21 '17 at 23:04

0

You are checking an equals condition on String values with a courseInput which is supposed to be an int.

Also, You are returning -1 for whatever happens in the if condition. Are you supposed to return -1 if the value of key doesn't equal the one in the list?

Also the courseInput variable is local to the method. Make it global by declaring it in the class as a member variable.

answered Feb 21 '17 at 23:04

Roe hit

457
1
7
23

Yes, I am supposed to produce and error if it cannot find the searched keyword. – Vykn Feb 21 '17 at 23:31
Add an else statement if(condition){ //code } else { return -1 } – Roe hit Feb 21 '17 at 23:40

score 0 · Answer 2 · answered Feb 21 '17 at 23:08

0

You have double courseInput = input.nextDouble(), which will only accept doubles, not characters. You also have int[] list, which is an array of integers, so it cannot hold characters either. You'll have better luck if you put the whole course name (both letters and numbers) into a String. Then you can scan an entire line and store it in an array of Strings.

answered Feb 21 '17 at 23:08

bluecanary

81
5

Okay, String will accept the combination of characters & integers? – Vykn Feb 21 '17 at 23:18
Yes, though keep in mind the integers will not be in `int` form. I'm assuming your goal is simply to print out the name of the course, so yes, you will be able to print "COP 2800". If you need the integers for calculations (e.g. sorting them so that COP 1000 is displayed before 2800) then you'd need to convert them into ints – bluecanary Feb 21 '17 at 23:33

score 0 · Answer 3 · edited May 23 '17 at 11:46

I suggest you do a bit more work on the basic data types of Java before attempting this. Java is a fairly strictly typed language which means that in general (there are plenty of exceptions) you need to decide what type of value you are storing in a variable ahead of time. You are mixing many types in your code: integers, doubles and strings.

So start by deciding if a course's identifier is a number of a name. If it's a number you can use an int to store it. If it's a name you can use a String. You note that a subject includes letters and numbers. But as long as you don't need to compare the numbers directly then you can store the entire identifier in a String. There's no need to split them up.

Another common trap for new starters is that primitive types (such as int) behave quite differently than objects (such as String). An immediate difference is the meaning of ==. For primitive types it compares the values while for objects it checks if the left and right side refer to the same object.

I suspect you want your subjects to be identified by name (i.e. String). In which case your code might look something like:

String[] subjects = {"COP2800", "PSY1012", "EVR2001"};

and

for (int i = 0; i < subjects.length; i++) {
    if (subjects[i].equals(name))
        return i;
}
return -1;

Searching an Array for a combination of letters and numbers

3 Answers3