Вход на сайт
IT образование в Германии
NEW 30.06.13 20:38
По истории системы контроля бывает не легко найти автора кода. Как минимум приходиться тратить время. А бывает вообще невозможно, если commit делал не автор.
в ответ scorpi_ 30.06.13 19:02
В ответ на:
Для этого система контроля версий.
Мы работаем с TFS. Но там можно поставить нотификацию только на изменение в файле целиком. Нет возможности контролировать только одну функцию к примеру или даже часть функции в несколько строк.Для этого система контроля версий.
В ответ на:
Лишнего в коде быть не должно. Blame, история изменений, толковые submit-messages дают полную информацию и не засоряют код.
Это верно что лишнего ничего не должно быть. Авторская подпись - это не лишнее. Т.к. ясно к кому обращаться с вопросами если возникнут. Кроме этого (кроме тривиальных случаев конечно) надо указать почему были внесены изменения и ссылочку на дополнительные описания(у нас это TFS ID, где обычно еще и доплнительные файлы ложатся, картинки, видео и прочее).Лишнего в коде быть не должно. Blame, история изменений, толковые submit-messages дают полную информацию и не засоряют код.
По истории системы контроля бывает не легко найти автора кода. Как минимум приходиться тратить время. А бывает вообще невозможно, если commit делал не автор.
Проклят нарушающий межи ближнего своего (Втор.27:17)
NEW 30.06.13 22:33
Что такое нотификация? Что значит "контролировать"?
Кто автор класса или функции если я изменил половину? Современные методики не предполагают индивидуального авторства.
http://msdn.microsoft.com/en-us/library/bb385979.aspx
Коммитить полагается не мегабайтами, а отдельными фичами/багами/whatever; с понятными описаниями коммита.
Это что за бардак?
в ответ anly 30.06.13 20:38
В ответ на:
Мы работаем с TFS. Но там можно поставить нотификацию только на изменение в файле целиком. Нет возможности контролировать только одну функцию к примеру или даже часть функции в несколько строк.
Мы работаем с TFS. Но там можно поставить нотификацию только на изменение в файле целиком. Нет возможности контролировать только одну функцию к примеру или даже часть функции в несколько строк.
Что такое нотификация? Что значит "контролировать"?
В ответ на:
Авторская подпись - это не лишнее
Авторская подпись - это не лишнее
Кто автор класса или функции если я изменил половину? Современные методики не предполагают индивидуального авторства.
В ответ на:
Т.к. ясно к кому обращаться с вопросами если возникнут
Т.к. ясно к кому обращаться с вопросами если возникнут
http://msdn.microsoft.com/en-us/library/bb385979.aspx
В ответ на:
надо указать почему были внесены изменения
надо указать почему были внесены изменения
Коммитить полагается не мегабайтами, а отдельными фичами/багами/whatever; с понятными описаниями коммита.
В ответ на:
А бывает вообще невозможно, если commit делал не автор
А бывает вообще невозможно, если commit делал не автор
Это что за бардак?
NEW 30.06.13 23:55
Ну еще бывает что с коллегой обсуждал, и закоммители вместе на то том компе где сидели, ну а авторство владельцу компа приписывается.
В ответ на:
Что такое нотификация? Что значит "контролировать"?
нотификация - уведомление. В TFS можно сделать чтоб письмо высылалось. Контролировать - здачит знать об изменениях и что изменения корректны. Некорректным может быть даже просто стиль. Напр. у нас некоторые до сих пор объявляют переменные в начале функции, а не поближе к использованию. Или переменную цикла переиспользуют несколько раз в нескольких циклах, или "просто так" сделают функцию виртуальной. Или не заботятся о контантности функции, если она таковой по смыслу является (тоже относится и к аргументам). Вот чтобы не захламить хотябы мой код чужими вставками я его контролирую.Что такое нотификация? Что значит "контролировать"?
В ответ на:
Кто автор класса или функции если я изменил половину? Современные методики не предполагают индивидуального авторства.
наполовину ты. Но такое допустимо, ежели предыдущий
автор вообще покинул команду, или если все в комманде строго следят за правилами. Я напр. стараюсь чтобы функции были маленькими, а кто-то возьмёт да в ставит 200 строчек в мою фунцию, вместо того чтобы эти 200 строк отдельной функцией(лучше штук пять) и вставить только вызов. Много есть моментов, которые я считаю бардаком и хотябы мой код берегу от него. Хотябы на уровне функций стараюсь сохранить индивидуальное авторство.Кто автор класса или функции если я изменил половину? Современные методики не предполагают индивидуального авторства.
В ответ на:
http://msdn.microsoft.com/en-us/library/bb385979.aspx
там тоже есть недостатки. Даже если кто только пробельчик для красоты добавил или удалил, или орфографическую ошибку в комментарии исправил: annotate теперь его как автора показывать будет. Хотя вещь полезная.http://msdn.microsoft.com/en-us/library/bb385979.aspx
В ответ на:
Коммитить полагается не мегабайтами, а отдельными фичами/багами/whatever; с понятными описаниями коммита.
но и мегабайтами тоже приходится. Например
при смене системы контроля. Мы уже на третью переходим.Коммитить полагается не мегабайтами, а отдельными фичами/багами/whatever; с понятными описаниями коммита.
В ответ на:
Это что за бардак?
один товарищ у нас работает с парнями из России. Они коммитят в отдельную ветку, а мерджем в основную ветку занимается он. И автором всегда он получается.Это что за бардак?
Ну еще бывает что с коллегой обсуждал, и закоммители вместе на то том компе где сидели, ну а авторство владельцу компа приписывается.
Проклят нарушающий межи ближнего своего (Втор.27:17)
NEW 01.07.13 11:38
в ответ Kabal 01.07.13 09:43
как то пытались, да не прижилось это. Делаем только если сам девелопер захочет, чувствуя что может допустить какую лажу в чужом коде. Тогда он обращается к тому чей код, с просьбой проверить. Но редко бывает.
Проклят нарушающий межи ближнего своего (Втор.27:17)
NEW 08.07.13 07:45
вы очень уверены в себе.
ещё врагов среди коллег не нажили?
в ответ anly 30.06.13 11:19
В ответ на:
писать так чтобы было приятно смотреть. .....Т.к. я написал утилиту которая парсит все проэкты и сообщает мне о любых измениниях как в моих файлах, так и в моих вставках в чужих файлах. И ежели я не согласен со стилем изменений (с правильностью кода - само собой) то как минимум добавляю коммент - кто там напартачил.
писать так чтобы было приятно смотреть. .....Т.к. я написал утилиту которая парсит все проэкты и сообщает мне о любых измениниях как в моих файлах, так и в моих вставках в чужих файлах. И ежели я не согласен со стилем изменений (с правильностью кода - само собой) то как минимум добавляю коммент - кто там напартачил.
вы очень уверены в себе.

ещё врагов среди коллег не нажили?
NEW 18.07.13 01:28
в ответ spaceX 08.07.13 07:45
оставлять свой автограф в комменте к изменениям - это требование Code Guidelines. Правда, я заметил что не все во всём им следуют (а некоторые похоже вообще на них забили).
Поэтому даже если я за когото добавлю его автограф к его изменениям - то это корректировка для соответсвия Code Guidelines.
Такие поправки случаются весьма редко. Так что пока никто не возмущался.
Зато что хорошо с этим парсером, что я всегда вкурсе если ктото влез в мой код. Даже если какую мою лажу исправят - я сразу об этом узнаю. Ну а если наоборот лажу внесут, то чем раньше её убрать, тем лучше.
Поэтому даже если я за когото добавлю его автограф к его изменениям - то это корректировка для соответсвия Code Guidelines.
Такие поправки случаются весьма редко. Так что пока никто не возмущался.
Зато что хорошо с этим парсером, что я всегда вкурсе если ктото влез в мой код. Даже если какую мою лажу исправят - я сразу об этом узнаю. Ну а если наоборот лажу внесут, то чем раньше её убрать, тем лучше.
Проклят нарушающий межи ближнего своего (Втор.27:17)
NEW 18.07.13 14:59
Дикари. Система поверсионного хранения сырцов спасет отцов русской демократии - сразу будет видно, кто версию чек-инил.
в ответ anly 18.07.13 01:28
В ответ на:
оставлять свой автограф в комменте к изменениям - это требование Code Guidelines
оставлять свой автограф в комменте к изменениям - это требование Code Guidelines
Дикари. Система поверсионного хранения сырцов спасет отцов русской демократии - сразу будет видно, кто версию чек-инил.
- Живем один раз!
- Нет, мы умираем один раз. А живем мы каждый день.
NEW 18.07.13 20:45
Кстати не так давно перешли c TFS на SVN (точнее нынешние версии так и остались на TFS, а следующяя и будущие версии уже будут на SVN). В результате вся история накрылась.
А еще более ранние версии хранятся в SourceSafe, при переходе на TFS тоже история пошла с нуля.
в ответ Kabal 18.07.13 14:59
В ответ на:
Система поверсионного хранения
ты имеешь в виду типа TFS, SVN?Система поверсионного хранения
Кстати не так давно перешли c TFS на SVN (точнее нынешние версии так и остались на TFS, а следующяя и будущие версии уже будут на SVN). В результате вся история накрылась.
А еще более ранние версии хранятся в SourceSafe, при переходе на TFS тоже история пошла с нуля.
Проклят нарушающий межи ближнего своего (Втор.27:17)
18.07.13 22:17
причину этого я знаю только поверхностно, т.к. не я это решал а высшее руководство. Ну а причина (как я понял) что SVN хорошо работает с Polarion (в чём это выражается я пока не знаю), наша шеф-фирма от которой мы сильно зависим уже давно с Polarion работает давно и от нас того же желает. Да вроде еще Polarion для ведения докомунтации получше TFSа. Хотя мои познания Polarion и SVN пока не глубоки.
Проклят нарушающий межи ближнего своего (Втор.27:17)
NEW 19.07.13 00:03
у нас есть один коллега который пришел к нам не так давно, как раз в то время как об SVN заговорили. Так он удивлённо поведал, что на предыдушей фирме перешли с SVN на TFS, посчитав TFS лучше...
Начальству виднее. Мне лично почти всё равно. Почти, т.к. под TFS я yже наработал несколько полезных утелиток интегрировав их в VisualStudio как AddIn-ы. Для SVN придётся переделывать.
Начальству виднее. Мне лично почти всё равно. Почти, т.к. под TFS я yже наработал несколько полезных утелиток интегрировав их в VisualStudio как AddIn-ы. Для SVN придётся переделывать.
Проклят нарушающий межи ближнего своего (Втор.27:17)