BfwSql\Queries\Parts\ColumnList

Complexity

Total Complexity

8

Size/Duplication

Total Lines	76
Duplicated Lines	18.42 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getTable()	0	4	1
A	__invoke()	0	12	3
A	generate()	14	14	3

<?php

namespace BfwSql\Queries\Parts;

use \BfwSql\Queries\AbstractQuery;

class ColumnList extends AbstractList
{
    /**
     * @var \BfwSql\Queries\Parts\Table $table The table where are columns
     */
    protected $table;
    
    /**
     * {@inheritdoc}
     */
    protected $separator = ',';
    
    /**
     * {@inheritdoc}
     * 
     * @param \BfwSql\Queries\AbstractQuery $querySystem
     * @param \BfwSql\Queries\Parts\Table $table The table where are column
     */
    public function __construct(AbstractQuery $querySystem, Table $table)
    {
        parent::__construct($querySystem);
        $this->table = $table;
    }
    
    /**
     * Getter accessor to property table
     * 
     * @return \BfwSql\Queries\Parts\Table
     */
    public function getTable(): Table
    {
        return $this->table;
    }
    
    /**
     * Magic method __invoke, used when the user call object like a function
     * @link http://php.net/manual/en/language.oop5.magic.php#object.invoke
     * 
     * @param array $columns The list of columns to declare
     *  The key into the array is the shortcut of the column.
     *  The value is the column name.
     *  If no key is defined (so integer), the column will not have shortcut.
     * 
     * @return void
     */
    public function __invoke(array $columns)
    {
        foreach ($columns as $shortcut => $name) {
            if (is_int($shortcut)) {
                $column = new Column($this->table, $name);
            } else {
                $column = new Column($this->table, $name, $shortcut);
            }
            
            $this->list[] = $column;
        }
    }
    
    /**
     * {@inheritdoc}
     */
    public function generate(): string

This method seems to be duplicated in your project.

    {
        $sqlPart = '';
        
        foreach ($this->list as $index => $column) {
            if ($index > 0) {
                $sqlPart .= $this->separator;
            }
            
            $sqlPart .= $column->generate();
        }
        
        return $sqlPart;
    }
}