Snaky (10.11.2010 13:35, просмотров: 157) ответил bialix на diff изменений записывается в текстовый файл, затем в нужных местах добавляются комментарии к изменениям: что хорошо, что плохо, как сделать лучше. очень просто и надежно. именно так делается code review через e-mail в open-source проектах. еслиDRC придумали трусы
-
- речь шла о рецензировании кода. рецензируют всегда изменения или новый код. - bialix(10.11.2010 13:40)
- По-моему, техника code review в общем случае не подразумевает внесение изменений рецензентом. Достаточно просто указать на слабое место. - amusin(10.11.2010 13:45)
- а я что сказал? - bialix(10.11.2010 13:51, ссылка)
- Что-то у нас логические базисы разные. :-) - amusin(10.11.2010 13:59)
- я много лет работал с описанным code review over e-mail в проекте Bazaar. Как это выглядит и работает знаю. Я и сам делал такое review и сам получал такое review от других. Я знаю как это работает не по наслышке. Могу поискать, чтобы показать bialix(137 знак., 10.11.2010 14:02)
- Собсно, я написал то же, что и Snaky... Прогуглил, попал на -> amusin(166 знак., 10.11.2010 14:12, ссылка)
- я много лет работал с описанным code review over e-mail в проекте Bazaar. Как это выглядит и работает знаю. Я и сам делал такое review и сам получал такое review от других. Я знаю как это работает не по наслышке. Могу поискать, чтобы показать bialix(137 знак., 10.11.2010 14:02)
- Что-то у нас логические базисы разные. :-) - amusin(10.11.2010 13:59)
- а я что сказал? - bialix(10.11.2010 13:51, ссылка)
- По-моему, техника code review в общем случае не подразумевает внесение изменений рецензентом. Достаточно просто указать на слабое место. - amusin(10.11.2010 13:45)
- не пользоваться системами контроля версий в наши дни -- это как минимум не эффективно. если уже учить выпускника ВУЗа, то начинать с этого, а не с алгоритмов. - bialix(10.11.2010 13:39)
- Да слушайте, выпускник вуза во всю уже пользуется системой контроля версий. Вопрос был не в этом. Текстовый файл это хорошо, но для меня не наглядно. И я не хочу сажать рядом с собой никого, мне просто надо иметь возможность в любое время написать Alex B.(34 знак., 10.11.2010 15:40)
- я не про это. а про то что в вашем варианте надо самому исправить код сначала, а потом на отличиях показывать как правильно. а автор топика хотел просто при просмотре кода добавить замечания, не трогая сам код, как я понял. - Snaky(10.11.2010 13:44)
- самому править не обязательно. можно словами объяснить что не так. - bialix(10.11.2010 13:52)
- чем спец комментарии отличаются от просто комментариев? добавлением маркера XXX или подобного? или что? - bialix(10.11.2010 13:38)
- Vim можно научить выделять комментарии типа WTF - fk0(10.11.2010 14:43)
- WTF is vim? - Snaky(10.11.2010 14:49)
- Средство помощи бедным детям из Уганды. Умеет бибикать и всё портить. - fk0(10.11.2010 15:15)
- WTF is vim? - Snaky(10.11.2010 14:49)
- тем что можно видеть только их, легко перемещаться между ними и не смешивать со своими комментариями. ну хотя бы ХХХ. или некоторые IDE находят в комментах фразу TODO и позволяют прыгать на эти места (как вариант). - Snaky(10.11.2010 13:43)
- Так XXX вроде как бывает сразу в списке рядом с TODO, FIXME, PENDING. Если IDE позволяет добавлять метки, то добавить WTF :-) - ReAl(10.11.2010 13:59)
- когда кол-во WTF per KLOC становится < 10 студент переходит на второй уровень. - Snaky(10.11.2010 14:02)
- +1 :-) - bialix(10.11.2010 14:04)
- когда кол-во WTF per KLOC становится < 10 студент переходит на второй уровень. - Snaky(10.11.2010 14:02)
- Так XXX вроде как бывает сразу в списке рядом с TODO, FIXME, PENDING. Если IDE позволяет добавлять метки, то добавить WTF :-) - ReAl(10.11.2010 13:59)
- Vim можно научить выделять комментарии типа WTF - fk0(10.11.2010 14:43)
- речь шла о рецензировании кода. рецензируют всегда изменения или новый код. - bialix(10.11.2010 13:40)