Modelクラス（MVC）は静的メソッドまたはインスタンスメソッドを使用する必要がありますか？

Question

MVCフレームワークの場合、静的メソッドまたはインスタンスメソッドを使用する必要がありますか？

Usersクラスと、クラスを返すメソッドgetUserById()を想定します。どれが良い選択ですか？

Users users = new Users(); 
User ret = users.getUserById(123); 

または

User ret = Users.getUserById(123); 

より良い選択である1 、クラスUsersにはインスタンス変数がないと仮定？

Answer 1

私はインスタンス変数の方に傾いています。単にテストを書くのが簡単になるからです。さらに、現在の多くのサーバー技術（Spring、JavaEEなど）は、Bean /リソースの注入を非常にうまくサポートしています。静的メソッドではなく、これをサポートします。

Answer 2

UserクラスとProductクラスがあり、そこにオブジェクトIDを持つオブジェクトがある場合は、 'User'と 'Category'に 'getById'メソッドを1つ拡張することをお勧めします。実行される$ idを受け取りました。

このようにして、2つの異なるタイプのオブジェクトで同じ方法を使用できます。

私は、この例では、いくつかの理にかなって願っています：

ふてくされ

class User extends SuperClass { 
    public function getTableName() { 
     return 'table_name_for_user'; 
    } 
    public function getPK() { 
     return 'primary_key_for_user'; 
    } 
} 

class Category extends SuperClass { 
    public function getTableName() { 
     return 'table_name_for_category'; 
    } 
    public function getPK() { 
     return 'primary_key_for_category'; 
    } 
} 

class SuperClass { 
    public function getById($id) { 
     $query = $this->db->query("SELECT * FROM " . $this->getTableName() . " WHERE " . $this->getPK() . " = $id"); 
     return $query->result(); 
    } 
} 

Answer 3

いいえ。実際には、DAO（Data Access Object）パターンを調べる必要があります。

モデルクラス自体は、1つのロジックインスタンスから別のロジックインスタンスに情報を転送することのみを担当し、geterメソッドとsetterメソッドのみを含む必要があります。

public class BookDAO { 

    private PreparedStatement saveStmt; 
    private PreparedStatement loadStmt; 

    public DBBookDAO(String url, String user, String pw) { 
    Connection con = DriverManager.getConnection(url, user, pw); 
    saveStmt = con.prepareStatement("INSERT INTO books(isbn, title, author) " 
            +"VALUES (?, ?, ?)"); 
    loadStmt = con.prepareStatement("SELECT isbn, title, author FROM books " 
            +"WHERE isbn = ?"); 
    } 

    public Book loadBook(String isbn) { 
    Book b = new Book(); 
    loadStmt.setString(1, isbn); 
    ResultSet result = loadStmt.executeQuery(); 
    if (!result.next()) return null; 

    b.setIsbn(result.getString("isbn")); 
    b.setTitle(result.getString("title")); 
    b.setAuthor(result.getString("author")); 
    return b; 
    } 

    public void saveBook(Book b) { 
    saveStmt.setString(1, b.getIsbn()); 
    saveStmt.setString(2, b.getTitle()); 
    saveStmt.setString(3, b.getAuthor()); 
    saveStmt.executeUpdate(); 
    } 
}