Skip to end of metadata
Go to start of metadata

Code Conventions

(+) (#)

Source code and documentation contributed to the Dubbo repositories should observe the:

as core references regarding the formatting of code and documentation.

  • 异常和日志:
    • 尽可能携带完整的上下文信息,比如出错原因,出错的机器地址,调用对方的地址,连的注册中心地址,使用Dubbo的版本等。
    • 尽量将直接原因写在最前面,所有上下文信息,在原因后用键值对显示。
    • 抛出异常的地方不用打印日志,由最终处理异常者决定打印日志的级别,吃掉异常必需打印日志。
    • 打印ERROR日志表示需要报警,打印WARN日志表示可以自动恢复,打印INFO表示正常信息或完全不影响运行。
    • 建议应用方在监控中心配置ERROR日志实时报警,WARN日志每周汇总发送通知。
    • RpcException是Dubbo对外的唯一异常类型,所有内部异常,如果要抛出给用户,必须转为RpcException。
    • RpcException不能有子类型,所有类型信息用ErrorCode标识,以便保持兼容。
  • 配置和URL:
    • 配置对象属性首字母小写,多个单词用驼峰命名(Java约定)。
    • 配置属性全部用小写,多个单词用"-"号分隔(Spring约定)。
    • URL参数全部用小写,多个单词用"."号分隔(Dubbo约定)。
    • 尽可能用URL传参,不要自定义Map或其它上下文格式,配置信息也转成URL格式使用。
    • 尽量减少URL嵌套,保持URL的简洁性。
  • 单元和集成测试:
    • 单元测试统一用JUnit和EasyMock,集成测试用TestNG,数据库测试用DBUnit。
    • 保持单元测试用例的运行速度,不要将性能和大的集成用例放在单元测试中。
    • 保持单元测试的每个用例都用try...finally或tearDown释放资源。
    • 减少while循环等待结果的测试用例,对定时器和网络的测试,用以将定时器中的逻辑抽为方法测试。
    • 对于容错行为的测试,比如failsafe的测试,统一用LogUtil断言日志输出。
  • 扩展点基类与AOP:
    • AOP类都命名为XxxWrapper,基类都命名为AbstractXxx。
    • 扩展点之间的组合将关系由AOP完成,ExtensionLoader只负载加载扩展点,包括AOP扩展。
    • 尽量采用IoC注入扩展点之间的依赖,不要直接依赖ExtensionLoader的工厂方法。
    • 尽量采用AOP实现扩展点的通用行为,而不要用基类,比如负载均衡之前的isAvailable检查,它是独立于负载均衡之外的,不需要检查的是URL参数关闭。
    • 对多种相似类型的抽象,用基类实现,比如RMI,Hessian等第三方协议都已生成了接口代理,只需将将接口代理转成Invoker即可完成桥接,它们可以用公共基类实现此逻辑。
    • 基类也是SPI的一部分,每个扩展点都应该有方便使用的基类支持。
  • 模块与分包:
    • 基于复用度分包,总是一起使用的放在同一包下,将接口和基类分成独立模块,大的实现也使用独立模块。
    • 所有接口都放在模块的根包下,基类放在support子包下,不同实现用放在以扩展点名字命名的子包下。
    • 尽量保持子包依赖父包,而不要反向。
Labels:
None
Enter labels to add to this page:
Please wait 
Looking for a label? Just start typing.