Fonction et procedure

Marsh Posté le 04-11-2015 à 15:13:16

Bonjour,

J'ai un problème sur un exercice, voici les consignes :

Les procédures et fonctions qui seront écrites dans cet exercice devront être compilées séparément.
Votre projet devra donc comporter un fichier "main. cpp il dans lequel on trouvera la fonction
main, un fichier "fonctions.h" dans lequel se trouveront les déclarations de toutes les fonctions que
l'on vous demande d'écrire, et un fichier "fonctions.cpp" dans lequel se trouvera le corps de ces
fonctions
o Ecrire une procédure permettant la saisie d’une chaine.
o Ecrire une fonction permettant de déterminer la longueur d'une chaine.
Elle calculera et retournera la longueur de la chaîne de caractères.
o Ecrire une procédure permettant de convertir tous les caractères Majuscule d’une chaine en
minuscule.
o Ecrire une fonction permettant de rechercher un caractère dans une chaine
Si le caractère donné est trouvé, la fonction retournera son adresse dans la chaine ; sinon elle
retournera le pointeur NULL.

Ecrire une procédure qui permette de remplacer tous les caractères égaux à un caractère donné,
par un autre caractère donné. Par exemple le remplacement des ‘e’ par ‘o’ dans la chaine
« telephone » donnera la chaine « tolophono
o Ecrire une procédure qui permette d’éliminer tous les caractères égaux à un caractère donné, par
un autre caractère donné. Par exemple l’élimination des ‘e’ dans la chaine « telephone » donnera
la chaine « tlphon ».
o Ecrire un programme permettant de tester chacune de ces procédures et fonctions

Voici mon header :

Code :

typedef char chaine[100];
int LongueurChaine(char *);
char *RechercheCaractere(char *);
void SaisieChaine(char *);
void Convertion(char *);
void ChangementCaractere(char *);
void SuppresionCaractere(char *);

mon cpp avec les fonctions et procédure :

Code :

#include <iostream>
#include <conio.h>
using namespace std;
void SaisieChaine(char *chaine) //Première procédure pour l'insertion de la chaine de caractère
{
cout << "Inserez votre chaine de caractere puis appuyez sur ENTRER" << endl;
cin >> chaine;
_getch();
system("cls" );
}
int LongueurChaine(char *chaine)
{
int longueur = 0;
longueur = strlen(chaine); // strlen permet de compter le nombre de caractère présente dans une chaine
return longueur;
}
void Convertion(char *chaine)
{
for (int i = 0; i < strlen(chaine); i++)
{
if (chaine[i] >= 'A' && chaine[i] <= 'Z')
{
chaine[i] = chaine[i] - 'A' + 'a';
}
}
cout << "la nouvelle chaine est :\n";
for (int i = 0; i < strlen(chaine); i++)
{
cout << chaine[i];
}
}
char *RechercheCaractere(char *chaine)
{
char choix;
cout << "\nQuel Caractere cherchez vous \n";
cin >> choix;
int i = 0;
while (choix != chaine[i] && chaine[i])
{
i++;
}
if (!chaine[i])
{
return NULL;
}
else
{
return chaine + i;
}
}
void ChangementCaractere(char * chaine)
{
char base;
char nouvelle;
cout << "\n Quelle lettre voulez vous remplacer :\n";
cin >> base;
cout << "\n Par quelle lettre voulez vous la remplacer : \n";
cin >> nouvelle;
for (int i = 0; i < strlen(chaine); i++)
{
if (chaine[i] == base)
{
chaine[i] = nouvelle;
}
}
cout << "Votre nouvelle chaine est :\n";
for (int i = 0; i < strlen(chaine); i++)
{
cout << chaine[i];
}
}
void SuppresionCharactere(char *chaine)
{
char suppression;
cout << "\n Quelle lettre voulez vous supprimer? :\n";
cin >> suppression;
for (int i = 0; i < strlen(chaine); i++)
{
if (chaine[i] == suppression)
{
for (int j = i; j <= strlen(chaine); j++)
{
chaine[j] = chaine[j + 1];
}
}
}
cout << "Voici votre nouvelle phrase :\n";
for (int i = 0; i < strlen(chaine); i++)
{
cout << chaine[i];
}
}

et enfin mon cpp main :

Code :

#include <iostream>
#include <conio.h>
#include "fonction.h"
using namespace std;
void main()
{
const int Cmax = 100;
int choix;
chaine a, b, c, d, e, f;
SaisieChaine(a); // procédure saisie de la chaine de caractère
for (int i = 0; i <= Cmax; i++)
{
b[i] = a[i];
c[i] = a[i];
d[i] = a[i];
e[i] = a[i];
f[i] = a[i];
}
cout << "***************************************************************************" << endl;
cout << "* Choisissez ce que vous voulez faire parmis les propositions suivantes *" << endl;
cout << "* 1. Calculer la longueur de la chaine. *" << endl;
cout << "* 2. Convertir les majuscules de votre phrase en minuscule. *" << endl;
cout << "* 3. Rechercher un caractere dans votre chaine. *" << endl;
cout << "* 4. Effectuer un changement de caractere dans votre phrase. *" << endl;
cout << "* 5. Effectuer une suppression de caractere dans votre phrase. *" << endl;
cout << "***************************************************************************" << endl;
cin >> choix;
switch (choix)
{
case 1: cout << "la taille de la chaine est : " << LongueurChaine(b) << endl; // Fonction Longueur de la chaine
break;
case 2: Convertion(c); // Convertion Majuscule --> Minuscule
break;
case 3: cout << "l'adresse du caractere voulus dans la chaine est : " << RechercheCaractere(d); // Fonction Recherche de caractère
break;
case 4: ChangementCaractere(e);
break;
case 5: SuppresionCaractere(f);
break;
default: cout << " Vous n'avez pas choisit une option du tableau, au revoir." << endl;
}
_getch();
}

Sa mafiche deux erreurs que je ne comprend pas, Quelqu'un voit le probleme svp?

Cordialement

Message cité 1 fois

Reply

Marsh Posté le 04-11-2015 à 15:13:16

Reply

Marsh Posté le 04-11-2015 à 15:52:47

Déjà: mise en page du code à revoir...

sylargh a écrit :

Sa mafiche deux erreurs que je ne comprend pas, Quelqu'un voit le probleme svp?
Cordialement

Ca serait bien de nous dire quelles erreurs...

Bon, MinGW me dit

Citation :

D'après internet faut rajouter

Code :

#include <cstdlib>
#include <cstring>

dans fonctions.cpp, ce qui résout le problème. Entre parenthèses, le system() c'est probablement pas très élégant, il n'y a pas mieux en C++? (Je connais juste le C.)

Après il y a une faute de frappe #include "fonction.h" il manque un 's' et une autre erreur facile à corriger:

Citation :

main.cpp|5|error: '::main' must return 'int'|

et puis

Citation :

main.o:main.cpp: (.text.startup+0x1b8)||undefined reference to `SuppresionCaractere(char*)'|

problème de nom dans fonctions.cpp: SuppresionCharactere vs SuppresionCaractere

Restent les warnings (si il y en a pas faut indiquer -Wall au GCC/MinGW!):

Citation :

fonctions.cpp|21|warning: comparison between signed and unsigned integer expressions [-Wsign-compare]|

à de nombreuses reprises, "unsigned" résout le problème.

et puis

Citation :

main.cpp|13|warning: array subscript is above array bounds [-Warray-bounds]|

à plusieurs reprises. Ca c'est plus grave et effectivement le programme plante rapidement. Je te laisse vérifier ton code/algo.

edit: D'ailleurs, si c'est du C++, il y a plus élégant/pratique que des array of char il me semble, mais bon, je ne sais pas ce que veut le prof...

Message édité par rat de combat le 04-11-2015 à 15:56:16

Reply

Marsh Posté le 04-11-2015 à 16:25:43

Quelque remarques en vrac...

Dans convertion() (qui d'ailleurs s'écrit conversion en français...):

Code :

cout << "la nouvelle chaine est :\n";
for (unsigned int i = 0; i < strlen(chaine); i++)
{
cout << chaine[i];
}

:ouch:

Code :

cout << "la nouvelle chaine est :"<<chaine<<"\n";

idem dans ChangementCaractere() et SuppresionCaractere()...

RechercheCaractere():

Code :

{
char choix;
cout << "\nQuel Caractere cherchez vous \n";
cin >> choix;
unsigned int i = 0;
while (choix != chaine[i] && chaine[i])
{
i++;
}

Danger: Si le caractère cherché n'est pas dans la chaîne au dernier tour de la boucle on dépasse les bornes de la chaîne de caractère (out of bound). Il faut d'abord vérifier si la chaîne n'est pas terminée et ensuite faire la comparaison pour voir si on a trouvé le caractère demandé.

main():

Code :

cout << "l'adresse du caractere voulus dans la chaine est : "<< RechercheCaractere(d);

n'affiche pas une adresse comme demandé. Je ne parle pas C++, je ne sais donc pas comment faire avec cout.

Pour le warning dont je parlais, ta boucle for() (toujours dans main) est fausse et inutile, si a chaque exécution du programme on ne peut choisir qu'une seule opération il est inutile de sauvegarder / faire des copies de la chaîne initiale (ce qui en plus peut se faire de manière plus élégante qu'une boucle).

Ah oui, je pense que par "déterminer la longueur d'une chaine" le prof entends autre chose qu'un appel de strlen() non?

Et perso je virerais les system("cls" ) et les _getch().

Reply

Marsh Posté le 04-11-2015 à 22:35:41

Le problème de ton exercice, c'est que soit tu fais du C, ou les chaines sont des suites de caractères accédées par pointeur char *, soit tu fais du C++ moderne, ou tu utilises le type standard string et tu as la librairie standard.
Mais il faut surtout pas mélanger les deux.

Typiquement, en C++ (11), on ferait un truc du genre

Code :

#include <algorithm>
#include <cctype>
#include <iostream>
#include <string>
void get_input(std::string& s) {
std::cout << "Taper une phrase : " << std::flush;
std::getline(std::cin, s);
}
size_t get_length(std::string& s) {
return s.size();
}
void upcase(std::string& s) {
std::transform(s.begin(), s.end(), s.begin(), ::toupper);
}
char *find_char(std::string& s, char c) {
size_t pos = s.find(c);
return (pos == std::string::npos)?NULL:&s[pos];
}
void replace_char(std::string& s, char from, char to) {
std::replace_if(s.begin(), s.end(), [from](char c) {return (c == from);}, to);
}
void remove_char(std::string& s, char from) {
std::string::iterator nend = std::remove_if(s.begin(), s.end(), [from](char c) {return (c == from);});
s.resize(nend - s.begin());
}
int main() {
std::string input;
get_input(input);
std::string test = input;
std::cout << "Phrase entree: " << test << std::endl;
std::cout << "Nombre de caracteres: " << get_length(test) << std::endl;
upcase(test);
std::cout << "En majuscules: " << test << std::endl;
test = input;
std::cout << "Saisir un caractere a chercher: " << std::flush;
std::string s;
std::cin >> s;
char c = *s.begin();
char *epos = find_char(test, c);
if (epos == NULL) {
std::cout << "Pas de caractere " << c << " dans la phrase" << std::endl;
}
else {
std::cout << "Le caractere " << c << " est a la position " << (epos - &test[0] + 1) << " dans la phrase" << std::endl;
std::cout << "et son adresse est: " << std::hex << std::showbase << ((int)epos) << std::endl;
}
replace_char(test, c, '$');
std::cout << "Remplacement de " << c << " par $: " << test << std::endl;
test = input;
remove_char(test, c);
std::cout << "Suppression de " << c << " : " << test << std::endl;
return 0;
}

l'intérêt de la chose, c'est que par construction, c'est bien plus blindé contre les erreurs que la même chose écrite en C.

Bon, évidemment, si tu as un prof qui a pas compris que le C++ c'est pas du C, et qui attends que tu réécrives a la main un parcours de chaine de caractère un a un pour comparer avec un caractère recherché, ça sera instructif pour apprendre le C (quoique tu aies strchr en standard), mais pas vraiment pour apprendre le C++.

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 07-11-2015 à 11:32:10

Bonjour merci de vos réponses, le programme marche apres avoir enlever la faute "H" a Caractere, merci
Oui mon prof date un peu, ( dans les 60 ans ), avec mes camarades
nous connaissons deja String mais ils refusent de nous le faire utiliser pour le moment... & pour les pointeurs et reference je suis encore en train d'apprendre, je me suis pas beaucoup exercé ^^, par exemple mon probleme de renvoie d'adresse qui ne fonctionne pas, je ne sais pas trop comment faire pour le moment.
Merci de l'aide apporter rapidement sur ce Forum :hello:

Message cité 1 fois

Reply

Marsh Posté le 08-11-2015 à 13:14:45

> nous connaissons deja String mais ils refusent de nous le faire utiliser pour le moment
Ce qui le disqualifie d'emblée pour enseigner le C++ (les char * ça doit être vu après les strings, pas avant, en C++).

A+,

---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --

Reply

Marsh Posté le 11-11-2015 à 22:20:33

sylargh a écrit :

Bonjour merci de vos réponses, le programme marche apres avoir enlever la faute "H" a Caractere, merci

J'espère que tu as aussi pris en compte les autres remarques/warnings. :ouch:

Citation :

Bon, évidemment, si tu as un prof qui a pas compris que le C++ c'est pas du C, et qui attends que tu réécrives a la main un parcours de chaine de caractère un a un pour comparer avec un caractère recherché, ça sera instructif pour apprendre le C (quoique tu aies strchr en standard), mais pas vraiment pour apprendre le C++.

J'ai l'impression qu'il y a pas mal de profs qui pensent qu'il suffit de faire s/printf/cout/g (idem cin) pour transformer du C en C++, parce que "C++" ça doit faire mieux que "C" sur les présentations de la formation et le CV des étudiants.

Reply

Fonction et procedure

Sujets relatifs:

Leave a Replay