Текущее время: Пт, июл 28 2017, 08:54

Часовой пояс: UTC + 4 часа


Правила форума


ВНИМАНИЕ!

Вопросы по SAP Query и Quick View - сюда



Начать новую тему Ответить на тему  [ Сообщений: 44 ]  На страницу Пред.  1, 2, 3
Автор Сообщение
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, мар 15 2016, 16:14 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
hugo788 писал(а):
Kengur писал(а):
IF_EX_CTS_REQUEST_CHECK


Как я понял это внедрение в процесс создание/деблокирования запросов.

Тут я могу свою логику написать, но это не совсем то, что мне нужно.

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


Разобрался, извиняюсь был не прав, это то, что нужно, нужен как раз IF_EX_CTS_REQUEST_CHECK~CHECK_BEFORE_RELEASE_SLIN

Но не работает, создал внедрение, поставил точку, не попало, при разборе нашел в инклуде LSCTS_RELEASEF12:

Код:
IF pv_dialog = 'X'.
    lt_e071[] = ps_request-objects[].

    CALL FUNCTION 'TRINT_INSPECT_OBJECTS'
         EXPORTING
              iv_mode             = 'T'
              it_e071             = lt_e071
              it_e071k            = lt_e071k               " GTABKEY
              is_header           = ps_request-h           " GTABKEY
              IV_GTABKEY_CHK_ONLY = lv_gtabkey_check_only  " GTABKEY
         IMPORTING
              ev_error_type = lv_error_type.

*-------call user-exit  for approval ---------------
* user exit , possible to change lv_error_type = '1'
*  to  '2' as exception
*   IF ps_request-h-trstatus NA sctsc_states_released. ***TODO****
*    PERFORM user_exit_before_release_slin      USING  lv_current
*                                                      lv_error_type
*                                                      ps_request  .
*   ENDIF.

      IF lv_error_type = '1'.          "relevant faults for release


Как раз тут должен вызываться user-exit , который менят статус ошибка/инфо (lv_error_type ), но почему он закомментирован??


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Ср, мар 23 2016, 15:38 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
Нашел еще один существенный недостаток инструмента, во всяком случае для нашей фирмы - обязательная проверка происходит при деблокировании запроса, не задачи.

У нас запрос создает проектировщик, затем в нем программист в задаче пишет код, может быть несколько программистов, соотвественно несколько задач в одном запросе.
Так зачем делать проверки кода сразу по всем задачам и в момент деблокирования всего запроса???? Это будет уже происходить после тестирования, непосредственно перед переносом программы в продуктив. И что, проектировщик, при обнаружении ошибок, будет анализировать предупреждения кода программ? искать какие ошибки от какой задаче, тогда, когда уже надо будет нести разработку в продуктив? Очень странно....


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Ср, мар 23 2016, 23:26 
Почетный гуру
Почетный гуру
Аватара пользователя

Зарегистрирован:
Чт, окт 06 2005, 17:44
Сообщения: 2881
Откуда: Москва
hugo788 писал(а):
Нашел еще один существенный недостаток инструмента, во всяком случае для нашей фирмы - обязательная проверка происходит при деблокировании запроса, не задачи.

У нас запрос создает проектировщик, затем в нем программист в задаче пишет код, может быть несколько программистов, соотвественно несколько задач в одном запросе.
Так зачем делать проверки кода сразу по всем задачам и в момент деблокирования всего запроса???? Это будет уже происходить после тестирования, непосредственно перед переносом программы в продуктив. И что, проектировщик, при обнаружении ошибок, будет анализировать предупреждения кода программ? искать какие ошибки от какой задаче, тогда, когда уже надо будет нести разработку в продуктив? Очень странно....

А какая проблема включить проверку при деблокировании задачи? Мы для этой цели использовали BAdI CTS_REQUEST_CHECK (метод IF_EX_CTS_REQUEST_CHECK~CHECK_BEFORE_RELEASE) и ФМ TR_INSPECT_OBJECTS.
И да, деблокирование запроса делает не постановщик, а администратор разработок. :wink:

_________________
С уважением,
Удав.


Принять этот ответ
Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вс, мар 27 2016, 14:32 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
Удав писал(а):
И да, деблокирование запроса делает не постановщик, а администратор разработок. :wink:


Это когда/где как :)

Спасибо, тоже с помощью этого ФМ и внедрения реализовал проверки на уровне деблокирования задач, в любом случае считаю, что это правильней,
как минимум не нужна волокита с поиском (администратором, постановщиком, не важно) ответственных за объекты своей конкретной задачи.


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, сен 27 2016, 11:40 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
Интересует вопрос по псевдокомментариям инспектора кода.

Если с помощью возможностей ATC(запросы менеджерам качества ПО на исключение проверок инспектора кода в некоторых случаях) можно обойти проверки инспектора - что настроено и работает,
то в свою очередь возникает вопрос, как можно отключить возможность использование программистами псевдокомментарий?


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, сен 27 2016, 13:17 
Специалист
Специалист

Зарегистрирован:
Чт, май 12 2011, 17:06
Сообщения: 220
Как-то неэтично спрашивать у программистов, каким образом они предлагают надежно закрыть самим себе возможность обойти проверки. :)


Принять этот ответ
Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, сен 27 2016, 14:13 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
LAT писал(а):
Как-то неэтично спрашивать у программистов, каким образом они предлагают надежно закрыть самим себе возможность обойти проверки. :)


:oops: :D

Ну просто мы используем функционал ATC, который итак позволяет действительно при необходимости их обойти, но он теряет свой смысл для проверок на которые действуют псевдокомментарии....


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, сен 27 2016, 16:00 
Почетный гуру
Почетный гуру
Аватара пользователя

Зарегистрирован:
Чт, авг 19 2004, 18:37
Сообщения: 1755
Откуда: Москва
Пол: Мужской
hugo788 писал(а):
LAT писал(а):
Как-то неэтично спрашивать у программистов, каким образом они предлагают надежно закрыть самим себе возможность обойти проверки. :)


:oops: :D

Ну просто мы используем функционал ATC, который итак позволяет действительно при необходимости их обойти, но он теряет свой смысл для проверок на которые действуют псевдокомментарии....

Строго говоря, если программист поставил такой комментарий, значит знает что делает, соответственно отвечает за свои действия.
Я сталкивался с такой ситуацией на проекте, когда любое сообщение проверки принималось за ошибку и проверяющие тупо требовали исправить. Сие есть полное мракобесие!
Приведу простой пример. Зачастую в ФМ я описываю особые ситуации, и часто сам же их не использую - всё зависит от ситуации.
В самом ФМ я инициирую особые ситуации через "Message ... raising...", создавая некоторую избыточность, указывая одновременно особую ситуацию и номер сообщения, а также тип и доп.поля.
Если мне необходимо только отобразить или сохранить в журнал сообщение об ошибке из ФМ, то я пользуюсь "exceptions others" и универсальной подпрограммой журналирования сообщений из SY-MSG*. А если нужно обработать конкретную ошибку и направить алгоритм по другому руслу, тогда обрабатываю конкретный "exceptions".
Поэтому сообщение проверки о том, что я не все ситуации обрабатываю, не говорит об ошибке! Более того, требование проверяющих исправить это ухудшит качество разработки, т.к. больше кода - больше ошибок.

_________________
"For all entries" не в SAP-ах, "for all entries" в головах! :)


Принять этот ответ
Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, сен 27 2016, 17:42 
Специалист
Специалист

Зарегистрирован:
Чт, май 12 2011, 17:06
Сообщения: 220
Parazit писал(а):
Я сталкивался с такой ситуацией на проекте, когда любое сообщение проверки принималось за ошибку и проверяющие тупо требовали исправить. Сие есть полное мракобесие!
Полностью поддерживаю. Чрезмерное увлечение контролем за качеством кода вредит самому качеству кода.


Принять этот ответ
Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Ср, сен 28 2016, 13:21 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
Parazit писал(а):
Строго говоря, если программист поставил такой комментарий, значит знает что делает, соответственно отвечает за свои действия.


Эх...как бы хорошо было бы если все так работали :)
В общем возможности значит такой нет, спасибо.


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Пт, мар 31 2017, 08:43 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
При проверки ATC ФМ-ов, проверяется вся группа функций.
При проверки ATC методов - проверяется весь класс.

Есть где то настройки ограничить, что если я проверяю в se37 допустим ФМ, чтоб проверялся лишь код этого ФМ?


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Пт, мар 31 2017, 13:54 
Почетный гуру
Почетный гуру
Аватара пользователя

Зарегистрирован:
Чт, окт 06 2005, 17:44
Сообщения: 2881
Откуда: Москва
Обычно ФМ использует подпрограммы и данные, описанные в группе функций.
Вы предлагаете их не проверять? :roll:

_________________
С уважением,
Удав.


Принять этот ответ
Вернуться к началу
 Профиль  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Вт, апр 11 2017, 10:43 
Ассистент
Ассистент

Зарегистрирован:
Пт, ноя 07 2014, 21:19
Сообщения: 48
Тоже верно.

Может можно как то настроить, если метод не вызывает другие, если ФМ не вызывает объекты с гф, то не проверять....)


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
 Заголовок сообщения: Re: ATC (ABAP Test Cockpit) - проверка кода
СообщениеДобавлено: Пн, апр 17 2017, 19:53 
Начинающий
Начинающий

Зарегистрирован:
Вт, окт 16 2012, 10:57
Сообщения: 6
Нету такого и не нужно. Смысл ставить и настраивать проверку ATC, если она использоваться будет частично? Если проверяется ФМ - значит должна проверяться вся ГФ. Если вам не нужно проверять ГФ, значит вам эта проверка вообще не нужна - отключите проверку модулей совсем :roll:


Принять этот ответ
Вернуться к началу
 Профиль Отправить email  
 
Показать сообщения за:  Поле сортировки  
Начать новую тему Ответить на тему  [ Сообщений: 44 ]  На страницу Пред.  1, 2, 3

Часовой пояс: UTC + 4 часа


Кто сейчас на конференции

Сейчас этот форум просматривают: Yandex [Bot] и гости: 12


Вы не можете начинать темы
Вы не можете отвечать на сообщения
Вы не можете редактировать свои сообщения
Вы не можете удалять свои сообщения
Вы не можете добавлять вложения

Найти:
Перейти:  
Powered by phpBB © 2000, 2002, 2005, 2007 phpBB Group
Русская поддержка phpBB