«Вот тут бы ещёпорефакторить».Вот тутпереименуйкласс/переменную/функцию"нет проверкиуказателя" вмаленькой функции,которая вызываетсяодин раз из другойфункции и там этотуказательпроверяется."Я не верю, чтоэто будетработать вслучае..." послечего описываетсякакой-тоебанутый случай.— Всё хорошо, держиаппрув* Спустя какое-товремя *— Ну что выливаем?— Ой, а я тут ещё 20комментариевоставил, резолвиУ тебятутпробелы,а не табы«Мы же устнопроговаривалиполгода назадчто мы такделать небудем»Давай мы сейчасна тестовыйзальем, в тестотдадим, а я покатвои замечанияисправлю!- Это не поофициальномуcode convention- Но у нас вкомпании естьсвой, вот ссылка...- Мне не нравится"x ==SOME_CONSTнадо заменитьнаSOME_CONST== x"Нетдокументации.Не покод-стайлу.Не теокончаниястрок.BOM"Добавьлоггирование"- ключи в объектахдолжны быть поалфавиту, но idдолжен бытьпервым, поэтому мыне используемлинтер. отсортируйпожалуйста руками.«Вот тут бы ещёпорефакторить».Вот тутпереименуйкласс/переменную/функцию"нет проверкиуказателя" вмаленькой функции,которая вызываетсяодин раз из другойфункции и там этотуказательпроверяется."Я не верю, чтоэто будетработать вслучае..." послечего описываетсякакой-тоебанутый случай.— Всё хорошо, держиаппрув* Спустя какое-товремя *— Ну что выливаем?— Ой, а я тут ещё 20комментариевоставил, резолвиУ тебятутпробелы,а не табы«Мы же устнопроговаривалиполгода назадчто мы такделать небудем»Давай мы сейчасна тестовыйзальем, в тестотдадим, а я покатвои замечанияисправлю!- Это не поофициальномуcode convention- Но у нас вкомпании естьсвой, вот ссылка...- Мне не нравится"x ==SOME_CONSTнадо заменитьнаSOME_CONST== x"Нетдокументации.Не покод-стайлу.Не теокончаниястрок.BOM"Добавьлоггирование"- ключи в объектахдолжны быть поалфавиту, но idдолжен бытьпервым, поэтому мыне используемлинтер. отсортируйпожалуйста руками.

Code Review Bingo - Call List

(Print) Use this randomly generated list as your call list when playing the game. There is no need to say the BINGO column name. Place some kind of mark (like an X, a checkmark, a dot, tally mark, etc) on each cell as you announce it, to keep track. You can also cut out each item, place them in a bag and pull words from the bag.


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
  1. «Вот тут бы ещё порефакторить».
  2. Вот тут переименуй класс/переменную/функцию
  3. "нет проверки указателя" в маленькой функции, которая вызывается один раз из другой функции и там этот указатель проверяется.
  4. "Я не верю, что это будет работать в случае..." после чего описывается какой-то ебанутый случай.
  5. — Всё хорошо, держи аппрув * Спустя какое-то время * — Ну что выливаем? — Ой, а я тут ещё 20 комментариев оставил, резолви
  6. У тебя тут пробелы, а не табы
  7. «Мы же устно проговаривали полгода назад что мы так делать не будем»
  8. Давай мы сейчас на тестовый зальем, в тест отдадим, а я пока твои замечания исправлю!
  9. - Это не по официальному code convention - Но у нас в компании есть свой, вот ссылка... - Мне не нравится
  10. "x == SOME_CONST надо заменить на SOME_CONST == x"
  11. Нет документации.
  12. Не по код-стайлу.
  13. Не те окончания строк.
  14. BOM
  15. "Добавь логгирование"
  16. - ключи в объектах должны быть по алфавиту, но id должен быть первым, поэтому мы не используем линтер. отсортируй пожалуйста руками.