Appel de 2 fonctions

Version imprimable

Bonsoir,

J'essaye actuellement de compter le nombre d’occurrences d'un caractère (une lettre) dans une suite de lettres et je veux ensuite afficher la masse de cette lettre (puisque cette lettre correspond a un acide aminé).
Le nombre d’occurrence marche et est affiché, cependant, à la suite de cela le programme fait comme si je n'avais rien mis après alors que je veux afficher une certaine phrase, savez vous d'ou cela pourrait venir ?

Voici mon programme :

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
46
47
48
49
50
51
 
#include <cstdio>
#include <cstdlib>
#include <string>
#include <iostream>
#include <cmath>
#include <conio.h>
 
using namespace std; 
 
double A,B,C,D,E,F,G,H,I,J,K,L,M,N,P,Q,R,S,T,V,W,Y;
float i, n, res=1;
int p;
 
K = 128.094 * res;
 
int nb_occ(char* chaine,char car)                                              
{
    int c,nbocc = 0,
    size = strlen(chaine);                                       //Taille de la chaine
 
    for (c = 0 ; c < size ; c++)
    {
        if (chaine[c] == car)                                                  
            nbocc ++;                                                           
    }
 
    return nbocc;                                                              
}
 
int main(void)                                                         //Fonction principale
{
 
n = 1.66054 * 10;                                                               
p = -27;
if (p>0) for (i=0; i<p; i++) res=res*n;
else for (i=0; i<(-p); i++) res=res/n;
 
getch();
 
    char maChaine[] = "MTYVKKYHPTTKYYGIIHGEKYELQDVLF";
    int nb = nb_occ(maChaine,'K');                                             
 
    cout<<"La chaine : "<<maChaine<<" contient "<<nb<<" fois le caractere 'K'"<<endl;
    cout<<endl;
    fin = nb*K;
    cout<<"La masse de cet acide aminé est : "<<fin<<" Kilogrammes";
 
    system ("PAUSE");
    return 0;
}

Cordialement.

22/01/2014, 20h05
Neckara

Bonjour,

Plusieurs remarques :

Code:

double A,B,C,D,E,F,G,H,I,J,K,L,M,N,P,Q,R,S,T,V,W,Y;

Utilise un tableau.

En C++, on utilise plus des char * mais des std::string.
En C++, il existe des algorithmes comme count, count_if.

Tu as oublié un std::endl à la fin de ton programme.

Salut

Je passe rapidement, c'est dommage de s'embêter avec du C quand on peut faire hyper simple en C++11.

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
#include <unordered_map>
#include <iostream>
#include <algorithm>
#include <string>
 
size_t nb_occ(std::string const& chaine, char needle) {
  return std::count(chaine.begin(), chaine.end(), needle);
}
 
int main() {
  std::string chaine = "MTYVKKYHPTTKYYGIIHGEKYELQDVLF";
 
  // Methode 1 : on compte par caractère isolé
  std::cout << "La chaîne " << chaine << " contient " << nb_occ(chaine,'K') << " fois le caractère 'K'." << std::endl;
 
  // Methode 2 : on compte toutes les occurences de tous les caractères d'un coup
  std::unordered_map<char,size_t> occurences;
  for(char c : chaine) ++occurences[c];
 
  std::cout << "La chaîne " << chaine << " contient " << occurences['K'] << " fois le caractère 'K'." << std::endl;
  std::cout << "La chaîne " << chaine << " contient " << occurences['Y'] << " fois le caractère 'Y'." << std::endl;
 
  return 0;
}

23/01/2014, 06h22
Neckara

Pourquoi lui donner une solution toute faite ?
Ce n'est pas comme cela qu'il progressera :?
23/01/2014, 07h58
Ehonn

Citation:

Envoyé par Neckara

Utilise un tableau.

Petite précision : les tableaux en C++ c'est std::vector ou std::array si la taille est connue à la compilation (et qu'il tiens sur la pile).
23/01/2014, 12h09
Neckara

Petite précision pour la précision d'Ehonn, std::array vient du C++11, (option -std=c++11 avec gcc).

Donc si on veut un tableau de taille fixe alloué automatiquement sans faire du C++11 (compilateur trop vieux par exemple), utiliser un tableau n'est pas un drame ;).
23/01/2014, 19h46
Ehonn

Et à défaut d'utiliser un compilateur à jour ou n'importe quel équivalent à boost::array, on choisira d'encapsuler le tableau dans une classe pour accéder facilement à sa taille de façon sûre via une méthode (voire on ajoutera les méthodes pour avoir des itérateurs sur ce tableau).
23/01/2014, 20h05
Neckara
Citation:

Envoyé par Ehonn

Et à défaut d'utiliser un compilateur à jour ou n'importe quel équivalent à boost::array, on choisira d'encapsuler le tableau dans une classe pour accéder facilement à sa taille de façon sûre via une méthode (voire on ajoutera les méthodes pour avoir des itérateurs sur ce tableau).

Taille de façon sûre :
Code:

1 2 int tab[54]; size_t size = sizeof(tab)/sizeof(*tab);
Itérateurs :
Code:

1 2 3 int tab[54]; int * iterator_begin = tab; // normalement ça devrait fonctionner avec les algo de la stl int * iterator_end = tab + taille_tab;
Pourquoi s'enquiquiner à faire une classe/réinventer std::array ?
23/01/2014, 20h30
Ehonn

Ton size est même pas const ! :p
Des méthodes .size() .begin() .end() .cbegin() .cend() sont plus simples à utiliser et à se souvenir, elles sont "STL-friendly".

Citation:

Envoyé par Neckara

Pourquoi s'enquiquiner à faire une classe/réinventer std::array ?

Car je me place dans ton contexte, pas de support de C++11. Pas envie d'utiliser boost::array ou équivalement.
std::array n'a pas été normalisé sur un coup de tête.
23/01/2014, 20h42
Neckara

Citation:

std::array n'a pas été normalisé sur un coup de tête.

Je ne dis pas le contraire.

Mais sans C++11 et si tu tiens absolument à avoir begin()/end()/... autant utiliser un std::vector non?

Pourquoi perdre du temps à développer une classe qui ne t'apportera au final presque rien ?

Parce que bon, dans les rares cas où tu as besoin d'un tableau constant avec impérativement begin() et compagnie... et que tu ne veux absolument pas d'allocation dynamique... ça se compte sur les doigts d'une main.
23/01/2014, 20h55
Ehonn

Citation:

Envoyé par Neckara

Mais sans C++11 et si tu tiens absolument à avoir begin()/end()/... autant utiliser un std::vector non?

Plutôt d'accord ;)

Citation:

Envoyé par Neckara

Pourquoi perdre du temps à développer une classe qui ne t'apportera au final presque rien ?

Parce que bon, dans les rares cas où tu as besoin d'un tableau constant avec impérativement begin() et compagnie... et que tu ne veux absolument pas d'allocation dynamique... ça se compte sur les doigts d'une main.

Citation:

Envoyé par Neckara

Citation:

Envoyé par Ehonn

std::array n'a pas été normalisé sur un coup de tête.

Je ne dis pas le contraire.

Si ça été normalisé, c'est que ça doit servir ;)
On aura même peut être std::dynarray en C++17

(C'est surtout la dissociation entre le tableau et sa taille qui me dérange.)
23/01/2014, 20h59
Neckara

Citation:

Envoyé par Ehonn

Si ça été normalisé, c'est que ça doit servir ;)

Je ne dis pas le contraire.
Mais en C++11, le rapport coût/bénéfice de l'utilisation de std::array est quasi-nul.
23/01/2014, 21h08
Ehonn

Citation:

Envoyé par Neckara

Mais en C++11, le rapport coût/bénéfice de l'utilisation de std::array est quasi-nul.

Bench (si tu parles de performence) !
23/01/2014, 21h19
Neckara

Je parlais de coût en développement (quasi-nul car on a juste un #include à rajouter)