Tableaux et fonctions en C

Bonsoir,


On m'a d'abord demandé de définir un type nommé TABCHAR permettant de déclarer des tableaux de 100 caractères.
Voici ce que j'ai fais :

Code:

---

typedef int TABCHAR[100];

---

Ensuite, l'énoncé demande, en considérant un tableau t de type TABCHAR, de chercher si le caractère ‘$’ est mémorisé dans ce tableau. Il faut écrire des lignes d’instructions permettant d’afficher la valeur -1 si le caractère ‘$’ n’est pas présent dans t, et sinon, l’indice d’une case du tableau où le caractère ‘$’ apparaît pour la première fois s’il est mémorisé dans t.

Voici ce que j'ai fais :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

typedef int TABCHAR[100];   TABCHAR t; int i, taille=100; /*est-ce inutile ici de préciser la taille du tableau?*/ char n='$';   for (i=0;i<taille;i++) { if (t[i]==n) /*t[i] est-il correct ? pourrait-il y avoir une confusion avec le t de TABCHAR t?*/ printf("%d",i); else puts("-1"); }

---

Toujours dans la même idée, il faut écrire une fonction F15 correspondant au traitement réalisé par les lignes d’instructions précédentes! Plutôt que de procéder à un affichage, il faudra que la fonction ait pour résultat -1 quand -1 était affiché, et sinon l’indice de la case qui était affiché.
Voici ce que j'ai fais :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

typedef TABCHAR[100]   char F15(TABCHAR t) /*TABCHAR peut-il être un paramètre ?*/ { int tailleTableau = 100, i; /*Y a-t-il un interêt à préciser la taille du tableau étant donné que TABCHAR le fait déjà ?*/ char n='$';   for (i=0;i<tailleTableau;i++) { if (t[i]==n) return i; else return -1; }

---

Est-ce correct s'il vous plait?

Merci d'avance

Peux-tu commencer par éditer ton post en évitant la double-imbrication de balises CODE?

Première remarque: Tu veux un tableau de cent char, pas un tableau de 100 int.

Pour éviter de se répéter quant aux tailles de tableau, la méthode classique, c'est de définir une macro pour la taille:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#define TAILLE_TABLEAU1 100 typedef char TABCHAR[TAILLE_TABLEAU1];   ...   TABCHAR t; int i char aTrouver='$';   for (i=0 ; i<TAILLE_TABLEAU1 ; i++) { if (t[i]==aTrouver) /*t[i] est-il correct ? pourrait-il y avoir une confusion avec le t de TABCHAR t?*/ printf("%d",i); else puts("-1"); }

---

Tu vas vite voir le problème de ce code: Il ne se limite pas au premier caractère trouvé. Il faut donc interrompre la boucle (traditionnellement, ça se fait avec break) après avoir trouvé le premier caractère.

Citation:

---

Envoyé par Médinoc

Première remarque: Tu veux un tableau de cent char, pas un tableau de 100 int.

Pour éviter de se répéter quant aux tailles de tableau, la méthode classique, c'est de définir une macro pour la taille:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#define TAILLE_TABLEAU1 100 typedef char TABCHAR[TAILLE_TABLEAU1];   ...   TABCHAR t; int i char aTrouver='$';   for (i=0 ; i<TAILLE_TABLEAU1 ; i++) { if (t[i]==aTrouver) /*t[i] est-il correct ? pourrait-il y avoir une confusion avec le t de TABCHAR t?*/ printf("%d",i); else puts("-1"); }

---

Tu vas vite voir le problème de ce code: Il ne se limite pas au premier caractère trouvé. Il faut donc interrompre la boucle (traditionnellement, ça se fait avec break) après avoir trouvé le premier caractère.

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#define TAILLE_TABLEAU1 100 typedef char TABCHAR[TAILLE_TABLEAU1];       TABCHAR t; int i; char aTrouver='$';   for (i=0 ; i<TAILLE_TABLEAU1 ; i++) { if (t[i]==aTrouver) printf("%d",i); break; /*oui car en effet il peut y avoir d'autres '$'...*/ else puts("-1"); }

---

Et pour ce qui est du paramètre pris pour la fonction F15 :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

#define TAILLE_TABLEAU 100   typedef char TABCHAR[100]   char F15(TABCHAR t) /*TABCHAR peut-il être un paramètre ?*/ { int tailleTableau = 100, i; char n='$';   for (i=0;i<tailleTableau;i++) { if (t[i]==n) return i; else return -1; }

---

Attention, ne confonds pas C et Python: En C, ce n'est pas l'indentation qui fait les structures de contrôles, ce sont les accolades.
Un tel code ne fait pas ce que tu crois (et il est mal indenté, d'ailleurs):

Code:

---

1 2 3

if(a) b(); c();

---

De plus, tes deux fonctions présentent un problème: La boucle s'arrête dès le premier caractère rencontré, qu'il s'agisse du caractère recherché ou non. Tu ne dois interrompre la boucle que quand c'est le caractère recherché!

Vous dites que la boucle s'arrête dès le premier caractère rencontré, qu'il s'agisse du caractère recherché ou non. Pourtant j'ai bien précisé que si t[i]==n, on obtiendra l'indice i en retour ? Ce n'est pas correct ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

#define TAILLE_TABLEAU 100 typedef char TABCHAR[100] char F15(void) /*pas de paramètres alors ?*/ { int tailleTableau = 100, i; char n='$'; for (i=0;i<tailleTableau;i++) { if (t[i]==n) return i; else return -1; } }

---

Regarde où se trouve ton return -1;...

Citation:

---

Envoyé par Médinoc

Regarde où se trouve ton return -1;...

---

Ah... donc d'emblée la boucle affiche -1 ?

D'emblée la boucle se termine et la fonction -1 sauf si $ est le premier caractère (auquel cas la fonction retourne 0).
C'est l'appelant qui affiche la valeur retournée, et non pas la fonction elle-même.

En y réfléchissant bien, les lignes du programme sont aussi fausses ! Vu que dans ce cas aussi, si on obtient pas le caractère recherché du premier coup, on affiche -1...

Voici de nouveau ma fonction :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#define TAILLE_TABLEAU 100 typedef char TABCHAR[100] char F15(void) /*sans paramètres alors ?*/ { int i; char n='$'; for (i=0;i<TAILLE_TABLEAU;i++) { if (t[i]==n) return i; if (t[i]!=n) i++; else return -1; } }

---

ça me semble toujours incorrect...

Pour bien faire, t devrait être un paramètre. aTrouver aussi, d'ailleurs.
Et comme d'habitude, tu en fais trop dans la boucle.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#define TAILLE_TABLEAU 100   typedef char TABCHAR[TAILLE_TABLEAU];   int F15(TABCHAR t, char aTrouver) { int i;   for (i=0 ; i<TAILLE_TABLEAU ; i++) { if (t[i]==aTrouver) return i; } return -1; }

---

Citation:

---

Envoyé par Médinoc

Pour bien faire, t devrait être un paramètre. aTrouver aussi, d'ailleurs.
Et comme d'habitude, tu en fais trop dans la boucle.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#define TAILLE_TABLEAU 100   typedef char TABCHAR[TAILLE_TABLEAU];   int F15(TABCHAR t, char aTrouver) { int i;   for (i=0 ; i<TAILLE_TABLEAU ; i++) { if (t[i]==aTrouver) return i; } return -1; }

---

---

Bonsoir,


J'aurais une question, s'il vous plait : pourquoi t et aTrouver sont des paramètres ?
Aussi, veuillez excuser les grossières erreurs que je commet...

Parce que sinon, comment la fonction sait-elle sur dans quel tableau (et contenant quoi) tu veux chercher un caractère?

Franchement, on pourrait se passer de mettre aTrouver en paramètre si la fonction, si on choisit de le hard-coder dans, par exemple, une fonction int chercheDollar(TABCHAR t).

Citation:

---

Envoyé par MayaKobayashi

Bonsoir,


On m'a d'abord demandé de définir un type nommé TABCHAR permettant de déclarer des tableaux de 100 caractères.
Voici ce que j'ai fais

---

tu a utiliser

Code:

---

for

---

dans le programme, ce qui une erreur. tu doit plutôt utiliser

Code:

---

while

---

par ce que quant tu auras retrouver l'élément en question il doit te renvoyer sont indice et sortir la boucle