Помогите пожалуйста. Не получается с самой маленькой строчкой....

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

Решен

package com.javarush.task.task07.task0712;

import com.sun.org.apache.xerces.internal.xs.StringList;

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

/*
Самые-самые
*/

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

        ArrayList<String> list = new ArrayList<String>(10);

        BufferedReader stroki5 = new BufferedReader(new InputStreamReader(System.in));          //Конструктор для считвывания с клваиатуры



        String a1 = "";
        String b1 = "11111111111111111";            //String b1 = list.get(0);

        String[] li = new String[10];                                                            //Создаем новый массив на 5 строк;
        for (int i = 0; i < li.length; i++) {                                                   //Цикл для массива
            String s = stroki5.readLine();// Считываем строки с клавы;
            list.add(s);
            li[i] = s;

            if (s.length() > a1.length()) //Максимальное занчение
            {
                a1 = s;
            }

            else if (s.length() < b1.length()) //Минимальное значение
            {
                b1 = s;
            }

        }
        for (int i = 0; i < li.length; i++ )

        if (a1.length() == li[i].length())
        {
            System.out.println(a1);
            break;
        }
       else if ((b1.length() == li[i].length())){
            System.out.println(b1);
            break;
        }
    }
}

1. Создай список строк.
2. Добавь в него 10 строчек с клавиатуры.
3. Узнай, какая строка в списке встретится раньше: самая короткая или самая длинная.
Если таких строк несколько, то должны быть учтены самые первые из них.
4. Выведи на экран строку из п.3. Должна быть выведена одна строка.

Требования:

Объяви переменную типа список строк и сразу проинициализируй ee.
Программа должна считывать 10 строк с клавиатуры и добавлять их в список.
Программа должна выводить на экран самую короткую строку, если она была раньше самой длинной.
Программа должна выводить на экран самую длинную строку, если она была раньше самой короткой.
Должна быть выведена только одна строка.

/*package com.javarush.task.task07.task0712; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Самые-самые public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код } } */ package com.javarush.task.task07.task0712; import com.sun.org.apache.xerces.internal.xs.StringList; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Самые-самые */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> list = new ArrayList<String>(10); BufferedReader stroki5 = new BufferedReader(new InputStreamReader(System.in)); //Конструктор для считвывания с клваиатуры String a1 = ""; String b1 = "11111111111111111"; //String b1 = list.get(0); String[] li = new String[10]; //Создаем новый массив на 5 строк; for (int i = 0; i < li.length; i++) { //Цикл для массива String s = stroki5.readLine();// Считываем строки с клавы; list.add(s); li[i] = s; if (s.length() > a1.length()) //Максимальное занчение { a1 = s; } else if (s.length() < b1.length()) //Минимальное значение { b1 = s; } } for (int i = 0; i < li.length; i++ ) if (a1.length() == li[i].length()) { System.out.println(a1); break; } else if ((b1.length() == li[i].length())){ System.out.println(b1); break; } } } //напишите тут ваш код /* Самые-самые 1. Создай список строк. 2. Добавь в него 10 строчек с клавиатуры. 3. Узнай, какая строка в списке встретится раньше: самая короткая или самая длинная. Если таких строк несколько, то должны быть учтены самые первые из них. 4. Выведи на экран строку из п.3. Должна быть выведена одна строка. Требования: 1. Объяви переменную типа список строк и сразу проинициализируй ee. 2. Программа должна считывать 10 строк с клавиатуры и добавлять их в список. 3. Программа должна выводить на экран самую короткую строку, если она была раньше самой длинной. 4. Программа должна выводить на экран самую длинную строку, если она была раньше самой короткой. 5. Должна быть выведена только одна строка.*/

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

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

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

Riccio

Уровень 35

Master

12 апреля 2019, 22:01

решение

Вы не понимаете, что и для чего пишите.

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

        //почитайте про списки. цифра 10 здесь не нужна
        ArrayList<String> list = new ArrayList<String>(10);

        //что за название для ридера - stroki5?
        BufferedReader stroki5 = new BufferedReader(new InputStreamReader(System.in));


        String a1 = "";

        //это хардкод. это неправильное решение, так делать нельзя
        String b1 = "11111111111111111";

        //что такое li? как мне понять, спустя пару строк - что это за li? название переменной ниочём
        String[] li = new String[10];

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

            //почему stroki5, если считывается 10 раз?!
            String s = stroki5.readLine();

            //зачем добавлять считанную строку и в массив, и в список??
            list.add(s);
            li[i] = s;

            //"если длина элемента из списка больше нулевой длинны"
            //да почти однозначно, что введенная строка будет больше
            //но если она больше a1.length() - сравнение не переходит к else
            if (s.length() > a1.length()) //Максимальное занчение
            {
                a1 = s;

                //а всё, строка сюда не дошла - она же была больше длинны a1
                //хотя может быть она была минимальной, по длинне, строкой
            } else if (s.length() < b1.length())
            {
                b1 = s;
            }
        }

        for (int i = 0; i < li.length; i++)
            if (a1.length() == li[i].length()) {
                System.out.println(a1);
                break;
            } else if ((b1.length() == li[i].length())) {
                System.out.println(b1);
                break;
            }
    }
}

Riccio

Уровень 35

Master

12 апреля 2019, 22:02

Подходит сын к програмисту и спрашивает: — Папа, а что самое сложное в твоей работе? — Самое сложное, сынок — придумывать имена переменным Вам стоит внимательно изучить эту лекцию.

Antonio Abelardo

Уровень 15

13 апреля 2019, 08:20

Спасибо, согласен с многим. Понимаю что нужно писать понятным языком, else это не уважение других. Но на данном этапе пытаюсь понять синтаксис , а почистить котов думаю после этого. Хотя взять за правило стоит!

Antonio Abelardo

Уровень 15

13 апреля 2019, 08:33

Решение проходит, но только с 1111111111111111 Подскажи чем можно заменить

//это хардкод. это неправильное решение, так делать нельзя
   String b1 = "11111111111111111";

я пробывал брать из списка первую строку при помощи

String b1 = list.get(0);

но не проходит... почему тоже не пойму.

Riccio

Уровень 35

Master

13 апреля 2019, 08:45

1) Можно использовать за эталон длину первой строки. 2) Можно создать два списка (массива): один со строками, другой с длинами строк. Потом отсортировать список с длинами и взять крайние значения за min и max. Почему у вас не получилось - смотреть надо конкретный код, а не одну строку.

Antonio Abelardo

Уровень 15

13 апреля 2019, 09:22

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

        ArrayList<String> list = new ArrayList<String>();

        BufferedReader stroki10 = new BufferedReader(new InputStreamReader(System.in));          //Конструктор для считвывания с клваиатуры



        String a1 = "";
        String b1 = "1111111111111111";            //String b1 = list.get(0);

        String[] li = new String[10];                                                            //Создаем новый массив на 5 строк;
        for (int i = 0; i < li.length; i++) {                                                   //Цикл для массива
            String s = stroki10.readLine();// Считываем строки с клавы;
            list.add(s);
            li[i] = s;

            if (s.length() < b1.length()) //Максимальное занчение
            {
                b1 = s;
            }

            if (s.length() > a1.length()) //Минимальное значение
            {
                a1 = s;
            }

        }
        for (int i = 0; i < li.length; i++ )

        if (b1.length() == li[i].length())
        {
            System.out.println(b1);
            break;
        }
       else if ((a1.length() == li[i].length())){
            System.out.println(a1);
            break;
        }
    }
}

Riccio

Уровень 35

Master

13 апреля 2019, 15:47

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

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int max = -1;
        int min = -1;
        String[] array = new String[10];
        for (int i = 0; i < array.length; i++) {

            array[i] = reader.readLine();

            if (i == 0) {
                max = array[i].length();
                min = array[i].length();
            }

            if (array[i].length() < min) {
                min = array[i].length();
            }

            if (array[i].length() > max) {
                max = array[i].length();
            }

        }
        for (int i = 0; i < array.length; i++)
            if (array[i].length() == min || array[i].length() == max) {
                System.out.println(array[i]);
                break;
            }
    }
}