Discussion :

[Arvi89]

Bonjour,
j'ai fait une application grace à gtk pour générer un fichier de configuration pour un routeur CISCO.

Seulement, je voulais vérifier que les champs IP soient comprit entre 0 et 255.
J'ai d'abord testé dans le programme(ip[0] une variable gchar) :

if (atoi(ip[0]) > 0 && atoi(ip[0]) < 255)
{
}
else
{
ip[0] = "255";
}

Ca fonctionne. Seulement, j'ai 48 champs à vérifier, je ne vais pas tous les faire les uns après les autres, et c'est là que je bloque

En effet, j'ai essayé de faire une fonction ou je passe ip[0] en argument, mais ça me met des trucs du genre pointeur error, et sinon qq chose comme ça :
ip[0] = verif(atoi(ip[0]))
Où la fonction verif fait un return.
Seulement, il faudrait remettre en chaîne de caractère, et je ne suis pas sur que ça marche même comme ça.

Si quelqu'un voit ce que je veux dire, et qu'il peut m'aider, merci ^^

[Fredy Kruger]

Salut,

et comment est la declaration des champs ip ?
L'instruction ip[o] = "255" est un peu suspecte.

Fredy

[Copros]

Salut,
déjà un guchar serait plus approprié qu'un gchar me semble-t-il, afin de coder des valeurs de 0 à 255.
Ensuite ton utilisation de atoi() est mauvaise : elle est censée convertir une chaine de caractères (char* ou gchar*) en int. Par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
char* s = "25";
int n = atoi("25") /* n vaut 25 */

Or tu manipules des caractères isolés. Tu peux manipuler directement les valeurs des caractères, par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
if(ip[0] >= 255)
  ip[0] = 255

Cerise sur le gâteau : le domaine des unsigned char ou guchar s'étend pile-poil de 0 à 255. Ta vérification est donc inutile vu que le seul fait que ta variable soit déclarée guchar t'assure qu'elle est comprise entre 0 et 255 (inclus).
De manière plus générale, la GLib fournit une macro CLAMP qui permet de vérifier qu'une variable est bien comprise entre 2 bornes.

[gege2061]

Cerise sur le gâteau : le domaine des unsigned char ou guchar s'étend pile-poil de 0 à 255.

Pas forcement : http://emmanuel-delahaye.developpez.com/init_c.htm#type

atoi saimal !

[Copros]

Je suis pourtant à peu près sûr qu'à la différence des autres types, le domaine indiqué pour les signed / unsigned char est le domaine exact, et non minimal. À vérifier.
Dans le cas contraire, la macro CLAMP devrait convenir au désir de notre PO.

[Arvi89]

Je vous explique pk c'était gchar ^^
Parmi la dose de variable, il y a ip[0]
const gchar *ip[0], *ip[1]......;
ip[0] = gtk_entry_get_text(GTK_ENTRY(app->ip[0]));
ip[1] = gtk_entry_get_text(GTK_ENTRY(app->ip[1]));
.
.
.

j'ai pas trouvé autre chose que atoi, mais je vais essayer de sauvegarder dans un guchar pour voir ce que ça donne

[Copros]

Ah donc en fait, il ne s'agit pas de gchar (que je te recommandais de remplacer par un guchar pour stocker ta valeur) mais de gchar* (la chaine issue du GtkEntry).

Le plus simple à mon avis est alors de remplacer ton GtkEntry par son dérivé GtkSpinButton, dans lequel on ne peut entrer que des nombres dans un certain intervalle :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
app->ip[0] = gtk_spin_button_new_with_range(0., 255., 1.);
/* ... */
ip[0] = gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(app->ip[0]));

et si tu veux une seconde sécurité, je reste fidèle à mon CLAMP :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ip[0] = CLAMP(ip[0], 0, 255)

N.B. Tout cela ne te dispense pas de revoir comment on manipule les chaines de caractères.