Discussion :

[madden44]

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 :

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

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.

[Neckara]

Bonjour,

Plusieurs remarques :

Code :

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

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.

[jblecanard]

Salut

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

Code :

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

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;
}

[Neckara]

Pourquoi lui donner une solution toute faite ?
Ce n'est pas comme cela qu'il progressera

[Ehonn]

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).

[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 .

[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).

[Neckara]

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 :

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

1
2
int tab[54];
size_t size = sizeof(tab)/sizeof(*tab);

Itérateurs :

Code :

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

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 ?

[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".

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.

[Neckara]

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.

[Ehonn]

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

Plutôt d'accord

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.

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.)

[Neckara]

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.

[Ehonn]

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) !

[Neckara]

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