Рев'ю

Що таке рев'ю?

Рев'ю (код-рев'ю) — це коли хтось аналізує ваш код та видає на нього рецензію, де вказує на слабкі місця та дає поради щодо оптимізації стилю й логіки.

Пам'ятайте: Якщо код не проходить хоча б один тест — він неробочий. Якщо є питання по неробочому коду — вам потрібно не рев'ю, а підтримка Ш++.

Рев'ю — це про повністю робочий код та його оптимізацію.

Як отримати рев'ю?

На курсі Zero рев'ю — текстові, у Discord. Якщо хочете рев'ю — пишіть до чату #zero-review повідомлення (запит рев'ю).

Приклад гарного запиту:

@zero-volo прошу рев'ю задачі Числовий ряд

Інтуїтивно мені здається, що у коді проблеми з побудовою логіки циклів: мабуть, можна спростити, але я поки не розумію, як саме. Тому буду вдячний за поради щодо елегантнішого циклобудування :) Ну і якщо є інші проблеми/недоліки — теж вкажіть на них, будь ласка.

Ось посилання на код: https://js.ide.shpp.me/q1w2e3r4t5y6

Тобто в одному повідомленні:

  • Тегніть волонтерів @zero-volo (без цього запит може загубитись в чаті).

  • Попросіть провести рев'ю.

  • Напишіть текстом назву задачі.

  • Додайте посилання на код рішення (через кнопку в Ш++ IDE).

  • Вкажіть мету рев'ю: поясніть, навіщо конкретно скидаєте код на перевірку, та щодо чого саме хочете отримати поради.

Ліміти рев'ю

Протягом курсу Zero можна отримати до 5 рев'ю (включно). Кількість проведених вам рев'ю позначатиметься ρ-приміткою до нікнейму в Discord. Наприклад, якщо волонтери провели 2 рев'ю — буде ρ2.

Які саме задачі скидати на рев'ю (та чи скидати взагалі) — вирішуєте ви. Можна використати всі 5 на перших же задачах. Можна пройти весь курс без жодного рев'ю. Робіть так, як вам зручно.

Емодзі статусу рев'ю

Для зручної навігації волонтери відмічають запити на рев'ю в чаті спеціальними емодзі:

Причини, чому можуть відхилити запит:

  • У запиті — більше одної задачі.

  • Запит на рев'ю - не в чаті #zero-review

  • Відсутнє посилання на код

  • Відсутня мета рев'ю

  • В автотестах в Ш++ IDE за посиланням обрана не та задача, на яку просите рев'ю.

  • Один або більше автотестів в Ш++ IDE не проходить

  • Вичерпаний ліміт рев'ю

Last updated