caxapa.ru :: Хочу странного - удобного инструмента для рецензирования кода

Alex B. (02.11.2010 00:52, просмотров: 30825)

Хочу странного - удобного инструмента для рецензирования кода Но нужно это не для хитрых методик групповой работы, а больше для обучения. Вижу это как текстовый редактор, одна половина которого - исходник, другая - комментарии редактора. Лучше, конечно, онлайн. В слике есть заметки - но они не очень подходят. И, да, нахаляву или за смешные деньги (ну, 5 баксов в месяц готов отдать)

Согласуйте импеданс!

Ответить

- Попался на глаза интересный плагинчик к редмайну - =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)
- diff изменений записывается в текстовый файл, затем в нужных местах добавляются комментарии к изменениям: что хорошо, что плохо, как сделать лучше. очень просто и надежно. именно так делается code review через e-mail в open-source проектах. если bialix(241 знак., 10.11.2010 13:21)
  - чтобы сделать дифф надо как минимум иметь две версии (т.е. внести эти изменения), а речь вроде шла просто как добавить спец. комментарии для автора - 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)
        
        Ну возьмите Source Insight. Инсталлятор весит 3 МБ. Он даже препроцессор чутка понимает: Alex B.(106 знак., 12.11.2010 18:54)
        
        Про финских студентов еще забыли вставить. - 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)
        
        Это не делает вам чести как руководителю. Лучше бы вы настаивали на использовании статической проверки - это гораздо умнее - 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, )
        
        Это ошибка в Windows. В Unix данной ошибки не возникает. - fk0(12.11.2010 12:18)
        
        :>>>> - Snaky(12.11.2010 12:19)
        
        Не всегда. Неправильно оформленный комментарий может сломать программу. - 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)
        
        шутка. компилятор не видит комментов - их откусывает препроцессор - Snaky(10.11.2010 14:12)
    - +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)
- это что-то из разряда тулзов для 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, ссылка)

Средства и методы разработки