ABAP的手动代码评论和注释

2020-09-05 06:00发布

         点击此处--->   EasySAP.com群内免费提供SAP练习系统(在群公告中)

加入QQ群:457200227(SAP S4 HANA技术交流) 群内免费提供SAP练习系统(在群公告中)


你好,

尽管我们拥有SLIN,SCI,ATC等自动代码审查工具,但我敢肯定,您还可以通过比较不同系统中的不同版本来进行手动检查。

当您这样做时,作为审阅者,您如何传达对更改的感受?

并不是每个人都能拥有整个开发团队并排坐在同一时区的奢侈。 有些人必须"异步"进行此评论,并将评论清楚地传达给我们心爱的开发人员,以免他们再犯同样的错误。

  • 您是否使用简单的.txt文件或添加了评论的电子邮件,并稍后与开发人员讨论?
  • 如果您的团队中有太多开发人员,并且与所有人一对一地工作是不切实际的怎么办?
  • 如果您需要在评论中包括屏幕截图,该怎么办?
  • 如果您需要在当前评论中引用您之前的评论怎么办?
  • 如何跟踪与每个开发人员讨论的所有评论?

有了像Gerrit这样的高级工具可用于云,我觉得使用电子邮件/记事本记录我的评论非常荒谬。

你们是如何做到的?

         点击此处--->   EasySAP.com群内免费提供SAP练习系统(在群公告中)

加入QQ群:457200227(SAP S4 HANA技术交流) 群内免费提供SAP练习系统(在群公告中)


你好,

尽管我们拥有SLIN,SCI,ATC等自动代码审查工具,但我敢肯定,您还可以通过比较不同系统中的不同版本来进行手动检查。

当您这样做时,作为审阅者,您如何传达对更改的感受?

并不是每个人都能拥有整个开发团队并排坐在同一时区的奢侈。 有些人必须"异步"进行此评论,并将评论清楚地传达给我们心爱的开发人员,以免他们再犯同样的错误。

  • 您是否使用简单的.txt文件或添加了评论的电子邮件,并稍后与开发人员讨论?
  • 如果您的团队中有太多开发人员,并且与所有人一对一地工作是不切实际的怎么办?
  • 如果您需要在评论中包括屏幕截图,该怎么办?
  • 如果您需要在当前评论中引用您之前的评论怎么办?
  • 如何跟踪与每个开发人员讨论的所有评论?

有了像Gerrit这样的高级工具可用于云,我觉得使用电子邮件/记事本记录我的评论非常荒谬。

你们是如何做到的?

付费偷看设置
发送
6条回答
亦是此间程序员
1楼-- · 2020-09-05 06:16
追夢秋陽
2楼-- · 2020-09-05 06:13

嗨,

一个简单的方法是 使用excel记录问题。

-尝试对常见问题进行分类。

-在月末,编译所有常见的经常性问题,并进行简短的培训以增加他们的技术知识。

开发。

樱桃小丸子0093
3楼-- · 2020-09-05 06:25

也许不是直接在这里回答您的问题,但是,为项目中的所有开发人员编写一些遵循的编码标准又如何呢?

野沐沐
4楼-- · 2020-09-05 06:25

首先,虽然它看起来有点像穴居人,但使用电子邮件和记事本并不需要太多额外的培训;)严重的是,如果连接"太大" 跨时区的团队是一个问题,要确保他们都接受"还需要其他工具管理"方面的培训,这是一个更大的问题(更不用说跟踪落后者并促使他们完成了。)

第二,正如许多其他人提到的那样,遵守单一代码标准对于为任何类型的代码审查(手动或自动)建立框架至关重要。 Clean ABAP框架给我留下了深刻的印象[ https://blogs.sap.com/2019/05/03/clean-abap/]。 作者已经为建立扎实的工作基础进行了大量的思考和努力,并且这种努力正在进行中。

第三,正如其他人在这里提到的那样,任何采用标准和方法的方法都将非常耗时。 当然,纠正错误的代码很费时,但是时间和精力却埋在了"正常维护"中。 此外,管理层还必须考虑额外的计划时间,这些时间将通过进行代码审查和修订来吸收-并且他们必须相信,一次构建可靠的代码比一次又一次地修补和修补错误的代码要少(且最终便宜)。

最后,无论采用哪种方法/标准,团队和管理层都必须做出承诺:从团队中遵守既定规则; 并从管理层中删除拒绝遵守这些规定的员工,无论是员工还是承包商。 多年来,这个社区中也讨论了这一要求,但是这里分享的轶事证据表明管理层缺乏后续行动。 要求大多数程序员遵守标准,但允许其他人炫耀规则,则不可避免地导致标准被扔到垃圾堆中,而人员则流失到更好的管理项目中。

只有我的0.02美元。

渐行渐远_HoldOn
5楼-- · 2020-09-05 06:24

首先,请记住,这将花费很多时间

开发人员需要释放开发任务来修复 审查过程中的代码

开发人员发送审查请求

审查可能是带有严重性注释列表的文档(用于管理,如果失败的话) 审查和紧急请求)

开发人员更正代码,要求进行第二次审查

第一个文档已更新

如果您使用Solman/Charm,则可以 为审阅者在任务级别上设置代码版本

:从不进行谈判,开发人员就像孩子一样,如果他们觉得自己可以绕过法律,他们会尝试

祝你好运

Cikesha
6楼-- · 2020-09-05 06:13

Loyd Enochs 感谢您向我介绍Clean ABAP。

一周热门 更多>