1C-Company / v8-code-style

Расширение для 1C:EDT, которое помогает разрабатывать конфигурации/приложения по стандартам 1С для платформы "1С:Предприятие 8".
Eclipse Public License 2.0
175 stars 53 forks source link

Исправление NPE #1316

Closed iArtemv closed 1 year ago

iArtemv commented 1 year ago

Что сделано

Чек-лист

Общее:

Если применимо:

Закрываемые задачи

Closes #??

sonarcloud[bot] commented 1 year ago

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

65.5% 65.5% Coverage
0.0% 0.0% Duplication

marmyshev commented 1 year ago

@iArtemv уточни, для каких случаев в коде - отсутствие имени является приемлемым?

iArtemv commented 1 year ago

В процессе редактирование или замены имени метода, экскпшн с npe. В целом отсутствие у метода имени неприемлемо, но кратковременно при изменение вполне рабочий процесс.

marmyshev commented 1 year ago

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

Выглядит как что-то неправильное, а стрельнуть так может в любом объекте EMF по любому свойству. Поэтому меня реквест этот смущает.

iArtemv commented 1 year ago

Да, наверное, можно обсудить и задачу на EDT