xml解析错误处理

Question

我使用tinyxml 来解析xml 文件，并且我发现这里的错误处理适合箭头代码。我们的错误处理只是向文件报告一条消息。

这是一个例子：

  const TiXmlElement *objectType = dataRoot->FirstChildElement( "game_object" );
  if ( objectType ) {
    do {
      const char *path = objectType->Attribute( "path" );
      if ( path ) {
        const TiXmlElement *instance = objectType->FirstChildElement( "instance" ); 
        if ( instance ) {
          do {
            int x, y = 0; 
            instance->QueryIntAttribute( "x", &x );
            instance->QueryIntAttribute( "y", &y );
            if ( x >= 0 && y >= 0 ) {
              AddGameObject( new GameObject( path, x, y ));
            } else {
              LogErr( "Tile location negative for GameObject in state file." );
              return false;
            }
          } while ( instance = instance->NextSiblingElement( "instance" ));
        } else {
          LogErr( "No instances specified for GameObject in state file." );
          return false;
        }
      } else {
        LogErr( "No path specified for GameObject in state file." );
        return false;
      }
    } while ( objectType = objectType->NextSiblingElement( "game_object" ));
  } else {
    LogErr( "No game_object specified in <game_objects>. Thus, not necessary." );
    return false;
  }
  return true;

我并没有对此气喘吁吁，但如果有人能想出一种更干净的方法来实现这一点，我将不胜感激。

PS 例外不是一个选项。

编辑：

这样的事情会更好吗？

if ( !path ) {
  // Handle error, return false
}
// Continue

这消除了箭头代码，但箭头代码有点将所有错误记录放在一处。

Answer 1

使用返回值作为错误代码只会导致这样的代码，它无法改进太多。一种稍微简洁的方法是使用 goto 将所有错误处理分组到单个块中并减少块的嵌套。

然而，这并没有解决实际问题，即使用返回值作为错误代码。在 C 中，没有其他选择，但在 C++ 中，异常是可用的并且应该使用。如果它们不是一个选择，你就会被现有的东西所困。

Answer 2

您可以为此创建一个宏，它封装了 if (!var) { .. return false; } 和错误报告。

然而，我不认为这可以得到多大改善；事情就是这样。 这就是生活。这是代码...

Answer 3

我并没有为此气喘吁吁，
但如果有人能想到一个清洁工
实现这一目标的方法是
赞赏。

我已经将嵌套的 if 替换为错误时的 return 语句（这使得代码“向下流动”而不是“箭头形状”。我还用 for 循环替换了 do 循环（这样我可以更好地理解它）。

这是什么？你想要吗？

const TiXmlElement *objectType = dataRoot->FirstChildElement( "game_object" );
if ( !objectType ) {
    LogErr( "No game_object specified in <game_objects>. Thus, not necessary." );
    return false;
}

for(; objectType != 0; objectType = objectType->NextSiblingElement( "game_object" )) {
    const char *path = objectType->Attribute( "path" );
    if ( !path ) {
        LogErr( "No path specified for GameObject in state file." );
        return false;
    }

    const TiXmlElement *instance = objectType->FirstChildElement( "instance" ); 
    if ( !instance ) {
        LogErr( "No instances specified for GameObject in state file." );
        return false;
    }

    for(; instance != 0; instance = instance->NextSiblingElement( "instance" )) {
        int x, y = 0; 
        instance->QueryIntAttribute( "x", &x );
        instance->QueryIntAttribute( "y", &y );
        if ( x >= 0 && y >= 0 ) {
            AddGameObject( new GameObject( path, x, y ));
        } else {
            LogErr( "Tile location negative for GameObject in state file." );
            return false;
        }
    }
}
return true;

Answer 4

我知道有点晚了，但我知道 QueryIntAttribute 返回一个值，可用于错误处理，以防您也希望将其用于属性。

if (instance->QueryIntAttribute("x",&x)!=TIXML_SUCCESS)
    cout << "No x value found";