Codeigniter SQLでWHERE句の条件について

Question

お世話になります。
CodeigniterでDBの接続を行っているのですが、
同じSQLで、1行だけ取得したい時と、全件取得したい時、それぞれメソッドを用意するのでしょうか？
DBへの接続と、データの取得はできています。
MVC的な考え方がまだ分かっていません。

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

###該当のソースコード

class Test_model extends CI_Model {

    function __construct() {
         parent::__construct();
         $this->load->database('default', FALSE, TRUE);
    }


    //一行だけ取得したい時
    function getTest($data) {
        $this->db->select('*');
        $this->db->from ('test');
        $this->db->order_by('test_sort', 'ASC');
        $this->db->limit(1);
        $query = $this->db->get();

        return $query->result_array();
    }

  //全件取得したい時
    function getAll($data) {
        $this->db->select('*');
        $this->db->from ('test');
        $this->db->order_by('test_sort', 'ASC');
        $query = $this->db->get();

        return $query->result_array();
    }

}

###試したこと

  //同じqueryを省略したいため、結果を配列に入れてみました。
    function getAll($data) {
        $this->db->select('*');
        $this->db->from ('test');
        $this->db->order_by('test_sort', 'ASC');
        $query = $this->db->get();

        $query_data['first_row'] = $query->first_row();
        $query_data['all'] = $query->result_array();

        return $query_data;
  }

###補足情報(言語/FW/ツール等のバージョンなど)
php5.4
Codeigniter3.1.2
Postgresql

Answer 1

ちょっとエスパーな回答になりますが、こんなことをイメージしてます？

CodeIgniter の Model を一工夫して使いやすくする方法

Modelを拡張する場合は、メソッドの分割は一般的になるかと思います。

【MY_Model extends CI_Model】で検索するといろいろ出てきます。
参考にされては？

Comments

[退会済みユーザー]

違ったようですね＾＾；

[pluto22]

言葉足らずで申し訳ございません！
同じテーブルに何度も問い合わせて良いのか分からず悩んでおりました。

[退会済みユーザー]

全件取得が前提ならプロパティとして全件データを持っておき、全件返却するメソッドと1件返却するメソッドをモデル内に作成してやれば良いです。

[pluto22]

確認が遅くなってしまいすみません、ご回答ありがとうございます！
プロパティとして持つという事は、
function __construct() {}内に共通処理として全件データを取得するSQLを置いておくと言う認識であっていますでしょうか？
お時間あるときで構いませんので教えて頂ければ幸いです。

[退会済みユーザー]

CIのお作法的にどのような実装が正しいかアドバイスできませんが、書かれたやり方でも実装できます。
ただ、あんまり悩むとこでもないと思いますよ。自身ができるところで実装してしまえば良いです。
正直それで悩んで、前に進めないほうが時間の無駄です。

[pluto22]

ありがとうございます！
確かに、ここで悩みすぎて他が進まなくなったら元も子もないですよね。
共通処理に置いて、各メソッドを作成して進めていこうと思います。ありがとうございました。

Answer 2

状況がよくわからないのですが、ORDER BYしたうえでLIMIT 1を指定して
全件取得すれば1行のみ取得になるのでは？
取得したけど参照しないというのはあまり効率がいいとは言えません

Comments

[pluto22]

説明が足らず申し訳ございませんでした。
ORDER BYが抜けていたため、質問文のSQLも編集いたします。

あるコントローラーからは新着の1件のみ、他のコントローラーからは全件取得したい状況です。
同じテーブルに何度も問い合わせるよりは、まとめた方がいいのかと悩んでおります。