PHPExcel_Style_Fill无限递归

Question

PHPExcel_Style_Fill无限递归

8

我使用库PHPExcel 1.7.9处理Excel文件。首先，我创建一个模板，进行样式和排版上的优化。接着，为了避免硬编码样式，我使用上述库打开那个模板，更改一些数值并将其保存为一个新的.xlsx文件。

首先，我们获取单元格中的样式。

$this->styles = array() ;
$this->styles['category'] = $sheet->getStyle("A4");
$this->styles['subcategory'] = $sheet->getStyle("A5");

这是一个递归函数，用于显示类别和子类别。

private function displayCategories($categories, &$row, $level = 0){
    $sheet = $this->content ;

    foreach($categories as $category){
        if ($category->hasChildren() || $category->hasItems()){ //Check if the row has changed.
            $sheet->getRowDimension($row)->setRowHeight(20);
            $sheet->mergeCells(Cell::NUMBER . $row . ":" . Cell::TOTAL . $row) ;

            $name = ($level == 0) ? strtoupper($category->name) : str_repeat(" ", $level*6) ."- {$category->name}" ;
            $sheet->setCellValue(Cell::NUMBER . $row, $name) ;
            $sheet->duplicateStyle((($level == 0) ?  $this->styles['category'] : $this->styles['subcategory']), Cell::NUMBER . $row);

            $row++ ;
            if ($category->hasChildren()){
                $this->displayCategories($category->children, $row, $level+1);
            }
        }
    }   
}

问题

如果使用$sheet->duplicateStyle()，由于无限递归，将无法保存文档。

已达到“200”个最大函数嵌套级别，中止！<- 致命错误

问题出现在下面的代码片段中，位于PHPExcel_Style_Fill类内部，一个对象反复引用自己。

public function getHashCode() { //class PHPExcel_Style_Fill
    if ($this->_isSupervisor) { 
        var_dump($this === $this->getSharedComponent()); //Always true 200 times
        return $this->getSharedComponent()->getHashCode();
    }
    return md5(
          $this->getFillType()
        . $this->getRotation()
        . $this->getStartColor()->getHashCode()
        . $this->getEndColor()->getHashCode()
        . __CLASS__
    );
}

有没有解决这个问题的方法？我也接受任何关于如何将一个单元格的完整样式应用到另一个单元格的想法。

解决方案：

正如@MarkBaker在评论中所说，GitHub上的分支develop确实包含了修复该错误的补丁。

- sybear

你能不能把 $sheet->duplicateStyle() 移到 if ($category->hasChildren() 上面？基本上就是在 foreach 循环中，但不在条件语句内部。 - Anthony

请尝试从Github获取最新的develop分支代码，该分支已经实现了一些对duplicateStyle()方法的修复，这些修复将在下一个版本中发布。 - Mark Baker

不，它是标记为“develop”的分支。 - Mark Baker

好的，我会看一下。 - sybear

如果您正在使用xdebug，可以在php.ini文件中添加“xdebug.max_nesting_level=500”一行。请执行此操作并告诉我。 - Jigar Chaudhary

显示剩余5条评论

2个回答

1

在不了解该库的具体细节的情况下...

这样改变一下如何：

public function getHashCode() { //class PHPExcel_Style_Fill
    if ($this->_isSupervisor) {

转换为：

public function getHashCode() { //class PHPExcel_Style_Fill
    if ($this->_isSupervisor && ( $this != $this->getSharedComponent() ) ) {

如果在if语句后的哈希码逻辑不适用于_isSupervisor，则添加另一种逻辑语句并返回一个固定值，如下所示：

public function getHashCode() { //class PHPExcel_Style_Fill
    if ($this->_isSupervisor) { 
        if ( $this == $this->getSharedComponent() )
            return md5(0);

- ash

这可能是个好主意，我也试过了，但由于某些原因还有另一个递归循环。不过还是给你点赞尝试。实际上，如果我们像那样改变很多代码，那么这可能会起作用:D - sybear

如果是这种情况，那么_isSupervisor看起来应该与$this == $this->getSharedComponent()互斥。这可能是一个好地方去看看 - 这种关系是什么，它是否允许引用自身？也许代码中的错误就在于将其带到了那个位置。祝你好运。 - ash

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Adam Zielinski · Accepted Answer

编辑我已经提交了一个修复的拉取请求：https://github.com/PHPOffice/PHPExcel/pull/251

当您尝试将单元格的样式复制到相同的单元格时，就会出现这种情况；请看下面的示例：

$phpe = new PHPExcel();
$sheet = $phpe->createSheet();

$sheet->setCellValue('A1', 'hi there') ;
$sheet->setCellValue('A2', 'hi again') ;

$sheet->duplicateStyle($sheet->getStyle('A1'), 'A2');

$writer = new PHPExcel_Writer_Excel2007($phpe);
$writer->save('./test.xlsx');

它会正常工作。但是，如果我再添加一行像这样的内容：

$sheet->duplicateStyle($sheet->getStyle('A1'), 'A1');

然后，一旦调用save方法，就会开始无限递归。

要修复代码，您应该修改此部分：

$sheet->duplicateStyle((($level == 0) ?  $this->styles['category'] : $this->styles['subcategory']), Cell::NUMBER . $row);

要翻译为类似以下内容：

$style = ($level == 0) ?  $this->styles['category'] : $this->styles['subcategory'];
$targetCoords = Cell::NUMBER . $row;
if($style->getActiveCell() != $targetCoords) {
    $sheet->duplicateStyle($style, $targetCoords);
}