< codereview />: IT карьера под ключ

2021-09-26 19:15:00 Основной посыл комментариев

коммит #4
#php

1. Использование getter и setter
$pdo изменить с public на private или protected, и работать с ним в других классах только через getter и setter.

2. Типизация параметров
Использовать типизацию везде, где это возможно.

3. Строгое сравнение
Старайтесь придерживаться строго сравнения ===.

4. Использование встроенных функций
Например вместо !isset лучше использовать метод empty.

5. Использование отдельного файла для настроек
Не надо хардокдить настройки подключения к DB в функции. Лучше вынести все параметры в отдельный env файл.

6. Проверка уровня доступа
Вместо строк SIUD лучше использовать числовые значения. И вместо if используйте elseif или switch.

Автор кода выделил следующих ревьюеров

1. SnusnumrConceit
2. R1KO
3. lis-dev Открыть/Комментировать

2021-09-26 13:28:25 Основной посыл комментариев

пулл-реквест #3
#java

1. Отдавать предпочтение примитивным типам данных
Если нам не требются свойства или методы таких класов как Boolean или, например, Integer, вместо них имеет смысл использовать примитивные типы данных boolean и int.

2. Использовать Lombok
Эта библиотека содержит много аннотаций, которые упростят написание конструкторов, геттеров, сеттеров (и не только их).

3. Не забывать о проверках на null
NullPointerException - распространенная ошибка. Следует написать проверки на получение данных из DB и проверять полученные значения на null.

4. Обращать внимание на нейминг
Такие названия классов как Start, AppException или BaseResponse не совсем раскрывают свою суть. Следует подумать над более удачными альтернативами.

Автор кода выделил следующих ревьюеров

1. driversti
2. lixach93 Открыть/Комментировать

2021-09-26 12:00:50 Основной посыл комментариев

пулл-реквест #1
#python

1. Стукрута проекта
Лучше всего создать папку src, разбить логику по моделям, и сервисам и вынести все из корневой папки.

2. Доработать модели
Вынести BaseModel в отдельный файл. В Vacancy добавить employment type с EnumField.

3. Нейминг
Избавиться от Helpers, чтобы поднять читаемость кода.

коммит #2
#typescript #angular

1. Задать точный тип возврата функции
Функция calculateWinner вернет X | O | null.

2. Избегать повторяемость кода
X и O используется в нескольких местах. Лучше вынести в качестве констант.

3. Неявное приведение объекта к boolean
Вместо проверки if(Winner) => isWinnerExists, и вместо this.squares[idx] => используй includes.

4. Избегать логических ошибок
makeMove будет вызывать calculateWinner, даже если ничего не изменилось.

Хотел бы выделить следующих ревьюеров

fallncrlss - за информативный и структурированный комментарий для python.
QoreCode - за большой объем полезных комментариев для TypeScript Открыть/Комментировать

2021-09-25 21:20:41 Итоги сессии #2

Прошло 72 часа с начала второй код-ревью сессии, и можно подвести итоги.

Я вижу, как пополняется наш состав ревьюеров - это не может не радовать! Но еще приятнее - видеть позитивные отзывы людей, которые присылают нам свои коммиты. Это значит, что фидбэк, который вы им даете, имеет для них большое значение!

Это только начало, и нам еще многое предстоит сделать.

Если говорить о недочетах, сейчас у нас образовалось большое PHP комьюнити, которому уже явно не хватает свободного места на одном коммите. Это значит, что на следующую ревью сессию мы возьмем как минимум два коммита на PHP.

Второй момент - соотношение PHP ревьюеров с ревьюерами на других языках. Нам явно стоит поработать над расширением Python, Java, и фронтенд комьюнити. Этим я и займусь на следующей неделе.

Таблица топ-ревьюеров уже обновлена, а завтра я выложу небольшую выжимку по коммитам.

Надеюсь, что вторая сессия понравилась вам не меньше первой, и от себя хочу поблагодарить всех, кто принимает в этом участие! Открыть/Комментировать

2021-09-24 11:05:00 Напоминаю, что ревью сессия в самом разгаре.

В коммите на PHP вы уже вряд ли найдете местечко, но вот на других коммитах еще очень много пространства для творчества. Товарищи питонисты, джависты и фронтендеры, накидайте свой фидбэк на коммиты в предыдущем посте, а также не забудьте поставить под полезными комментариями.

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

#код_ревью_сессия Открыть/Комментировать