于东辉code review问题清单.xlsVIP

Sheet3

Sheet2

Sheet1

程序名

开发员

错误描述（含bug,及performance问题）

错误分类

处理方式

处理状态

完成日期

序号

行号

解决方案描述

QA人员

ZFIR0002

王学会/徐博

others

相应的变量常量注释

添加注释解释

取消注释掉的代码

于东辉

open

textsymbols

把相关的MESSAGE消息文本放到textsymbols中

把相关的全局声明或定义放到程序DATADECLARE把相关的局部声明或定义放到当前程序前。

把相关的全局声明或

定义放到程序DATADECLARE

把相关的局部声明或定义放

到当前程序前。

82/127

在loop前和append后或者modify后，清空工作区

在loop前和append后

或者modify后，清空

工作区

40/229

ZFIR0005

133-191

41/47

ZFIR0008

徐博

该注释过长，如果没用就去掉，如果有用，建议

用单独的块注释段

取消或者更改

增加注释

需要加注释解释‘9*’

197/249/

注释过长，建议改成*注释

更改注释

select字段最好不易横行过长，日后调试都

不容易看到

横行改为纵向

掉alv_grid_display函数，最好去掉不用的参数同时打开系统message，因为有时候可能有问题是通过系统message报出来的

去掉不用的参数，

打开message

ZFII0007

建议模块化

建议模块化

ZFIR0009

ZFIR0009_SSCREEN该子程序并没有用到，取消

取消或者吧screen改放

在该子程序下面

6/120

ZFIE0001

增加注释解释method

ZFII0002

readtable是否成功常用sy-subrc去判断

建议使用sy-subrc

ZFIF0003

screen建议使用block

增加规范，只是建议

内表定义应该规范一些，it1_itab，不建议在字符中加数字方式定义

更改

select看起来非常不清晰，建议改为纵向

建议改为纵向

去掉无用的空格，缩排程序

缩排程序

ZFIF0003_EXCEL取消

1.00

13.00

?2009?/?8?/?26

?2009?/?8?/?26

24.00

?2009?/?8?/?26

86.00

?2009?/?8?/?26

?2009?/?8?/?26

2.00

?2009?/?8?/?26

?2009?/?8?/?26

3.00

28.00

?2009?/?8?/?26

44.00

?2009?/?8?/?26

109.00

?2009?/?8?/?26

?2009?/?8?/?26

315.00

?2009?/?8?/?26

48.00

?2009?/?8?/?26

4.00

1.00

?2009?/?8?/?27

167.00

?2009?/?8?/?27

5.00

1.00

?2009?/?8?/?27

?2009?/?8?/?27

6.00

450.00

?2009?/?8?/?27

495.00

?2009?/?8?/?27

126.00

?2009?/?8?/?27

7.00

76.00

?2009?/?8?/?27

49.00

?2009?/?8?/?27

8.00

1.00

?2009?/?8?/?27

17.00

?2009?/?8?/?27

36.00

?2009?/?8?/?27

67.00

?2009?/?8?/?27

100.00

?2009?/?8?/?27

111.00

?2009?/?8?/?27

243.00

?2009?/?8?/?27

?2009?/?8?/?27

?2009?/?8?/?27