Code Inspector - проверка модулей на скрытые ошибки
Комментарии
В избранное
Подписаться на ответы
Сортировка:
Древо развёрнутое
Свернуть все
(2) TrinitronOTV,
может использоваться на любой машине, но только Вами. Лицензия недоступна для организаций, а также не может быть использована одновременно в нескольких терминальных сессиях на одном сервере. Предусмотрена безопасная активация на один сеанс работы или удаление данных ключа лицензии, при использовании на чужих компьютерах.
может использоваться на любой машине, но только Вами. Лицензия недоступна для организаций, а также не может быть использована одновременно в нескольких терминальных сессиях на одном сервере. Предусмотрена безопасная активация на один сеанс работы или удаление данных ключа лицензии, при использовании на чужих компьютерах.
(6) tormozit, по возможности да, но пока не на 100%
ложные срабатывания могут быть, но на моей практике и судя по отзывам пользователей их совсем немного
планирую еще добавить функцию Suppress, когда можно будет добавлять в исключения ложное срабатывание в определенном модуле. Ну и дальше увеличивать точность определения.
ложные срабатывания могут быть, но на моей практике и судя по отзывам пользователей их совсем немного
планирую еще добавить функцию Suppress, когда можно будет добавлять в исключения ложное срабатывание в определенном модуле. Ну и дальше увеличивать точность определения.
Классная штука для рефакторинга...
Но есть вопрос.
Хорошо, когда инспектор сообщает "Ошибок не обнаружено".
Но хотелось бы иметь полное описание всех потенциальных ошибок, которые может обнаруживать инспектор.
А то не совсем ясно, от каких ошибок свободен модуль после ревизии.
А вообще идея бомбовая... еще бы структуру объектов/модулей/форм/процедур/функций выгружать в виде дерева.
Хотя бы в эксель. Чтобы можно было плюсиками разворачивать дерево по веткам.
Но есть вопрос.
Хорошо, когда инспектор сообщает "Ошибок не обнаружено".
Но хотелось бы иметь полное описание всех потенциальных ошибок, которые может обнаруживать инспектор.
А то не совсем ясно, от каких ошибок свободен модуль после ревизии.
А вообще идея бомбовая... еще бы структуру объектов/модулей/форм/процедур/функций выгружать в виде дерева.
Хотя бы в эксель. Чтобы можно было плюсиками разворачивать дерево по веткам.
(8) Evgen.Ponomarenko, спасибо за отзыв!
По поводу «Ошибок не обнаружено».
Это как в юнит тестах. Если тесты говорят что все ОК – это еще не значит что ошибок нет. Но если тесты предупреждают что есть ошибка – скорее всего ошибка есть и нужно проверять. Так и здесь.
Самыми распространенными ошибками, которые обнаруживает Code Inspector являются:
По выгрузке структуры модулей.
Планируется, со временем, добавление разных новых возможностей, возможно будет и эта :)
По поводу «Ошибок не обнаружено».
Это как в юнит тестах. Если тесты говорят что все ОК – это еще не значит что ошибок нет. Но если тесты предупреждают что есть ошибка – скорее всего ошибка есть и нужно проверять. Так и здесь.
Самыми распространенными ошибками, которые обнаруживает Code Inspector являются:
- Присвоили значение переменной, а использовать забыли. Или сделали опечатку в имени переменной при присвоении значения, я такое встречал даже в типовых.
- В функции есть различные условные ветки, но не все они возвращают значение. При каких то условиях функция может вернуть Неопределено. Code Inspector это обнаружит.
- Бывают ошибки, когда значение переменной Неопределено, но используется. Например, для необъявленной переменной можно написать ИмяПеременной = ИмяПеременной + 1. Проверка синтаксиса это пропустит, но в процессе выполнения произойдет исключения. Code Inspector выдаст предупреждение в таком случае.
- Code Inspector обнаружит неиспользуемые параметры и объявленные переменные, а так же локальные функции.
По выгрузке структуры модулей.
Планируется, со временем, добавление разных новых возможностей, возможно будет и эта :)
(20) Evgen.Ponomarenko, вот группы ошибок на текущий день
"{{{1}}}: переменной '{0}' присвоено значение, но оно никогда не используется"
"{{{1}}}: возможно Неопределено значение переменной '{0}'"
"{{{1}}}: переменная '{0}' объявлена, но никогда не используется"
"{{{1}}}: параметр '{0}' никогда не используется"
"Локальная функция '{0}' никогда не используется"
"'{0}' Не все части функции возвращают значение"
"{{{1}}}: переменной '{0}' присвоено значение, но оно никогда не используется"
"{{{1}}}: возможно Неопределено значение переменной '{0}'"
"{{{1}}}: переменная '{0}' объявлена, но никогда не используется"
"{{{1}}}: параметр '{0}' никогда не используется"
"Локальная функция '{0}' никогда не используется"
"'{0}' Не все части функции возвращают значение"
(25) СергейКа, мне уже писали в ЛС с пожеланием добавить API. И тогда пользователи смогут добавлять свои проверки, построения графов и т.д. Мне нужно время - повнимательней изучить этот вариант.
По поводу параметров проверки - сейчас уже есть возможность настройки вывода групп ошибок, они сохраняются при следующем вызове.
По поводу параметров проверки - сейчас уже есть возможность настройки вывода групп ошибок, они сохраняются при следующем вызове.
(27) Еще немного :)
Для организаций будет интересней следующий функционал:
1. Возможность проверки извне или пакетно, с указанием проверяемых модулей.
2. Формирование отчетов по ошибкам, возможность их сравнения для проверки исправлений ранее найденных ошибок.
Для организаций будет интересней следующий функционал:
1. Возможность проверки извне или пакетно, с указанием проверяемых модулей.
2. Формирование отчетов по ошибкам, возможность их сравнения для проверки исправлений ранее найденных ошибок.
1. Было бы хорошо получать статитику по подсистемам в следующих разрезах:
Сообщения:
Предупреждения(Warnings)
Ошибки (Errors)
Фатальные ошибки (Fatal errors)
Статистика:
Средняя вложенность условий (Average nesting conditions)
Средняя вложенность функций (Average nesting function)
Количество операторов (Total operators)
Это позволило бы сформировать комплексную оценку кода по формальным признакам.
2. Было бы хорошо добавить три предупреждения:
1. Большая вложенность условий (средняя>1)
2. Большая вложенность функций (средняя>2)
3. Оператор ИЛИ в условиях
3. Было бы хорошо добавить две ошибки:
1. Слишком большая вложенность условий (максимальная>2)
2. Слишком большая вложенность функций (максимальная>3)
Сообщения:
Предупреждения(Warnings)
Ошибки (Errors)
Фатальные ошибки (Fatal errors)
Статистика:
Средняя вложенность условий (Average nesting conditions)
Средняя вложенность функций (Average nesting function)
Количество операторов (Total operators)
Это позволило бы сформировать комплексную оценку кода по формальным признакам.
2. Было бы хорошо добавить три предупреждения:
1. Большая вложенность условий (средняя>1)
2. Большая вложенность функций (средняя>2)
3. Оператор ИЛИ в условиях
3. Было бы хорошо добавить две ошибки:
1. Слишком большая вложенность условий (максимальная>2)
2. Слишком большая вложенность функций (максимальная>3)
(34) CheBurator, УТ не проверял. Другие типовые пробовал проверять - да есть ошибки. Встречались также опечатки в именах переменных. В основном много неиспользуемых значений переменных и параметров. Ну и функции, не всегда возвращающие значения тоже встречаются.
Добрый день
При запуске выдает :
"Не удалось получить лицензию. Сервер нарушил протокол. Section=ResponseStatusLine"
.NET Framework 3.5 Client Profile скачал при запуске файла "dotnetfx35setup.exe" диалоговых окон нет
Win 7 x64 во включении и отключении компонент ".NET Framework 3.5" включена.
Как запустить инспектора ?
При запуске выдает :
"Не удалось получить лицензию. Сервер нарушил протокол. Section=ResponseStatusLine"
.NET Framework 3.5 Client Profile скачал при запуске файла "dotnetfx35setup.exe" диалоговых окон нет
Win 7 x64 во включении и отключении компонент ".NET Framework 3.5" включена.
Как запустить инспектора ?
"С ростом функционала будет расти и цена. Самое время для выгодной покупки!"
Но не сказано, что будет расти цена на НОВЫЕ версии.
"Один год поддержки и обновлений"
Каждый год покупать? Зарегистрированным пользователям скидки не будет? Тогда вообще сейчас нет смысла покупать, а подождать, когда функционал будет увеличен (API, пакетный режим работы и т.д.), чтоб не вкладывать два раза в один и тот же продукт.
А вообще - подумайте над отдельной лицензией на продление, как у Касперского - существенно дешевле, чем приобретение новой лицензии.
Но не сказано, что будет расти цена на НОВЫЕ версии.
"Один год поддержки и обновлений"
Каждый год покупать? Зарегистрированным пользователям скидки не будет? Тогда вообще сейчас нет смысла покупать, а подождать, когда функционал будет увеличен (API, пакетный режим работы и т.д.), чтоб не вкладывать два раза в один и тот же продукт.
А вообще - подумайте над отдельной лицензией на продление, как у Касперского - существенно дешевле, чем приобретение новой лицензии.
+(45)
>Code Inspector (лицензия для индивидуальных разработчиков) 1 990,00 руб.
Мало функционала для такой цены!!!
Из чего данная цена складывается?
Например годовая лицензия на "MS Office 365 для дома расширенный" составляет всего то 2 499,00 руб ( и это включая НДС)!!! И это целый мс оффиссс.
>На данный момент Инспектор уже умеет выдавать следующие предупреждения:
>Локальная процедура или функция нигде не используется
>Переменная или параметр нигде не используется
>Переменной было присвоено значение, но оно нигде не используется
Это можно с помощью простейшей обработки реализовать
>Не все части функции возвращают значение
>Значение переменной Неопределено
На самом деле, так как правила проверки не переопределяются в пользовательском режиме, это огромный минус данной программы. Если бы код был открытым...
Например базовый функционал (реализация интерфейса API с конфигуратором) может продаваться в виде dll, а правила проверки кто может напишет свои...
>Code Inspector (лицензия для индивидуальных разработчиков) 1 990,00 руб.
Мало функционала для такой цены!!!
Из чего данная цена складывается?
Например годовая лицензия на "MS Office 365 для дома расширенный" составляет всего то 2 499,00 руб ( и это включая НДС)!!! И это целый мс оффиссс.
>На данный момент Инспектор уже умеет выдавать следующие предупреждения:
>Локальная процедура или функция нигде не используется
>Переменная или параметр нигде не используется
>Переменной было присвоено значение, но оно нигде не используется
Это можно с помощью простейшей обработки реализовать
>Не все части функции возвращают значение
>Значение переменной Неопределено
На самом деле, так как правила проверки не переопределяются в пользовательском режиме, это огромный минус данной программы. Если бы код был открытым...
Например базовый функционал (реализация интерфейса API с конфигуратором) может продаваться в виде dll, а правила проверки кто может напишет свои...
(46) GoodBeaver,
ИМХО
Сколько вы потратите время на написание такой обработки с вашим опытом? Точно не менее 2-х часов. И что на выходе? Копируете текст, переходите в предприятие, открываете обработку, вставляете текст - выполнить.
Но самое интересное - как вы получите список реквизитов объекта и формы контекста?
Более того, функционал инспектора не ограничен только 3-я пунктами. Нора намного глубже, чем кажется на первый взгляд.
Тем более что обработкой вы не получите никакой интеграции, тем более такой.
Что касается цены - Офис - продукт массового потребления. могут себе позволить демпинг.
Эта разработка является коммерческой большими с планами по развитию со всеми вытекающими. Посмотрите цены на инфостарте. минимум 100р. и это то что вы сможете как раз часа за 4 сделать.
Сори, если мой коммент прозвучал резко.
ИМХО
Это можно с помощью простейшей обработки реализовать
Сколько вы потратите время на написание такой обработки с вашим опытом? Точно не менее 2-х часов. И что на выходе? Копируете текст, переходите в предприятие, открываете обработку, вставляете текст - выполнить.
Но самое интересное - как вы получите список реквизитов объекта и формы контекста?
Более того, функционал инспектора не ограничен только 3-я пунктами. Нора намного глубже, чем кажется на первый взгляд.
Тем более что обработкой вы не получите никакой интеграции, тем более такой.
Что касается цены - Офис - продукт массового потребления. могут себе позволить демпинг.
Эта разработка является коммерческой большими с планами по развитию со всеми вытекающими. Посмотрите цены на инфостарте. минимум 100р. и это то что вы сможете как раз часа за 4 сделать.
Сори, если мой коммент прозвучал резко.
(47) MarSeN, в добавок (46) GoodBeaver,
В данном случае покупка данного продукта - это отражение вашей уверенности в будущем проекта,
как следствие вы поддерживаете автора материально.
А МС офис (2003) вообще должен продаваться по цене памперсов, уже сто раз окупился.
Как по мне МС офис (2007 и 2013) утопия - и рубля жалко. Так, что сравнение не уместное.
А вот автора поддержать нужно, те более, что в этой разработке вагон опыта и море человеко часов.
Что касается цены - Офис - продукт массового потребления.
В данном случае покупка данного продукта - это отражение вашей уверенности в будущем проекта,
как следствие вы поддерживаете автора материально.
А МС офис (2003) вообще должен продаваться по цене памперсов, уже сто раз окупился.
Как по мне МС офис (2007 и 2013) утопия - и рубля жалко. Так, что сравнение не уместное.
А вот автора поддержать нужно, те более, что в этой разработке вагон опыта и море человеко часов.
(46) GoodBeaver, есть очень интересный прием, для тех, кто "купил бы, если"
Им предлагают купить за цену, которую они считают справедливой, для такого продукта в текущем его виде.
Обычно либо идет ответ: "цена - 1 рубль (копейка), покупаю", либо собеседник теряется из виду.
Им предлагают купить за цену, которую они считают справедливой, для такого продукта в текущем его виде.
Обычно либо идет ответ: "цена - 1 рубль (копейка), покупаю", либо собеседник теряется из виду.
Объявление
Завтра готовится выход новой версии, содержащей довольно много исправлений. В программе предусмотрена автоматическая проверка обновлений, так что те у кого она включена - о выходе узнают. Обновление очень рекомендовано к скачиванию, особенно для тех, кто использует системы с включенным UAC, а так же по какой-то причине имел проблемы с активацией через сервер лицензий.
Завтра готовится выход новой версии, содержащей довольно много исправлений. В программе предусмотрена автоматическая проверка обновлений, так что те у кого она включена - о выходе узнают. Обновление очень рекомендовано к скачиванию, особенно для тех, кто использует системы с включенным UAC, а так же по какой-то причине имел проблемы с активацией через сервер лицензий.
Вышла новая версия 1.0.5129.32197
Добавлен конфигурационный файл, для возможности работы программы через прокси-сервер
Добавлена возможность ручного получения лицензии через браузер, при невозможности автоматического получения по какой-либо причине
Исправлена неправильная работа уведомлений о наличии новых версий.
Исправлена ошибка, приводившая к замене содержимого буфера обмена, при навигации по предупреждениям.
Исправлена ошибка, когда программе не удавалось дождаться появления окна Служебных сообщений, при включенном UAC.
Исправлена ошибка, приводившая к блокировке клавиатуры и мыши, в режиме с включенным UAC.
Исправлена ошибка, когда программа переставала работать при многократном вызове Инспектора в одном и том же модуле, при включенном UAC
Исправлена работа навигации по предупреждениям, при включенном UAC.
Внесены разные мелкие исправления.
(54) zoytsa, да этот баг был исправлен. А так же, для других подобных случаев была улучшена система аварийной разблокировки через 30 секунд. Все это уже есть в текущем релизе.
На всякий случай, если что-то подобное еще всплывет, то можно нажать NumLock пару раз и клавиатура разблокируется.
На всякий случай, если что-то подобное еще всплывет, то можно нажать NumLock пару раз и клавиатура разблокируется.
Ну и вдогонку теме цен:
Начальники, ну когда, ёмоё, минимальная цена на инфостарте снизится до хотя бы 100 рублей??? Ведь это реальный стимул для авторов продавать чаще, а для покупателей - не прижимать деньги из-за высоковатой цены. Сегментирование рынка же! А инфостарту никто не запрещает брать комиссию с продаж.
Начальники, ну когда, ёмоё, минимальная цена на инфостарте снизится до хотя бы 100 рублей??? Ведь это реальный стимул для авторов продавать чаще, а для покупателей - не прижимать деньги из-за высоковатой цены. Сегментирование рынка же! А инфостарту никто не запрещает брать комиссию с продаж.
Вышла версия 1.1.5140.17883
Добавлена новая проверка "Обнаружен недостижимый участок кода".
Добавлена поддержка получения измерений и ресурсов для форм регистров.
Исправлена ошибка с сохранением настроек фильтрации.
Исправлена ошибка в проверке функции на возврат значения.
Внесены мелкие исправления.
Добавлена новая проверка "Обнаружен недостижимый участок кода".
Добавлена поддержка получения измерений и ресурсов для форм регистров.
Исправлена ошибка с сохранением настроек фильтрации.
Исправлена ошибка в проверке функции на возврат значения.
Внесены мелкие исправления.
строишь AST и смотришь - на указаный узел хоть как-то передается управление или нет ?
и кстати - в 1С нормально не будет все равно никогда работать, потому что выяснить результат чаще всего нельзя не проведя интерпретацию кода, например
если выполнить("1 = 2") тогда
//// тут очень много кода, но любой анализатор скажет что код используется.
конецесли;
и кстати - в 1С нормально не будет все равно никогда работать, потому что выяснить результат чаще всего нельзя не проведя интерпретацию кода, например
если выполнить("1 = 2") тогда
//// тут очень много кода, но любой анализатор скажет что код используется.
конецесли;
Вышла версия 1.2.5150.27244
Code Inspector теперь использует API TurboConf 3, с новой системой взаимодействия с Конфигуратором.
Добавлена полная поддержка систем с включенным UAC.
Добавлен выбор режима отображения окна Инспектора - встроенное в окне Служебных сообщений или отдельное окно.
Если результаты проверки Code Inspector выводятся в отдельное окно, то это окно можно перемещать и закреплять на экране.
Изменена форма автоматического выбора режима работы программы.
В Code Inspector, при определении неиспользуемых значений переменных и параметров, добавлен анализ строк внутри операторов Выполнить и Вычислить.
Code Inspector теперь использует API TurboConf 3, с новой системой взаимодействия с Конфигуратором.
Добавлена полная поддержка систем с включенным UAC.
Добавлен выбор режима отображения окна Инспектора - встроенное в окне Служебных сообщений или отдельное окно.
Если результаты проверки Code Inspector выводятся в отдельное окно, то это окно можно перемещать и закреплять на экране.
Изменена форма автоматического выбора режима работы программы.
В Code Inspector, при определении неиспользуемых значений переменных и параметров, добавлен анализ строк внутри операторов Выполнить и Вычислить.
(80) ZhokhovM, спасибо за отзыв
1. Вроде не сталкивался с такой необходимостью, но если очень нужно, то можно будет добавить
2. Попробуйте использовать TurboConf Ultimate . Там выходит много исправлений и улучшений для CodeInspector.
3. Подумаю над добавлением этой возможности
1. Вроде не сталкивался с такой необходимостью, но если очень нужно, то можно будет добавить
2. Попробуйте использовать TurboConf Ultimate . Там выходит много исправлений и улучшений для CodeInspector.
3. Подумаю над добавлением этой возможности
4.Не может определить тексты модуля находящиеся на стыке инструкций препроцессоров #Область ИмяОбласти и #КонецОбласти. Заменил их комментариями типа // ИмяОбласти и // в отдельном блокноте, все ок. Очень смущает когда я делаю так.
Да, очень нужно.
1. Вроде не сталкивался с такой необходимостью, но если очень нужно, то можно будет добавить
Да, очень нужно.
(0) вот если бы это было вот так - мы были готовы и купить и подключить к своим open source проектам.
В текущем варианте использования применимость пока неявная - и как вы будете с Graphit'ом работать тоже неясно.
P.S. Это я не в порядке критики, а в порядке "поделиться" вариантами использования для Ваших потенциальных клиентов
В текущем варианте использования применимость пока неявная - и как вы будете с Graphit'ом работать тоже неясно.
P.S. Это я не в порядке критики, а в порядке "поделиться" вариантами использования для Ваших потенциальных клиентов
(93) lustin, спасибо за отзыв
Этот инструмент больше как проверка Resharper'a, а не как CodeClimate или PVS-Studio и т.д. Т.е. проверка непосредственно в процессе разработки.
Если такая необходимость возникнет, с Графитом думаю будет легче подружиться, чем с закрытым Конфигуратором.
Вообще я конечно подумаю и над Вашим вариантом.
Если будет спрос на мой проект Инспектора, как отдельного приложения или сервиса, я хочу добавить проверку cf файлов. Много наработок для этого у меня уже есть. Так что кому интересно, пожалуйста сообщите.
Этот инструмент больше как проверка Resharper'a, а не как CodeClimate или PVS-Studio и т.д. Т.е. проверка непосредственно в процессе разработки.
Если такая необходимость возникнет, с Графитом думаю будет легче подружиться, чем с закрытым Конфигуратором.
Вообще я конечно подумаю и над Вашим вариантом.
Если будет спрос на мой проект Инспектора, как отдельного приложения или сервиса, я хочу добавить проверку cf файлов. Много наработок для этого у меня уже есть. Так что кому интересно, пожалуйста сообщите.
Вопросы с вознаграждением
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|
||
|
|
|