Массив

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

Решен

Почему то не печатается последнее числов массиве, и не могу понять как сделать 2 и 3 пункт, подскажите пожалуйста

1. В методе initializeArray():
1.1. Создай массив на 20 чисел
1.2. Считай с консоли 20 чисел и заполни ими массив
2. Метод max(int[] array) должен находить максимальное число из элементов массива

Требования:

Метод initializeArray должен создавать массив из 20 целых чисел.
Метод initializeArray должен считать 20 чисел и вернуть их в виде массива.
Метод max должен возвращать максимальный элемент из переданного массива.
Метод main изменять нельзя.

package com.javarush.task.task07.task0701; import java.io.IOException; /* Массивный максимум */ import java.util.Scanner; public class Solution { public static void main(String[] args) throws Exception { int[] array = initializeArray(); int max = max(array); System.out.println(max); } public static int[] initializeArray() throws Exception { Scanner scanner=new Scanner(System.in); int array[]=new int[20]; for(int i=1;i<array.length ;i++){ int number=scanner.nextInt(); array[i]=number; System.out.println(array[i]); } // создай и заполни массив return null; } public static int max(int[] array) { // найди максимальное значение return 0; } }

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

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

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

Прыгун

Уровень 22

13 июля 2020, 09:29

2) ты считываешь в методе initializeArray 19 чисел, не учитываешь первое число массива(оно должно иметь индекс 0, а не один как у тебя в цикле) 3) ну сравни первое и второе число массива, найди из них максимальное, потом его сравни с третьим числом и так далее, используя цикл

Максим Носов

Уровень 11

13 июля 2020, 09:34

А почему в конце выводится 0?? 5 10 16 17 9 3 4 46 97 89 12 45 98 111 42 98 98 -9 71 -150 0

Максим Носов

Уровень 11

13 июля 2020, 09:37

package com.javarush.task.task07.task0701;

import java.io.IOException;

/*
Массивный максимум
*/
import java.util.Scanner;
public class Solution {
    public static void main(String[] args) throws Exception {
        int[] array = initializeArray();
        int max = max(array);
        System.out.println(max);
    }

    public static int[] initializeArray() throws Exception {
        Scanner scanner=new Scanner(System.in);
        int array[]=new int[20];

        for(int i=0;i<array.length ;i++){

            int number=scanner.nextInt();
            array[i]=number;
            System.out.println(array[i]);

        }



        // создай и заполни массив
        return null;
    }

    public static int max(int[] array) {
        int min=-10001;
        for(int i=0;i<array.length;i++){
            if(array[i]>min){
                min=array[i];
            }

        }




        // найди максимальное значение
        return 0;
    }
}

Прыгун

Уровень 22

13 июля 2020, 09:41

потому что ты в методе max возвращаешь 0, а не максимальное число(return 0 поменяй на return min;;) убери мин=-10001 и сравнивай в цикле array[i] и array[i+1], чтобы не вылез за пределы цикла, делай цикл пока i<array.length-1 метод initializeArray() должен возвращать массив а не ноль

Максим Носов

Уровень 11

13 июля 2020, 10:10

package com.javarush.task.task07.task0701;

import java.io.IOException;

/*
Массивный максимум
*/
import java.util.Scanner;
public class Solution {
    public static void main(String[] args) throws Exception {
        int[] array = initializeArray();
        int max = max(array);
        System.out.println(max);
    }

    public static int[] initializeArray() throws Exception {
        Scanner scanner=new Scanner(System.in);
        int array[]=new int[20];

        for(int i=0;i<array.length ;i++){

            int number=scanner.nextInt();
            array[i]=number;
            System.out.println(array[i]);

        }



        // создай и заполни массив
        return array;
    }

    public static int max(int[] array) {
        int max=-10001;
        for(int i=0;i<array.length-1;i++){
            if(array[i]>array[i+1]){
                max=array[i];
            }

        }




        // найди максимальное значение
        return max;
    }
}
Теперь только 3 пункт не проходит

Прыгун

Уровень 22

13 июля 2020, 10:19

int max=-10001; я ж говорил вот этого не надо😁 а вдруг у тебя все числа будут меньше чем -10001? сделай просто int max = array[0]; а в сравнении

if(array[i]<array[i+1]){
                max=array[i+1];
            }

Максим Носов

Уровень 11

13 июля 2020, 11:33

package com.javarush.task.task07.task0701;

import java.io.IOException;

/*
Массивный максимум
*/
import java.util.Scanner;
public class Solution {
        public static void main(String[] args) throws Exception {
        int[] array = initializeArray();
        int max = max(array);
        System.out.println(max);
    }

    public static int[] initializeArray() throws Exception {
        Scanner scanner=new Scanner(System.in);
        int array[]=new int[20];

        for(int i=0;i<array.length ;i++){

            int number=scanner.nextInt();
            array[i]=number;
            System.out.println(array[i]);

        }



        // создай и заполни массив
        return array;
    }

    public static int max(int[] array) {


        for(int i=0;i<array.length-1;i++){
            int max=array[0];
            if(array[i]<array[i+1]){
                max=array[i+1];


            }


        }




        // найди максимальное значение
        return max;
    }
}

Максим Носов

Уровень 11

13 июля 2020, 11:34

Не идет по 3 пункут(

Артем Divertitto Senior Android-разработчи в United Tech

Master

13 июля 2020, 13:11

У тебя код не скомпилится, ты объявил переменную max внутри цикла, она будет недоступна за его пределами

Артем Divertitto Senior Android-разработчи в United Tech

Master

13 июля 2020, 13:14

Ты сравниваешь два соседних числа, а нужно сравнивать с max

array[i]<array[i+1]