11

Зачастую при любой разработке возникает необходимость написать код, про который известно заранее что он:

  1. Крайне не гибок. Или..
  2. Может вызвать баги, проблемы в дальнейшем. Или..
  3. Совершенно не читаем.

Или такой код достаётся по наследству, и хочется пометить места, которые надо затем выправить. Такой код называется по-русски костыль. К сожалению на англоязычных сайтах трудно найти информацию об этом, так как у них термин "костыль" - отсутствует в принципе. Рекомендуют переводить, как "kludge" - но это не часто понимают.

Вопрос - как помечать костыли в коде комментариями, чтобы было просто составить список из всех костылей в проекте, и крайне желательно - чтобы была их обозримость в IDE, может даже расстановка приоритетов? Меня больше интересует PhpStorm, но другим будет интересно узнать про их IDE.

P.S.
Есть конечно спец-метка в комментах TODO, но хотелось бы представлять костыли и TODO отдельно друг от друга, всё таки они сильно разные по сути. Поэтому некоторые договариваются о специальных комментах вроде:

/* TRASHCODE {why below code is bad} */
...
/* TRASHCODE end*/

Потом по коду выискать просто, но помощи IDE в обозрении костылей в таком случае нет.

1

3 ответа 3

13

хотелось бы представлять костыли и TODO отдельно друг от друга

Во-первых, все приличные IDE кроме TODO понимают "из коробки" ещё и FIXME.

но помощи IDE в обозрении костылей в таком случае нет

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

PhpStorm (и любые другие джетбрейновские):

введите сюда описание изображения

Eclipse (можно кастомизировать даже для каждого языка/проекта):

введите сюда описание изображения

для FIXME и TODO - можно обозначать конец? То есть выделить с помощью комментов блок.

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

CLion:

введите сюда описание изображения

В закладке:

введите сюда описание изображения

Eclipse:

введите сюда описание изображения

В закладке:

введите сюда описание изображения

3
  • А не подскажешь, для FIXME и TODO - можно обозначать конец? То есть выделить с помощью комментов блок. 4 окт 2016 в 1:30
  • 1
    @ГончаровАлександр, добавил в ответ.
    – PinkTux
    4 окт 2016 в 1:48
  • @PinkTux дату не исползуете в теге? И если вы не один разработчик, то и имя: TODO (Вася-1212172221), штамп времини потом используется как ID фикса
    – Hellseher
    12 дек 2017 в 22:23
1

Ахахах...Нет ничего более постоянного, чем временное решение

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

Вы немного неправы про отсутствие костыля в английском языке, оно часто называется kludge or workaround

Куски кода которые требуется исправить часто отмечают с помощью FIXME, которую поддерживают многие редакторы и PHPSTORM тоже, оно отмечается в TODO. FIXME ставиться непосредственно перед проблемным участно и может содержать описание в каких строках код содержит проблемы и что за проблемы, которые нужно исправить, ограничителя нет.

FIXME: строки 38-120 имеют низкую абстракцию. 

Опыт

Могу сказать, что я и коллеги, мы редко помечаем костыли. Часто они сразу переписываются без пометок в момент новой задачи или поиска ошибки (класс уже несколько раз стрельнул, даже после исправления), потому что все эти пометки с недостатком времени превращаются в постоянные. Переписываются естественно костыли без фанатизма, в целях экономии средств бизнеса на дальнейшую разработку и поддержку продукта.

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

P.S Избегайте лишних парадигм: костылирование, инкостыляция и поликостылизм, пишите красивый код приложив к нему хотя бы минутку проектирования.

7
  • IMHO, TODO и FIXME - совершенно нормальное явление при "практике частых коммитов". В пуш их, конечно, попадать должно как можно меньше, но в процессе работы их бывает немало, что в этом плохого? Это не более чем ещё один инструмент для навигации по "сырому" исходнику.
    – PinkTux
    4 окт 2016 в 1:54
  • >Если Вы не можете писать лаконичный и гибкий код для определенной задачи, то у вас проблемы с проектированием архитектуры и стоит подтянуть свои скилы, такие проблемы у начинающих разработчиков из-за недостатка опыта. 4 окт 2016 в 2:20
  • 3
    @Firepro ну плохое начало ответа, что за гордливость, а) Кто не понимает почему необходимы костыли - тот не бывал на передовой. Потому что заказчику что-то припёрло, потому что денег на всех не хватает, потому что ПМ-а нет в принципе, потому что проект горит, и что-то надо срочно делать, потому что кто-то хочет заработать. 4 окт 2016 в 2:22
  • @Firepro и кстати не ходит у них в обиходе kludge , по крайней мере знакомые иностранцы недоумевали, что я под этим имею в виду. Насчёт workaround не слышал. 4 окт 2016 в 2:37
  • @ГончаровАлександр более часто используют workaround (ru.wikipedia.org/wiki/…) и оно часто встречается в статьях. Я могу понять костыль от недостатка времени или обхода временного бага в сторонней библиотеке, или когда весь проект уже в костылях, и писать адекватное решение уже нет смысла(дорого), но все остальное связано с недостатком опыта. Хотя даже когда времени мало (не экстремально мало и это фатальный баг), все-таки стоит основываться на уже реализованных решениях и это позволит сократить костыльность.
    – Firepro
    4 окт 2016 в 11:32
1

PHP

#region FIXME Bug text
...
#endregion

Javascript

//region FIXME Bug text
...
//endregion

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.