Работа с чужим кодом

Не относящиеся к фреймворку и программированию вопросы
Ответить
dmg
Сообщения: 685
Зарегистрирован: 2012.10.15, 03:09

Работа с чужим кодом

Сообщение dmg »

Существую ли какие-то правила или традиции по такому вопросу.
Работаю с чужим кодом. Заказчик просит внести правки. Все хорошо.
Как оставить правильно оставить инфу для потомков?
1. Когда не знал про svc/git - писал комменты, "исправлено то то таким@такой.ру по просьбе заказчика скайп такойто. дата.
2. теперь делаю то же самой через git.
3. Может надо еще в корне проекта оставлять какой то readme/change log ?
4. Корректо ли оставлять какие -то рекомендации по заказчику - какой он хороший или, например, - жмот.
mickgeek
Сообщения: 957
Зарегистрирован: 2014.05.31, 20:50
Откуда: Санкт-Петербург
Контактная информация:

Re: Работа с чужим кодом

Сообщение mickgeek »

3. Большинство разработчиков не занимается подобными вещами. Ценят время (и/или ленятся). Даже если один из фрилансеров создаст такой файл, далеко не факт, что следующий продолжит его. Но в крупных проектах, в принципе, документирование некоторых изменений не помешало бы.
4. Похоже на лицемерие, некрасиво. Да и смысл? Не каждый разработчик смотрит код перед тем, как согласиться на доработку. Умные фрилансеры отсеивают заказчиков во время обсуждения проекта или аккуратно прекращают сотрудничество после определённого этапа.

Лучше просто грамотно писать код, согласно рекомендациям фреймворка. Синтаксис, комментарии, авторство и прочее.
Аватара пользователя
samdark
Администратор
Сообщения: 9489
Зарегистрирован: 2009.04.02, 13:46
Откуда: Воронеж
Контактная информация:

Re: Работа с чужим кодом

Сообщение samdark »

2. Нормально.
3. Если есть что написать именно по проекту.
4. В коде — нет.
dmg
Сообщения: 685
Зарегистрирован: 2012.10.15, 03:09

Re: Работа с чужим кодом

Сообщение dmg »

Спасибо.
так и делаю - где нет git - делаю git init и далее свои commit с комментариями.
Аватара пользователя
ifelse
Сообщения: 227
Зарегистрирован: 2013.02.05, 13:05

Re: Работа с чужим кодом

Сообщение ifelse »

На своем опыте- как правило когда доделываешь чужой проект доступа к оригинальному репозиторию нет, в итоге создаю локальный для себя. Кто-нибудь заливает на хостинг папку .git?
lynicidn
Сообщения: 2222
Зарегистрирован: 2014.05.24, 15:12

Re: Работа с чужим кодом

Сообщение lynicidn »

на хостинге делает гит клоне - папка сама появится
Аватара пользователя
chungachguk
Сообщения: 435
Зарегистрирован: 2012.07.17, 11:52

Re: Работа с чужим кодом

Сообщение chungachguk »

ifelse писал(а):На своем опыте- как правило когда доделываешь чужой проект доступа к оригинальному репозиторию нет, в итоге создаю локальный для себя. Кто-нибудь заливает на хостинг папку .git?
Зачем?
Аватара пользователя
ifelse
Сообщения: 227
Зарегистрирован: 2013.02.05, 13:05

Re: Работа с чужим кодом

Сообщение ifelse »

chungachguk писал(а): Зачем?
Чтобы другим разрабам жить было проще
Аватара пользователя
chungachguk
Сообщения: 435
Зарегистрирован: 2012.07.17, 11:52

Re: Работа с чужим кодом

Сообщение chungachguk »

ifelse писал(а):
chungachguk писал(а): Зачем?
Чтобы другим разрабам жить было проще
Дргуим разрабам можно и на битбакете сделать репозиторий. А так вагон места на хостинге знаимать, плюс кто-то может захотеть уязвимости поискать, с историей изменений ему будет проще.
Аватара пользователя
ifelse
Сообщения: 227
Зарегистрирован: 2013.02.05, 13:05

Re: Работа с чужим кодом

Сообщение ifelse »

chungachguk писал(а): Дргуим разрабам можно и на битбакете сделать репозиторий. А так вагон места на хостинге знаимать, плюс кто-то может захотеть уязвимости поискать, с историей изменений ему будет проще.
Ну вот сдается проект и вы с клиентом разбегаетесь, что дальше? :)
Аватара пользователя
samdark
Администратор
Сообщения: 9489
Зарегистрирован: 2009.04.02, 13:46
Откуда: Воронеж
Контактная информация:

Re: Работа с чужим кодом

Сообщение samdark »

Это дело клиента. Хочет репозиторий — давайте, не знает — предлагайте.
Аватара пользователя
ifelse
Сообщения: 227
Зарегистрирован: 2013.02.05, 13:05

Re: Работа с чужим кодом

Сообщение ifelse »

Спасибо
Ответить