Утилита для анализа кода.

[Deimos747]

Доброго времени суток! Как-то на форуме, я натыкался на описание утилиты для as3, которая анализирует код и в результате пишет какие переменные и функции не используются, а также рекомендации по качеству кода. Не подскажете, существует ли реально такая утилита, и если то как называется? А то я сам найти не могу.

[bav]

Во FlashDevelop это Tools > Flash Tools > Analyze Project Source Code...

[caseyryan]

Эти анализаторы такую чушь выдают, что просто пипец )
Например, что-то типа:
functions should only have one exit point. Хотя очевидно, что это не так. Могут быть определенные условия, при которых функция должна возвращаться одно значение, при других - дургое.
Или что названия переменных слишком короткие (даже в циклах).
В общем, бредогенератор. Не советую.

[Inet_PC]

Тоже самое только для FB

functions should only have one exit point

Имеется ввиду что функция должна иметь только 1 return, а не то что Вы написали, все "бредовые" проверки можно отключить и оставить только те, что Вас устраивают.

[iNils]

Сообщение от caseyryan

Эти анализаторы такую чушь выдают, что просто пипец )
Например, что-то типа:
functions should only have one exit point. Хотя очевидно, что это не так. Могут быть определенные условия, при которых функция должна возвращаться одно значение, при других - дургое.
Или что названия переменных слишком короткие (даже в циклах).
В общем, бредогенератор. Не советую.

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

[Deimos747]

Спасибо за ссылки. А для FDT ничего такого нет? Меня в первую очередь интересуют неиспользуемые public функции и переменные.

Добавлено через 1 минуту
Всем спасибо, FlexPMD то что мне нужно.

[КорДум]

IDEA имеет такую возможность.

[caseyryan]

Сообщение от Inet_PC

Тоже самое только для FB

Имеется ввиду что функция должна иметь только 1 return, а не то что Вы написали,

Нифига. Я об этом говорю. Return'ов может быть несколько. И срабатывать будут те, что подходят по условиям.
Как на счет такого варианта:

private function getItemByCode(code:int):Item {
     for each (var item:Item in items) {
           if (item.code == code) return item;
     }
     return null;
}

?
Можно конечно написать и так

private function getItemByCode(code:int):Item {
     var i:Item = null; 
     for each (var item:Item in items) {
           if (item.code == code) {
                i = item;
                break;
           }
     }
     return i;
}

Но первый вариант короче и удобнее.

все "бредовые" проверки можно отключить и оставить только те, что Вас устраивают.

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

То есть сделать так, чтобы код соответствовал личным или командным нормативам. Так что вещь полезная.

Ну, разве что для командной работы

[Hauts]

iNils, подскажите, как настроить?

[iNils]

Сообщение от Hauts

iNils, подскажите, как настроить?

Tools > Flash Tools > Code Analyzer Ruleset Creator...
Загрузится страница, там куча различных правил которые можно отключить или включить, поменять приоритеты. Можно свои создать на основе регулярных выражений. Потом экспортировать изменения в C:\Program Files\FlashDevelop4\Tools\flexpmd\default-ruleset.xml или в другой файл, а в настройках плагина указать где он находится.

Добавлено через 5 минут

Сообщение от caseyryan

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

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

И стили у все разные, тот же return. Автор этих проверок считал, что он должен быть один, а ты нет. Возьми и отключи.

Это лишь проверка самого себя, можно на нее вообще забить.