Во время работы в LinkedIn большая часть моей работы составляло код-ревью. Вышло так, что некоторые рекомендации я давал много раз, поэтому я решил составить список, которым поделился с командой.
Вот мои 3 (+1 бонусная) наиболее распространенные рекомендации по код-ревью.

Рекомендация 1: Выбрасывайте исключения, если что то идет не так
Зачастую паттерн выглядит так:
List<String> getSearchResults(...) {
try {
List<String> results = // make REST call to search service
return results;
} catch (RemoteInvocationException e) {
return Collections.emptyList();
}
}
Этот паттерн вызвал перебои в одном из мобильных приложений, над которыми я работал. Поиск на стороне сервера, который мы использовали, начал выбрасывать исключения. Оказалось, на серверном API приложения был некоторый код, похожий на приведенный выше. Поэтому приложение получало 200 ответ сервера и с радостью показывало пустой список для каждого поискового запроса.
Читать полностью »







![[Ответ программиста из Google] Есть ли жизнь после 35-40 лет? (для разработчика софта) - 1 image](https://www.pvsm.ru/images/2017/11/20/otvet-programmista-iz-Google-est-li-jizn-posle-35-40-let-dlya-razrabotchika-softa.jpeg)