Pb code please help

biohazard -  
 loupius -
Bonjour,
pourriez vous me dire pourquoi ca marche pas? Je veux faire un programme qui convertisse une chaine d'ADN en ARN. Voila mon code:
int main(void)
#include <stdio.h>
int main(void)
{
int a;
char dna[256];
scanf("%s", &dna);
for(a=0; a<256; a++)
{
if(dna[a]!='A' && dna[a]!='T' && dna[a]!='C' && dna[a]!='G')
{
printf("Invalid dna");
}
else
{
if(dna[a]=='A')
dna[a]='U';
else
if(dna[a]=='T')
dna[a]=='A';
else
if(dna[a]=='C')
dna[a]='G';
else
dna[a]='C';
}
}
return 0;
}
Configuration: Windows Vista
Firefox 3.0.8

8 réponses

  1. loupius
     
    Tu boucles de 0 à 255, est-tu ceratin que la longueur de la chaîne saisie aura cette taille... moi pas !
    Je te laisses trouver une bonne solution.
    0
  2. biohazard
     
    merci
    donc y faut que je déclare le tableau sans préciser sa taille? comme ca: char dna[]?
    0
    1. loupius
       
      Ca ne changera rien, car ensuite il faudra allouer de la place pour ta chaîne et à ce moment là il faudra que tu rentres une taille...
      Non, je vais t'aider un petit peu: il faut que la boucle soit limitée à la longueur de la chaîne entrée.
      0
  3. Mehdi4.0 Messages postés 54 Date d'inscription   Statut Membre 13
     
    Verifie par le biait d'un affichage si le tableau DNA contient bien la chaine entrée par l'utilisateur !
    La programmation est l'art de programmer des semaines pour gagner quelques microsecondes.
    0
  4. biohazard
     
    En fait la taille c'est pas très important, ce qui m'embete le plus, c'est que le programme ne marche pas, il m'affiche toujours Invalid dna, meme si je rentre une chaine correcte d'adn.
    0
  5. Vous n’avez pas trouvé la réponse que vous recherchez ?

    Posez votre question
  6. biohazard
     
    Ok je vois ce qui faut faire pour la taille merci. Mais reste toujours le pb de fond qui est que le programme marche pas.
    0
    1. loupius
       
      Mais si, le programme marche !!! Seulement il ne faut pas tester en dehors de la chaîne saisie, sinon on est certain que le test:
      if(dna[a]!='A' && dna[a]!='T' && dna[a]!='C' && dna[a]!='G') sera satisfait !
      0
      1. KiLuKru.. > loupius
         
        Slt,

        En C, il ne faut pas utiliser strcmp() pour tester l'égalité entre deux chaines de caractères normalement ?..

        Salutations.
        0
      2. KiLuKru.. > loupius
         
        Ouaip mais non.

        En comparant seulement sur un octet - un simple test d'égalité semble suffisant.

        Autant pour moi.
        0
  7. biohazard
     
    Je croit que j'ai compris, il faut faire entrer la taille de la chaine par l'utilisateur, c ca?
    0
  8. Mehdi4.0 Messages postés 54 Date d'inscription   Statut Membre 13
     
    De preferable non.
    Bon juste fixer une taille maximale pour la chaîne de caractères puis lorsque tu va parcourir ta chaine, tu va aller de 0 jusqua la taille rééel de ta chaine pas la taille maximal.

    La programmation est l'art de programmer des semaines pour gagner quelques microsecondes.
    0
  9. loupius
     
    Au vu du reste de ton programme, je pensais que tu connaissais un peu plus le C; mais au vu des réponses 5 et 11, j'ai l'impression que tu piges pas.
    C'est pourtant très simple: Il ne faut pas tester en dehors de la chaîne; c'est une nécessité.
    Alors comment faire, il y a 3 solutions:
    1) Demander à l'utilisateur de saisir la longueur de la chaîne: c'est absurde, que se passera-t-il s'il se trompe ? Et puis on va pas lui demander un travail que l'on peut faire.
    Il faut savoir qu'en C les chaînes de caractères sont terminées par un '\0' (caractère nul qui vaut zéro 0h):
    2) Il existe une fonction (strlen) qui cherche ce zéro et qui donne la longueur de la chaîne; ainsi on va faire une boucle for (a=0; a<strlen(dna); a++)
    3) Il existe une autre manière de faire une boucle, c'est avec 'while (condition)', la condition étant le caractère qui ne devra pas être nul: a=0; while (dna[a]) { ... a++; }
    Ici, pour moi, la réponse la mieux adaptée est la n° 3.
    Bon choix.
    0