Posprint\Connectors\File

Complexity

Total Complexity

16

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	3
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	14	5
A	__destruct()	0	6	2
A	close()	0	7	2
A	write()	0	6	3
A	read()	0	12	4

<?php

namespace Posprint\Connectors;

/**
 * Class File
 * Create a binary file and writes the data line by line.
 * And it can also be used to provide direct connections to USB ports.
 *
 * @category  NFePHP
 * @package   Posprint
 * @copyright Copyright (c) 2016
 * @license   http://www.gnu.org/licenses/lesser.html LGPL v3
 * @author    Roberto L. Machado <linux.rlm at gmail dot com>
 * @link      http://github.com/nfephp-org/posprint for the canonical source repository
 */

use Posprint\Connectors\ConnectorInterface;
use RuntimeException;
use InvalidArgumentException;

class File implements ConnectorInterface
{
    /**
     * @var resource The file pointer to send data to.
     */
    protected $resource = false;

    /**
     * Construct new connector, given a filename
     * If created a binary file must be granted the necessary
     * permissions to create and write the system file
     *
     * @param string $filename
     */
    public function __construct($filename = null)
    {
        if (is_null($filename) || empty($filename)) {
            throw new InvalidArgumentException("A filepath must be passed!");
        }
        $command = 'rb+';
        if (! is_file($filename)) {
            $command = 'wb+';    
        }
        $this->resource = @fopen($filename, $command);
        if ($this->resource === false) {
            throw new RuntimeException("Failed to open the file. Check the permissions!");
        }
    }

    /**
     * Destruct conection closing the file
     */
    public function __destruct()
    {
        if ($this->resource != false) {
            $this->close();
        }
    }
      
    /**
     * Close file pointer
     */
    public function close()
    {
        if (is_resource($this->resource)) {
            @fclose($this->resource);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

            $this->resource = false;

It seems like false of type false is incompatible with the declared type resource of property $resource.

        }
    }

    /**
     * Write data to the file
     *
     * @param string $data
     */
    public function write($data = '')
    {
        if (is_resource($this->resource) && !empty($data)) {
            return fwrite($this->resource, $data);
        }
    }
    /**
     * Read some bytes from file
     * 
     * @param  int $len
     * @return string
     */
    public function read($len = null)
    {
        $data = '';
        if (!is_null($len) && is_numeric($len)) {
            $len = ceil($len);
            return fread($this->resource, $len);
        }
        while (!feof($this->resource)) {
            $data .= fread($this->resource, 4096);
        }
        return $data;
    }
}