ベストプラクティスMVCとPHP関数を持つOOP？

Question

OOPの初心者ですので、ベストプラクティスを理解しようとしています。このコードは、私が追加している既存のスクリプトに基づいています。この質問と

スレッドの大半は、ようにコーディングするポスターを伝える：

function ($arg1, $arg2){ 
//some code 
} 

およびコール：

function($a1, $a2); 

を私はOOPベースの機能（働き）を持っていますが、それはdoesnのかなりうまく見えて、私はそれを提案された方法として呼び出そうとすると、次のようになります。

配列から文字列への変換....オンラインで配列

ここで出力を収集し、私の（作業）関数です：これは、作業（醜い）です

public function getMail($type, $id = 0) { 
     $query = $this->db->query("SELECT * FROM km_mail WHERE id = '" . (int)$id . "' AND `type` = '" . $this->db->escape($type) . "'"); 

     foreach ($query->rows as $result) { 
      $mail_data[$result['title']] = $result['value']; 
     } 

    return $mail_data; 
} 

一部 - これは、要求されたデータベースの列を返します（ただし、間違って見える？）：

$this->model_setting_mail->getMail('order')['update_link']; 

私はそうのような列を要求しようとすると、文字列変換エラーに配列が発生します。

$this->model_setting_mail->getMail('order','update_link'); 

私の例では、$ idが渡されない限り、の順 = $ type、update_link = $ result ['value']と$ id = 0がデフォルトです。

Answer 1

最初に示す例は、関数から返された値から配列要素を選択する簡単な方法です。

$result = $this->model_setting_mail->getMail('order'); 
print $result['update_link']; 

2番目の例では、関数に2つの値を渡している。

$this->model_setting_mail->getMail('order')['update_link']; 

は同じです。

これらは全く異なっています。