Laravel 集合 map 在同一个 class 中使用私有 function 而不是关闭

Question

我使用map function循环collections的每个object。关闭方法太长，我试图将其设为私有function。但我无法成功调用function。

这是我的示例代码。 （注意：function 测试在实际情况下可能会很长）

<?php

class cls
{
    private function test($a) {
        return ($a + 1);
    }

    public function run1() {
        return ($this->test(5));
    }

    public function run2() {
        $col = Collect([1,2,3]);
        return ($col->map($this->test()));
    }

    public function run3() {
        $col = Collect([1,2,3]);
        $mycls = $this;
        return ($col->map(function ($c) use ($mycls) {
            return($mycls->test($c));
        }));
    }
}

$c = new cls;
$c->run1(); # 6
$c->run2(); # Error: Too Few Argements
$c->run3(); # [2,3,4]

我使用 function: run1 来测试私有 function 是否可调用，但我在 function: run2 失败了。 虽然 function: run3 使代码变短了。 似乎有些多余。 我怎样才能使 run2 工作？

更新

我的Laravel版本是6.2

更新

我尝试了@xenooooo 的回答。 它适用于公共 function，但我使用私有方法得到了不同的错误代码。

Answer 1

简单的回答： $this->test()需要一个参数，而你只是没有传递任何东西给它。

您还可以修改run2/run3方法以执行以下操作：

public function run2() {
   $col = collect([1,2,3])->map(function ($value) {
      return $this->test($value);
   });
        
   return $col; //returns a collection of items
   //return $col->toArray(); //(extra option) returns an array of collected items
}

结果：

Illuminate\Support\Collection {#872 ▼ 
  #items: array:3 [▼
    0 => 2
    1 => 3
    2 => 4
  ]
  #escapeWhenCastingToString: false
}

您可以在此处阅读有关 collections 的更多信息： https://laravel.com/docs/9.x/collections#introduction

Answer 2

run2中的问题是您直接调用该方法并且需要传递 an 参数，因为test需要参数$a 。 要将该方法用作回调 function，您需要将其作为数组传递，第一个值是$this ，第二个是方法名称test ：

public function test($a) {
    return ($a + 1);
}

public function run2()
{
    $collection = collect([1,2,3]);

    $newCollection = $collection->map([$this,'test']);

    return $newCollection;
}

更新

它仅在您将调用的方法是公共方法时才有效，如果您使用私有或受保护方法则它将无效。 如果你使用 private 或 protected if 将抛出BadMethodCallException