Условие выполняется, но валидатор не пропускает. Что опять я упустил?)

Вопрос по задаче

Решен

Решение не хочу смотреть, медленно но уверенно решаю сам) Но вот здесь именно в упор не понимаю почему валидатор не пропускает и пишет: "Метод sort должен сортировать переданный массив по убыванию, а не по возрастанию." Хотя программа выводить числа согласно условию!!

package com.javarush.task.task07.task0728;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.Arrays;

/*
В убывающем порядке
*/

public class Solution {
    public static int array_size = 7;
    public static int[] array = new int[array_size];

    public static void main(String[] args) throws Exception {
        Solution.sort(Solution.array);
    }

    public static void sort(int[] array) throws IOException {
        //напишите тут ваш код
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int[] result = new int[array_size];

        //Input
        for (int i = 0; i < array.length; i++)
            Solution.array[i] = Integer.parseInt(reader.readLine());

        //Sort
        Arrays.sort(array);

        //Show
        for (int i = array.length-1; i >= 0; i--)
            System.out.println(array[i]);

    }
}

Задача: Написать программу, которая вводит с клавиатуры 20 чисел и выводит их в убывающем порядке.

Требования:

Программа должна считывать 20 целых чисел с клавиатуры.
Программа должна выводить 20 чисел.
В классе Solution должен быть метод public static void sort(int[] array).
Метод main должен вызывать метод sort.
Метод sort должен сортировать переданный массив по убыванию.

package com.javarush.task.task07.task0728; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.Arrays; /* В убывающем порядке */ public class Solution { public static int array_size = 20; public static int[] array = new int[array_size]; public static void main(String[] args) throws Exception { Solution.sort(Solution.array); } public static void sort(int[] array) throws IOException { //напишите тут ваш код BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int[] result = new int[array_size]; //Input for (int i = 0; i < array.length; i++) Solution.array[i] = Integer.parseInt(reader.readLine()); //Sort Arrays.sort(array); //Copy //int index = 0; for (int i = array.length-1; i >= 0; i--) //array[index++] = array[i]; System.out.println(array[i]); /* array = result.clone(); //Print for(int num : array) System.out.println(num);*/ } }

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #2322530

Уровень 41

25 ноября 2020, 10:40

1) статичных переменных тут быть не должно

public static int array_size = 20;
public static int[] array = new int[array_size];

2) это должно быть в методе main

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
int[] result = new int[array_size];

//Input
for (int i = 0; i < array.length; i++)
    Solution.array[i] = Integer.parseInt(reader.readLine());

3) метод sort у тебя сортирует по возрастанию. а выводит на экран по убыванию. метод sort должен сортировать по убыванию. а выводить на экран по порядку.

Arrays.sort(array);

//Copy
//int index = 0;
for (int i = array.length-1; i >= 0; i--)
    //array[index++] = array[i];
    System.out.println(array[i]);

ForJavaRush

Уровень 15

25 ноября 2020, 10:44

1. Про статические переменные не написано. 2. Негде не указанно что массив должен заполнятся именно в main.

hidden #2322530

Уровень 41

25 ноября 2020, 10:48

1. если метод main запустить несколько раз (не перезапуская программу) то у тебя из-за статичных переменных возникнут проблемы. во всех задачах, если не указано что надо использовать статичные переменные, то надо стараться избегать их. 2. метод sort() должен сортировать список, а не создавать его. но даже если взять в пример что валидатор не докопается к тому, где ты создаёшь массив. то у тебя сортировка по возрастанию идёт. а должна быть по убыванию.

Arrays.sort(array);

для валидатора важно каким будет массив по результату работы sort(), а не как ты его выводишь.

ForJavaRush

Уровень 15

25 ноября 2020, 10:54

Вот!! в последнем предложении вся суть) ибо я думал что сортировку могу осуществлять НЕ обязательно меняя элементы между собой местами(на мой взгляд это очень важная детать именно в этой задаче.) Думал что цель это отобразить данные согласно условию(пробовал даже их писать в отдельный массив и потом его уже выводить)

hidden #2322530

Уровень 41

25 ноября 2020, 10:57

как бы условие задания гласит очень четко что надо сделать. Метод sort должен сортировать переданный массив по убыванию.

ForJavaRush

Уровень 15

25 ноября 2020, 12:01

Спасибо за все ответы выше)
Намекните что здесь не верно...


package com.javarush.task.task07.task0728;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

/*
В убывающем порядке
*/

public class Solution {

    public static void main(String[] args) throws Exception {

        int[] array = new int[20];
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        //Input
        for (int i = 0; i < array.length; i++)
            array[i] = Integer.parseInt(reader.readLine());

       Solution.sort(array);
    }

    public static void sort(int[] array) throws IOException {

        //Sort
        int max = 0;
        for(int i=0; i < array.length; i++) {
            for(int j=0; j < array.length; j++) {
              if(array[i] > array[j]) {
                  max = array[j];
                  array[j] = array[i];
                  array[i] = max;
                  break;
              }
            }
        }

        //Print
        for(int value : array)
            System.out.println(value);

    }
}

hidden #2322530

Уровень 41

25 ноября 2020, 12:05

это лишнее.

break;

совет на будущее, прежде чем отправлять на проверку, запустить по возможности программу и посмотреть как она работает.

ForJavaRush

Уровень 15

25 ноября 2020, 15:41

Не внимательность и желание пройти больше уровней за меньший промежуток времени)) Спасибо!!