TaiWan Vat Declarition refactor

1. 现状

由于收到一个ATC issue:


image

在ADT中查看该CDS view的dependency analyzer:


image

access的db table 多达220!


image

超出了VDM CDS Metadata Checks (POC_ANNOTA) rule:
“Consumption view selects from 224 used tables (max allowed 100)”。
不过根据经验,猜测只有当access db超过200时才会报atc issue。

2.分析

当前cds 的架构如图:


CDS architecture.PNG

P_TW_TaxItemReversal 由P_TW_TaxItemConsume 和I_TW_TaxItemCube join而来。而 I_TW_TaxItemCube 本身又是从P_TW_TaxItemConsume 推演出来的,P_TW_TaxItemConsume 的复杂度被放大了2倍。

仔细看下业务逻辑:

  • P_TW_TaxItemReversal是为了按document class汇总生成pdf提供数据。
  • I_TW_TaxItemCube经过底层的计算已经得到了比较准确的document class以及其它一些信息,故可以保留这个cds view。但此cds view里面的凭证可能是reversal凭证,并不知道对应的原始凭证
  • join 另一边的P_TW_TaxItemConsume其实只是为了拿到reversal凭证对应的原始凭证

因此可以看出,P_TW_TaxItemConsume的目的只是为了带出bset中行项目上的ReverseDocument。这完全可以从bset 和bkpf推演出来而不用再去重复一遍P_TW_TaxItemConsume繁杂的推演逻辑。

解决

新的架构如下:


new CDS architecture.PNG

修改P_TW_TaxItemConsume:

define view P_TW_TaxItemConsume
  as select from I_TaxItem

{

  key CompanyCode,
  key AccountingDocument,
  key FiscalYear,
  key AccountingDocumentItem,
      case _AccountingDocument.IsReversal
        when 'X'
        then _AccountingDocument.ReverseDocument
        else _AccountingDocument.AccountingDocument
        end                                                                                               as OriginalDocument,
      case _AccountingDocument.IsReversal
      when 'X' then cast((cast(_AccountingDocument.ReverseDocumentFiscalYear as abap.int4) - 1911) as sstring)
      else cast((cast(_AccountingDocument.FiscalYear as abap.int4) - 1911) as sstring)
      end
                                                                                                          as OriginalDocumentTWYear,
      cast(decimal_shift( amount => TaxAmountInCoCodeCrcy,
      currency => CompanyCodeCurrency  ) as  abap.dec(12, 0))                                             as TaxAmount, // 9. Tax Amount
      cast(decimal_shift( amount => TaxBaseAmountInCoCodeCrcy,
      currency => CompanyCodeCurrency  ) as  abap.dec(12, 0))                                             as TaxBaseAmount

}

修改P_TW_TaxItemReversal, 去掉P_TW_TaxItemConsume的参数

define view P_TW_TaxItemReversal
  with parameters
    P_StatryRptgEntity  : srf_reporting_entity,
    P_StatryRptCategory : srf_rep_cat_id,
    P_StatryRptRunID    : srf_report_run_id,
    P_StatryRptRunType  : srf_report_run_type
  as select from P_TW_TaxItemConsume as taxItem
    inner join I_TW_TaxItemCube(
                    P_StatryRptgEntity  :$parameters.P_StatryRptgEntity ,
                    P_StatryRptCategory :$parameters.P_StatryRptCategory,                 
                    P_StatryRptRunID    :$parameters.P_StatryRptRunID,
                    P_StatryRptRunType  :$parameters.P_StatryRptRunType
                                                     )
                                            as cube on taxItem.CompanyCode             = cube.CompanyCode
                                            and taxItem.OriginalDocumentTWYear                = cube.TaiwaneseCalendarYear
                                            and taxItem.OriginalDocument                      = cube.AccountingDocument
                                            and taxItem.AccountingDocumentItem                = cube.AccountingDocumentItem
{
...
}

经过此次修改,dependency analyzer中查看cds C_TW_VATDeclarationItem 复杂度从220降为了119,应该可以通过atc check了


dependency AFTER.png
©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 174,310评论 25 709
  • 有些爱情平淡得千金不换, 有些人愿意竭尽所能,把生活画成有你的模样, 他叫孟睐。 1. 子玮是孟睐介绍给我认识的,...
    他们叫我大恒阅读 239评论 0 0
  • 这个世界上没有被物质满足到欢愉的灵魂。因为物质无止境,折腾就无止境,折磨就无止境。所以,最盛大的富有,便是内心丰盈...
    淑雲阅读 1,013评论 0 0