SAP R/3 форум ABAP консультантов
Russian ABAP Developer's Club

Home - FAQ - Search - Memberlist - Usergroups - Profile - Log in to check your private messages - Register - Log in - English
Blogs - Weblogs News

Качаство стандартного кода :(



 
Post new topic   Reply to topic    Russian ABAP Developer's Club Forum Index -> Talks | Беседка
View previous topic :: View next topic  
Author Message
XXX_:)
Аналитик
Аналитик


Age: 41
Joined: 01 Feb 2008
Posts: 387
Location: Воронеж
PostPosted: Fri Jun 05, 2009 5:52 pm    Post subject: Качаство стандартного кода :( Reply with quote
Есть такой HR HRULPFP1, печатная форма.
И есть там подпрограмма, которая делает проверки собирает сообщения. Количество сообщений собирает в l__warn_counter.
Потом сравнивает его в 15 и зарубает выгрузку текущего сотрудника. Если больше 15, то проблем нет Sad. Было бы смешно, если не было бы так грустно, помимо приведенного примера, достаточно кода не лучшего качества. Программа достаточно свежая сентябрь 2008 да и печатные формы для HR пишут в основном программисты с русскими фамилиями, а эта подозрительно не подписаная. В общем расстроился, единственный выход это либо в Z копировать, либо пользоваться implicit enhancement...

Code:
  case p_adv.
    when 1.
      l__adv_counter = 15.
    when 2.
      l__adv_counter = 15.
    when 3.
      if person-snomer is initial.
        l__adv_counter = l__warn_counter.
      else.
        l__adv_counter = -1.
      endif.
  endcase.
  if l__warn_counter eq l__adv_counter.
** log
    call method g_log->add_message_to_log_and_table
      exporting
        im_pernr      = pernr
        im_msg_type   = 'E'
        im_msg_number = '084'.
    call method g_log->add_employee_node
      exporting
        im_pernr = pernr.
    reject.
  endif.


Кому интересно это в HRUIPFP3_3 form check_for_reject
_________________
Hормальные люди делают вещи намного более безумные чем всё, что делают сумасшедшие (c) С.Лем
Back to top
View user's profile Send private message Blog
Display posts from previous:   
Post new topic   Reply to topic    Russian ABAP Developer's Club Forum Index -> Talks | Беседка All times are GMT + 4 Hours
Page 1 of 1

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You can download files in this forum


All product names are trademarks of their respective companies. SAPNET.RU websites are in no way affiliated with SAP AG.
SAP, SAP R/3, R/3 software, mySAP, ABAP, BAPI, xApps, SAP NetWeaver and any other are registered trademarks of SAP AG.
Every effort is made to ensure content integrity. Use information on this site at your own risk.