Skip to main content
replaced http://pt.stackoverflow.com/ with https://pt.stackoverflow.com/
Fonte Link

O problema inicial é que você definiu duas vezes a propriedade password como já dito em outras respostas.

O problema inicial é que você definiu duas vezes a propriedade password como já dito em outras respostas.

removeu 1 caractere do conteúdo
Fonte Link
gmsantos
  • 17,6mil
  • 7
  • 51
  • 91
// Arquivo Users.php
class Users 
{
    public $name; 
    public $idade;
    public $email;
    private $senha;

    public function __construct($name, $idade, $email, $senha){ 
        $this->name = $name;
        $this->idade = $idade;
        $this->email = $email;
        $this->setPassword($senha);
        echo "O objeto foi contruido!";
    }

    public function setPassword($senha) {
        if (strlen($senha) > 8 and strlen($senha) < 13) {
            $this->senha = password_hash($senha, PASSWORD_DEFAULT);
        } else {
            thrownthrow new InvalidArgumentException(
                'Sua senha deve conter entre 8 e 13 caracteres'
            );
        }
    }
}

// Arquivo index.php
try {
    $pessoa = new Users("Flavio", 19, "[email protected]", "testando123");
    var_dump($pessoa);

    // Aqui vai dar erro
    $pessoa->setPassword("ba");
    var_dump($pessoa);

catch (InvalidArgumentException $e) {
    // Faz alguma magia negra.

    echo $e->getMessage();
}
// Arquivo Users.php
class Users 
{
    public $name; 
    public $idade;
    public $email;
    private $senha;

    public function __construct($name, $idade, $email, $senha){ 
        $this->name = $name;
        $this->idade = $idade;
        $this->email = $email;
        $this->setPassword($senha);
        echo "O objeto foi contruido!";
    }

    public function setPassword($senha) {
        if (strlen($senha) > 8 and strlen($senha) < 13) {
            $this->senha = password_hash($senha, PASSWORD_DEFAULT);
        } else {
            thrown new InvalidArgumentException(
                'Sua senha deve conter entre 8 e 13 caracteres'
            );
        }
    }
}

// Arquivo index.php
try {
    $pessoa = new Users("Flavio", 19, "[email protected]", "testando123");
    var_dump($pessoa);

    // Aqui vai dar erro
    $pessoa->setPassword("ba");
    var_dump($pessoa);

catch (InvalidArgumentException $e) {
    // Faz alguma magia negra.

    echo $e->getMessage();
}
// Arquivo Users.php
class Users 
{
    public $name; 
    public $idade;
    public $email;
    private $senha;

    public function __construct($name, $idade, $email, $senha){ 
        $this->name = $name;
        $this->idade = $idade;
        $this->email = $email;
        $this->setPassword($senha);
        echo "O objeto foi contruido!";
    }

    public function setPassword($senha) {
        if (strlen($senha) > 8 and strlen($senha) < 13) {
            $this->senha = password_hash($senha, PASSWORD_DEFAULT);
        } else {
            throw new InvalidArgumentException(
                'Sua senha deve conter entre 8 e 13 caracteres'
            );
        }
    }
}

// Arquivo index.php
try {
    $pessoa = new Users("Flavio", 19, "[email protected]", "testando123");
    var_dump($pessoa);

    // Aqui vai dar erro
    $pessoa->setPassword("ba");
    var_dump($pessoa);

catch (InvalidArgumentException $e) {
    // Faz alguma magia negra.

    echo $e->getMessage();
}
Fonte Link
gmsantos
  • 17,6mil
  • 7
  • 51
  • 91

O problema inicial é que você definiu duas vezes a propriedade password como já dito em outras respostas.

Como você está começando, gostaria de complementar com alguns pontos legais de boas práticas.

O primeiro é o uso do : no if e em outras estruturas. Apesar de funcionar, o foco do : é para utilizar em templates:

<?php if ($var instanceof Legume): ?>
    Pode ser uma batata!
<?php else: ?>
    Não é uma batata
<?php endif; ?>

Num arquivo onde só existe código, prefira usar as {}

if ($var instanceof Legume) {
    Pode ser uma batata!
} else {
    Não é uma batata
}

Outro ponto é a visibilidade dos métodos. O PHP usa function para definir os métodos e não é obrigatório definir a visibilidade. Caso nada seja definido o método é tratado como public. Esse comportamento vem do PHP 4, e hoje em dia, é melhor deixar explícito que algo é public.

Para ter uma ideia melhor de como escrever o código, você pode se basear no code style do PSR-2. Esse é o padrão adotado por vários projetos em PHP e isso garante que o código escrito é parecido visualmente, independente de quem escreveu o código.

Por fim, prefira disparar uma Exception ao invés de fazer um die e encerrar seu script todo.

Veja algumas referencias sobre Exceptions:

Por fim, se código pode chegar a ficar dessa forma:

// Arquivo Users.php
class Users 
{
    public $name; 
    public $idade;
    public $email;
    private $senha;

    public function __construct($name, $idade, $email, $senha){ 
        $this->name = $name;
        $this->idade = $idade;
        $this->email = $email;
        $this->setPassword($senha);
        echo "O objeto foi contruido!";
    }

    public function setPassword($senha) {
        if (strlen($senha) > 8 and strlen($senha) < 13) {
            $this->senha = password_hash($senha, PASSWORD_DEFAULT);
        } else {
            thrown new InvalidArgumentException(
                'Sua senha deve conter entre 8 e 13 caracteres'
            );
        }
    }
}

// Arquivo index.php
try {
    $pessoa = new Users("Flavio", 19, "[email protected]", "testando123");
    var_dump($pessoa);

    // Aqui vai dar erro
    $pessoa->setPassword("ba");
    var_dump($pessoa);

catch (InvalidArgumentException $e) {
    // Faz alguma magia negra.

    echo $e->getMessage();
}