Thread: Calculation looks like it should work, but doesn't.

System.out.println("\r" + "\n\tSorted by Rating\n");
 
	 for (int count = 0; count < bookL.size(); count++) {
		    for (int in = 0; in < bookList.size() - 1; in++) 
	            if (videoList.get(in).getRating() < bookList.get(in + 1).getRating()) {
	                Book temp = bookList.get(in);
	                bookList.set(in, bookList.get(in+1));
	                bookList.set(in+1, temp);
 
	            }
 
		    System.out.println(bookList.get(count).getRating() + "   " + bookList.get(count).getTitle());
 
	        }

I was told that I could easily alter the code to sort alphabetically as well.

 for (int count = 0; count < bookList.size(); count++) {
		    for (int in = 0; in < bookList.size() - 1; in++) 
		    	if (bookList.get(in).getTitle().compareTo(bookList.get(in + 1).getTitle()) < 0) {
	                Book temp = bookList.get(in);
	                bookList.set(in, bookList.get(in+1));
	                bookList.set(in+1, temp);
 
	            }
 
		    System.out.println(bookList.get(count).getRating() + "   " + bookList.get(count).getTitle()+"\t");
 
	        }

Why doesn't this work?

Can you post the code's output that shows what you are talking about.
Also print the contents of the arraylist before the sort and after the sort.

It's printing the content in the order it appears in the code.

--- Update ---

Do I need a compareTo bit in my Book class. Like with all this stuff?

String title;
    int rating;
 
    public Video(String pTitle, int pRating) {
    	title = pTitle;
    	rating = pRating;}
 
    public String getTitle() {
        return title;}
 
    public int getRating() {
        return rating;}

I've been trying to add a compareTo method in here but I don't really know how.

Please copy what is printed out and paste it here.
Print the contents of the arrraylist before the sort
and after the sort.

Also print out the contents of the arraylist inside of the if statement after each swap is done.