PHP Aplicabilidade de métodos PHP

[Fertiin]

A fim de estudos, eu fiz o seguinte código que está funcionando perfeitamente, porém gostaria de saber se os métodos empregados nele foram aplicados da forma correta:

<?php



  class Carro {



  private $modelo;

  private $cor;

  private $ano;



  public function __construct(){

    if(isset($_GET['carro'])){

    $this->modelo = $_GET['carro'];

    $this->cor = $_GET['cor'];

    $this->ano = $_GET['ano'];

    }

  }



  public function setCor(){

    if($this->ano < 2015 && $this->cor === "Preto" ) {

    $this->cor = "Cor não disponível para o ano selecionado";

    }else if($this->ano >= 2015 && $this->cor === "Branco"){

    $this->cor = "Cor não disponível para o ano selecionado";

    }else {

    $this->cor;

    }



  	return $this->cor;

  }



  public function setAguardaURL(){

    if(isset($_GET['carro'])){

    return "Modelo: " . ($this->modelo) . "<br/> Cor: " . ($this->setCor()) . "<br/> Ano: " . ($this->ano);

    }

    }

  }



  $landRover = new Carro();

  echo $landRover->setAguardaURL();



?>

 

[makotsunami]

@Fertiin Não existe maneira "correta" de programar, mesmo sendo orientada a objetos, em projetos grandes você sempre vai se encontrar em situações onde você terá que resolver um problema de uma maneira que nem sempre é a mais certa. 

 

Mas um coisa é certa, eu nunca vi classes como a sua na minha vida e você provavelmente você nunca vai ver isso caso um dia trabalhe na área de desenvolvimento, normalmente os valores são resgatados antes de se construir a classe.

 

Normalmente teríamos algo assim:

<?php

$cor = isset($_GET['cor']) ? $_GET['cor'] : false;
$ano = isset($_GET['ano']) ? $_GET['ano'] : false;
$modelo = isset($_GET['modelo']) ? $_GET['modelo'] : false;

$landRover = new Carro($cor, $ano, $modelo);

public function __construct($cor, $ano, $modelo) {
	if ($cor && $ano && $modelo) {
		$this->ano = $ano;
		$this->cor = $cor;
		$this->modelo = $modelo;
	}
}

 

Uma das coisas que eu acho mais legais em Java do que em PHP é que em Java podemos ter vários construtores, então não precisaríamos fazer essas verificações chatas dentro do método