Pagina 1 di 2 1 2 ultimoultimo
Visualizzazione dei risultati da 1 a 10 su 12
  1. #1

    domanda su codice dubbio

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

    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)...

    codice:
    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 ?

  2. #2
    io la differenza non la trovo
    « Se nella prima mezz'ora non capisci chi è il pollo, allora il pollo sei tu. » [Thomas "Amarillo Slim" Preston, campione del mondo di poker]

  3. #3
    Nemmrno io
    http://www.anobii.com/isalreadyinuse

  4. #4
    Moderatore di CMS L'avatar di kalosjo
    Registrato dal
    Jul 2001
    residenza
    In culo alla luna
    Messaggi
    1,999
    if($controller = JRequest::getWord('controller')){

    assegna e verifica

    La verifica è sempre true!!

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

    verifica solo, senza assegnare

    Questa è quella giusta.....
    Scusate i puntini di sospensione...... La verità è che non ho argomenti....

  5. #5
    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
    « Se nella prima mezz'ora non capisci chi è il pollo, allora il pollo sei tu. » [Thomas "Amarillo Slim" Preston, campione del mondo di poker]

  6. #6
    Moderatore di CMS L'avatar di kalosjo
    Registrato dal
    Jul 2001
    residenza
    In culo alla luna
    Messaggi
    1,999
    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.......
    Scusate i puntini di sospensione...... La verità è che non ho argomenti....

  7. #7
    grazie mi sono tolto un dubbio....

  8. #8
    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:
    Codice PHP:
    $controller JRequest::getWord('controller');
    if (
    $controller){ 
    se avessi voluto effettuare un confronto io avrei scritto:
    Codice PHP:
    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

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

  10. #10
    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
    « Se nella prima mezz'ora non capisci chi è il pollo, allora il pollo sei tu. » [Thomas "Amarillo Slim" Preston, campione del mondo di poker]

Permessi di invio

  • Non puoi inserire discussioni
  • Non puoi inserire repliche
  • Non puoi inserire allegati
  • Non puoi modificare i tuoi messaggi
  •  
Powered by vBulletin® Version 4.2.1
Copyright © 2025 vBulletin Solutions, Inc. All rights reserved.