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
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 216,496评论 6 501
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 92,407评论 3 392
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 162,632评论 0 353
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 58,180评论 1 292
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 67,198评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 51,165评论 1 299
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 40,052评论 3 418
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,910评论 0 274
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 45,324评论 1 310
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,542评论 2 332
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,711评论 1 348
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,424评论 5 343
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 41,017评论 3 326
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,668评论 0 22
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,823评论 1 269
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,722评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,611评论 2 353

推荐阅读更多精彩内容

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