Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / ASP.NET [игнор отключен] [закрыт для гостей] / Пометить объект / 25 сообщений из 49, страница 1 из 2
  1  все
10.12.2015, 11:58
    #39124736
Фёдор К
Фёдор К
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Здравствуйте.

Подскажите, есть-ли способ в M$VS пометить к примеру поле или метод с сообщением типа "Реализация временная и должна быть пересмотрена".
Ну, часто бывает нужно бросить г-код без обработок иксепшнов, оптимизаций и др.

Чтобы не забыть и как-то напоминало себе и другим?

Спасибо.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:04
    #39124749
Axeleron
Axeleron
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Фёдор К,
Чем ObsoleteAttribute не устраивает?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:06
    #39124754
Алексей К
Алексей К
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Использование списка задач
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:14
    #39124768
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Фёдор К,

Код: c#
1.
2.
3.
4.
5.
// TODO: вернуться и всё переделать!

// HACK: я тут херню сделяль...

// UNDONE: тут ещё пахать и пахать :'(
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:15
    #39124772
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
AxeleronФёдор К,
Чем ObsoleteAttribute не устраивает?

Это вообще из другой оперы.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:17
    #39124778
Axeleron
Axeleron
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttФёдор К,

Код: c#
1.
2.
3.
4.
5.
// TODO: вернуться и всё переделать!

// HACK: я тут херню сделяль...

// UNDONE: тут ещё пахать и пахать :'(


Красиво, конечно, только компилятору будет пофиг на все это. Разве что ТС захочет открывать каждый класс и пролистывать, чтобы найти это все 'добро'...
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:19
    #39124790
Zoria
Zoria
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
AxeleronhVosttФёдор К,

Код: c#
1.
2.
3.
4.
5.
// TODO: вернуться и всё переделать!

// HACK: я тут херню сделяль...

// UNDONE: тут ещё пахать и пахать :'(



Красиво, конечно, только компилятору будет пофиг на все это. Разве что ТС захочет открывать каждый класс и пролистывать, чтобы найти это все 'добро'...
Я так понимаю, таски при этом в помощь.
А компилятору должно быть и так хорошо. Так как смысл в том, что когда-нибудь.. будет ой как много свободного времени и хорошее настроение и можно будет заняться рефакторингом. :)
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:20
    #39124791
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
У решарпера ещё есть крутая поддержка:
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:21
    #39124794
Изопропил
Изопропил
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttAxeleronФёдор К,
Чем ObsoleteAttribute не устраивает?

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

TODO - это хорошо, но в ряде случаев видеть то хочется не весь говнокод, а только реально вызываемый
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:21
    #39124796
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
AxeleronРазве что ТС захочет открывать каждый класс и пролистывать, чтобы найти это все 'добро'...

Не надо, студия из коробки может показывать весь список. Решапер это показывает прям в дереве, как в солюшене. Можно настроить при желании хуки, да много что можно сделать.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 12:24
    #39124801
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
ИзопропилTODO - это хорошо, но в ряде случаев видеть то хочется не весь говнокод, а только реально вызываемый

(dot/N)Cover +NUnit и всё-всё можно видеть, чево-каво и как и почему, зачем, как жить дальше )) ещё поднять сонаркуб и кайфовать от +100500 метрик кода.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 13:11
    #39124863
Фёдор К
Фёдор К
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Спасибо всем.

Я имел ввиду, чтобы как Обслит глаза мозолил в коде.
Но придется работать со списком задач. В конце-концов это дисциплинирует. :)

С уважением.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 14:28
    #39124967
super-code
super-code
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Ха, да вы все ламеры!
Надо так:

Код: c#
1.
if (new Random().next(100) > 97) throw new Exception("TODO: Не забыть исправить кривой алгоритм поиска!");



Тогда будет напоминание такое, что по любому не забудешь исправить! :)

По моему многие так делают, бывает упадет приложение, потом запустишь сделаешь тоже самое, а все ок - без ошибок. Это значит они TODO в лог записали
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 16:27
    #39125128
skyANA
skyANA
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
С руководством договорится, что начиная с такого-то числа, если не исправил, то минус 5% от зарплаты каждый месяц, пока не исправил :)

Ну или если не бабками, то на рабочем месте стоит табличка: "Говнокодер", - пока не исправил.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 17:47
    #39125188
super-code
super-code
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
skyANA,

>то на рабочем месте стоит табличка: "Говнокодер", - пока не исправил.
Как Маконнелл писал, над рабочим местом повесить леденец, типа сосунок.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 20:07
    #39125318
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
skyANAот зарплаты каждый месяц, пока не исправил :)

Стёр коммент и всё пучком?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.12.2015, 22:12
    #39125383
skyANA
skyANA
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttskyANAот зарплаты каждый месяц, пока не исправил :)

Стёр коммент и всё пучком? Пул реквест не пройдёт ревью и не получит апрува :)
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
11.12.2015, 00:52
    #39125433
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
skyANAПул реквест не пройдёт ревью и не получит апрува :)

Офигенная автоматизация процесса разработки

Юнит-тест упал, и ничего никуда не пройдёт автоматически.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
11.12.2015, 00:59
    #39125436
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
Фёдор КСпасибо всем.

Я имел ввиду, чтобы как Обслит глаза мозолил в коде.
Но придется работать со списком задач. В конце-концов это дисциплинирует. :)

С уважением.

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

Аналогом TODO комментов могли бы быть записи в блокнотике на столе, типа: «вернуться к файлу XXXXX/YYYYY/XXXXX/AAAA.cs к строке 2456, допилить эту фигню». Это не более, чем закладка с пояснением, личный инструмент разработчика.

Если так хочется мозолилку, пишешь свой кастомный атрибут типа [VzorviMneMozgAttribute] и один юнит-тест, который будет валиться, если найдёт хоть один такой атрибут. Если тесты не используешь, можно зафигачить билд-таск который будет ворнинги толкать в консоль или вообще валиться с ошибкой, зависит от степени мазохизма
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
11.12.2015, 07:59
    #39125484
skyANA
skyANA
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttskyANAПул реквест не пройдёт ревью и не получит апрува :)

Офигенная автоматизация процесса разработки

Юнит-тест упал, и ничего никуда не пройдёт автоматически.Юнит-тест упал из-за стёртого комментария?

Офигенное модульное тестирование.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
12.12.2015, 09:43
    #39126295
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
skyANAЮнит-тест упал из-за стёртого комментария?

Нет, тест упал, потому что метод работает не так как надо. На комментарии вообще пофигу
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
12.12.2015, 10:50
    #39126312
skyANA
skyANA
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttskyANAЮнит-тест упал из-за стёртого комментария?

Нет, тест упал, потому что метод работает не так как надо. На комментарии вообще пофигу
У ТСа метод вроде работает как надо, проблема в том, что это "г-код без обработок иксепшнов, оптимизаций и др".
То есть таки нужно code review при pull request-ах
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
12.12.2015, 10:58
    #39126314
Фёдор К
Фёдор К
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttФёдор КСпасибо всем.

Я имел ввиду, чтобы как Обслит глаза мозолил в коде.
Но придется работать со списком задач. В конце-концов это дисциплинирует. :)

С уважением.

Странный кейс какой-то. Если ты написал TODO в коде, значит что-то не доделал, значит что-то будет не работать, или работать не так. Это и без всяких мозолилок будет очевидно.
- Да согласен я, согласен...
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
12.12.2015, 11:03
    #39126316
hVostt
hVostt
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
skyANAпроблема в том, что это "г-код без обработок иксепшнов, оптимизаций и др".

Код: c#
1.
Assert.That(() => myObject.MyMethod(null), Throws.TypeOf<ArgumentNullException>());



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


skyANAТо есть таки нужно code review при pull request-ах

А у вас есть code review по комментариям?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
12.12.2015, 13:15
    #39126349
skyANA
skyANA
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Пометить объект
hVosttskyANAпроблема в том, что это "г-код без обработок иксепшнов, оптимизаций и др".

Код: c#
1.
Assert.That(() => myObject.MyMethod(null), Throws.TypeOf<ArgumentNullException>());



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


skyANAТо есть таки нужно code review при pull request-ах

А у вас есть code review по комментариям?у нас есть code review pull request-ов
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
  1  все
Форумы / ASP.NET [игнор отключен] [закрыт для гостей] / Пометить объект / 25 сообщений из 49, страница 1 из 2
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]