Goto也有它的好

十月 17, 2006

1 条评论

最近真是忙的昏头转向，再加上天气逐渐转冷，很是有些不爽。今天dreamhead提醒我好久不更新Blog了，我也想更新，但是写点什么呢，工作相关的吧。

今天抽出一点儿时间来改一个系统的大Bug，这个问题早已定性，只是由于修改工作量较大，范围较广，而不敢轻易修改。不过眼看系统要上线测试，不改也不行了。

问题主要是由于系统锁资源使用不当，导致有时一些指针在无锁保护的情况下’裸奔’，解决方案就是在业务一层加锁，将底层的接口实现替换为无锁。在修改的过程中常常会遇到这样的情况：

rv = action1
if (rv != X_SUCCESS) {
return xx;
}

rv = action2
if (rv != X_SUCCESS) {
return xx;
}

action1和action2是需要在锁保护下的两个操作，对于上面的代码在任何一个可能退出该函数的出口，我们都需要进行解锁操作，所以一般方法就是：

locked;

rv = action1
if (rv != X_SUCCESS) {
unlocked;
return rv;
}

rv = action2
if (rv != X_SUCCESS) {
unlocked;
return rv;
}

unlocked
return rv;

这时候一旦还有更多的action或者是嵌套actions，代码维护起来就很是不方便，比如：
rv = action1
if (rv != X_SUCCESS) {
  if (….) {
   ….
  } else {
   ….
  }

}

rv = action2
if (rv != X_SUCCESS) {
  return rv;
} else {
  rv = action3
   if (rv != X_SUCCESS) {
    return rv;
   }
}
…
return rv;

这时候我们会想到在一个’关键路径’上加上解锁操作即可，这段代码在离开该函数前必将被执行，锁资源也必将被释放，而实现这种方式的一个好的方法就是使用goto，我们看一下用goto后的代码。
locked;

rv = action1
if (rv != X_SUCCESS) {
goto over;
}

rv = action2
if (rv != X_SUCCESS) {
goto over;
}

over:
unlocked
return rv;
看起来的确简单直观，逻辑清晰，整个函数就唯一出口。有人说goto不好，影响程序的结构化，其实goto很好，关键要看你如何用，滥用当然不好，像那种从前goto到后面，在从后面goto回来的代码是极力不推荐的。记得刚入司时只是教条的记得书本上说不要使用goto，甚至有的C编码规范’三令五申’不许使用goto，看了公司的代码中居然大量使用goto，当时感到不解，在以后的工作中渐渐体会到goto也有它的好处。任何事物都有其两面性，关键看你如何对待了。

饿了!回去煮面吃－肉酱面:)