-
- Попался на глаза интересный плагинчик к редмайну - =AlexD=(16.11.2010 09:30, ссылка)
- Спасибо, работает клево - Alex B.(18.03.2011 16:59)
- На текущей конторе для code review используем Мурзик Васильевич(84 знак., 12.11.2010 16:28,
)
- Спасибо, но более уебищный интерфейс трудно себе представить. Хотя принцип понятен - bialix его описал. У всех ревьюиров он, наверное, одинаковый. - Alex B.(12.11.2010 16:44)
- На мультимиллионном проекте Codestriker справляется со своими обязанностями на 5+, так что, как говорится, up to you... :) - Мурзик Васильевич(13.11.2010 15:46,
)
- да хоть на мультимиллиардном, какая мне разница. Мне надо решить свои задачи, а вы начинаете себя пяткой в грудь бить... Alex B.(92 знак., 13.11.2010 16:55 - 17:11)
- вы-то сами на чем остановились? или еще в поиске? bialix(100 знак., 15.11.2010 12:21)
- Пока ни на чем, присматриваюсь к Crucible. А так конечно сообщу. - Alex B.(15.11.2010 16:21)
- на видео выглядит красиво, плюс полная интеграция с JIRA. по сути принцип работы такой же. - bialix(16.11.2010 11:15)
- Пока ни на чем, присматриваюсь к Crucible. А так конечно сообщу. - Alex B.(15.11.2010 16:21)
- вы-то сами на чем остановились? или еще в поиске? bialix(100 знак., 15.11.2010 12:21)
- да хоть на мультимиллиардном, какая мне разница. Мне надо решить свои задачи, а вы начинаете себя пяткой в грудь бить... Alex B.(92 знак., 13.11.2010 16:55 - 17:11)
- На мультимиллионном проекте Codestriker справляется со своими обязанностями на 5+, так что, как говорится, up to you... :) - Мурзик Васильевич(13.11.2010 15:46,
- Спасибо, но более уебищный интерфейс трудно себе представить. Хотя принцип понятен - bialix его описал. У всех ревьюиров он, наверное, одинаковый. - Alex B.(12.11.2010 16:44)
- diff изменений записывается в текстовый файл, затем в нужных местах добавляются комментарии к изменениям: что хорошо, что плохо, как сделать лучше. очень просто и надежно. именно так делается code review через e-mail в open-source проектах. если bialix(241 знак., 10.11.2010 13:21)
- чтобы сделать дифф надо как минимум иметь две версии (т.е. внести эти изменения), а речь вроде шла просто как добавить спец. комментарии для автора - Snaky(10.11.2010 13:35)
- речь шла о рецензировании кода. рецензируют всегда изменения или новый код. - 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)
- чтобы сделать дифф надо как минимум иметь две версии (т.е. внести эти изменения), а речь вроде шла просто как добавить спец. комментарии для автора - Snaky(10.11.2010 13:35)
- Как показывает моя практика, надо быть проще :) Красный Свин(940 знак., 02.11.2010 01:44,
)
- Вот мне и надо проще. Alex B.(333 знак., 02.11.2010 10:39)
- ..заметки в комментариях сильно перегружают. Код, как ни странно, плохо читается. blackbit(483 знак., 02.11.2010 04:14)
- Да-да, а еще комменты внутри циклов замедляют их исполнение.. Если не хочется видеть много блочных комментов - откройте для себя редакторы с Code Folding, типа Ультраедита. - Красный Свин(02.11.2010 23:40,
)
- Уже не раз всречаю эту фразу. Объясните, пожалуйста, как комменты могут влиять на работу программы?! - mazur(10.11.2010 14:05)
- это придется вам постичь самому. - bialix(10.11.2010 14:25)
- грамматические ошибки в комментах раздражают компилятор и он от огорчения может забыть оптимизировать что-нибудь - Snaky(10.11.2010 14:08)
- Если что-то коммент, то компилятору пох что там написано. Хоть мурку там наизусть пиши. Ваша фраза - шутка? Мож я чего-то и не знаю..... - mazur(10.11.2010 14:10)
- Вот пример, когда не пох (довольно частая ошибка): testerplus(185 знак., 12.11.2010 12:16)
- Из-за чего здесь будет ошибка? Попробую угадать. Два варианта. readme.txt c:\doc\ - mazur(12.11.2010 12:26)
- Обртный слеш в конце строки говорит за то, что следующая строка - продолжение комментария в текущей. Дргуми словами "return BAD" окажется частью комментария - testerplus(12.11.2010 12:58)
- Тонко подмечено! Вот почему я настаиваю, чтобы наши программеры все каменты оформляли /* ... */ - Evgeny_CD(12.11.2010 13:08)
- При оформлении в таком стиле сложнее временно целыми блоками комментарить. Приходится что-то типа #if 0 использовать, но при этом блок уже не подсвечивается как закомментированный. - rezident(12.11.2010 15:14)
- За "закоментаривание" вместо #if 0 -- людей сжигать нужно (C). Если не подсвечивается -- нужно взять номальный редактор. А то в wordpad вообще ничего не подсвечивается. За закоментаривание путём // -- сжигать 2 раза. - fk0(12.11.2010 15:51)
- большая часть редакторов подсвечивающий каменты в стиле // /* всякие умные #if 0 не понимают. городить монстра типа сликэдит - муторно. - Mahagam(12.11.2010 18:50)
- Это не повод говнокодить. И без подсветки жить можно. Большая часть -- это wordpad и notepad? Ибо подсвечивающих редакторов как бы не больше, чем не подсвечивающих... Редакторов, а не ноутпадов. - fk0(13.11.2010 15:30)
- обосновать гавнокодистость или кошерность разного вида закомментирования неиспользуемых кусков кода можете? - Mahagam(15.11.2010 13:09)
- C-комментарии не могут быть вложенные и вызывают глюки в таком случае. C++ комментарии вызывают сложности при обратном раскомментаривании (лишнего, вложенного и т.п.) и портят diff'ы. Речь, конечно не о закомментаривании 2-х строчек. - fk0(15.11.2010 13:20)
- вложенное закомментирование исходного кода - само по себе жуть. del на левый код и дело с концом. - Mahagam(15.11.2010 15:45)
- C-комментарии не могут быть вложенные и вызывают глюки в таком случае. C++ комментарии вызывают сложности при обратном раскомментаривании (лишнего, вложенного и т.п.) и портят diff'ы. Речь, конечно не о закомментаривании 2-х строчек. - fk0(15.11.2010 13:20)
- обосновать гавнокодистость или кошерность разного вида закомментирования неиспользуемых кусков кода можете? - Mahagam(15.11.2010 13:09)
- Ну возьмите Source Insight. Инсталлятор весит 3 МБ. Он даже препроцессор чутка понимает: Alex B.(106 знак., 12.11.2010 18:54)
- Это не повод говнокодить. И без подсветки жить можно. Большая часть -- это wordpad и notepad? Ибо подсвечивающих редакторов как бы не больше, чем не подсвечивающих... Редакторов, а не ноутпадов. - fk0(13.11.2010 15:30)
- Про финских студентов еще забыли вставить. - rezident(12.11.2010 16:04)
- Советским профессиональным программистам, увы, до финских студентов ой как далеко. - fk0(12.11.2010 17:30)
- Имя нормального редактора? - AlexBi(12.11.2010 15:55)
- NetBeans понимает #if*, только для дефайнов в makefile ему нужно вручную в code assistance прописать то, что в makefile. Ну и то, что компилятор от себя ставит по ключам, скажем, __AVR_ATmega328P__. С #if 0, естественно, проблем вообще нет. - -ReAl-(12.11.2010 17:04,
)
- SlickEdit 15 понимает #if 0 - Alex B.(12.11.2010 16:22)
- Попробовал в 14-ом. Тоже понимает, а я не знал. - AlexBi(12.11.2010 16:25)
- (G)Vim - fk0(12.11.2010 15:58)
- NetBeans понимает #if*, только для дефайнов в makefile ему нужно вручную в code assistance прописать то, что в makefile. Ну и то, что компилятор от себя ставит по ключам, скажем, __AVR_ATmega328P__. С #if 0, естественно, проблем вообще нет. - -ReAl-(12.11.2010 17:04,
- большая часть редакторов подсвечивающий каменты в стиле // /* всякие умные #if 0 не понимают. городить монстра типа сликэдит - муторно. - Mahagam(12.11.2010 18:50)
- За "закоментаривание" вместо #if 0 -- людей сжигать нужно (C). Если не подсвечивается -- нужно взять номальный редактор. А то в wordpad вообще ничего не подсвечивается. За закоментаривание путём // -- сжигать 2 раза. - fk0(12.11.2010 15:51)
- Это не делает вам чести как руководителю. Лучше бы вы настаивали на использовании статической проверки - это гораздо умнее - Alex B.(12.11.2010 15:04, ссылка)
- тесты, и только автоматические тесты можно считать стоящей проверкой. - bialix(15.11.2010 12:24)
- Вы выкладывали несколько старый, но рабочий релиз этой замечательной штуковины. Она используется частью программеров у нас (остальных недодавил). - Evgeny_CD(12.11.2010 16:06 - 16:11)
- static analysys - good(12.11.2010 15:20,
)
- При оформлении в таком стиле сложнее временно целыми блоками комментарить. Приходится что-то типа #if 0 использовать, но при этом блок уже не подсвечивается как закомментированный. - rezident(12.11.2010 15:14)
- Тонко подмечено! Вот почему я настаиваю, чтобы наши программеры все каменты оформляли /* ... */ - Evgeny_CD(12.11.2010 13:08)
- Обртный слеш в конце строки говорит за то, что следующая строка - продолжение комментария в текущей. Дргуми словами "return BAD" окажется частью комментария - testerplus(12.11.2010 12:58)
- Это ошибка в Windows. В Unix данной ошибки не возникает. - fk0(12.11.2010 12:18)
- :>>>> - Snaky(12.11.2010 12:19)
- Из-за чего здесь будет ошибка? Попробую угадать. Два варианта. readme.txt c:\doc\ - mazur(12.11.2010 12:26)
- Не всегда. Неправильно оформленный комментарий может сломать программу. - Ruslan(10.11.2010 14:13, ссылка)
- А нечего в комментах фигню писать, и будет усе кучеряво...... - mazur(10.11.2010 14:24)
- т.е. мурку - можно, а фигню - нечего? - Snaky(10.11.2010 14:27)
- Ептыть, ну ясен же пень, что мурка-это образно. Комменты должны кратки, лаконичны, но емки. Иначе и действительно, комменты засоряют полезную информацию (код). В основном, комментов у меня вообще нет. Только в самых важных местах. - mazur(10.11.2010 14:31)
- А тост, должен быть коротким как выстрел :) Скрипач(115 знак., 12.11.2010 13:03)
- Ептыть, ну ясен же пень, что мурка-это образно. Комменты должны кратки, лаконичны, но емки. Иначе и действительно, комменты засоряют полезную информацию (код). В основном, комментов у меня вообще нет. Только в самых важных местах. - mazur(10.11.2010 14:31)
- т.е. мурку - можно, а фигню - нечего? - Snaky(10.11.2010 14:27)
- А нечего в комментах фигню писать, и будет усе кучеряво...... - mazur(10.11.2010 14:24)
- шутка. компилятор не видит комментов - их откусывает препроцессор - Snaky(10.11.2010 14:12)
- Вот пример, когда не пох (довольно частая ошибка): testerplus(185 знак., 12.11.2010 12:16)
- Если что-то коммент, то компилятору пох что там написано. Хоть мурку там наизусть пиши. Ваша фраза - шутка? Мож я чего-то и не знаю..... - mazur(10.11.2010 14:10)
- Уже не раз всречаю эту фразу. Объясните, пожалуйста, как комменты могут влиять на работу программы?! - mazur(10.11.2010 14:05)
- +2. И вообще не надо много думать о коде. Есть ещё и алгоритмы. Просто говнокодить не нужно. - fk0(02.11.2010 11:13)
- +1, особенно если гора комментов написанных "под doxygen". иногда хочется отделить мух от котлет (i.e. разнести код и комменты по разным файлам). - Snaky(02.11.2010 04:38)
- просто надо уметь их готовить - index.html(12.11.2010 03:00,
)
- ну так покажи как, она
нистним - Alex B.(12.11.2010 11:30)
- ну так покажи как, она
- просто надо уметь их готовить - index.html(12.11.2010 03:00,
- Да-да, а еще комменты внутри циклов замедляют их исполнение.. Если не хочется видеть много блочных комментов - откройте для себя редакторы с Code Folding, типа Ультраедита. - Красный Свин(02.11.2010 23:40,
- это что-то из разряда тулзов для code review наверное - Snaky(02.11.2010 01:30, ссылка)
- У нас в конторе ReviewBoard используется. Для начинающих будет недостатком сильная привязанность к системам контроля версий и консольный интерфейс для генерации и заливки кода на сервер. - Sergey_N(02.11.2010 20:35)
- Я просто перевел сразу =) Обзор 2008 года, может чо посимпатичнее есть? - Alex B.(02.11.2010 01:37)
- плагин к эклипсу есть - Snaky(02.11.2010 01:47, ссылка)
- Попался на глаза интересный плагинчик к редмайну - =AlexD=(16.11.2010 09:30, ссылка)