How to implement this method?

Question about the task

Under discussion

The for loops work just fine! The issue arises at temp part... "number(k) now found". how to solve this.

package com.codegym.task.task06.task0622; import java.io.BufferedReader; import java.io.InputStreamReader; import java.io.*; import java.util.*; /* Ascending numbers */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); ArrayList<Integer> number = new ArrayList<Integer>(5); for (int i=0; i<5; i++){ String num = reader.readLine(); int Num = Integer.parseInt(num); number.add(Num) ; } ArrayList<Integer> temp = new ArrayList<Integer>(5); for(int k = 0;k <number.size(); k++) { for(int j = 1; j<number.size(); j++) { if (number.get(k) < number.get(j)) temp = number(k); number(k) = number(j); number(j) = temp; } } Arrays.sort(numbers); for(int j = 0; j<numbers.length; j++){ System.out.println(Arrays.toString(list)); }//write your code here System.out.println(number); } }

Comments (10)

Popular
New
Old

You must be signed in to leave a comment

~mK

Level 7 , Princeton, NJ, usa

3 April 2020, 23:28

// Why do have to work so f--n hard? // ( remember the kiss principle? : KISS = Keep it simple Stupid // Hint** you need to import something! public class Solution { public static void main(String[] args) throws Exception { int[] myList = new int[5]; Scanner kb = new Scanner(System.in); for (int i = 0; i < myList.length; i++) { myList[i] = kb.nextInt(); } for (int i = 0; i < myList.length; i++) { Arrays.sort(myList); System.out.println(myList[i]); } } }

Guadalupe Gagnon

Level 37 , Tampa, United States

16 October 2019, 13:24

public static void main(String[] args) throws Exception {
   BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
   ArrayList<Integer> number = new ArrayList<Integer>(5);

   for (int i=0; i<5; i++){
      String num = reader.readLine();
      int Num = Integer.parseInt(num);
      number.add(Num) ;
   }
   /* ArrayList<Integer> temp = new ArrayList<Integer>(5);

   for(int k = 0;k <number.size(); k++) {
      for(int j = 1; j<number.size(); j++) {
         if (number.get(k) < number.get(j))
            temp = number(k);
            number(k) = number(j);
            number(j) = temp;
         }
      }
   } erase these lines, you don't need them */
   Arrays.sort(numbers); // this sorts the array in ascending order

   for(int j = 0; j<numbers.length; j++){
      System.out.println(Arrays.toString(list));
   } // change this array list to output the last 5 numbers in the numbers list

        //System.out.println(number);
    }

hidden #10462592

Level 8

17 October 2019, 08:05

Thanks! in-built functions to the rescue!

hidden #10462592

Level 8

17 October 2019, 08:40

I tried this method, and it displays this error stating that --> no suitable method found for toString(java.util.ArrayList<java.lang.Integer>) method java.util.Arrays.toString(long[]) is not applicable (argument mismatch; java.util.ArrayList<java.lang.Integer> cannot be converted to long[]) method java.util.Arrays.toString(int[]) is not applicable (argument mismatch; java.util.ArrayList<java.lang.Integer> cannot be converted to int[]) method java.util.Arrays.toString(short[]) is not a ...... and so on, the list continues. I imported various packages like Collection, Arraylist, list and tried using Collections.sort(number) .. but I faced the same issue. Kindly let me know how you solved this error.

Guadalupe Gagnon

Level 37 , Tampa, United States

17 October 2019, 13:10

my bad, that line 21 should be number.sort(), not Array.sort(number);

hidden #10462592

Level 8

17 October 2019, 13:28

so I found this... when I create an array in this method : int[] arr = {21,34,3,435}; Arrays.sort(arr); ---> This works fine. but then, when we use for loop and make the array entry dynamic the sort function doesn't work. Do you know why???

hidden #10462592

Level 8

17 October 2019, 13:31

number.sort() ... Arrays.sort(number)...... Collection.sort(number) ... all these give the same error --> "no suitable sort method found" .. So I think in order to sort an array, the array should be initialised in the program and must not be filled using a for loop! This works fine --> int[] number = {15,48,3,11,9}; Arrays.sort(number); System.out.println(Arrays.toString(number));

Guadalupe Gagnon

Level 37 , Tampa, United States

17 October 2019, 13:36

Collections.sort(number) make sure its Collections, not Collection.

Guadalupe Gagnon

Level 37 , Tampa, United States

17 October 2019, 13:54

I can't remember all these rules, but Intellij helps so: number.sort() would work but it needs a Comparator to tell it how to sort, so the correct line should have been: number.sort(Comparator.naturalOrder()); (there is also one for reverseOrder) Collections.sort() and Arrays.sort() are the same thing, except one takes a List(like an ArrayList) while the other only takes an Array. These use a Comparator too, but defaults to naturalOrder. With Arrays.Sort you are not able to change the order with Arrays of primitives, like ints. You would need an Integer[] to change the order. Collections.sort() wont have that problem because you can't make primitive's lists. If you tried ArrayList<int> temp = new ArrayList<>(), the code would fail.

hidden #10462592

Level 8

17 October 2019, 14:11

Thank you so much!! although that didn't work I learnt new concepts. this is what finally did the job! int a = Integer.parseInt(reader.readLine()); int b = Integer.parseInt(reader.readLine()); int c = Integer.parseInt(reader.readLine()); int d = Integer.parseInt(reader.readLine()); int e = Integer.parseInt(reader.readLine()); int[] numbers = new int[]{a,b,c,d,e}; Arrays.sort(numbers); Why is the other method where we used "for-loop" didn't work with sort function??