Regexp per rilevare il passaggio del tempo di chiamata per riferimento nel codice sorgente PHP

Question

Sto cercando un'espressione regolare che identifichi accuratamente qualsiasi passaggio di tempo di chiamata PHP mediante riferimenti nel codice sorgente per facilitare la migrazione a PHP 5.3.

Attualmente, ho [^=&]\s*&\s*\$, ma questo non esclude i casi di assegnazione ($var = &$othervar;).

Questa espressione regolare deve essere compatibile con eclipse (mi spiace, non sono sicuro di quale sia l'analisi di regexp eclipse).

Edit: Questo è un po 'più vicino (anche se un po' di un hack): (?<!([&=]\s{0,15}))&\s*\$

Answer 1

Per questo è possibile utilizzare phpcs. Ha un rule to detect Call Time Pass by References:

Assicura che le variabili non vengano passate per riferimento quando si chiama una funzione.

C'è anche un plugin to integrate phpcs into Eclipse

Generazione di insiemi di regole per PHPCS (e PMD) è facile con questo generatore di linea:

• http://edorian.github.io/php-coding-standard-generator/#phpcs

Answer 2

Non è possibile ottenere quelli con espressioni regolari. Utilizzare invece lo . Sarà necessario cercare '&' dove il prossimo '(' a sinistra (risolvere parentesi mentre si cammina lì) è preceduto da T_STRING ma non da T_FUNCTION.

$tokens = new TokenStream($source); 
foreach ($tokens as $i => $token) { 
    if ($token->is(T_AMP)) { 
     while ($i--) { 
      if ($tokens[$i]->is(T_CLOSE_ROUND, T_CLOSE_SQUARE, T_CLOSE_CURLY)) { 
       $i = $tokens->complementaryBracket($i); 
      } elseif ($tokens[$i]->is(T_OPEN_ROUND)) { 
       if ((($tokens[--$i]->is(T_WHITESPACE) && $tokens[--$i]->is(T_STRING)) 
        || $tokens[$i]->is(T_STRING)) 
        && !$tokens[--$i]->is(T_WHITESPACE) 
        && !$tokens[--$i]->is(T_FUNCTION) 
       ) { 
        throw new Exception('Call-time pass by reference'); 
       } 
       break; 
      } 
     } 
    } 
} 

Questo utilizza my TokenStream wrapper. Con l'output nativo diventerà un po 'più difficile;)

Answer 3

php -l (php-linter) trova errori di riferimento pass-by-reference, ho usato

find -name '*.php' -exec php -l '{}' \; >/dev/null 

in linux

Answer 4

^(?!^.*(function|foreach|array)).*\(.*\&\$.*\) 

Questo dovrebbe aiutare.

Answer 5

È possibile utilizzare questo modello:

/(->|::|call_user_func|call_user_func_callable).*\(.*\&\$/ 

abbinerà le seguenti stringhe:

'->($arg1, &$arg2)' 
'->(&$arg1, $arg2)' 
'::($arg1, &$arg2)' 
'::(&$arg1, $arg2)' 
'call_user_func($callback, &$arg2)' 
'$callback, &$arg2)' 
'call_user_func_callable $callback, &$param_arr)' 

In caso di call_user_func_callable non è necessario verificare se la matrice di parametri contiene i riferimenti. Il passaggio dei riferimenti all'interno di un array non è considerato tempo di chiamata per riferimento e assolutamente soddisfacente.