引数のScopeについて

Question

Githubにあるサンプルを見ながら掲示板の開発を行っております。
不明点があり、どなたかご教示頂ければ幸いです。

下記のように、作者はController→Serviceクラス→repositoryクラスへ処理を流しておられます。
質問したい部分は、controllerでdestroyの処理を行う際、リポジトリクラスのdeleteCommentを呼び出し→
deleteCommentは更に同リポジトリクラスのgetCommentByIdを呼び出し、となっていますが、この引数である"id"は、どうしてdeleteCommentの中のgetCommentByIdでも使うことができるのか、ということです。
私の考えではdeleteComment→getCommentByIdを呼び出した際、引数には何も渡って来ず、Call to a member function delete() on nullとなるのではと考えております。

よろしくお願いいたします。

https://github.com/owl/owl

<?php namespace Owl\Repositories\Eloquent;

use Owl\Repositories\CommentRepositoryInterface;
use Owl\Repositories\Eloquent\Models\Comment;

class CommentRepository implements CommentRepositoryInterface
{
    protected $comment;

    public function __construct(Comment $comment)
    {
        $this->comment = $comment;
    }

    /**
     * Delete a comment.
     *
     * @param $id int
     * @return boolean
     */
    public function deleteComment($id)
    {
        $comment = $this->getCommentById($id);
        $comment->delete();

        return $comment;
    }

    /**
     * Get a comment by comment id.
     *
     * @param $id int
     * @return Illuminate\Database\Eloquent\Model
     */
    public function getCommentById($id)
    {
        $comment = $this->comment->with('user')->find($id);
        if (!empty($comment)) {
            return $comment;
        }
        return false;
    }
}

<?php namespace Owl\Http\Controllers;

use Illuminate\Contracts\Events\Dispatcher;
use Owl\Services\UserService;
use Owl\Services\ItemService;
use Owl\Services\CommentService;
use Owl\Events\Item\CommentEvent;

class CommentController extends Controller
{
    protected $userService;
    protected $itemService;
    protected $commentService;
    private $status = 400;

    public function __construct(
        UserService $userService,
        ItemService $itemService,
        CommentService $commentService
    ) {
        $this->userService = $userService;
        $this->itemService = $itemService;
        $this->commentService = $commentService;
    }



    public function destroy()
    {
        if ($comment = $this->commentService->getCommentById(\Input::get('id'))) {
            $this->commentService->deleteComment($comment->id);
            $this->status = 200;
        }
        return  \Response::make("", $this->status);
    }
}

<?php namespace Owl\Services;

use Owl\Repositories\CommentRepositoryInterface;

class CommentService extends Service
{
    protected $commentRepo;

    public function __construct(
        CommentRepositoryInterface $commentRepo
    ) {
        $this->commentRepo = $commentRepo;
    }

    /**
     * Delete a comment.
     *
     * @param $id int
     * @return boolean
     */
    public function deleteComment($id)
    {
        return $this->commentRepo->deleteComment($id);
    }

    /**
     * Get a comment by comment id.
     *
     * @param $id int
     * @return Illuminate\Database\Eloquent\Model
     */
    public function getCommentById($id)
    {
        return $this->commentRepo->getCommentById($id);
    }
}

Comments

[motuo]

「私の考えではdeleteComment→getCommentByIdを呼び出した際、引数には何も渡って来ず」の考えの根拠を詳しく教えて頂けますか？プログラム内のコメントなどでその根拠を示して頂けると分かり易いと思います。

Answer 1

ちょっと何を言いたいのか、私の方で理解が追いつかなかったのですが

        if ($comment = $this->commentService->getCommentById(\Input::get('id'))) {
            $this->commentService->deleteComment($comment->id);
            $this->status = 200;
        }

この処理ですが、まずCommentServiceからコメント情報を取得して、$comment変数に入れています。
ここで、コメントが取得できずnullの場合はこのifの中を通ることはありません。($comment変数の中身はnullとなるのでif文はfalseになります)

コメント情報が取得できた場合は、実際に削除処理を行います。
CommentServiceのdeleteCommentに取得できたコメント情報のidを渡してDBからデータを削除します。(ここでは$comment->idとなっていますが、\Input::get('id')でも良いでしょう。使わなかった理由はメソッド呼び出しが発生するからだと思います。)

Comments

[AlwaysOreHungry]

ご回答ありがとうございました。fagaiさんのアンサーを読んでいるうちに、色々と私自身の勘違いに気付けました。