0

我正在尝试创建一种闪存消息类,该类将在第一次获取消息时“刷新”消息。因此,例如,如果从一个控制器操作转发到另一个(同一个 HTTP 请求),则不在下一个 HTTP 请求上

这是我的班级设计:

<?php
namespace MartynBiz;

/**
 * Flash messages. Slight variation in that this will store a message until it
 * is accessed - whether that is a next http request, or same request. Simply
 * when get method is called the message is wiped from session.
 * TODO move this to martynbiz\php-flash
 */
class Flash
{
    /**
     * Message storage
     *
     * @var ArrayObject
     */
    protected $storage;

    /**
     * @param string $storage Name to store messages in session
     */
    public function __construct($storage=null)
    {
        // if storage is not defined, create ArrayObject (not persistent)
        if (is_null($storage)) {
            $storage = new \ArrayObject();
        }

        $this->storage = $storage;
    }

    /**
     * Add flash message
     *
     * @param string $key The key to store the message under
     * @param string $message Message to show on next request
     */
    public function addMessage($key, $message)
    {
        // create entry in the session
        $this->storage[$key] = $message;
    }

    /**
     * Get flash messages, and reset storage
     * @return array Messages to show for current request
     */
    public function flushMessages()
    {
        $messages = $this->storage->getArrayCopy();

        // clear storage items
        foreach ($this->storage as $key => $value) {
            unset($this->storage[$key]);
        }

        return $messages;
    }
}

我编写了一些 PHPUnit 测试,也演示了如何使用 Flash 类:

<?php

// TODO test with an ArrayAccess/Object $storage passed in

use MartynBiz\Flash;
use Zend\Session\Container;

class FlashTest extends PHPUnit_Framework_TestCase
{
    protected $flash;

    public function testInstantiation()
    {
        $flash = new Flash();

        $this->assertTrue($flash instanceof Flash);
    }

    public function testGettingSetting()
    {
        $flash = new Flash();

        $flash->addMessage('key1', 'value1');
        $flash->addMessage('key2', 'value2');
        $flash->addMessage('key2', 'value3');

        $expected = array(
            'key1' => 'value1',
            'key2' => 'value3',
        );

        // assert first time to access messages
        $messages = $flash->flushMessages();
        $this->assertEquals($expected, $messages);

        // assert messages have been cleared
        $messages = $flash->flushMessages();
        $this->assertEquals(array(), $messages);
    }

    public function testCustomStorage()
    {
        $container = new Container('mycontainer');
        $flash = new Flash($container);

        $flash->addMessage('key1', 'value1');

        $expected = array(
            'key1' => 'value1',
        );

        // assert first time to access messages
        $messages = $flash->flushMessages();
        $this->assertEquals($expected, $messages);

        // assert messages have been cleared
        $messages = $flash->flushMessages();
        $this->assertEquals(array(), $messages);
    }
}

您可以看到我还传入了一个自定义 $storage(Zend 会话容器实例),这是我想要的一个选项。这些测试似乎通过了,但是我在其他测试中收到以下错误,我真的不明白问题是什么:

$ ./vendor/bin/phpunit tests/library/FlashTest.php 1
PHPUnit 4.8.21-4-g7f07877 by Sebastian Bergmann and contributors.

.E.

Time: 68 ms, Memory: 4.50Mb

There was 1 error:

1) FlashTest::testGettingSetting
MartynBiz\Flash::flushMessages(): ArrayIterator::next(): Array was modified outside object and internal position is no longer valid

/var/www/crsrc-slimmvc/app/library/MartynBiz/Flash/Flash.php:53
/var/www/crsrc-slimmvc/tests/library/FlashTest.php:33

FAILURES!
Tests: 3, Assertions: 3, Errors: 1.

我已经四处寻找这个错误并尝试了一些替代方法(例如 $this->storage->getIterator() ),但我仍然得到同样的错误。有什么想法我哪里出错了吗?我想我对 ArrayObject 有点陌生。

4

0 回答 0