Funzione non restituisce valore

[foreach]

Salve a tutti,

ho creato una funzione in php che mi permette di vedere se un indirizzo IPv4 fa parte di un determinato range:

function ipfaparte($ip, $range) {
list($rete, $bit) = explode('/', $range);
$n1 = substr(decbin(ip2long($ip)), 0, $bit);
$n2 = substr(decbin(ip2long($rete)), 0, $bit);
if($n1 == $n2) {
return(true);
} else {
return(false);
}
} 

dove per $ip gli scrivo "192.168.54.32" e per $range scrivo "192.168.32.0/12", ma quando provo a scrivere questo:

echo (ipfaparte("192.168.54.32", "192.168.32.0/12") ? "si" : "no"); 

non mi stampa niente.

Però ho notato che se io aggiungo un var_dump() all'interno della funzione, facendola diventare così:

function ipfaparte($ip, $range) {
list($rete, $bit) = explode('/', $range);
$n1 = substr(decbin(ip2long($ip)), 0, $bit);
$n2 = substr(decbin(ip2long($rete)), 0, $bit);
var_dump($ip, $range, $n1, $n2);
if($n1 == $n2) {
return(true);
} else {
return(false);
}
} 

mi fa vedere il dump delle variabili e il sì/no come risultato della funzione così:

string(10) "87.16.78.0"
string(2) "25"
string(25) "1010111000100000100111000"
string(25) "1010111000100000100111000"
si

(vabbè in questo risultato non ho messo i valori ip e range che ho scritto sopra)


Come posso risolvere questo?






Grazie,
foreach

[alemoppo]

A me scrive "si":

<?php
function ipfaparte($ip, $range) { 
list($rete, $bit) = explode('/', $range); 
$n1 = substr(decbin(ip2long($ip)), 0, $bit); 
$n2 = substr(decbin(ip2long($rete)), 0, $bit); 
if($n1 == $n2) { 
return(true); 
} else { 
return(false); 
} 
}
echo (ipfaparte("192.168.54.32", "192.168.32.0/12") ? "si" : "no");
?>

Non sono entrato nel merito della funzione, ma mi è balzato all'occhio che:

if($n1 == $n2) { 
return(true); 
} else { 
return(false); 
} 

Puoi scriverlo semplicemente:

return ($n1 == $n2); 

Ciao!

[karl94]

A parte quanto già scritto da Alemoppo, perché tutte quelle parentesi? Alcune non servono: return non è una funzione, ma una istruzione e non necessita di quelle parentesi. Lo stesso vale per echo.
Le parentesi possono sì servire quando si hanno operatori con diversa precedenza o per rendere più chiaro a chi legge un'espressione aritmetica o booleana, o spesso ancora per evidenziare le precedenze di alcuni operatori poco usati, come quelli bit a bit, per evitare di commettere errori.
Ma nei casi che ti ho scritto sopra non vedo perché inserirle.

[foreach]

Ho creato il file a.php con questo codice:

<?php
error_reporting(E_ALL);

function ipfaparte($ip, $range) {
list($rete, $bit) = explode('/', $range);
$n1 = substr(decbin(ip2long($ip)), 0, $bit);
$n2 = substr(decbin(ip2long($rete)), 0, $bit);
return($n1 == $n2);
}

echo(ipfaparte("192.168.54.32", "192.168.32.0/12") ? "si" : "no");

if(isset($_GET['dump'])) {
var_dump(ipfaparte('87.16.78.50', '87.16.78.0/25'));
}

?>

Allora innanzitutto vorrei sapere dove posso togliere le parentesi inutili, e poi ho notato che visitando il file a.php normalmente non mi stampa niente, ma visitandolo a.php?dump mi stampa sibool(true), il file è raggiungibile qui, ora ditemi voi se ho sbagliato qualcosa io o è il server (non AV) che fa brutti scherzi. Se proprio serve, posso anche farvi vedere phpinfo() del mio sito.




Grazie,
foreach

[karl94]

Sicuro non sia problema di cache? Da come lo descrivi sembra proprio così. Comunque le parentesi superflue sono quelle che ti ho indicato: dopo echo e dopo return.

[foreach]

Può essere, perché ho notato che visualizzandolo da un altro computer mi dice "si" chiamando il file a.php senza ?dump. Ho notato anche se vedo il codice sorgente di a.php in chrome, mi dà file vuoto. E' un po' strano secondo me.

Però ora quella funzione mi serve per un controllo serio, non per un semplice echo, e facendo una cosa tipo così:

if(ipfaparte("192.168.54.32", "192.168.32.0/12") || altri controlli) {
header("HTTP/1.1 303 See Other");
header('Location: errore.php');
die();
} 

e vedo che anche se l'indirizzo IP fa parte del range scritto, non mi manda all'errore e quindi continua lo script dando per buono una cosa che non è.

Può essere anche qui un problema di cache?



Grazie,
foreach

[karl94]

Credo proprio di sì, la cosa più semplice per verificarlo è provare usando la navigazione anonima.