Код-ревью сессия #1 🧑💻
Ну что же, мы начинаем нашу первую сессию. (немного волнуюсь.)
👉 Коммит #1
Первый коммит прислал нам Rustam. Проект написан на PHP + Laravel. Это открытая CRM, которая облегчает работу с клиентами, позволяет вести проекты и создавать таски.
Основное ревью будет по TaskController (CRUD тасков в приложении) и ClientController (CRUD клиентов). В нем также будут классы типа ClientStoreRequest или TaskStoreRequest, которые отвечают за валидацию реквеста.
В этом коммите есть много Blade файлов, но мне кажется ревью их будет довольно скучным, но если кому-то захочется, то welcome.
👉 Коммит #2
Следующий коммит нам прислал Wterh. Реальный пример из его проекта (биржа копирайтинга), который написан на чистом PHP. В нем всего один файл с одним методом, который отвечает за поиск свободных ссылок в базе, и если их нет, генерирует новую ссылку. Используется при изменении статуса задания.
❓Все вопросы по ходу сессии можете задавать в нашем чате.
#код_ревью_сессия #php
Ну что же, мы начинаем нашу первую сессию. (немного волнуюсь.)
👉 Коммит #1
Первый коммит прислал нам Rustam. Проект написан на PHP + Laravel. Это открытая CRM, которая облегчает работу с клиентами, позволяет вести проекты и создавать таски.
Основное ревью будет по TaskController (CRUD тасков в приложении) и ClientController (CRUD клиентов). В нем также будут классы типа ClientStoreRequest или TaskStoreRequest, которые отвечают за валидацию реквеста.
В этом коммите есть много Blade файлов, но мне кажется ревью их будет довольно скучным, но если кому-то захочется, то welcome.
👉 Коммит #2
Следующий коммит нам прислал Wterh. Реальный пример из его проекта (биржа копирайтинга), который написан на чистом PHP. В нем всего один файл с одним методом, который отвечает за поиск свободных ссылок в базе, и если их нет, генерирует новую ссылку. Используется при изменении статуса задания.
❓Все вопросы по ходу сессии можете задавать в нашем чате.
#код_ревью_сессия #php
Друзья, напоминаю!
Осталось меньше 24 часов, чтобы еще успеть оставить свое ревью на коммит#1 на PHP + Laravel и коммит#2 на PHP, а также проголосовать за понравившиеся вам комментарии (👍/👎). Вся инфа в посте выше.
А уже завтра мы подведем итоги нашей первой ревью сессии, выделим ключевые моменты из оставленных комментариев и отметим наиболее активных ревьюеров, которые принимали участие в сессии.
P.S
Накидайте ребятам звезд ⭐ на их проекты на гитхабе за то, что они первые решились отдать свой код п̶о̶д̶ ̶о̶б̶с̶т̶р̶е̶л̶ на ревью, им будет приятно.
#код_ревью_сессия #php #laravel
Осталось меньше 24 часов, чтобы еще успеть оставить свое ревью на коммит#1 на PHP + Laravel и коммит#2 на PHP, а также проголосовать за понравившиеся вам комментарии (👍/👎). Вся инфа в посте выше.
А уже завтра мы подведем итоги нашей первой ревью сессии, выделим ключевые моменты из оставленных комментариев и отметим наиболее активных ревьюеров, которые принимали участие в сессии.
P.S
Накидайте ребятам звезд ⭐ на их проекты на гитхабе за то, что они первые решились отдать свой код п̶о̶д̶ ̶о̶б̶с̶т̶р̶е̶л̶ на ревью, им будет приятно.
#код_ревью_сессия #php #laravel
Код-ревью сессия #2🧑💻
А вот и вторая ревью сессия подлетела!
👉 Пулл-реквест #1
#python
Откроет нашу сессию пулл-реквест на Python от saintbyte. Простая тулза для сбора вакансий из RSS ленты хабр карьеры.
👉 Коммит #2
#typescript #angular
Коммит прислал нам mbogomazov. Он написал игру «крестики-нолики» на Angular.
👉 Пулл-реквест #3
#java
TimurBaldin прислал нам целый пулл-реквест на Java. Проект для генерации тестовых данных для ручного и автоматизированного тестирования.
👉 Коммит #4
#php
И последний небольшой коммит от andreaskosten на PHP. Класс DB реализует посредством гарантированно единственного экземпляра подключение к БД mysql через юзера с указанным набором привилегий.
❓Все вопросы по ходу сессии можете задавать в нашем чате.
⏳ У нас есть 72 часа, погнали!
#код_ревью_сессия
А вот и вторая ревью сессия подлетела!
👉 Пулл-реквест #1
#python
Откроет нашу сессию пулл-реквест на Python от saintbyte. Простая тулза для сбора вакансий из RSS ленты хабр карьеры.
👉 Коммит #2
#typescript #angular
Коммит прислал нам mbogomazov. Он написал игру «крестики-нолики» на Angular.
👉 Пулл-реквест #3
#java
TimurBaldin прислал нам целый пулл-реквест на Java. Проект для генерации тестовых данных для ручного и автоматизированного тестирования.
👉 Коммит #4
#php
И последний небольшой коммит от andreaskosten на PHP. Класс DB реализует посредством гарантированно единственного экземпляра подключение к БД mysql через юзера с указанным набором привилегий.
❓Все вопросы по ходу сессии можете задавать в нашем чате.
⏳ У нас есть 72 часа, погнали!
#код_ревью_сессия
💬 Основной посыл комментариев
👉 коммит #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
👉 коммит #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
Код-ревью сессия #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 часа
Сегодня у нас аж 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 часа
Код-ревью сессия #4 👨💻
🚀 Сегодня стартует первая октябрьская сессия. Все ревью будут также зачитываться в октябрьский рейтинг ревьюеров. В конце месяца ревьюер, занявший 1 место получит небольшое вознаграждение в виде 25$. Всем удачи!)
—— Свои проекты ——
👉 Коммит #1
#php
От завсегдатого andreaskosten.
👉 Коммит #2
#python #анон
👉 Коммит #3
#php #анон
👉 Коммит #4
#php
Сортировщик фотографий от Savin-Igor. Выкладываем целый репо на ревью с нашего гитхаба.
—— Решения заданий ——
👉 решение #1
#python
Прислал mkile
👉 решение #2
#sql
Прислал Kp0c
👉 решение #3
#php
Прислал SnusnumrConceit
👉 решение #4
#sql #aнон
👉 решение #5
#сsharp
Прислал CustomRider
❓Все вопросы по ходу сессии можете задавать в нашем чате.
⏳ У нас 72 часа!
🚀 Сегодня стартует первая октябрьская сессия. Все ревью будут также зачитываться в октябрьский рейтинг ревьюеров. В конце месяца ревьюер, занявший 1 место получит небольшое вознаграждение в виде 25$. Всем удачи!)
—— Свои проекты ——
👉 Коммит #1
#php
От завсегдатого andreaskosten.
👉 Коммит #2
#python #анон
👉 Коммит #3
#php #анон
👉 Коммит #4
#php
Сортировщик фотографий от Savin-Igor. Выкладываем целый репо на ревью с нашего гитхаба.
—— Решения заданий ——
👉 решение #1
#python
Прислал mkile
👉 решение #2
#sql
Прислал Kp0c
👉 решение #3
#php
Прислал SnusnumrConceit
👉 решение #4
#sql #aнон
👉 решение #5
#сsharp
Прислал CustomRider
❓Все вопросы по ходу сессии можете задавать в нашем чате.
⏳ У нас 72 часа!
Код-ревью сессия #5 👨💻
🚀 Ну что же, начинаем нашу пятую код-ревью сессию! Сегодня у нас будет аж два новых языка: Go и Kotlin.
💰Напоминаю, что самый топовый ревьюер октября получит $25 в конце месяца. Поднажмите!
Коммит #1
#Kotlin
dimagor555 прислал на ревьюе приложение, которое позволяет создавать и сохранять факт карты.
Коммит #2
#Java
Аутентификация через Facebook на Java от maxappdev.
Коммит #3
#Go
Проект на GO от fallncrlss. Описание будет тут.
Коммит #4
#PHP
Коммит из рабочей библиотеки andreaskosten.
Коммит #5
#PHP #анон
Довольно большой проект скинули анонимно на ревью. Описание найдете тут
⏳Итоги подведем в вск. Поехали!
🚀 Ну что же, начинаем нашу пятую код-ревью сессию! Сегодня у нас будет аж два новых языка: Go и Kotlin.
💰Напоминаю, что самый топовый ревьюер октября получит $25 в конце месяца. Поднажмите!
Коммит #1
#Kotlin
dimagor555 прислал на ревьюе приложение, которое позволяет создавать и сохранять факт карты.
Коммит #2
#Java
Аутентификация через Facebook на Java от maxappdev.
Коммит #3
#Go
Проект на GO от fallncrlss. Описание будет тут.
Коммит #4
#PHP
Коммит из рабочей библиотеки andreaskosten.
Коммит #5
#PHP #анон
Довольно большой проект скинули анонимно на ревью. Описание найдете тут
⏳Итоги подведем в вск. Поехали!
Ревью сессия #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, которое позволяет видеть всю структуру репозитория, и удобно перемещаться между файлами.
⏳Итоги сессии подведем в вск вечером.
Друзья, мы начинаем седьмую ревью сессию!!
👉 Коммит #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, которое позволяет видеть всю структуру репозитория, и удобно перемещаться между файлами.
⏳Итоги сессии подведем в вск вечером.
GitHub
redux-eggs · YourCodeReview/review_sessions@c3cbb6b
fostyfost - redux-eggs
Новое видео на YouTube 📺
📺 Друзья, для тех кто не смог поучаствовать в нашем вчерашнем стриме мы выложили полную запись стрима на нашем YouTube канале.
💬 Вместе с R1KO и QoreCode мы разобрали генератор картинок на PHP и решение задания #4 на TypeScript. Дали небольшие советы по архитектуре кода и ООП, а также разобрали основные ошибки, которые можно избегать.
👉 Поэтому переходите и смотрите, надеюсь будет полезно и интересно!
❗️Не забудьте написать ваш фидбэк по поводу стрима в комментариях к видео, а также поставить ему 👍 и подписаться на наш YouTube канал.
📺 Друзья, для тех кто не смог поучаствовать в нашем вчерашнем стриме мы выложили полную запись стрима на нашем YouTube канале.
💬 Вместе с R1KO и QoreCode мы разобрали генератор картинок на PHP и решение задания #4 на TypeScript. Дали небольшие советы по архитектуре кода и ООП, а также разобрали основные ошибки, которые можно избегать.
👉 Поэтому переходите и смотрите, надеюсь будет полезно и интересно!
❗️Не забудьте написать ваш фидбэк по поводу стрима в комментариях к видео, а также поставить ему 👍 и подписаться на наш YouTube канал.
YouTube
Code Review на PHP и TypeScript в прямом эфире.
Улучшили код на TypeScript и PHP в прямом эфире.
В чт на нашей сессии мы разобрали генератор картинок на PHP, а также решение одной задачи с нашего канала на TypeScript. Дали советы по ООП и архитектурным решениям, а также разобрали ошибки в коде, которые…
В чт на нашей сессии мы разобрали генератор картинок на PHP, а также решение одной задачи с нашего канала на TypeScript. Дали советы по ООП и архитектурным решениям, а также разобрали ошибки в коде, которые…