besoin d'aide pour un programme en c++

Version imprimable

Bonjour , jai un programme en c++ à faire . Je suis debutante.
cela devrais me retourner un tableau ayant les valeurs 0,8,31 (soit l'endroit ou le mot bon se retrouve dans la phrase) et se terminer par -1 (valeur supplementaire obligatoire) Je peux seulement utiliser strcpy, strlen, strcmp et strcat. Quelles modifications dois-je apporter à ma fonction indexe pour que cela fonctionne svp? je dois remettre ceci dans moins de 24 h . Merci de m'aider
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 int main(){ char texte[]="bonjour bonsoir cava tres bien bon bye"; char mot[]="bon"; int *tab = indexe(texte,mot); } int *indexe(char *s1, char *s2){ int i=0; int j=0; int n; n = strlen(s1); char *texte; s1=texte; char *mot; s2=mot; int* tabposition = (int *) malloc(strlen(s1)*sizeof(int)); while(s1[i]!='\0' && i<=n ){ if (i==n){ tabposition[j]=-1; }else{ if (strncmp(s1,s2,3)){ i++ }else{ //=0 tabposition[j]=i; j++; i++ } return tabposition; } }

Bonjour,
Quelques commentaires :
Code:

1 2 int n; n = strlen(s1);
** Tu peux mettre les deux lignes précédentes en une seule pour plus de concision (et de clarté), les séparer n'est utile que si tu veux séparer les déclarations du reste du code, ce qui n'est apparemment pas le cas ici.
Code:

1 2 3 4 char *texte; s1 = texte; char *mot; s2 = mot;
** Les variables texte et mot sont inutiles ici, tu ne t'en sers pas par la suite. Il vaut mieux nommer explicitement s1 et s2, ça permettra d'auto-documenter la fonction et d'avoir un code plus clair.

Code:

int *tabposition = (int *)malloc(strlen(s1) * sizeof(int));

** Normalement il faut toujours tester le retour de malloc au cas où ...

Code:

if (strncmp(s1,s2,3)){

** Plusieurs choses ici :
* Tu compares toujours les mêmes chaines, quelles que soient les valeurs de i et de j, le résultat du test est donc toujours le même.
* De plus pourquoi le strncmp ? et pourquoi le 3 ? Tu voulais certainement utiliser la longueur du mot recherché ("bon" dans ton exemple) mais alors il vaut certainement mieux utiliser strlen(s2).
* Enfin il faut que les tests soient le plus explicite possible, ici il vaut mieux écrire :
if (strncmp(...) != 0)
Utiliser le fait qu'un entier non nul soit vrai est à utiliser de préférence quand le nom de l'entier (ou de la fonction le renvoyant) est explicite exemple :
if (estPositif(x)) { /* Faire quelque chose */ }

Code:

return tabposition;

** Dès que i est différent de n, la fonction retourne, donc dès le premier tour de boucle, si la chaîne est de longueur non nulle.

** Il manque une accolade.
** Pas de libération (free()) du tableau alloué dynamiquement.
** Il ne faut pas hésiter à utiliser l'arithmétique des pointeurs exemple si texte = "bonjour bonsoir cava tres bien bon bye"
alors texte + 1 = "onjour bonsoir cava tres bien bon bye"
de plus les chaînes de caractères sont des tableaux de chars terminés par '\0' donc peut faire les choses suivantes :
Code:

1 2 3 4 5 6 7 char texte[] = "bonjour bonsoir cava tres bien bon bye"; char c = texte[4]; texte[4] = '\0'; // ici texte contient la chaîne "bonj" texte[4] = c; // ici la texte contient à nouveau la chaîne "bonjour bonsoir cava tres bien bon bye"
Je pense t'avoir donné des éléments pour t'en sortir, n'hésite pas si quelque chose n'est pas clair.

Merci huit_six ,
j'ai essayer de modifier ma boucle, sans succès, le tableau retourner me donne toujours 0.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
include <iostream>
#include <string.h>
using namespace std;
 
int *indexe(char * , char *);
 
int main(){
char texte[]="bonjour bonsoir cava tres bien bon bye";
char mot[]="bon";
int *tab = indexe(texte,mot);
cout << *tab<< endl;
 
}
 
int *indexe(char *s1, char *s2){
    int i,j;
    int n,q;
    n = strlen(s1);
    q = strlen(s2);
 
 
    int* tabposition = (int *) malloc(strlen(s1)*sizeof(int)); 
    for(j=0, i=0 ; s1[i]!='\0'  && i<=n ;i++){
 
 
        if (strncmp(s1,s2,strlen(s2)))!=0{  
 
            s1=s1+1;
 
 
        }else{
            tabposition[j]=i;
            j++;
            s1=s1+1;
 
	}
 	if (i!=n){ 
 
            }else{
		tabposition[j]=-1;  
        }  
 
     }
return tabposition;	
   }

Pour ce qui est du free(), je ne sais trop comment ca marche.
et if (strncmp(s1,s2,strlen(s2)))!=0{ me renvoie comme erreur à la compilation :
programme3.cpp:26: erreur: expected primary-expression before ?!=? token
programme3.cpp:26: erreur: expected `;' before ?{? token

09/12/2009, 17h46
psychedelia14

oups pour le if, je lai mal fait , erreur stupide ;) mais bon , ca me retourne toujours 0
10/12/2009, 22h58
AndreiS

La fonction retourne un tableau de 'int' mais tu affiche seulement son premier élément malgré qu'elle marche presque bien. 'indexe' doit retourner aussi le nombre d'endroits ou le mot cherché se trouve et il faut que tu l'utilise pour boucler et afficher tous les valeurs. Pour vérifier ça, change l'affichage à

cout << tab[1] << endl;

Si tu ne veux pas retourner le nombre des indexes, met -1 dans tous les éléments de tabposition juste après sa déclaration et vérifie cette condition quand tu l'affiche.

Aussi, la boucle que tu utilise ne marche pas bien, efface ' s1[i]!='\0' ' et change la condition à ' i<=n - q ' - ça suffira pour trouver tous les endroits que tu cherche.
11/12/2009, 09h46
3DArchi

hormis l'utilisation du std::cout, ce code n'est qu'un problème de C. C'est pourquoi la discussion a été déplacée ici.
11/12/2009, 19h31
psychedelia14

Citation:

Envoyé par AndreiS

cout << tab[1] << endl;

Si tu ne veux pas retourner le nombre des indexes, met -1 dans tous les éléments de tabposition juste après sa déclaration et vérifie cette condition quand tu l'affiche.

Aussi, la boucle que tu utilise ne marche pas bien, efface ' s1[i]!='\0' ' et change la condition à ' i<=n - q ' - ça suffira pour trouver tous les endroits que tu cherche.

Merci AndreiS, cela semble si évident, je ne comprend pas pourquoi je n'y ai pas pensé avant !
11/12/2009, 22h36
fearyourself

Citation:

Envoyé par huit_six

Code:

int *tabposition = (int *)malloc(strlen(s1) * sizeof(int));

** Normalement il faut toujours tester le retour de malloc au cas où ...

- Il faudrait aussi enlever le cast
- Mettre sizeof (*tabposition) à la place de sizeof (int)

Jc