Я должен фиксировать косметические изменения?

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

Официасируйте их следующим большим изменением в качестве Sidenote. По крайней мере, это то, что я сделал.

Это «несовершеннолетние» исправляют? Если да, совершите их. Если нет, нет.

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

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

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

Я чувствую, что важно исправить все, что «неправильно» с кодовой базой, даже если это чисто незначительные вещи, такие как отступ.

Не совершайте их вместе с несвязанными исправлениями.

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

Вы можете использовать префикс, например [Cleanup] .

[cleanup] Removed some whitespace
[cleanup] Changed format
Fixed some major bug.
[cleanup] Corrected indentation

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

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

Четкое руководство, прозрачное для всех, является наилучшим способом решения этих вопросов и избежания путаницы в будущем.

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

Зачем работать так усердно, придумывая трудный для чтения образец? Просто найдите все вхождения образца и подсчитайте, сколько вы найдете.

len(re.findall("A", "AbcAbcAbcA")) % 2 == 0

Это должно быть мгновенно понятно всем опытным программистам, в то время как образец типа "(?

Проще.

Определенно совершите их. Если вы принимаете их вместе с реальными изменениями кода и вам приходится откатывать эти изменения, то вы теряете косметические исправления.

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

Есть пара вопросов.

Во-первых, не делайте изменения в коде, потому что вам скучно и не хватает реальных задач. Если это так, перейдите к своему менеджеру проекта и получите некоторые реальные задачи, назначенные вам, что-то со значением.

Другими словами, не переходите кода ради изменения. Всегда добавляйте некоторое значение к коду в процессе.

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

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

«Хорошо, так что вы исправили ошибку 7711, а также изменились около 100 других файлов. Приятно, так что на самом деле здесь bugfix?»

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

Удачи.

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

В Code Complete есть отличный раздел о достоинствах различных стилей кодирования. Если вы можете заставить вашу команду прочитать этот раздел перед встречей стиля кодирования, это может помочь вывести вас всех из встречи живыми сфокусировать дискуссию.

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

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

Окончательное руководство, которое является прозрачным для всех, является лучшим способом справиться с этими вопросами и избежать путаницы в будущем.

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

Официасируйте их, с комментарием комментария соответствующим образом помечали, чтобы облегчить игнорировать при скинге через список изменений.

Не совершайте их в той же операции, что и изменение функциональности. Таким образом, если вы что-то сломаете, легче сузить то, что сломало его, и легко вернуть только рефакторинг, если это необходимо.

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

Короче говоря: совершать часто и всегда документировать изменения. Когда есть огромные изменения, пометите его.

Посмотрите на Max

Функция MAX, одна из функций Excel статистические функции, используется для поиска наибольшее или максимальное число в данный список значений или аргументов.

=MAX( argument1, argument2, ... argument30 ) 

SQL Server сохраняет временную часть как число 1/300 длинных делений с полуночи.

23:59: 59.999 округляется до ближайшей галочки, которая приходится на 00:00: 00.000 следующего дня.

SELECT  CAST(CAST('2009-12-01 00:00:00.000' AS DATETIME) AS BINARY(8)),
        CAST(CAST('2009-12-01 23:59:59.997' AS DATETIME) AS BINARY(8)),
        CAST(CAST('2009-12-01 23:59:59.999' AS DATETIME) AS BINARY(8))



0x00009B8F 00000000    0x00009B8F 018B81FF    0x00009B90 00000000

В первом значении позиция даты 0x9B8F ( 39823 ) - это количество дней с 1 января 1900 , а позиция времени 0 - это количество засечек с полуночи.

Во втором значении 0x018B81FF ( 25919999 или 24 * 60 * 60 * 300 - 1 ) - максимально возможное число клещей с полуночи.

Наконец, третье значение имеет значение 0 в позиции времени и позиции даты, увеличенное на единицу.

Не фиксируйте только ради фиксации. Обычно я добавляю их для кода, над которым работаю. Например, если я исправляю ошибку в методе A, я также обязательно внесу все косметические изменения.

IMO вам, ребята, не хватает инструментов кодирования, таких как PMD, JIndent и т.д., которые заботятся об этих проблемах, когда вы кодируете. Некоторые среды IDE как Netebeans показывают эти «проблемы» как предупреждения. Таким образом, его не случайное/личное изменение соответствует стандартам.