Код-ревью сессия #2🧑‍💻

А вот и вторая ревью сессия подлетела!

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

Откроет нашу сессию пулл-реквест на Python от saintbyte. Простая тулза для сбора вакансий из RSS ленты хабр карьеры.

👉 Коммит #2
#typescript #angular

Коммит прислал нам mbogomazov. Он написал игру «крестики-нолики» на Angular.

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

TimurBaldin прислал нам целый пулл-реквест на Java. Проект для генерации тестовых данных для ручного и автоматизированного тестирования.

👉 Коммит #4
#php

И последний небольшой коммит от andreaskosten на PHP. Класс DB реализует посредством гарантированно единственного экземпляра подключение к БД mysql через юзера с указанным набором привилегий.

❓Все вопросы по ходу сессии можете задавать в нашем чате.

⏳ У нас есть 72 часа, погнали!

#код_ревью_сессия

💬 Основной посыл комментариев

👉 пулл-реквест #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

Код-ревью сессия #3

Сегодня у нас аж 8 коммитов на разных языках. Ловите!

👉 Коммит #1
#C

Откроет нашу сессию анонимный коммит на С. Симулятор песка, воды и стен.

👉 ПР #2
#python

Коммит прислал нам uni-jacob. Бот, упрощающий рутину старост групп.

👉 Коммит #3
#JS

Dsgnfox прислал нам конструктор добавляющий загрузку и отправку файлов из формы.

👉 Коммит #4
#TS

Наш ревьюер QoreCode прислал свой коммит на ревью. Описание найдете в README.md

👉 ПР #5
#php #laravel

И еще один наш PHP ревьюер SnusnumrConceit прислал нам проект предназначенный для импорта Records в Excel-формате в таблицу.

👉 Коммит #6
#CSharp #задания

А вот и задания. Первым свое задание на ревью прислал нам CustomRider. Описание задания вы найдете здесь.

👉 ПР #7
#typescript #задания

Еще один наш ревьюер - Kp0c прислал задание на JS + Svelte. Пощупать его можно по этой ссылке.

👉 ПР #8
#Java #задания

Закроет нашу сессию ПР с заданием от ревьюера driversti.

❓Все вопросы по ходу сессии можете задавать в нашем чате.

⏳ 72 часа

Ревью сессия #7 👨‍💻

Друзья, мы начинаем седьмую ревью сессию!!

👉 Коммит #1
#typescript

FostyFost написал небольшую библиотеку, которая позволяет улучшить и упростить работу с redux-store. FostyFost хочет получить ревью на блоки Core и Next. Описание проекта можно найти тут. Для тех, кто хочет лучше понять структуру проекта и развернуть его у себя - ссылка на сам проект.

👉 Коммит #2
#Java

Rabestro прислал нам еще одну написанную им игру-задание для Hyperskill. Игра Minesweeper (Миноискатель) на Java. Думаю каждый из вас хоть раз в жизни в нее играл. Проект состоит из Game — основной класс игры. Реализует Runner интерфейс. Метод run запускает игру. Board — поле игры. CellState — состояние клетки поля. И GameState — состояние игры.

Rabestro хотел бы узнать у ревьюеров - на сколько использование одномерного массива в Board понятно. На сколько код чистый и понятный.

Описание тут. Развернуть сам проект можно по этой ссылке. Все ревью сюда.

👉 Коммит #3
#JavaScript

Maxi прислал нам свой слайдер фотографий написанный на JS. Пощупать и посмотреть его можно по этой ссылке. Макса интересует общий фидбэк по слайдеру, качество кода и что можно улучшить. Этот проект мы разберем сегодня в 20:30 на нашем стриме вместе с R1KO, QoreCode и SnusnumrConceit.

👉 Коммит #4
#HTML/CSS

Небольшой коммит HTML кода, который прислал нам Иван Музыка 🙂 Парню 13 лет, а он уже сделал свой сайт! Посмотреть его можно тут.

👉 ПР #5
#Swift

Сегодня выкладываем также наш первый ПР на Swift. Его прислал нам DimkaIsALifelongLearner.

👉 Коммит #6
#php #laravel

Ну и в завершение мой коммит на PHP Laravel, который я писал в самом начале моей IT карьеры)) Ревью этого коммита состоится на нашем сегодняшнем стриме, в 20:30 по мск! Приходите, если интересно взглянуть на говнокод!)

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

⏳Итоги сессии подведем в вск вечером.

Новое видео на YouTube 📺

📺 Друзья, для тех кто не смог поучаствовать в нашем вчерашнем стриме мы выложили полную запись стрима на нашем YouTube канале.

💬 Вместе с R1KO и QoreCode мы разобрали генератор картинок на PHP и решение задания #4 на TypeScript. Дали небольшие советы по архитектуре кода и ООП, а также разобрали основные ошибки, которые можно избегать.

👉 Поэтому переходите и смотрите, надеюсь будет полезно и интересно!

❗️Не забудьте написать ваш фидбэк по поводу стрима в комментариях к видео, а также поставить ему 👍 и подписаться на наш YouTube канал.