PHP 7.4.13 の環境にて https://zenn.dev/sogaoh/scraps/0898c2ef617dcf のような経緯でデバッグを行っているのですが、 題名の通りの問題が発生してしまっています。

すべては書けないのですが、渡ってきているデータは以下のようなものです

array(1) {
  ["v0"]=>
  array(22) {
    ["i_type"]=>
    int(1)
    ["reason"]=>
    string(1) "f"
    ["fk_m_id"]=>
    int(1)
    ["fk_p_id"]=>
    int(1)
    ["b_type"]=>
    int(2)
    ["b_date"]=>
    string(19) "2019-12-09 00:00:00"
    ["b_end_date"]=>
    string(19) "2019-12-09 00:00:00"
    ["net"]=>
    int(10000)
    ["gross"]=>
    int(100000)
    ["remarks"]=>
    string(1) "t"
    ["user_type"]=>
    int(1)
    ["created_at"]=>
    object(Illuminate\Support\Carbon)#4192 (16) {
        //略
    }
    ["updated_at"]=>
    object(Illuminate\Support\Carbon)#4196 (16) {
        //略
    }
    ["id"]=>
    int(1)
    ["array_p"]=>
    array(12) {
      //int 8項目
      //string 4項目
    }
    ["array_a"]=>
    array(12) {
      //int 8項目
      //string 4項目
    }
    //その他、いくつか "array_p" や "array_a" のような連想配列
  }
}

array(1) {
  ["v1"]=>
  array(12) {
    ["id"]=>
    int(1)
    ["fk_a_id"]=>
    int(1)
    ["fk_c_id"]=>
    int(1)
    ["name"]=>
    string(17) "Isabella McKenzie"
    ["fk_g_id"]=>
    int(1)
    ["fk_ct_id"]=>
    int(1)
    ["c_p"]=>
    string(12) "xxxxxxxxxxxx"
    ["r_c_type"]=>
    int(0)
    ["user_type"]=>
    int(3)
    ["user_id"]=>
    int(1)
    ["created_at"]=>
    string(19) "2020-12-10 20:40:58"
    ["updated_at"]=>
    string(19) "2020-12-10 20:40:58"
  }
}

array(1) {
  ["k0"]=>
  int(0)
}

array(1) {
  ["k1"]=>
  int(0)
}

「落ちる」のは↓だと思っています。 https://github.com/akanehara/ginq/blob/master/src/Ginq/JoinSelector/DelegateJoinSelector.php#L46

解決方法を考えたいです。

*OrElse 系で代替値の評価自体を遅延したいケースもあるはずなので、 $default に関数もとれるようにする。

/**
 * @param mixed|\Closure $default () -> mixed
 * @param callable|null      $predicate (v, k) -> bool
 * @return mixed
 */
public function firstOrElse($default, $predicate = null);

/**
 * @param int                   $index
 * @param mixed|\Closure $default (i) -> mixed
 * @return mixed
 */
public function getValueAtOrElse($index, $default);

It would be amazing to add a shorthand notation for closures:

$xs = Ginq::from(array(1,2,3))->where('x % 2 != 0')->select('x * x');

and/or

$xs = Ginq::from(array(1,2,3))
         ->where(['x' => 'x % 2 != 0'])->select(['v, k' => 'k+v']);

See

• http://symfony.com/doc/current/components/expression_language/syntax.html
• https://github.com/michelsalib/BCCEnumerableUtility#use-of-functions

ソース見て、なんとなくそんな気はしてたけど、 \Ginq::from([])->any() エラーになるね、これ。

...
#2 {main}
  thrown in D:\work\tests\php\Ginq\Predicate\PredicateParser.php on line 34

Fatal error: Uncaught exception 'InvalidArgumentException' with message ''predicate' callable expected, got NULL'
...

ちなみに期待した結果は、false

https://github.com/akanehara/ginq/tree/feature/ginq-context

feature/ginq-context ブランチの

• Ginq\Context
• Ginq\OrderedContext
• Ginq\GroupingContext

を、それぞれ

• Ginq\Stream
• Ginq\OrderedStream
• Ginq\GroupingStream

にリネームするのはどうかという提案。

Ginq::join() がルックアップテーブルを作るのは、Ginq::join() 実行時ではなく、イテレータの riwind() であるべき。クエリを組み立てるだけで inner の全件走査が動作するのはできる限り実行を遅延するGinqのポリシーに反する。join を selectMany の再利用で実装するのをやめ、専用のイテレータを作る。

selectManyのセレクタで最初に空のリストが返却されるとselectManyの結果も空になります。

空のリストが含まれる場合でも次のような場合は

$data = array(
    array(
        'values' => array(1, 2)
    ),
    array(
        'values' => array()
    ),
    array(
        'values' => array(3, 4)
    )
);

print_r(Ginq::from($data)->selectMany('[values]')->toList());

以下のように正常な結果が得られます。

Array
(
    [0] => 1
    [1] => 2
    [2] => 3
    [3] => 4
)

しかし、最初に空のリストがある場合

$data = array(
    array(
        'values' => array()
    ),
    array(
        'values' => array(1, 2)
    ),
    array(
        'values' => array(3, 4)
    )
);

print_r(Ginq::from($data)->selectMany('[values]')->toList());

以下のように結果は空になってしまいます。

Array
(
)

Related to #34

In the Symfony is the PropertyAccess component, which provides function to read from an object or array using a simple string notation. The big advantage of this component is that it can call the corresponding methods (getters, hassers, issers, __get, __call) to accessing private properties of an object.

For more information see http://symfony.com/doc/current/components/property_access/introduction.html

Ginqがイテレータのkeyを意識していなかった頃のjoinの引数は

join($outerKeySelector, $innerKeySelector, $joinSelector)

で、keyという言葉がは結合キー「だけ」を指していた。

ところが、現在は反復子のキーにも key という言葉を使っているために

join(
    $outerKeySelector, $innerKeySelector,
    $valueJoinSelector, $keyJoinSelector
)

となってしまっており、どれが結合キーを表すのかとてもわかりづらい。

• ２値をとるセレクタを JoinSelector と呼ぶことの妥当性
• 結合キーセレクタを表す変数名 outerKeySelector, innerKeySelector の妥当性

これらについて考えたい。

Your competitor here. :grinning:

I've run performance tests on YaLinqo, Ginq and Pinq. Whenever I use property access syntax, performance of Ginq queries drops 2-5 times. See gist with performance results (results from YaLinqo master branch). You should cache functions or something. Using "string lambdas" in YaLinqo, which rely on create_function, does decrease performance, but not so significantly (since I've added caching, inner queries stopped killing performance too).

Just curious, is using explicit iterator classes instead of generators a deliberate design decision, or do you just aim for compatibility with older PHP versions?

P. S. Thanks for competition. Ginq and Pinq forced me to finally update my library: add missing methods, improve performance, add integration with continuous integration and code quality tools, put reference documentation online etc. :grinning:

「何を以って重複とするか」を指定できると便利だな、という場面がありました。 例えば「ある親IDに属するメンバーからひとつのみ選択」という条件を適用する場合

$picked_parent_ids = [];
Ginq::from($members)
    ->where(function(Members $m) use (&$picked_parent_ids) {
        if (!in_array($m->parent_id, $picked_parent_ids)) {
            $picked_parent_ids[] = $m->parent_id;
            return true;
        } else {
            return false;
        }
    })

のようにフィルタリングする必要がありますが、これを以下のように表現できるとスッキリします。

Ginq::from($members)
    ->distinctBy('parent_id');

I want to use own Ginq class that adds a custom method, but it's impossible. It would be useful to have a environment object that can be passed to each iterator:

$ginq = new Environment(['Ging' => 'Vendor\Ginq', 'OrderingGinq' => ...]);
$ginq->from($data)->...;

It can be used pimple DI container - https://github.com/fabpot/Pimple.