Fichtre, pas mal de choses à corriger (qui a dit tout ?)
Dans ta fonction "relance_id", tu as comme paramètre $row_ct, mais tu fais tes "if" sur $row_ct_env (que la fonction ne connaît pas, à moins que tu n'utilises des variables globales partout).
Tu n'affectes pas la valeur de retour de la fonction relance_id à une variable (donc elle ne sert à rien dans ta configuration).
Donc, pour rester proche de ce que tu fais
Code :
- while ($row_ct_emv = mysql_fetch_assoc($result_ct_emv)) {
- $country= relance_id($row_ct_emv['relances_id']);
- echo "<tr><td with=\"20px\"></td><td width=\"100px\">".$country."</td><td width=\"120px\"><b>".$row_ct_emv['planning_date']."</b></td></tr>";
- }
|
et ta fonction gagnerait en clarté avec l'utilisation d'un switch
Code :
- function relance_id($relance) {
- switch ($relance) {
- case 1:
- return 'france';
- break;//bon, le break doit être inutile, vu qu'on a un return...
- case 2:
- return 'amerique';
- break;//idem
- //etc.
- default:
- return 'norvège';
- break//idem
- }
- }
|
Cela dit, du moment que tu travailles avec une BDD, il pourrait être intéressant d'avoir une table pays_relance, ou un truc du genre, non ?
Message édité par deliriumtremens le 10-06-2009 à 17:03:12