SQL specified by Sql annotation/attribute

[koriym]

• インジェクションされる変数名と同じプロパティ名の@SqlがSQL文の特定に使われます。
• callableの代わりにInvokeInterfaceインターフェイスを使います。

PHP 7.x

class TodoRepository
{
    /**
     * @Sql("todo_insert")
     */
    public $todoCreate;

    /**
     * @Sql("todo_item_by_id")
     */
    public $todoItem;

    /**
     * @Sql("todo_list")
     */
    public $todoList;

    public function __construct(
        InvokeInterface $todoCreate,
        RowInterface $todoItem,
        RowListInterface $todoList
    ){
        $this->todoCreate = $todoCreate;
        $this->todoItem = $todoItem;
        $this->todoList = $todoList;
    }
}

$todoCreateにはtodo_insert.sqlSQL実行オブジェクトがインジェクトされます。

PHP8.x

<?php

declare(strict_types=1);

namespace Ray\Query;

use Ray\Query\Annotation\Sql;

class TodoRepository
{
    public function __construct(
        #[Sql('todo_insert')] public readonly InvokeInterface $todoCreate,
        #[Sql('todo_item_by_id')]  public readonly RowInterface $todoItem,
        #[Sql('todo_list')]  public readonly RowListInterface $todoList
    ){}
}

現状の@Namedの煩雑さと開発時の実行速度を改善します。

[jingu]

わかりやすく開発環境の速度向上も👍です。

[yuki777]

todo_insert.sqlを指定するのであれば、
@Sql("todo_insert.sql") や、
@Sql("path/to/todo_insert.sql") などはどうでしょうか？

[koriym]

拡張子をつけていない（＝ファイル指定ではなくIDの指定にしているのは）
sql以外のものに差し替える事ができることと、他のライブラリの整合性もあります。

• https://github.com/koriym/Koriym.QueryLocator
• https://github.com/ray-di/Ray.MediaQuery

[koriym]

そういう意味では@Sqlを@QueryIdにするのがいいのでしょうか？

[koriym]

ping @jingu @yuki777

[koriym]

Gentle reminder @jingu @yuki777