Избыточная проверка кода

Question

Избыточная проверка кода

2

Существуют ли какие-либо инструменты, которые могут найти какие-либо частные функции без каких-либо ссылок? (Избыточные функции)

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

Не было бы слишком сложно создать его самостоятельно, но мне было интересно, есть ли какие-либо доступные приложения, которые могли бы сделать это с файлами, содержащими код?

Мой код находится в С#, но я могу представить, что этот вопрос охватывает множество языков кодирования.

ThePower 25 авг. 2009, в 15:51

Источник

1

Дубликат stackoverflow.com/questions/614096/clean-up-unwanted-code-c/…
Nathan Koop 25 авг. 2009, в 13:22

Теги:

c#

refactoring

language-agnostic

4 ответа

3

ReSharper выполняет задание.

The Chairman 25 авг. 2009, в 11:50

0

Было бы полезно, если бы вы рассказали людям, как
MPritchard 26 авг. 2009, в 08:33
0

ReSharper помечает любую частную функцию, которая не вызывается: * Желтая метка в правой части окна кода VS. * Название метода серое. * Красная лампочка, если курсор находится на названии функции. ALT + RETURN предлагает «Удалить неиспользуемый метод». это оно
The Chairman 26 авг. 2009, в 11:12

0

Eclipse делает это автоматически для Java, не уверен, что вы можете иметь то же самое для С#.

Ivan Koblik 25 авг. 2009, в 11:40

0

Еще один вопрос может быть даже "Скомпилирует ли компилятор С# частные методы, которые фактически не используются?".

Моя догадка не будет, но вы никогда не знаете!

EDIT:

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

John Bubriski 25 авг. 2009, в 10:43

2

Было бы «плохо» (tm) для компилятора удалять методы, которые не используются, частные или иные, так как к ним все еще можно получить доступ и вызвать через Reflection. На самом деле, я считаю, что инфраструктура VS Unit Test генерирует код, который делает именно это, чтобы позволить вам тестировать модули private / internal.
Rob 25 авг. 2009, в 13:01

Ещё вопросы

Дубликат stackoverflow.com/questions/614096/clean-up-unwanted-code-c/…
Было бы полезно, если бы вы рассказали людям, как
ReSharper помечает любую частную функцию, которая не вызывается: * Желтая метка в правой части окна кода VS. * Название метода серое. * Красная лампочка, если курсор находится на названии функции. ALT + RETURN предлагает «Удалить неиспользуемый метод». это оно
Было бы «плохо» (tm) для компилятора удалять методы, которые не используются, частные или иные, так как к ним все еще можно получить доступ и вызвать через Reflection. На самом деле, я считаю, что инфраструктура VS Unit Test генерирует код, который делает именно это, чтобы позволить вам тестировать модули private / internal.

Rob · Accepted Answer · 2009-08-25T10-53-00.000Z

2

Лучший ответ

Если ваш код имеет Unit Tests (он делает, правильно?;-), то запуск NCover позволит вам определить методы, которые никуда не вызывают. Если у вас нет модульных тестов, то это хороший повод использовать для начала их создания.

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

Rob 25 авг. 2009, в 10:53

0

Согласен с вами об этом, но я также недавно начал проект с 0 тестами, но также хотел бы удалить такой «мертвый код». Это уменьшит объем кода, для которого я должен начать писать тесты!
MPritchard 25 авг. 2009, в 12:59
0

@Martin - нет тестов; ouchies! Если вы начнете писать модульные тесты для открытых методов, которые, вероятно, сделают частные методы, которые не вызываются, достаточно маленьким набором, вы можете использовать поиск Visual Studio для проверки их вызова. Тем не менее, я точно знаю , откуда вы пришли с этим! :)
Rob 25 авг. 2009, в 13:03
0

У меня нет модульных тестов в моем проекте, может быть потому, что это SharePoint, хотя :-D.
Colin 25 авг. 2009, в 13:19
2

Недостатком является то, что если закрытый метод / функция является результатом какого-то более раннего цикла разработки, основанной на поведении / тестировании, то может быть тест, охватывающий неиспользуемый в настоящее время закрытый метод с помощью другого метода, который не является закрытым или является угловым случаем для пример пустой проверки, которая в текущем воплощении программного обеспечения никогда не будет выполнена. Простое рассмотрение покрытия кода вам там не поможет.
Esko 25 авг. 2009, в 13:27
0

@ Эско - хорошая мысль! знак равно
Rob 25 авг. 2009, в 13:51

Показать ещё 3 комментария