Что я не учел?

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

Java Multithreading, 2 уровень, 9 лекция

Решен

Работает корректно на примерах Киев Нью-Йорк Амстердам Вена Мельбурн Нью-Йорк Новгород Дон Виен (это не совсем города, но цепочка показательна) Валидатор указывает на ошибки в последних двух пунктах, что не учел?

В методе main считай с консоли имя файла, который содержит слова, разделенные пробелом.
В методе getLine используя StringBuilder расставь все слова в таком порядке, чтобы последняя буква данного слова совпадала с первой буквой следующего не учитывая регистр.
Каждое слово должно участвовать 1 раз.
Считай, что абсолютно все слова из исходного списка могут (и должны!) быть включены в результат (лишних слов нет).
Метод getLine должен возвращать любой правильный вариант при наличии нескольких таковых (см. пример).
Слова разделять пробелом.
Вывести полученную строку на экран.

Пример тела входного файла:
Киев Нью-Йорк Амстердам Вена Мельбурн

Результат:
Амстердам Мельбурн Нью-Йорк Киев Вена
или
Вена Амстердам Мельбурн Нью-Йорк Киев
или
Мельбурн Нью-Йорк Киев Вена Амстердам
и т.п.

Требования:

Метод main должен считывать имя файла с клавиатуры.
В классе Solution не должно быть статических полей.
В методе getLine должен быть использован StringBuilder.
Метод getLine должен возвращать пустую строку (пустой StringBuilder) в случае если ему не были переданы параметры (слова).
Метод getLine не должен изменять переданные ему параметры (слова).
Все слова переданные в метод getLine должны быть включены в результирующую строку.
Вывод на экран должен соответствовать условию задачи.

Киев Нью-Йорк Амстердам Вена Мельбурн

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

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

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

JustinianJudge в Mega City One

Master

14 августа 2020, 12:18

решение

У меня твой код не отработал на:

париЖ амстердам Женева

По поводу кода:

for (String line : lines) arrayList.add(line);

циклы FOR/WHILE, оператор IF/ELSE IF/ELSE всегда пиши с фигурными скобками, есть очень редкие исключения для IF но придет время их выучишь, до тех пор пиши с фигурными скобками всегда, это маркер человека который делает первые шаги в джаве, твой уровень уже выше этого, не понижай качество своего кода на ровном месте

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

Для того чтобы ООП работало, его принципы нужно использовать, для полиморфизма коллекции объявляются через интерфейсы List<String> arrayList = ... + неинформативное имя, могут пистаь list, но оно 4 раза используется, многовато для временной переменной

ArrayList<String> arrayList= (ArrayList<String>) Arrays.stream(words)....

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

if (words.isEmpty()) return true;
if (s.toString().equals("")) lastChar = ' ';
 else lastChar = s.charAt(s.length() - 1);
if (!s.toString().equals("")) s.delete(s.length() - 1, s.length());

и так далее фигурные скобки

boolean flag = false;

есть JavaNamingConvention, имена boolean переменных в джаве начинаются либо с is, либо has + имя должно быть meaningful, раскрывать суть. То что это флаг, понятно с типа переменной, а что она делает, для чего предназначена?

public static boolean buildLine

метод с таким названием должен отдавать либо Билдер либо Строку. Имя метода должно быть другое, checkIfSmth, hasSmth и тд

Arseniy Zhuck

Уровень 30

14 августа 2020, 12:25

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

JustinianJudge в Mega City One

Master

14 августа 2020, 12:43

я не сильно вчитывался в код, если он там меняет по ссылке список, тогда ок. И снова-таки, в аргументах метода buildLine(StringBuilder s, ArrayList<String> words) должен быть List<String>

Arseniy Zhuck

Уровень 30

14 августа 2020, 11:10

Аман Донецк Дыра Куйбышев Новгород Новгород даже на таком наборе все корректно отрабатывает