Group by multiple columns in Laravel

前端 未结 4 685
小鲜肉
小鲜肉 2020-12-09 15:41

How to GROUP BY multiple column in Laravel? I tried this code:

$routes = DB::table(\'route\')
    ->groupBy(\'rte_origin\')
    ->groupBy(         


        
4条回答
  •  生来不讨喜
    2020-12-09 16:17

    It is incorrect to believe that the Database\Query\Builder::groupBy() method accepts arrays as arguments. Currently, it will only accept N number of string arguments.

    As of this writing, the current version of the Laravel Framework is: v4.2.4, and the code for the Database\Query\Builder::groupBy() method is as follows:

    /**
     * Add a "group by" clause to the query.
     *
     * @param  dynamic  $columns
     * @return \Illuminate\Database\Query\Builder|static
     */
    public function groupBy()
    {
        $this->groups = array_merge((array) $this->groups, func_get_args());
    
        return $this;
    }
    

    If you think about it, func_get_args() returns an array of all columns that might have been fed in as strings. Therefore, expected input of this function is:

    $builder->groupBy('column1', 'column2', ...);
    

    The resulting $this->groups property on the builder object should be an array of strings, like this:

    ['column1','column2']
    

    However, if we fed an array into the method above, like this:

    $builder->groupBy(['column1','column2']);
    

    the $this->groups property would end up with a nested array that looks like this:

    [['column1','column2']]
    

    The rest of the query builder framework expects the $builder->groups property to be a non-nested array of strings. Thus, when the framework tries to columnize and wrap table names in their proper escaped format (each database engine has a different table name escape operator), it tries to wrap an array instead of a string and you get your error.

    The offending error line is line 49 in Database\Grammar::wrap().

    If we were to modify the Database\Query\Builder::groupBy() method to make it accept arrays, we would rewrite it something like the following:

    public function groupBy()
    {
        $args = func_get_args();
        foreach($args AS $arg)
        {
            $arg = (is_array($arg)) ? $arg:[$arg];
            $this->groups = array_merge((array) $this->groups, $arg);
        }
    
        return $this;
    }
    

    This method accepts an array at any parameter index.

提交回复
热议问题