PHP、XML、访问属性

Question

我在访问 XML 中的属性时遇到一些问题。我的代码如下。最初我有两个循环，并且没有任何问题。

我首先获取图像名称，然后使用第二个循环获取故事标题和故事详细信息。然后将所有内容插入数据库。我想整理一下代码并只使用一个循环。我的图像名称存储在 Href 属性中。 ()

XML 布局示例 (http://pastie.org/1850682)。 XML 布局有点混乱，这就是使用两个循环的原因。

$xml = new SimpleXMLElement('entertainment/Showbiz.xml', null, true);

    // Get story images
    //$i=0;
    //$image = $xml->xpath('NewsItem/NewsComponent/NewsComponent/NewsComponent/NewsComponent/NewsComponent/ContentItem');
  //  foreach($image as $imageNode){
    //  $attributeArray = $imageNode->attributes(); 
    //  if ($attributeArray != ""){
    //      $imageArray[$i] = $attributeArray;
    //      $i++;
    //  }
    //}

// Get story header & detail
$i=0;
$story = $xml->xpath('NewsItem/NewsComponent/NewsComponent/NewsComponent');
foreach($story as $contentItem){
    //$dbImage = $imageArray[$i]['Href'];
    foreach($contentItem->xpath('ContentItem/DataContent/nitf/body/body.head/hedline/hl1') as $headline){
        $strDetail = "";
        foreach($contentItem->xpath('ContentItem/DataContent/nitf/body/body.content/p') as $detail){
            $strDetail .= '<p>'.$detail.'</p>';
            foreach($contentItem->xpath('NewsComponent/NewsComponent/ContentItem') as $imageNode){
                $dbImage = $imageNode->attributes();    
            }
        }

        $link = getUnique($headline);

        $sql = "INSERT INTO tablename (headline, detail, image, link) VALUES ('".mysql_real_escape_string($headline)."', '".mysql_real_escape_string($strDetail)."', '".mysql_real_escape_string($dbImage)."', '".$link."')";
        if (mysql_query($sql, $db) or die(mysql_error())){
            echo "Loaded ";
        }else{
            echo "Not Loaded "; 
        }

    }
    $i++;
}

我想我已经接近实现它了。我尝试在第四个嵌套的 foreach 循环中放入一些 echo 语句，但没有任何结果。所以它不执行该循环。我已经在这个问题上呆了几个小时了，也用谷歌搜索了，只是无法找到它。

如果所有其他方法都失败了，我将返回使用两个循环。

问候， 斯蒂芬

Answer 1

这很难理解。我简化了结构，以便我们可以看到我们关心的层次结构部分。

看起来，具有 Duid 属性的 NewsComponent 定义/包含一个完整的新闻片段。在其两个子级中，第一个子级 NewsComponent 包含摘要和文本，而第二个子级 NewsComponent 包含图像。

您的初始 XPath 查询针对 'NewsItem/NewsComponent/NewsComponent/NewsComponent'，它是第一个 NewsComponent 子组件（带有正文文本的组件）。您无法从该点找到该图像，因为该图像不在该 NewsComponent 内；你已经走得太深了。 （我得到一个 PHP 注意：未定义的变量：dbImage，这一事实让我得到了提示。）因此，将初始 XPath 查询降低一个级别，并在需要时将该额外级别添加到后续 XPath 查询中。

从这样：

$story = $xml->xpath('NewsItem/NewsComponent/NewsComponent/NewsComponent');
foreach($story as $contentItem){
  foreach($contentItem->xpath('ContentItem/DataContent/nitf/body/body.head/hedline/hl1') as $headline){
    foreach($contentItem->xpath('ContentItem/DataContent/nitf/body/body.content/p') as $detail){
      foreach($contentItem->xpath('NewsComponent/NewsComponent/ContentItem') as $imageNode){ /* ... */ }}}}

到这样：

$story = $xml->xpath('NewsItem/NewsComponent/NewsComponent');
foreach($story as $contentItem){
  foreach($contentItem->xpath('NewsComponent/ContentItem/DataContent/nitf/body/body.head/hedline/hl1') as $headline){
    foreach($contentItem->xpath('NewsComponent/ContentItem/DataContent/nitf/body/body.content/p') as $detail){
      foreach($contentItem->xpath('NewsComponent/NewsComponent/NewsComponent/ContentItem') as $imageNode){ /* ... */ }}}}

但是，此后图像仍然不起作用。由于您使用循环（有时不必要），$dbImage 会被重新分配给空字符串。第一个 ContentItem 具有 Href 属性，该属性被分配给 $dbImage。但随后它会循环到下一个 ContentItem，该 ContentItem 没有属性，因此会用空值覆盖 $dbImage。我建议修改该 XPath 查询以仅查找具有 Href 属性的 ContentItem，如下所示：

->xpath('NewsComponent/NewsComponent/NewsComponent/ContentItem[@Href]')

这样就可以了。

---

其他想法

如果可能的话，重构以清理此代码。

正如我所提到的，有时您会在不需要时进行循环和嵌套，最终会变得更难以遵循，并可能引入逻辑错误（如图所示）。看来这个文件的结构永远是一致的。如果是这样，您可以放弃一些循环，直接查找您要查找的数据。你可以这样做：

// Get story header & detail
$stories = $xml->xpath('/NewsML/NewsItem/NewsComponent/NewsComponent');
foreach ($stories as $story) {
    $headlineItem = $story->xpath('NewsComponent/ContentItem/DataContent/nitf/body/body.head/hedline/hl1');
    $headline = $headlineItem[0];

    $detailItems = $story->xpath('NewsComponent/ContentItem/DataContent/nitf/body/body.content/p');
    $strDetail = '<p>' . implode('</p><p>', $detailItems) . '</p>';

    $imageItem = $story->xpath('NewsComponent/NewsComponent/NewsComponent/ContentItem[@Href]');
    $imageAtts = $imageItem[0]->attributes();
    $dbImage = $imageAtts['Href'];

    $link = getUnique($headline);

    $sql = "INSERT INTO tablename (headline, detail, image, link) VALUES ('".mysql_real_escape_string($headline)."', '".mysql_real_escape_string($strDetail)."', '".mysql_real_escape_string($dbImage)."', '".$link."')";
    if (mysql_query($sql, $db) or die(mysql_error())) {
        echo "Loaded ";
    } else {
        echo "Not Loaded "; 
    }
}