HoaOption

HoaOptionSnippet.php

<?php

namespace Hoa\Option;

interface Option
{
    public function isSome() : bool;
    public function isNone() : bool;
}

class OptString implements Option
{
    /**
     * @var string
     */
    protected $value;

    public function __construct($value)
    {
        $this->value = $value;
    }

    public function isSome() : bool
    {
        return null !== $this->value && '' !== $this->value;
    }

    public function isNone() : bool
    {
        return null === $this->value || '' === $this->value;
    }

    public function __toString() : string
    {
        return $this->value;
    }
}

class MyClass
{
    /**
     * @var OptString
     */
    protected $title;

    public function testSome()
    {
        $this->title = new OptString('Foo Bar Baz');

        if ($this->title->isSome()) {
            echo "I got a string ! ", "\n", $this->title, "\n\n";
        }
    }

    public function testNone()
    {
        $this->title = new OptString(null);

        if ($this->title->isNone()) {
            echo "Sorry I've nothing to say", "\n\n";
        }
    }
}

$sut = new MyClass();
$sut->testSome();
$sut->testNone();

• isNone should be defined as return !$this->isSome(), simply.
• Why checking against the empty string too? Only null.
• What about all the other methods? Should be a pain to implement each time :-/.