PSR対応

[MugeSo]

namespaceの使い方などPSR対応してほしいです。

[akanehara]

Ginqオブジェクトがトップレベルでないほうがよい、という意味でしょうか？ あるいは、実装コード全体についてPSR準拠が弱い、ということでしょうか？ よろしくお願いいたします。

[MugeSo]

Ginqオブジェクトがトップレベルでないほうがよい、という意味でしょうか？

はい、そうです。PSR-0にしたがった名前空間は <ベンダー名>(<名前空間>)*<クラス名> という構造になります。

[akanehara]

なるほど、ありがとうございます。 現在のGinqの実装を Ginq\GinqContext （名前は暫定）に追いやって、フォーマルには

use Ginq\GinqContext as Ginq;

で使うことを推奨し、いっぽう、名前空間を使わないようなカジュアルなPHPプログラムでも気軽に使ってもらえるようにトップレベルのGinqも残しておく、というのはどうでしょうか。

つまりこんな感じです。ブランチ切ってみました。 https://github.com/akanehara/ginq/tree/feature/ginq-context/src/Ginq

[tanakahisateru]

\<Vendor Name>\(<Namespace>\)*<Class Name> なので良いと思います。

ただ、use Ginq\GinqContext as Ginq; の使い方では、Composerなどのクラスローダ使わないとautoloadが効かないことになってしまいます。

autoloadの実装を \Ginq の中に入れたPHPスクリプト (たとえばsrc/Ginq/autoload.php) に移し、それをユーザが任意で require_once できるようにしたらどうかと思いました。「内部のautoloadは欲しい。Composerは使わない。Ginqクラスが欲しいわけじゃない」という変わった人がいたらそれで対応できるので。

[akanehara]

こんな感じでどうでしょう。 https://github.com/akanehara/ginq/tree/feature/ginq-context

ついでに Ginq\GinqContext を Ginq\Context に変えました。

[tanakahisateru]

違うところで __DIR__ を使っちゃってたのでいっそこうしてしまうのはどうですか:

- require_once dirname(__FILE__) . "/Ginq/autoload.php";
+ require_once __DIR__ . "/Ginq/autoload.php";

[akanehara]

:+1:

[akanehara]

feature/ginq-context ブランチを push しました https://github.com/akanehara/ginq/tree/feature/ginq-context

32 も解決してから devel および master にマージしたいので、ご意見ください。