RT-Thread-rtgui api函数关于形参的一点修改建议RT-Thread问答社区

rtgui api函数关于形参的一点修改建议

发布于 2011-07-23 12:06:07 浏览：4159 订阅该版

在rtgui中很多api函数的形参使用的是结构体指针,这种方式使用的很频繁.
函数中的形参使用结构体指针,避免不了要在引用前先创建这些临时结构体变量,并为这些变量赋值,这些地方改进一下会大大提高函数的易用性.而且在很多GUI的api中已经形成一种默认标准.我认为rtgui有必要对这个问题作个讨论.
而且我实际测试了一下,在形参个数较多时,使不使用结构体指针并不会给性能带来提升,都会采用堆栈传递参数.但是使用结构体指针作为参数的话,会导致代码尺寸增大,原因是在函数被引用前都有处理这些结构体的代码,这些增加的代码代码是可以避免的.如果将rtgui中函数使用的结构体指针全部展开,编译之后代码尺寸可以较小几K,还是比较可观的.尺寸减小的程度与用户使用这些api的频率成正比.

看看实际使用这种函数的一个例子:
这段代码可以在一个控件上绘制一个十字架,并且不破坏控件原有的数据,会用到DC缓冲区,正好符合测试的要求.
static rt_int16_t bak_x=0,bak_y=0;
struct rtgui_dc_buffer *dc_grp; /* 绘图层缓冲区 */
struct rtgui_dc_buffer *dc_cross; /* 会在此缓冲区绘制十字线 */
#define USER_GRP_W  320
#define USER_GRP_H  240

/* 函数功能:初始化用户使用到的DC缓冲区 */
void user_dc_buffer_init(void)
{
    dc_grp = rtgui_dc_buffer_create(USER_GRP_W, USER_GRP_H);
    if(dc_grp == RT_NULL) return;
    dc_cross = rtgui_dc_buffer_create(20,20); /* 创建cross缓冲 */
    if(dc_cross == RT_NULL) return;
}

void user_draw_cross(rtgui_dc_t *dc)
{
    int w,h;
    rtgui_rect_t rect;
    
    rtgui_dc_get_rect(dc, &rect);
    w = rtgui_rect_width(rect);
    h = rtgui_rect_height(rect);
    RTGUI_DC_FC(dc) = blue;
    rtgui_dc_draw_hline(dc, 0, w, h/2-1);
    rtgui_dc_draw_vline(dc, w/2-1, 0, h);
}

/* 函数功能:在用户指定的控件上绘制一个十字线 */
void user_draw_cross_flag(PVOID wdt, int x, int y)
{
    rtgui_dc_t *dc;
    rtgui_rect_t rect;
    rtgui_point_t point;

RT_ASSERT(wdt != RT_NULL);
    if(dc_cross == RT_NULL || dc_grp == RT_NULL) return;

dc = rtgui_dc_begin_drawing(wdt);
    if(dc == RT_NULL) return;

rect.x1 = bak_x; rect.x2 = bak_x+20;
    rect.y1 = bak_y; rect.y2 = bak_y+20;
    point.x = bak_x; point.y = bak_y; 
    rtgui_dc_blt(dc_grp, &point, dc, &rect);
   
    point.x = x; point.y = y;
    rect.x1 = 0; rect.x2 = 20;
    rect.y1 = 0; rect.y2 = 20; 
    rtgui_dc_buffer_copy(dc_grp, &point, dc_cross, &rect);/* buffer copy */
 
    user_draw_cross(dc_cross);

rect.x1 = x; rect.x2 = x+20;
    rect.y1 = y; rect.y2 = y+20;
    rtgui_dc_blit(dc_cross, RT_NULL, dc, &rect);

rtgui_dc_end_drawing(dc);
    bak_x = x;
    bak_y = y;
}

rt_bool_t dc_grp_event_handle(PVOID wdt, rtgui_event_t *event)
{
    if(event->type == RTGUI_EVENT_PAINT)
    {
        rtgui_rect_t rect;
        rtgui_dc_t *dc = rtgui_dc_begin_drawing(wdt);
        
        rtgui_dc_get_rect(dc, &rect);
        rtgui_dc_blit(dc_grp, RT_NULL, dc, &rect);
        rtgui_dc_end_drawing(dc);
    }
    else
        return rtgui_view_event_handler(wdt, event);

return RT_TRUE;
}

用户可以使用user_draw_cross_flag(user_view, x, y);的方式在控件user_view上绘制一个十字架.如果看不懂这些代码,抱歉,我是在rtgui_win下编写的,移植到SVN_rtgui上也不难.
代码中红色的部分代码是完全可以优化的代码.
将
    rect.x1 = bak_x; rect.x2 = bak_x+20;
    rect.y1 = bak_y; rect.y2 = bak_y+20;
    point.x = bak_x; point.y = bak_y; 
    rtgui_dc_blt(dc_grp, &point, dc, &rect);
更改成
  rtgui_dc_blt(dc_grp, bak_x, bak_y, dc, bak_x, bak_y, 20, 20);
这种方式更便于理解,也可以避免犯错,第一种方式在代码很长时,一不小心就会导致传递参数错误或者参数赋值错误.
从性能上分析,这两种情况应该接近.分析错误了请别拍我. [s:175] 
第一种方式会开辟临时变量,第二种方式会在堆栈中存放参数,RAM消耗差不多吧.
从代码尺寸上分析,显然第二种更有优势,比第一种方式小64字节.如果这种优势扩展到整个rtgui,则更加可观了.
    综上所述,我的观点是,改进api函数中使用结构体指针作为形参的情况,因为在实际传递实参时影响不大,但是编程不方便,代码尺寸也会变大.
    希望各位多多讨论. [s:188]