Discussion :

[Nadess]

Bonjour,

je bloque un peu sur un exercice, je viens donc solliciter votre aide :

Réalisez une fonction nommée estADN qui vérifie si la chaîne de caractères passée en paramètre ne contient aucun autre caractère que les quatre bases A, C, G et T. Cette fonction retourne la valeur True si tel est le cas, et la valeur False dans le cas contraire. En particulier, elle renvoie True si la chaîne est vide.

j'ai donc écrit comme fonction :

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
def estADN(s):
    """ Fonction qui permet de vérifier si la chaîne de caractère contient les bases "A", "C", "G" et "T" pour être un ADN.
        :param s: (int) chaîne de caractère que l'on souhaite vérifier
        :return: (bool) Renvoie True si la chaîne s est un ADN, sinon false
        
        CU : aucune
        
        Exemple :
        
        >>> estADN("ACGGTAGCTAGTTTCGACTGGAGGGGTA")
        True
        >>> estADN("ACTZATCFDSFSZZG")
        False
        >>> estADN("GCTTYCFZGSGZG")
        False
        >>> estADN("AGCCGPQFQFZFZA")
        False
        >>> estADN("AGTVGSGEGTGA")
        False
        >>> estADN("")
        True
    """
    if (s == ""):
        res = True
    carac = "A" and "G" and "T" and "C"
    for c in s:
        if carac == c in s:
            res = True
        if carac != c in s:
            res = False
    return (res)
 
if __name__ == '__main__':
    import doctest
    doctest.testmod(optionflags=doctest.NORMALIZE_WHITESPACE | doctest.ELLIPSIS, verbose = True)

mon problème : la fonction ne fonctionne pas car même si la chaîne en paramètre contient les caractères A C G T, la fonction me renvoie False. J'en déduis donc que le code ne fonctionne pas mais j'ai essayé beaucoup de code sans succès à chaque fois.

Je me tourne vers vous pour comprendre tout cela et avoir le bon code. Je tiens à préciser que cet exercice m'est demandé dans un contexte de cours en Licence et je dois me limiter à ce que mon prof m'a apprît en cours : Données et expressions, Fonctions, Conditions, Chaînes de caractères.

Merci d'avance pour votre aide !

[wiztricks]

Salut,

Les pépites sont dans ces lignes:

Code :

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

1
2
3
4
5
6
7
    carac = "A" and "G" and "T" and "C"
    for c in s:
        if carac == c in s:
            res = True
        if carac != c in s:
            res = False
    return (res)

Déjà, vous pouvez tester ce que font les lignes carac = "A" and "G" and "T" and "C" et if carac == c in s à la console interactive pour vous assurer que çà fait bien ce que vous en attendez.
Ensuite vous réfléchissez à votre boucle "for" à sa sortie la valeur de "res" sera le résultat de la condition sur le dernier caractère alors que dès que vous trouver un caractère hors 'ACGT', pas la peine de continuer.

- W

[Nadess]

Salut,

Les pépites sont dans ces lignes:

Code :

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

1
2
3
4
5
6
7
    carac = "A" and "G" and "T" and "C"
    for c in s:
        if carac == c in s:
            res = True
        if carac != c in s:
            res = False
    return (res)

Déjà, vous pouvez tester ce que font les lignes carac = "A" and "G" and "T" and "C" et if carac == c in s à la console interactive pour vous assurer que çà fait bien ce que vous en attendez.
Ensuite vous réfléchissez à votre boucle "for" à sa sortie la valeur de "res" sera le résultat de la condition sur le dernier caractère alors que dès que vous trouver un caractère hors 'ACGT', pas la peine de continuer.

- W

Merci de votre réponse.

J'ai donc modifié dans un premier temps les lignes pour ACGT, j'ai donc ceci :

Code :

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

1
2
3
4
5
6
7
8
 
    if (s == ""):
       res = True
    A = "A"
    C = "C"
    T = "T"
    G = "G"
    carac = A, C, T, G

pour la boucle for :

Code :

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

1
2
3
4
5
6
 
    for c in s:
        if c == carac:
            res = True
        else:
            res = False

mais ce n'est toujours pas bon pour la boucle for et je ne vois pas pourquoi ce que j'ai écris est faux car je le comprend comme ceci :

pour les caractères dans la chaîne de caractère s :
si les caractères sont égales à la variable carac ("A", "C", "T", "G") alors res = True
sinon:
res = False

[wiztricks]

Salut,

Essayez de réfléchir un peu si carac = A, C, T, G comment voulez vous que la condition c == carac ait des chances d'être vraie?
"c" étant un simple caractère, impossible qu'il puisse être égal au tuple A, C, T, G...
Si vous savoir si "c" est l'un des "carac", difficile de ne pas écrire une boucle qui comparera chaque caractère de "carac" ou d'utiliser le mot clef "in" qui permet de tester c in carac (qui va faire faire cette boucle par Python).

Côté boucle "for":

Code :

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

1
2
3
4
5
    for c in s:
        if c == carac:
            res = True
        else:
            res = False

"res" vaudra toujours le résultat de la comparaison du dernier caractère.
Vous ce que vous voulez c'est vérifier que tous les caractères sont dans 'ACGT'...
Autrement dit, dès qu'on en trouve un de différent, on positionne res et on sort de la boucle avec un "break".

- W

[Nadess]

Code :

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

1
2
3
4
5
6
 
for c in s:
     if c != c in carac:
         res = False
else:
      res = True

Salut,

Je sais que ce n'est toujours pas bon mais j'en suis là

[wiztricks]

Code :

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

1
2
3
4
5
6
 
for c in s:
     if c != c in carac:
         res = False
else:
      res = True

Salut,

Je sais que ce n'est toujours pas bon mais j'en suis là

Essayez de relire ce que vous écrivez que peut bien vouloir dire c != c in carac?
Vous avez "c in carac" qui retourne un booléen le comparer à un caractère sera toujours faux (et n'importe quoi).
De plus la console Python vous permet de tester vos idées saugrenues et de constatez que:

Code :

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

1
2
3
4
5
>>> 'A' != 'A' in 'ACGT'
False
>>> 'Z' != 'Z' in 'ACGT'
False
>>>

donc si çà ne fait pas ce que vous voulez, vous retourner voir dans vos cours, tutos,... là où ils expliquent comment utiliser le mot clef "in".... Et si vous ne savez pas ou chercher, vous pouvez coder çà avec une boucle for, c'est pas très élégant mais çà fonctionne et c'est un bon exercice.

- W

[Sve@r]

Code :

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

1
2
if c != c in carac:
...

Bonjour

Je crois que ce que tu cherches à faire se traduit par if c not in carac. Comme le suggère wiztricks, les tutos ne sont pas fait que pour servir à caler les tables.

Accessoirement, la variable "res" est inutile. Si c n'est pas dans la liste des caractères autorisés, inutile de regarder les autres puisque de toute façon le résultat sera faux. Te suffit alors de faire un return False à cet endroit. Bon c'est vrai, certains puristes n'aiment pas les return intempestifs mais bon, comme disait ma mère au bridge "un bon coup d'oeil vaut mieux qu'une mauvaise impasse" (et mon prof de karaté disait à peu près la même chose avec sa devise "un point mal acquis vaut mieux que deux 'tu l'auras peut-être honnêtement' ")...