Search an arraylist for a string and print the string

Question

I'm trying to search an arrayList for a string specified by user input, then print the result of that search to the console. I understand I should be using toIndex() but I can't work out how to syntax it.

import java.util.*;

public class searchSongs {

    public static void main(String[]args){

        Scanner searchBar = new Scanner(System.in);
        System.out.println("Enter song title");
        String search = searchBar.nextLine().toUpperCase();

            for (int i = 0; i< MP3_catalogue.artist.size(); i++){
                if (MP3_catalogue.artist.contains(search)){
                    int SV = search.indexOf(search);
                    System.out.println(MP3_catalogue.title.get(SV));
                    System.out.println(MP3_catalogue.artist.get(SV));
                    System.out.println(MP3_catalogue.duration.get(SV));
            }
        }

        MP3_catalogue obj = new MP3_catalogue();
    }
}

EDIT: The main class is MP3_catalogue which contains the arrayLists. No need to do anything special with the other arrayLists they have the same index values as artist

import java.util.*;

public class MP3_catalogue {
    static String gotoMenu = new String();

//"gotoMenu" is variable used to return to menu after each interaction with the methods.

public static ArrayList<String> title = new ArrayList<String>(); 
public static ArrayList<String> artist = new ArrayList<String>();
public static ArrayList<String> duration = new ArrayList<String>();

//arrayLists for song elements

@SuppressWarnings("resource")
public static void main(String[]args){
    System.out.println("Welcome to your music catalogue. \n \n" + "Menu choices:");
    System.out.println("A to add songs\n" + "D to delete songs\n" + "S to search catalogue\n" + "C to change song\n" + "? to shuffle catalogue\n");

    gotoMenu = "Y"; 
    while (gotoMenu.equals("Y")){
    System.out.println("Enter your choice:"); 

    Scanner userOption = new Scanner(System.in); //scanner to choose menu option
    String choice = userOption.nextLine().toUpperCase();

        switch (choice) {//switch statement used to go to each menu option

        case "A": addSongs.main(args);//executes addSongs
            System.out.println("Would you like to return to menu? Press Y to return, press N to exit program.");//choice to return to menu
            String goback = userOption.nextLine().toUpperCase();
            if(goback.equals("N"))
            {
                gotoMenu = "N";
            }
            break;

        case "D": deleteSongs.main(args);
            System.out.println("Would you like to return to menu? Press Y to return, press N to exit program.");//choice to return to menu
            String returnMenu = userOption.nextLine().toUpperCase();
            if(returnMenu.equals("N"))
            {
            gotoMenu = "N";
            };
            break;

        case "S": searchSongs.main(args);
                    gotoMenu = "N";
            break;
        case "C": System.out.println("Change songs");
                    gotoMenu = "N";
            break;
        case "?": System.out.println("Shuffle time");
                    gotoMenu = "N";
            break;
        default: System.out.println("Doesn't match a menu choice. Type more carefully this time.");
            break;

        }
    }
}

}

stackoverflow.com/questions/2642589/… this might help understanding how the contains() method works — mastaH
– mastaH, Commented Nov 26, 2013 at 8:58

subash · Accepted Answer · 2013-11-26 08:32:33Z

2

this much enough. no need to for loop..

        if (MP3_catalogue.artist.contains(search)){
                int SV = MP3_catalogue.artist.indexOf(search);
                System.out.println(MP3_catalogue.title.get(SV));
                System.out.println(MP3_catalogue.artist.get(SV));
                System.out.println(MP3_catalogue.duration.get(SV));
        } else {
               System.out.println("not found");
        }

answered Nov 26, 2013 at 8:32

subash

3,1253 gold badges20 silver badges22 bronze badges

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

Comments

Scary Wombat · Accepted Answer · 2013-11-26 08:30:56Z

1

This looks wrong

int SV = search.indexOf(search);

You want to get the object from MP3_catalogue in the loop

for (int i = 0; i< MP3_catalogue.artist.size(); i++){
       Artist artist = MP3_catalogue.artist.get (i);
            if (artist.contains(search)){
                System.out.println(artist);
}

As I do not know your data structure, I can not say whether the above approach would also be OK for title and duration.

answered Nov 26, 2013 at 8:30

Scary Wombat

44.9k6 gold badges38 silver badges67 bronze badges

Comments

Satheesh Cheveri · Accepted Answer · 2013-11-26 09:53:20Z

I would prefer to write this way , encapsulate entire cataloge in to single bean,and access/use effectively .

class MP3Catalogue {
    private String title;
    private String artist;
    private String duration;

    public String getTitle() {
        return title;
    }

    public String getArtist() {
        return artist;
    }

    public String getDuration() {
        return duration;
    }
}



public class SearchSongs {
    public static ArrayList<MP3Catalogue> catelogs =  new ArrayList<MP3Catalogue>();
    public static void main(String[] args) {
        Scanner searchBar = new Scanner(System.in);
        System.out.println("Enter song title");
        String search = searchBar.nextLine().toUpperCase();

        for (MP3Catalogue cat : catelogs) {
            if (cat.getArtist().equalsIgnoreCase(search)) {
                System.out.println(" Title = " + cat.getTitle() +" Duration = " + cat.getDuration());
            }
        }
    }
}

ali4j · Accepted Answer · 2013-11-26 08:37:24Z

0

what's the usage of for loop in your code?

maybe you can change your code like this:

        for (int i = 0; i< MP3_catalogue.artist.size(); i++){
            if (MP3_catalogue.artist.get(i).equals(search)){
                int SV = i;
                System.out.println(MP3_catalogue.title.get(SV));
                System.out.println(MP3_catalogue.artist.get(SV));
                System.out.println(MP3_catalogue.duration.get(SV));
        }

answered Nov 26, 2013 at 8:37

ali4j

5223 silver badges15 bronze badges

2 Comments

Scary Wombat Over a year ago

why assign i to SV. This is just bad coding

ali4j Over a year ago

Yes you are right, the SV variable is additional, I just change your code to show the usage of for loop.

Collectives™ on Stack Overflow

Search an arraylist for a string and print the string

4 Answers 4

Comments

Comments

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related