JavaRush/Помощь по Java задачам/Не понимаю почему компиляцию не проходит.

19 уровень

06.12.2019
447просмотров
17комментариев

Не понимаю почему компиляцию не проходит.

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

Задача по алгоритмам Ӏ Java Syntax: 5 уровень, 12 лекция

Обсуждается

Прошу направить знающих, что то я не могу понять в чем ошибка.

Написать программу, которая:
1. считывает с консоли число N, которое должно быть больше 0
2. потом считывает N чисел с консоли
3. выводит на экран максимальное из введенных N чисел.

Требования:

Программа должна считывать числа с клавиатуры.
Программа должна выводить число на экран.
В классе должен быть метод public static void main.
Нельзя добавлять новые методы в класс Solution.
Программа должна выводить на экран максимальное из введенных N чисел.
Программа не должна ничего выводить на экран, если N меньше либо равно 0.

package com.javarush.task.task05.task0532; import java.io.*; /* Задача по алгоритмам */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int maximum = 0; while(true){ String s = reader.readLine(); int n = Integer.parseInt(s); if(n>maximum){ maximum = n; }else{ continue; } } System.out.println(maximum); } }

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

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

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

Anton

Уровень 13

6 декабря 2019, 12:03

Тут задание еще понять надо... Сначала ты вводишь общее количество чисел (N). То есть столько чисел, сколько хочешь ввести. Потом начинаешь вводить числа. Как только ввел N чисел, на экран будет выводится самое максимальное. В цикле обрабатываешь ввод чисел до N и в этом же цикле поиск максимального. PS. Используй int maximum = Integer.MIN_VALUE; (для сравнения в условии поиска максимума)

Максим Толстобров

Уровень 19

6 декабря 2019, 13:57

Правильно я понимаю, что тут должно быть два ввода, сперва N , потом все поштучно ? Просто я еще не сталкивался с кодом, в котором нужно два раза что то вводить, не совсем понимаю как это написать. Что то типа:

String s = Integer.parseInt(reader.readLine());
String n = Integer.parseInt(reader.readLine());

???

Anton

Уровень 13

7 декабря 2019, 06:04

Ну совсем когда замучаешься могу скинуть код конечно. А так: два ввода должно быть да. Только первый до цикла - это число N (количество введенных цифр), а второй уже в цикле. Цикл while (tempCount < N). И уже в цикле делаешь второй ввод и ищешь максимальное. Про проверку не забудь. PS. Разбивай задачу на более мелкие задачки. Сначала просто реализуй введение числа, потом просто вывод введенных чисел и т.д.

Sekator

Уровень 41

6 декабря 2019, 11:41

у тебя в цикле то что не надо в цикл

int maximum = 0;
        int N = Integer.parseInt(reader.readLine());
        while (N>0) {

начните так а потом уже все остальное

Ksenia Volkova Java Developer в DXC

Master

6 декабря 2019, 10:54

Сначала нужно считать число (N) Потом, если N > 0, ты считать еще N чисел. То есть если N = 5 - значит 5, если N = 8 - то 8. Никаких бесконечных циклов. Из этих введенных чисел (не включая само N) надо найти самое большое и вывести.

Максим Толстобров

Уровень 19

6 декабря 2019, 11:34

Я уже таким образом переписал, через ArrayList, устал от этой задачи, она мне никак не дается.

package com.javarush.task.task05.task0532;


import java.io.*;
import java.util.ArrayList;
import java.util.Arrays;

/* 
Задача по алгоритмам
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int maximum = 0;

        ArrayList<Integer> list = new ArrayList<Integer>();
        
        while (true){
        String s = reader.readLine();
        if (s.isEmpty()) break;
        list.add(Integer.parseInt(s));
        }
        
        for(int i=0; i<list.size(); i++){
            if(list.get(i)>maximum){
                maximum = list.get(i);
                
            }
        }
        System.out.println(maximum);
        
    }
}

hidden #598481

Уровень 27

6 декабря 2019, 11:41

так ты читай задание, прежде чем делать, и уставать будешь меньше Условие: 2. потом считывает N чисел с консоли Твой код:

while (true){
    String s = reader.readLine();
    if (s.isEmpty()) break;

Еще раз. Условие: N чисел с консоли Твой код:

while (true)

Ну неужели не очевидно, что ты какую-то свою задачу решаешь?

Максим Толстобров

Уровень 19

6 декабря 2019, 13:59

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

Максим Толстобров

Уровень 19

6 декабря 2019, 14:01

Ксения, то есть я делаю цикл, в котором я считываю число, получаю число, и в этом цикле делаю второй цикл в котором буду считывать числа полученное число в первом вводе ?

hidden #598481

Уровень 27

6 декабря 2019, 14:05

зачем два цикла? сначала просто вводишь число, потом стандартный цикл на n итераций: for (int i = 0; i < n; i++)

Максим Толстобров

Уровень 19

6 декабря 2019, 14:08

Переписал таким образом, все равно не находит.

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int maximum = 0;

        String s = reader.readLine();
        int n = Integer.parseInt(s);

        for(int i=0; i<n; i++){
            String s1 = reader.readLine();
            int n1 = Integer.parseInt(s);

            if(n1>maximum){
                maximum = n1;
            }else{
                continue;
            }
        }

        System.out.println(maximum);
    }
}

Ksenia Volkova Java Developer в DXC

Master

6 декабря 2019, 14:11

Не проходит, потому что если все числа (кроме N) будут отрицательными, то максимума ты не найдешь. И еще нет проверки на N > 0 else{continue;} - лишние строки, которые не имеют смысла

Максим Толстобров

Уровень 19

6 декабря 2019, 14:26

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

package com.javarush.task.task05.task0532;


import java.io.*;
import java.util.ArrayList;
import java.util.Arrays;

/*
Задача по алгоритмам
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));


        String s = reader.readLine();
        int n = Integer.parseInt(s);
        int maximum = n;
        if(n>0){
            for(int i=0; i<n; i++){
            String s1 = reader.readLine();
            int n1 = Integer.parseInt(s1);

            if(n1>maximum){
            maximum = n1;
                }
            }
        }else if(n<=0){
        }
        System.out.println(maximum);
    }
}

hidden #598481

Уровень 27

6 декабря 2019, 14:59

Нельзя объявлять максимум изначально ни равным нулю, ни равным n. Как вообще максимальное значение связано с количеством слов, которые мы намереваемся ввести? Объяви его равным MIN_VALUE, как обычно. А еще от тебя требуется ничего не выводить, если n не положительное. Почему у тебя вывод вне ифа?

else if(n<=0){
}

пустые блоки это мусор, удаляй

Максим Толстобров

Уровень 19

6 декабря 2019, 19:54

Спасибо, я завтра почитаю про MIN_VALUE , кажется не много проясняется, я не знал про него. Про пустые блоки тоже спасибо, я понимаю что они ничего не делают, но почему то пишу для проформы. Постараюсь переучиться

hidden #598481

Уровень 27

6 декабря 2019, 20:37

MIN_VALUE это просто константа, чтобы не 6углить каждый раз минимальное возможное значение переменной int и не писать руками число -2,147,483,648

Гофф

Уровень 20

Master

6 декабря 2019, 20:44

Есть ещё вариант, для которого не требуется никаких внешних констант (и поэтому он предпочтительней). Считываешь N, потом считываешь первое число и его значение присваиваешь maximum. Далее как у тебя - цикл, читаешь следующее, сравнение с максимумом, если, то...