基于官方arc-stable-9c57d86f66be,AUTOSAR版本3.1.5

基本问题

Arctic Core中使用了VALIDATE, VALIDATE_RV, VALIDATE_NO_RV等一系列宏简化参数判断。

#if ( CAN_DEV_ERROR_DETECT == STD_ON )
#define VALIDATE(_exp,_api,_err, ... ) \
if( !(_exp) ) { \
(void)Det_ReportError(CAN_MODULE_ID,,_api,_err); \
return __VA_ARGS__;\
} #define VALIDATE_NO_RV(_exp,_api,_err ) \
if( !(_exp) ) { \
(void)Det_ReportError(CAN_MODULE_ID,,_api,_err); \
return; \
} #define DET_REPORTERROR(_x,_y,_z,_q) (void)Det_ReportError(_x, _y, _z, _q)
#else
#define VALIDATE(_exp,_api,_err, ... )
#define VALIDATE_NO_RV(_exp,_api,_err )
#define DET_REPORTERROR(_x,_y,_z,_q)
#endif

方法是在每一个使用到这些宏的文件中都重新定义一遍。理由比较直接,因为每个模块是否打开这些宏,都有各自的开关。

简化的方法也很直接:重复的定义,提取到头文件(validate.h)中即可。

问题在于如何处理:

  1. 每个模块的开关宏;
  2. 每个模块不同的MODULE_ID;

因此,不能直接include该头文件就结束。

头文件的定义也依赖于这两个因素。

  1. 头文件中使用的开关宏取决于每个模块的开关宏;
  2. 头文件中使用的MODULE_ID取决于每个模块的MODULE_ID;

即:

#ifdef __THIS_MODULE_ENABLE_VALIDATE
#include <Det.h> #ifndef __THIS_MODULE_ID
#error "__THIS_MODULE_ID should be defined."
#endif #define VALIDATE(_exp, _api, _err) \
if( !(_exp) ) { \
(void)Det_ReportError(__THIS_MODULE_ID,,_api,_err); \
return __VA_ARGS__;\
} #define VALIDATE_RV(_exp, _api, _err, _rv) \
if( !(_exp) ) { \
Det_ReportError(__THIS_MODULE_ID, _instance, _api, _err); \
return _rv; \
} #define VALIDATE_NO_RV(_exp, _api, _err) \
if( !(_exp) ) { \
(void)Det_ReportError(__THIS_MODULE_ID,,_api,_err); \
return; \
} #define DET_REPORTERROR(_module, _instance, _api, _err) \
Det_ReportError(_module,_instance,_api,_err) #else #define VALIDATE(_exp, _api, _err)
#define VALIDATE_RV(_exp, _api, _err, _rv)
#define VALIDATE_NO_RV(_exp, _api, _err)
#define DET_REPORTERROR(_module, _instance, _api, _err) #endif

而使用的地方替换如下:

#if ( CAN_DEV_ERROR_DETECT == STD_ON )
#define __THIS_MODULE_ENABLE_VALIDATE 1
#define __THIS_MODULE_ID MODULE_ID_CAN
#endif
#include <validate.h>

如此每次需要使用VALIDATE_%类宏时,只需要提供所需的编译时信息即可。

提高复用,简洁明了。

更进一步

考虑到某些模块中报错是传递的不是本模块的MODULE_ID,有可能出现某次传递其他非本模块MODULE_ID的情况。为了提高灵活性,可以考虑提供外部传入全部参数的API。如下:

#ifndef __THIS_MODULE_ID
#error "__THIS_MODULE_ID should be defined."
#endif #define __VALIDATE(_exp, _module, _instance, _api, _err) \
if( !(_exp) ) { \
Det_ReportError(_module, _instance, _api, _err); \
} #define __VALIDATE_RV(_exp, _module, _instance, _api, _err, _rv) \
if( !(_exp) ) { \
Det_ReportError(_module, _instance, _api, _err); \
return _rv; \
} #define __VALIDATE_NO_RV(_exp, _module, _instance, _api, _err) \
if( !(_exp) ) { \
Det_ReportError(_module, _instance, _api, _err); \
return; \
} #define VALIDATE(_exp, _api, _err) \
__VALIDATE(_exp, __THIS_MODULE_ID, , _api, _err) #define VALIDATE_RV(_exp, _api, _err, _rv) \
__VALIDATE_RV(_exp, __THIS_MODULE_ID, , _api, _err, _rv) #define VALIDATE_NO_RV(_exp, _api, _err) \
__VALIDATE_NO_RV(_exp, __THIS_MODULE_ID, , _api, _err) #define DET_REPORTERROR(_module, _instance, _api, _err) \
Det_ReportError(_module,_instance,_api,_err) #else #define __VALIDATE(_exp, _module, _instance, _api, _err)
#define __VALIDATE_RV(_exp, _module, _instance, _api, _err, _rv)
#define __VALIDATE_NO_RV(_exp, _module, _instance, _api, _err)
#define VALIDATE(_exp, _api, _err)
#define VALIDATE_RV(_exp, _api, _err, _rv)
#define VALIDATE_NO_RV(_exp, _api, _err)
#define DET_REPORTERROR(_module, _instance, _api, _err)
#endif

如有需要,可以使用__VALIDATE_%类宏。

ArcticCore重构-VALIDATE_%的更多相关文章

  1. ArcticCore重构-问题列表1

    基于官方arc-stable-9c57d86f66be,AUTOSAR版本3.1.5 基本问题 Arctic Core中的代码组织有很多有待改进的地方,这里先提出几点: 1. 头文件引用混乱,所有头文 ...

  2. AUTOSAR ArcticCore重构 - for_each_HOH

    Arctic Core是AUTOSAR的实现,早期版本是开源的. 基本问题 在ARM架构下对CAN driver的实现(arch/arm/arm_cm3/drivers/Can.c)中,有这样一段代码 ...

  3. 记一次.NET代码重构

    好久没写代码了,终于好不容易接到了开发任务,一看时间还挺充足的,我就慢慢整吧,若是遇上赶进度,基本上直接是功能优先,完全不考虑设计.你可以认为我完全没有追求,当身后有鞭子使劲赶的时候,神马设计都是浮云 ...

  4. CSharpGL(17)重构CSharpGL

    CSharpGL(17)重构CSharpGL CSharpGL用起来我自己都觉得繁琐了,这是到了重构的时候. 下载 CSharpGL已在GitHub开源,欢迎对OpenGL有兴趣的同学加入(https ...

  5. ASP.NET MVC5+EF6+EasyUI 后台管理系统(58)-DAL层重构

    系列目录 前言:这是对本文系统一次重要的革新,很久就想要重构数据访问层了,数据访问层重复代码太多.主要集中增删该查每个模块都有,所以本次是为封装相同接口方法 如果你想了解怎么重构普通的接口DAL层请查 ...

  6. ASP.NET MVC5+EF6+EasyUI 后台管理系统(59)-BLL层重构

    系列目录 前言:  这应该是本系统最后一次重构,将重构BLL层和Model层.来完全取代代码生成器生成的BLL层和DAL层.完全废掉了代码生成器的DAL,BLL,MODEL层.  全自动生成增,删,改 ...

  7. 原生JS实现全屏切换以及导航栏滑动隐藏及显示——重构前

    思路分析: 向后滚动鼠标滚轮,页面向下全屏切换:向前滚动滚轮,页面向上全屏切换.切换过程为动画效果. 第一屏时,导航栏固定在页面顶部,切换到第二屏时,导航条向左滑动隐藏.切换回第一屏时,导航栏向右滑动 ...

  8. Android重构与设计之路,从整理提示弹窗(SmartAlertPop)开始

    封装一个独立弹窗Module,这里的弹窗包括普通的Dialog方式弹框和WindowManager方式弹窗.提供一种管理项目里面弹窗的方案,便于后期修改和维护. 首先描述一个在大项目中普遍存在的一个现 ...

  9. 产品前端重构(TypeScript、MVC框架设计)

    最近两周完成了对公司某一产品的前端重构,本文记录重构的主要思路及相关的设计内容. 公司期望把某一管理类信息系统从项目代码中抽取.重构为一个可复用的产品.该系统的前端是基于 ExtJs 5 进行构造的, ...

随机推荐

  1. ORACLE sid,pid,spid总结

    概念上:1.spid (system process id)  是操作系统层面的进程id .2.pid(process id)  这个是基于oracle的进程id个人理解为就是oracle给自己的进程 ...

  2. BAT Android工程师面试流程解析+还原最真实最完整的一线公司面试题

    尊重原创,转载请写明原文出处:http://blog.csdn.net/sk719887916/article/details/47040931 (skay) 求职和我们每个人息息相关,而求职也有门道 ...

  3. ffdshow 源代码分析 8: 视频解码器类(TvideoCodecDec)

    ===================================================== ffdshow源代码分析系列文章列表: ffdshow 源代码分析 1: 整体结构 ffds ...

  4. gcc学习(二)[第二版]

    头文件与库文件 1.概要 在使用C语言和其他语言进行程序设计的时候,我们需要头文件来提供对常数的定义和对系统及库函数调用的声明. 库文件是一些预先编译好的函数集合,那些函数都是按照可重用原则编写的.它 ...

  5. 【Android 应用开发】Android开发技巧--Application, ListView排列,格式化浮点数,string.xml占位符,动态引用图片

    一. Application用途 1. Application用途 创建Application时机 : Application在启动的时候会调用Application无参的构造方法创建实例; Appl ...

  6. winform编程设定listview选中行

    在做项目中,需要用到listview显示数据.同时,项目要求,通过检索用户输入的数据,程序通过搜索,确定数据所在的行并通过程序设定为选中状态并高亮显示.同时,正常响应鼠标单击响应的效果,单击时,程序设 ...

  7. 两种方法 更改 EBS R12界面LOGO以及内容

    from:metalink more: Note 174219.1 - How To Change The Logo In The Oracle Application Menu Note 84975 ...

  8. MIDle生命周期详解,以及工作原理

    当MIDlet被应用程序管理器成功地初始化之后,就开始展开了它的生命周期.MIDlet的生命周期完全由应用程序管理器控制,也就是说,当MIDlet要从一个状态变成另外一个状态时,应用程序管理器会调用对 ...

  9. Android开源经典项目

    目前包括: Android开源项目第一篇--个性化控件(View)篇   包括ListView.ActionBar.Menu.ViewPager.Gallery.GridView.ImageView. ...

  10. 【11】-java递归和非递归二叉树前序中序后序遍历

    二叉树的遍历 对于二叉树来讲最主要.最基本的运算是遍历. 遍历二叉树 是指以一定的次序访问二叉树中的每个结点.所谓 访问结点 是指对结点进行各种操作的简称.例如,查询结点数据域的内容,或输出它的值,或 ...