Соблюдаете ли вы соглашение об именах оригинального программиста?

Question

Если вы берете проект у кого-то для выполнения простых обновлений, соблюдаете ли вы соглашение об именах? Я только что получил проект, где предыдущий программист везде использовал венгерскую нотацию. У нашего основного продукта есть стандарт именования, но за многие годы у нас было много людей, готовых создавать собственные отчеты и делать то, что им хочется.

У меня нет времени, чтобы изменить все имена переменных, которые уже есть в коде.

Я склонен к читабельности, чтобы просто продолжить соглашение об именах.

Answer 1

К сожалению, в большинстве случаев ответ положительный. В большинстве случаев код не следует хорошим соглашениям, поэтому трудно следовать прецеденту. Но для удобства чтения иногда необходимо идти по течению.

Тем не менее , если это достаточно маленькое приложение, которое я могу реорганизовать для улучшения существующего кода, чтобы лучше «пахнуть», то я сделаю это. Или, если это часть большей переписки, я также начну кодирование с текущими стандартами кодирования. Но обычно это не так.

Answer 2

Если код уже имеет непротиворечивый стиль, включая присвоение имен, я стараюсь следовать ему. Если предыдущие программисты были непоследовательны, тогда я мог бы свободно применять корпоративный стандарт или мои личные стандарты, если нет какого-либо стандарта компании.

В любом случае я пытаюсь отметить сделанные мной изменения, обрамляя их комментариями. Я знаю, что с сегодняшними системами CVS это часто не делается, но я все же предпочитаю делать это.

Answer 3

Да, но, к сожалению, там, где до меня работали несколько разработчиков, которые не соответствовали этому правилу, поэтому у меня есть несколько соглашений об именах.

Но иногда у нас есть время, чтобы все исправить, так что в итоге все будет красиво и чисто.

Answer 4

Лично, когда я беру проект, имеющий другую схему именования переменных, я склоняюсь к той же схеме, которая использовалась предыдущим программистом. Единственное, что я делаю по-разному, это для любых новых переменных, которые я добавляю, я ставлю подчеркивание перед именем переменной. Таким образом, я могу быстро увидеть мои переменные и мой код, не заходя в историю исходников и не сравнивая версии. Но когда дело доходит до того, что я наследую просто нечитаемый код или комментарии, я обычно просматриваю их и очищаю, как могу, без переписывания всего этого (до этого дошло). Организация - это ключ к наличию расширяемого кода!

Answer 5

Я думаю сделать исправление ошибки как хирургическую процедуру. Садись, беспокойся как можно меньше, чини это, уходи, оставь как можно меньше следов своего бытия.

Answer 6

Когда в Риме делают, как делают римляне.

(За исключением имен переменных индекса, например, "iArrayIndex ++". Хватит потворствовать этому идиотизму.)

Answer 7

Да .. Есть несколько разочаровывающих тем, что вы попадаете в приложение с двумя совершенно разными стилями. В одном проекте, над которым я работал, было два разных способа манипулирования файлами, два разных способа реализации экранов, две разные фундаментальные структуры. Второй кодер даже зашел так далеко, что сделал новые функции частью библиотеки DLL, которая вызывается из основного кода. Техническое обслуживание было кошмарным, и мне пришлось выучить обе парадигмы и надежду, когда я был в одном разделе, я работал с правильным.

Answer 8

если я могу прочитать код, я (попробую) принять те же соглашения если он все равно не читается, мне нужно провести рефакторинг и, таким образом, изменить его (в зависимости от того, на что это похоже), значительно

Answer 9

Зависит. Если я создаю новое приложение и краду код из унаследованного приложения с именами переменных дерьма, я проведу рефакторинг, как только получу его в свое приложение.

Answer 10

Если в существующем приложении есть стандарт, я думаю, что лучше всего ему следовать. Если не существует стандарта (табуляция и пробелы смешаны, фигурные скобки везде ... о ужас), тогда я делаю то, что считаю лучшим, и обычно запускаю существующий код с помощью инструмента форматирования (например, Vim). Я всегда буду сохранять стиль заглавных букв и т. Д. Существующего кода, если есть согласованный стиль.

Мое единственное исключение из этого правила состоит в том, что я не буду использовать венгерскую нотацию, если у кого-то нет пистолета в мою голову. Я не буду тратить время на переименование существующего материала, но все, что я добавлю новое, не будет иметь никаких венгерских бородавок.