Size: a a a

Java/Kotlin and more

2021 February 28

AK

Anton Krasnov in Java/Kotlin and more
я не против такого,  я делал разные тестовые при прохождении собесов. И если бы мне сейчас предложили сделать такое я бы просто этого не делал т.к. это банально трата времени на создание того что уже создано. При том что тебе банально после того как отправишь могут не ответить даже
источник

VS

Vladimir Smirnov in Java/Kotlin and more
Anton Krasnov
я не против такого,  я делал разные тестовые при прохождении собесов. И если бы мне сейчас предложили сделать такое я бы просто этого не делал т.к. это банально трата времени на создание того что уже создано. При том что тебе банально после того как отправишь могут не ответить даже
Ну в общем случае, любое тестовое - это бесполезная трата времени)
источник

AK

Anton Krasnov in Java/Kotlin and more
если тестовое матчится с реальной жизнью то к нему нет вопросов
источник

R

Rus in Java/Kotlin and more
Vladimir Smirnov
Ну в общем случае, любое тестовое - это бесполезная трата времени)
мне больше нравятся тестовые чем тех собесы
источник

AK

Anton Krasnov in Java/Kotlin and more
или физбас написать пошарив экран при собесе
источник

AK

Anton Krasnov in Java/Kotlin and more
тоже и по врмени не затратно и изобретать ничего не нужно
источник

R

Roman in Java/Kotlin and more
Закрытие ресурсов вынеси в final блок либо юзай try-with-resources
источник

R

Rus in Java/Kotlin and more
Roman
Закрытие ресурсов вынеси в final блок либо юзай try-with-resources
уже сделано. еще?
источник

VS

Vladimir Smirnov in Java/Kotlin and more
Rus
мне больше нравятся тестовые чем тех собесы
Обычно не исключает одно другое
источник

VS

Vladimir Smirnov in Java/Kotlin and more
Что тестовое, что решение всяких задачек из хакерранка/кодворс/етс. не особо адекватно показывает уровень кандидата, но вариантов особо нет
источник

R

Roman in Java/Kotlin and more
Rus
уже сделано. еще?
API_KEY и т.д я бы сделал константами. + очень много кода ввалишь в один метод, попробуй вынести лишнюю логику в приватные методы
источник

R

Rus in Java/Kotlin and more
Roman
API_KEY и т.д я бы сделал константами. + очень много кода ввалишь в один метод, попробуй вынести лишнюю логику в приватные методы
принято. а вообще в общем по реализации правильно?
источник

AE

Alexandr Emelyanov in Java/Kotlin and more
1. *.iml файлы тоже в гитигнор
2. case без default ветки
3. writeResponse нарушает единственность ответственности, у тебя на уровень выше должен быть разбор сначала url, где определяется тип сервиса, потом для каждого сервиса разбор content type(whether service только json как я понял), потом отдельные обработчики для разных вызовов, которые вызывают конкретный сервис
4. Обработчики тоже в разные классы
источник

VS

Vladimir Smirnov in Java/Kotlin and more
Вообще по задаче не оч понятно, звучит как будто должен быть общий эмулятор, в который можно разные конфиги подставить и он будет работать по этим конфигам
В у тебя сейчас они по сути хардкодятся в эмуляцию
источник

СЕ

Сергей Ермаков... in Java/Kotlin and more
Rus
принято. а вообще в общем по реализации правильно?
Строковые литералы в константы по идее надо
источник

VS

Vladimir Smirnov in Java/Kotlin and more
+ Тестов нет
источник

R

Rus in Java/Kotlin and more
всем спасибо за помощь 👍👍👍 буду думать
источник

СЕ

Сергей Ермаков... in Java/Kotlin and more
Rus
всем спасибо за помощь 👍👍👍 буду думать
Я не супердавно работаю в этой сфере, но я считаю полной дичью делать таску на голой жабе, это все равно что предлагать работу на мерседесе на автомате и требовать опыт езды на камазе, зачем?
источник

AS

Anatoly Shirokov in Java/Kotlin and more
Сергей Ермаков
Я не супердавно работаю в этой сфере, но я считаю полной дичью делать таску на голой жабе, это все равно что предлагать работу на мерседесе на автомате и требовать опыт езды на камазе, зачем?
А что будешь делать, если мерс станет?
источник

R

Rus in Java/Kotlin and more
Alexandr Emelyanov
1. *.iml файлы тоже в гитигнор
2. case без default ветки
3. writeResponse нарушает единственность ответственности, у тебя на уровень выше должен быть разбор сначала url, где определяется тип сервиса, потом для каждого сервиса разбор content type(whether service только json как я понял), потом отдельные обработчики для разных вызовов, которые вызывают конкретный сервис
4. Обработчики тоже в разные классы
тут по 3 пункту с getQuery() должен принимать только WeatherService а с боди GeoLocationService
источник