Hợp lệ hóa __construct, đảm bảo vars của của đúng loại

Question

Viết đầu tiên thực PHP lớp của tôi và tình cờ gặp một vấn đề, sau đây là phương pháp __construct tôi:

public function __construct($foo,$bar) { 
     $this->foo = $foo; 
     $this->bar = $bar; 
} 

Cả $foo và $bar được yêu cầu, nếu không có họ, các phương pháp sẽ thất bại và cung cấp cho các lỗi php. OK khi chúng không được xác định khi đối tượng được khởi tạo:

$var = new Class(); 

Vì điều này sẽ gây ra lỗi (ví dụ: Lớp yêu cầu 2 thông số không được đặt). Nhưng nếu chúng được đặt nhưng không phải là loại dữ liệu chính xác như vậy:

$var = new Class('33','ddd'); 

Phương pháp của tôi vẫn sẽ thử và chạy sai biến, sau đó in ra lỗi.

Thực tiễn tốt nhất để xác thực những điều này là gì? Trong cấu trúc hoặc trong mỗi phương pháp?

Giải pháp của tôi mà tôi đang sử dụng ngay bây giờ làm việc, nhưng tôi không chắc chắn nếu đó là cách 'đúng':

//$foo needs to be a string with letters only 
//$bar needs to be an integer 
public function __construct($foo,$bar) { 
     $this->foo = $foo; 
     $this->bar = $bar; 
     if(!is_numeric($bar)){ 
     //print error msg 
     } 
     elseif(other validation case) 
     etc... 
} 

Các khái niệm về lập trình OO là khá mới mẻ với tôi, vì vậy liên kết bất kỳ tài liệu tham chiếu nào bạn đã sử dụng sẽ được đánh giá cao.

Answer 1

tôi có lẽ sẽ làm một cái gì đó như thế này để ngăn chặn lộn xộn bên trong ctor và cho phép các lớp để thiết lập chúng giá trị trong nội bộ:

class MyClass … 

    protected $_foo; 

    /** 
    * @param String $foo String with letters only 
    * @param Integer $bar Any Integer 
    * @return void 
    * @throws InvalidArgumentException when $foo is not letters only 
    * @throws InvalidArgumentException when $bar is not an Integer 
    */ 
    public function __construct($foo, $bar) 
    { 
     $this->_setFoo($foo); 
     $this->_setBar($bar) 
    } 

    /** 
    * @param String $foo String with letters only 
    * @return void 
    * @throws InvalidArgumentException when String is not letters only 
    */ 
    protected function _setFoo($foo) 
    { 
     if (FALSE === $this->_consistsOfLettersOnly($foo)) { 
      throw new InvalidArgumentException(
       '$foo should consists of letters only' 
      ); 
     } 
     $this->_foo = $foo; 
    } 

    … 

này có lợi thế thêm rằng nếu bạn cần phải công khai lộ setters sau đó, bạn chỉ cần thay đổi từ khóa hiển thị.

Việc xác thực thành phương pháp riêng của nó là không cần thiết, nhưng tôi nghĩ nó làm cho mã dễ đọc hơn. Nếu bạn tìm thấy một thuộc tính khác trong lớp đó cần xác nhận giống nhau, bạn cũng có thể sử dụng lại nó dễ dàng hơn.

Answer 2

Xác thực các loại trong hàm tạo (theo ví dụ của bạn) có vẻ là lựa chọn hợp lý đối với tôi, mặc dù hơi lạ nếu đối số là tùy chọn. Có lẽ bạn nên thiết lập giá trị mặc định an toàn nếu đây là trường hợp qua ...

public function __construct($foo=null, $bar=null)

Điều đó nói rằng, bạn có thể sử dụng type hinting cho các đối số trong PHP, nhưng tôi tin rằng điều này chỉ làm cho các mảng và các lớp học. (Không ngạc nhiên, vì không có thứ gì như một số nguyên hay chuỗi ký tự trong PHP.)

Như những người khác đã nói, bạn cũng nên đảm bảo rằng bạn xác thực trong bất kỳ trình cài đặt nào bạn có hoặc (tốt hơn) các setters từ bên trong constructor để đảm bảo không có sự trùng lặp mã.

Answer 3

Nếu cần $ foo và $ bar, hãy xác thực chúng trong __construct là nơi tốt nhất để làm như vậy (nếu không: sử dụng trình đặt và xác thực ở đó). Nếu bạn làm điều đó trong mọi phương pháp, bạn sẽ chỉ kết thúc với mã bị trùng lặp. Hãy tưởng tượng nếu một kiểu dữ liệu thay đổi, bạn phải thay đổi mọi phương thức ...

Nếu bạn muốn thực sự cầu kỳ, bạn có thể gọi is_numeric trước khi gán thanh $ đến $ this-> thanh này không cần thiết việc kiểm tra giá trị số không thành công.

Answer 4

Cách tốt nhất là sử dụng getter và setter - ví dụ:

private $_foo; 
private $_bar; 

public function __construct($foo,$bar) { 
    $this->setFoo($foo); 
    $this->setBar($bar); 
} 

public function setFoo($foo) { 
    // validate here 

    $this->_foo = $foo; 
} 

public function getFoo() { 
    return $this->_foo; 
} 

... 

sạch hơn nhiều theo cách này ...

Answer 5

vấn đề của bạn không bắt nguồn từ thực tế là bạn đang viết đối tượng định hướng mã - đó là vì PHP là loosely typed language. Điều này có nghĩa là các biến có thể chứa bất kỳ loại dữ liệu nào, từ int đến Object. Bạn có thể thực thi một số loại loại đúc như vậy:

$foo = (int)$foo; 
$bar = (str)$bar; 

Answer 6

Lập một chút về câu trả lời @ xil3 của tôi muốn nói tất cả Dan đang tìm kiếm/nhu cầu là:

private $_foo; 
private $_bar; 

public function __construct($foo,$bar) { 
    if(is_int($foo)) { 
    $this->setFoo($foo); 
    } 
    if(is_string($bar)) { 
    $this->setBar($bar); 
    } 
} 

public function setFoo(int $foo) { 
    $this->_foo = $foo; 
} 

public function setBar(string $bar) { 
    $this->_foo = $bar; 
} 

... 

Cho phép đối tượng của mình để được instantless instantless mà không gây tử vong, với các ràng buộc kiểu trên các setters.

Answer 7

Bạn không nên làm gì ngoài việc đặt giá trị trong hàm tạo. Nếu bạn cần phải xác nhận thông số nhà xây dựng như trong setters (có thể là tư nhân), hoặc sử dụng các tham số kiểu đặc biệt, vì vậy PHP sẽ xác nhận kiểu cho bạn, ví dụ:

__construct(SomeType $foo, AnotherType $bar); 

Ví dụ bạn có thể sử dụng SPL kiểu dữ liệu:

__construct(SplInt $integer, SplString $string); 

Read more about SPL data types