有没有更好的方法？ While循环并继续

Question

我所维护的代码中有许多函数，它们可能会被描述为样板重。这里是一个与游标处理数据库I/O时，重复整个应用程序的广告恶心的样板模式：

if(!RowValue(row, m_InferredTable->YearColumn(), m_InferredTable->YearName(), m_InferredTable->TableName(), value) 
     || !IsValidValue(value)) 
    { 
     GetNextRow(cursor, m_InferredTable); 

     continue; 
    } 
    else 
    { 
     value.ChangeType(VT_INT); 
     element.SetYear(value.intVal); 
    } 

的事情是不是所有的这些陈述是这样处理整数的，这个“元素”的对象， “年”专栏等。我被要求比现在更凝结它，我想不出一种办法。我继续跳过继续声明和各个类的访问器。

编辑：感谢所有那些评论。这就是为什么我喜欢这个网站。这里是一个扩大视图：

while(row != NULL) 
{ 
    Element element; 
    value.ClearToZero(); 
    if(!GetRowValue(row, m_InferredTable->DayColumn(), m_InferredTable->DayName(), m_InferredTable->TableName(), value) 
     || !IsValidValue(value)) 
    { 
     GetNextRow(cursor, m_InferredTable); 

     continue; 
    } 
    else 
    { 
     value.ChangeType(VT_INT); 
     element.SetDay(value.intVal); 
    } 

而事情继续这样下去。并非所有取自“行”的值都是整数。 while循环中的最后一个子句是“GetNextRow”。

Answer 1

好了，从你说过，你有一个结构是这样的：

while (row!=NULL) { 
    if (!x) { 
     GetNextRow(); 
     continue; 
    } 
    else { 
     SetType(someType); 
     SetValue(someValue); 
    } 
    if (!y) { 
     GetNextRow(); 
     continue; 
    } 
    else { 
     SetType(SomeOtherType); 
     SetValue(someOtherValue); 
    } 
// ... 

    GetNextRow(); 
} 

如果这是真的，我会摆脱所有GetNextRow调用除了最后一个。然后我会结构中的代码是这样的：

while (row != NULL) { 
    if (x) { 
     SetType(someType); 
     SetValue(someValue); 
    } 
    else if (y) { 
     SetType(someOtherType); 
     SetValue(SomeOtherValue); 
    } 
    // ... 
    GetNextRow(); 
} 

编辑：另一种可能性是写你的代码作为一个循环：

for (;row!=NULL;GetNextRow()) { 
    if (!x) 
     continue; 
    SetTypeAndValue(); 
    if (!y) 
     continue; 
    SetTypeandValue(); 
    // ... 

由于调用GetNextRow现在是循环的一部分，本身，我们不必每次都（明确地）调用它 - 循环本身会处理这个问题。下一步（如果你有足够的这样做值得）将缩短代码来设置类型和值。一种可能性是使用模板特：

// We never use the base template -- it just throws to indicate a problem. 
template <class T> 
SetValue(T const &value) { 
    throw(something); 
} 

// Then we provide a template specialization for each type we really use: 
template <> 
SetValue<int>(int value) { 
    SetType(VT_INT); 
    SetValue(value); 
} 

template <> 
SetValue<float>(float value) { 
    SetType(VT_FLOAT); 
    SetValue(value); 
} 

这可以让你结合一对呼叫设置类型和值到一个单一的通话。

编辑：至于切削加工短变，这取决于 - 如果解析列是昂贵的（足够的关心），你可以简单地嵌套的条件：

if (x) { 
    SetTypeAndValue(); 
    if (y) { 
     SetTypeAndValue(); 
     if (z) { 
      SetTypeAndValue(); 

等。这样做的一个主要缺点是，如果（如你所说）在单个循环中有20多个条件，它会变得非常深。既然如此，我可能会认真考虑上面给出的基于for循环的版本。

Answer 2

为什么不反转你的if-test？

if (RowValue(row, m_InferredTable->YearColumn(), m_InferredTable->YearName(), m_InferredTable->TableName(), value) 
    && IsValidValue(value)) 
    { 
    value.ChangeType(VT_INT); 
    element.SetYear(value.intVal); 
    } 
    else 
    { 
    GetNextRow(cursor, m_InferredTable); 
    } 

Answer 3

我本能的做法是在这里建立一个多态的方法，在那里你最终风做这样的事情（模数你的语言和详细的逻辑）：

db_cursor cursor; 

while(cursor.valid()) 
{ 
    if(cursor.data.valid()) 
    { 
    process(); 
    } 

    cursor.next(); 
} 

db_cursor将是一个基类，你不同的表类型继承自子类，并且子类将实现有效性函数。

Answer 4

将它移到一个模板函数中，以模板的元素类型（例如整数）为模板，您可以反复调用该函数。使用特征模板改变每种数据类型的行为。

template <typename T> struct ElemTrait<T> {}; 
template <> struct ElemTrait<int> { 
    static inline void set(Val &value, Elem &element) { 
     value.ChangeType(VT_INT); 
     element.SetYear(value.intVal); 
    } 
}; 
// template <> struct ElemTrait<float> { ... }; 

template <typename T> 
void do_stuff(...) { 
    // ... 

    if (!RowValue(row, 
     m_InferredTable->YearColumn(), 
     m_InferredTable->YearName(), 
     m_InferredTable->TableName(), value) 
     || !IsValidValue(value) 
    ) { 
     GetNextRow(cursor, m_InferredTable); 
     continue; 
    } else { 
     ElemTrait<T>::set(value, element); 
    } 

    // ... 
} 

Answer 5

你可以拿出所有的GetNextRow调用和else子句：

for (row = GetFirstRow() ; row != null ; GetNextRow()) 
{ 
    Element element; 
    value.ClearToZero(); 
    if(!GetRowValue(row, m_InferredTable->DayColumn(), m_MetInferredOutTable->DayName(), m_MetInferredOutTable->TableName(), value) 
      || !IsValidValue(value)) 
    { 
     continue; 
    } 

    value.ChangeType(VT_INT); 
    element.SetDay(value.intVal); 
} 

Answer 6

为什么不做一个功能来做所有的工作？

bool processElement(Element& element, Row* row, int value, Table& m_InferredTable, /*other params*/) 
{ 
    if(!GetRowValue(row, m_InferredTable->DayColumn(), m_InferredTable->DayName(), m_InferredTable->TableName(), value) 
      || !IsValidValue(value)) 
    { 
      GetNextRow(cursor, m_InferredTable); 
      return true; 
    } 
    else 
    { 
      value.ChangeType(VT_INT); 
      element.SetDay(value.intVal); 
    } 
    return false; 
} 

在你的循环

while (row != NULL) 
{ 
    if (processElement(element, row, value, m_InferredTable)) 
     continue; 
    // other code 
}