я говорю такой, тип смогу за спринг и хибу пояснить, а мне говорят, ты же на стажера/джуна устраиваешься забей))

всегда будут неожиданные моменты (=

всегда будет что-то забываться

думаю это было и к лучшему, ибо излишняя самоуверенность в такие моменты вредит ахах

опять же - если на интервью начинается какая-то странная свистопляска "а вот это знаешь? а это? а вот это? а ещё это? не знаешь? ууу..." - то ну его нафиг. ибо во время работы когда столкнёшься с чем-то новым (а столкнёшься 100%) - проще и продуктивнее работать без этого давления

опять же - если на интервью начинается какая-то странная свистопляска "а вот это знаешь? а это? а вот это? а ещё это? не знаешь? ууу..." - то ну его нафиг. ибо во время работы когда столкнёшься с чем-то новым (а столкнёшься 100%) - проще и продуктивнее работать без этого давления

+

cdn гитзаба на территории России заблочили

у меня лично стили гитхаба не грузятся

а у вас как?

грузится без проблем

``` можете временно прописать в /etc/hosts следующие строки: 151.101.12.133 assets-cdn.github.com 151.101.12.133 avatars0.githubusercontent.com 151.101.12.133 avatars1.githubusercontent.com 151.101.12.133 avatars2.githubusercontent.com 151.101.12.133 avatars3.githubusercontent.com 151.101.12.133 avatars4.githubusercontent.com 151.101.12.133 avatars5.githubusercontent.com 151.101.12.133 avatars6.githubusercontent.com 151.101.12.133 avatars7.githubusercontent.com 151.101.12.133 avatars8.githubusercontent.com 151.101.12.133 avatars9.githubusercontent.com 151.101.12.133 help.github.com 151.101.12.133 pages.github.com 151.101.12.133 raw.githubusercontent.com ```

хранить константы в интерфейсе — зашквар?

обсуждали уже

если классы, реализующие интерфейс юзают все его константы - то нормально, иначе - обычный класс со статик файнал константами

Только при условии, что от этого класса потом не будет наследования. Как, например, наследуясь от JFrame, JPanel и других, получаем ненужные константы WIDTH, HEIGHT от ImageObserver. https://docs.oracle.com/javase/8/docs/api/java/awt/image/ImageObserver.html#WIDTH

Купи ключ за полторы тыщи и не парь себе мозги

Где

Он же стоит 3к

Если полторы я куплю

че, кто хочет обоссать меня?

ну то код посмотреть

очередного тестового задания

ну давай, показывай

https://github.com/angrynaz/weather

да, пиздец

просто пиздец

Ну рассказывай

у меня даже через анонимайзер стили гитхаба не прогружаются

Main там если что удаляется

Ааа

ща, погодь

Первое

Ты апи кей выложил

В гит

Там местами есть копипаст кода, но я не знаю как от него избавиться когда перегружаешь методы

Да, я знаю

Ну как бы

Это сейчас не важно

1. Нейминг пакетов 2. I-интерфейс - это сисярпу оставь 3. Статики много

Апикей низя

я не понял из-за стилей

но у тебя там внутри класса еще классы

но у тебя там внутри класса еще классы

Круто же

Ну блин, Мне это скидывать чувакам, если они это потестить захотят то регаться им что ли

Апикей низя

придирки, не имеют отношение к чистоте кода

Там где классы внутри класса это для gson, можно разнести но смысла не вижу

Ну блин, Мне это скидывать чувакам, если они это потестить захотят то регаться им что ли

передал бы ключ в личке им

Да он получается в два клика

короче, в целом нормально

Че лишний раз напрягать пацанов

но я бы порефакторил на свой лад

и заюзал бы рест темплэйт

private ResponseMode responseMode = ResponseMode.JSON;

энам у тебя и так константы содержит, никакого смысла

так делать

плюс если прямо указывать в коде - намного читабельнее, чем выискивать переменную responseMode

это чтоб по дефолту у класса поле было определенное

или что ты имеешь в виду?

я тебе просто говорю, что обычно так не делают

Тебе когда отправлять задание?

в понедельник

но планировал сегодня

а с неймингом пакетов чо не так? camelCase забыл?

Он же стоит 3к

Написал в личку

ERROR: S client not available

https://github.com/angrynaz/weather

Это че тестовое?))

да)

Еее))

было тяжело делать учитывая что тз внятного не было. "Модуль который сервис тыкает и чтоб там было что-то вроде JSON"

У меня cdn не грузится

Роскомнадзор опять козлит чтоли?

придирки, не имеют отношение к чистоте кода

Имеют , конфига нет для приложения

Роскомнадзор опять козлит чтоли?

https://geektimes.ru/p/278244/

Заливай на битбакет)))

А мне вот это не понравилось: String getAPIAnswer(String city, APIKey key, String mode) throws IOException String getAPIAnswer(int id, APIKey key, String mode) String getAPIAnswer(int zip, String countryCode, APIKey key, String mode) String getAPIAnswer(int lat, int lon, APIKey key, String mode) Если потом вызывать эти методы, то неясно какой там нужный вызовется.

Имеют , конфига нет для приложения

Main где указан апиключ можно вообще удалить, он там не нужен. Вся основа в враппер-классе лежит, чтобы можно было как либу подключить и все такое

В общем, резюмирую, это только мое мнение, ты сам решаешь прислушиваться или нет. 1. Вынести в отдельный пэкдж трансферные сущности(твои поджики), энамы, константы. Например, beans/pojo/common/dto 2. В отдельный пакет логику работы с погодным апи. У тебя вроде как сделано, я бы назвал пакет api просто. 3. Отдельно пакет для классов, которые дергают твои классы, которые работают с погодным апи. Т.е. по сути слой бизнес логики, например service/logit etc.. 4. Используй проперти для конфигурации апи кея. 5. Избавься от статик методов, если возможно, исправь нейминг методов в интерфейсе апи, непонятно.

А мне вот это не понравилось: String getAPIAnswer(String city, APIKey key, String mode) throws IOException String getAPIAnswer(int id, APIKey key, String mode) String getAPIAnswer(int zip, String countryCode, APIKey key, String mode) String getAPIAnswer(int lat, int lon, APIKey key, String mode) Если потом вызывать эти методы, то неясно какой там нужный вызовется.

эти методы напрямую не вызываются, а во враппере все имена нормальные

Я за: getApiResponseByCity(String city, ...) getApiResponseByZip(int zip, ...)

Я за: getApiResponseByCity(String city, ...) getApiResponseByZip(int zip, ...)

+

ладно, сделаю так, если кому-то придет в голову копаться в коде

>Избавься от статик методов, если возможно ну дак класс утилитарный, нет смысла его создавать

окей, тогда заюзай статик импорт

будет совсем красиво

годная идея

и классы внутри классов в твоем случае лучше вынести все в отдельный файл

как сделать так, что если добавлять все это дело в виде либы, то можно было бы воздействовать только с классом-враппером?

чтобы нельзя было создать какой-нибудь класс типа APIRequest и не намудрить там

не хочешь создавать класс - приватный конструктор

методы пряч оберткой

если не указывать модификатор паблик, видимость на уровне пакета

и т.д.

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

и эти методы будут известны только врапперу

тем самым юзверь не сможет создать класс служебный

достаточно просто приватный конструктор