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();
}