Si la déclaration ne semble pas fonctionner correctement

Question

Je peux me tromper, mais jetez un coup d'œil. Si je code en dur la logique, cela fonctionne mais pas si j'essaie de l'utiliser comme variable.

if($range <= 50) { 
    $operator = "<="; 
} else { 
    $operator = ">="; 
} 

foreach($cursor as $s) { 
    $data = round($this->distance($zip_lat, $zip_lon, $s["lat"],$s["lon"]), 2); 

    if ($data .$operator. $range) { 
     $zipcodes[] = "$s[zipcode]"; 
    } 
}    

Je veux dire, je pourrais ajouter if/else à l'intérieur du foreach mais était pas sûr si elle ajoute une « surcharge ».

Answer 1

essayer:

if ($range <= 50 ? $data <= $range : $data >= $range) { 

} 

ou utiliser un eval()

Answer 2

Vous semblez manquer une fermeture}

Answer 3

Je pense que vous allez avoir besoin de le faire:

foreach ($cursor as $s) { 
    $data = round($this->distance($zip_lat, $zip_lon, $s["lat"],$s["lon"]), 2); 
    if ($range <= 50) { 
     if ($data <= $range) { 
      $zipcodes[] = "$s[zipcode]"; 
     } 
    } else { 
     if ($data >= $range) { 
      $zipcodes[] = "$s[zipcode]"; 
     } 
    } 
} 

Answer 4

if ($data .$operator. $range) 

Est toujours vrai, parce que utilisez-le est une chaîne, pas un null.

Vous pouvez trouver problème en utilisant ce code simple:

$data="0"; 
$operator=">="; 
$range="1"; 

if ($data .$operator. $range) { 
     echo $data .$operator. $range . " is true !"; 
}     

Answer 5

Vous ne pouvez pas avoir une variable représentant un opérateur d'évaluation. Vous devrez changer votre code à quelque chose comme:

foreach ($cursor as $s) { 
    $data = round($this->distance($zip_lat, $zip_lon, $s["lat"],$s["lon"]), 2); 
    if ($range <= 50 && $data <= $range) { 
     $zipcodes[] = $s['zipcode']; 
    } else if ($data >= $range) { 
     $zipcodes[] = $s['zipcode']; 
    } 
} 

Answer 6

Les « points » ne concaténer « string » régulière - vous ne pouvez pas les attendre (injecté sous forme de chaînes) à se comporter comme des opérateurs réguliers, « vrais » .

Pensez-y: si $data = 'data1' et $range = 50 votre instruction if devient:

if ('data1<=50') qui probablement évaluer à vrai ou faux, tout comme if ('yournamehere') ou if('randommumbojumbo')

Answer 7

Je suis très suspect que le cas est tout simplement évaluer la chaîne $ date. $ operator. $ range (qui retournera toujours true), car tout ce que vous faites est de concaténer l'opérateur et les opérandes ensemble.

En tant que tel, vous devrez peut-être eval (canard et couvrir les gens, le canard et la couverture) le contenu de l'if.