domanda su codice dubbio

[cpsio]

ho trovato questo codice relativo all'inclusione di file all'interno del codice:

if($controller = JRequest::getWord('controller')){
	$path = JPATH_COMPONENT . DS . 'controllers' . DS . $controller . '.php';
	if(file_exists($path)){
		require_once $path;
	} else {
		$controller = '';
	}
}

secondo me il codice corretto è il seguente (vedi if)...

if($controller == JRequest::getWord('controller')){
	$path = JPATH_COMPONENT . DS . 'controllers' . DS . $controller . '.php';
	if(file_exists($path)){
		require_once $path;
	} else {
		$controller = '';
	}
}

che ne pensate ?

[king size slim]

io la differenza non la trovo

[isAlreadyInUse]

Nemmrno io

[kalosjo]

if($controller = JRequest::getWord('controller')){

assegna e verifica

La verifica è sempre true!!

if($controller == JRequest::getWord('controller')){

verifica solo, senza assegnare

Questa è quella giusta.....

[king size slim]

Originariamente inviato da kalosjo
if($controller = JRequest::getWord('controller')){

assegna e verifica

La verifica è sempre true!!

if($controller == JRequest::getWord('controller')){

verifica solo, senza assegnare

Questa è quella giusta.....

già non ci ho fatto caso...

mi sfugge sempre... a volte anche nel mio codice

[kalosjo]

Originariamente inviato da king size slim
già non ci ho fatto caso...

mi sfugge sempre... a volte anche nel mio codice

Hai voglia quante volte a tutti.......

[cpsio]

grazie mi sono tolto un dubbio....

[MacApp]

Nell'originale dentro il primo "if" viene effettuato un assegnamento, nella versione modificata un confronto.

Per sapere quale sia giusto, bisgnerebbe conoscere cosa faccia la JRequest::getWord, e cosa volesse fare il programmatore.

In ogni caso sia che si volesse fare un assegnamento od un confronto lo sviluppatore ha sviluppato male.
Infatti se avessi voluto fare un assegnamento io avrei scritto:

$controller = JRequest::getWord('controller');
if ($controller){ 

se avessi voluto effettuare un confronto io avrei scritto:

if (JRequest::getWord('controller') == $controller){ 

in modo da enfatizzare al massimo, sia l'assegnamento che il confronto.

Invece così com'è, l'ambiguità è totale.

si veda anche un mio articolo (sito esterno):
Assegnamento per sbaglio

[cpsio]

sono d'accordo con te, ma sono anche convinto che un metodo vale l'altro...quando funziona

[king size slim]

Originariamente inviato da cpsio
sono d'accordo con te, ma sono anche convinto che un metodo vale l'altro...quando funziona

io non ne sarei così sicuro...

prova a scrivere il codice di un sito con un metodo qualunque purchè funzioni, codice sporco, mal commentato etc...

dopo 3 anni il cliente ti chiede una modifica e tu che fai??? ti spari in testa!

bisogna sempre cercare di scrivere il codice nel modo migliore possibile, prima o poi tornerà comodo