CPPPROGLIB Telegram 5766
Чек-лист для Code Review

Code review — один из самых эффективных способов повышения качества кода. Составили чек-лист для reviewer'ов и авторов PR.

1. Архитектура и дизайн

• Соответствует ли код общей архитектуре проекта?
• Не нарушаются ли SOLID принципы?
• Правильно ли выбраны абстракции и интерфейсы?

2. Логика и корректность

• Корректно ли реализована бизнес-логика?
• Обрабатываются ли все edge cases?
• Нет ли off-by-one ошибок в циклах?

3. Безопасность кода

• Правильно ли управляется память?
• Нет ли потенциальных buffer overflow'ов?
• Корректно ли обрабатываются исключения?

4. Производительность

• Нет ли ненужных копирований объектов?
• Эффективны ли используемые алгоритмы?
• Правильно ли используются контейнеры STL?

5. Читаемость и поддерживаемость

• Понятны ли названия переменных и функций?
• Достаточно ли комментариев для сложной логики?
• Не слишком ли сложные функции (соблюдается ли SRP)?

6. Соответствие стандартам

• Следует ли код style guide проекта?
• Используются ли современные возможности C++?
• Корректно ли оформлены заголовочные файлы?

7. Тестирование

• Покрыт ли новый код unit тестами?
• Обновлены ли существующие тесты?
• Протестированы ли edge cases?

8. Документация

• Обновлена ли документация API?
• Есть ли примеры использования для новых функций?
• Актуальны ли комментарии в коде?

9. Обратная совместимость

• нарушается ли API существующих компонентов?
• Совместимы ли изменения с используемыми версиями библиотек?

10. Финальная проверка

• Проходят ли все автоматические тесты?
• Нет ли compiler warnings?
• Готов ли код к production?

Библиотека C/C++ разработчика #буст
🔥6👍31😁1



tgoop.com/cppproglib/5766
Create:
Last Update:

Чек-лист для Code Review

Code review — один из самых эффективных способов повышения качества кода. Составили чек-лист для reviewer'ов и авторов PR.

1. Архитектура и дизайн

• Соответствует ли код общей архитектуре проекта?
• Не нарушаются ли SOLID принципы?
• Правильно ли выбраны абстракции и интерфейсы?

2. Логика и корректность

• Корректно ли реализована бизнес-логика?
• Обрабатываются ли все edge cases?
• Нет ли off-by-one ошибок в циклах?

3. Безопасность кода

• Правильно ли управляется память?
• Нет ли потенциальных buffer overflow'ов?
• Корректно ли обрабатываются исключения?

4. Производительность

• Нет ли ненужных копирований объектов?
• Эффективны ли используемые алгоритмы?
• Правильно ли используются контейнеры STL?

5. Читаемость и поддерживаемость

• Понятны ли названия переменных и функций?
• Достаточно ли комментариев для сложной логики?
• Не слишком ли сложные функции (соблюдается ли SRP)?

6. Соответствие стандартам

• Следует ли код style guide проекта?
• Используются ли современные возможности C++?
• Корректно ли оформлены заголовочные файлы?

7. Тестирование

• Покрыт ли новый код unit тестами?
• Обновлены ли существующие тесты?
• Протестированы ли edge cases?

8. Документация

• Обновлена ли документация API?
• Есть ли примеры использования для новых функций?
• Актуальны ли комментарии в коде?

9. Обратная совместимость

• нарушается ли API существующих компонентов?
• Совместимы ли изменения с используемыми версиями библиотек?

10. Финальная проверка

• Проходят ли все автоматические тесты?
• Нет ли compiler warnings?
• Готов ли код к production?

Библиотека C/C++ разработчика #буст

BY Библиотека C/C++ разработчика | cpp, boost, qt




Share with your friend now:
tgoop.com/cppproglib/5766

View MORE
Open in Telegram


Telegram News

Date: |

Just as the Bitcoin turmoil continues, crypto traders have taken to Telegram to voice their feelings. Crypto investors can reduce their anxiety about losses by joining the “Bear Market Screaming Therapy Group” on Telegram. Select “New Channel” Telegram message that reads: "Bear Market Screaming Therapy Group. You are only allowed to send screaming voice notes. Everything else = BAN. Text pics, videos, stickers, gif = BAN. Anything other than screaming = BAN. You think you are smart = BAN. Private channels are only accessible to subscribers and don’t appear in public searches. To join a private channel, you need to receive a link from the owner (administrator). A private channel is an excellent solution for companies and teams. You can also use this type of channel to write down personal notes, reflections, etc. By the way, you can make your private channel public at any moment. The Channel name and bio must be no more than 255 characters long
from us


Telegram Библиотека C/C++ разработчика | cpp, boost, qt
FROM American