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

[wiztricks]

Salut,

Puisque vous avez des soucis avec la boucle "for" peut être devriez vous commencer par écrire la fonction est_acgt(ch) qui retourne True si ch est dans 'ACGT'.
En fait, elle retournera True si ch est égal à un des caractères de 'ACGT', on sort de la boucle dès qu'on a trouvé une égalité. C'est un peu la même mécanique qui doit forcer la sortie de la boucle pour valider vos chaînes d'ADN.

- W

[Nadess]

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

Bonjour,

Effectivement en faite j'aimerai que ma fonction me renvoie False si il y a un caractère différent de "A", "C", "G", "T" dans la chaîne s, et True s'il n'y a aucun autre caractère que "A", "C", "G", "T".

Le soucis avec

Code :

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

if c not in carac:

est le suivant :

Failed example:
estADN("AGCCGPQFQFZFZA")
Expected:
False
Got:
True

Ensuite concernant les res = mon professeur nous oblige à n'utiliser qu'un seul return Bool

[Nadess]

Salut,

Puisque vous avez des soucis avec la boucle "for" peut être devriez vous commencer par écrire la fonction est_acgt(ch) qui retourne True si ch est dans 'ACGT'.
En fait, elle retournera True si ch est égal à un des caractères de 'ACGT', on sort de la boucle dès qu'on a trouvé une égalité. C'est un peu la même mécanique qui doit forcer la sortie de la boucle pour valider vos chaînes d'ADN.

- W

Bonjour,

le soucis est que même s'il y a ACGT dans la chaîne ça ne veut pas dire pour autant que c'est un ADN, hors ma fonction doit vérifié qu'il n'y a aucun autre caractère que ACGT pour vérifier si justement notre chaîne est bien un brin d'ADN qui ne peut contenir rien d'autre que les bases A, C, G, T.

Et c'est tout là mon problème c'est que je n'arrive pas à faire en sorte que la fonction vérifie chaque caractère jusqu'à la fin de la chaîne et dès qu'elle trouve un caractère différent de A, C, G, T renvoie False et si elle ne trouve rien d'autre que des A,C,G,T renvoie True.

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.

[Sve@r]

Bonjour,

le soucis est que même s'il y a ACGT dans la chaîne ça ne veut pas dire pour autant que c'est un ADN, hors ma fonction doit vérifié qu'il n'y a aucun autre caractère que ACGT pour vérifier si justement notre chaîne est bien un brin d'ADN qui ne peut contenir rien d'autre que les bases A, C, G, T.

Hé ben ? Suffit de prendre chaque caractère "c" de la chaine et regarder si c not in "ACGT" !!!

[Nadess]

Hé ben ? Suffit de prendre chaque caractère "c" de la chaine et regarder si c not in "ACGT" !!!

C'est bon, j'ai fini par comprendre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    res = True
    if (s == ""):
        res = True
    A = "A"
    C = "C"
    T = "T"
    G = "G"
    carac = A, C, T, G
    for c in s:
        if c not in carac:
            res = False
    return (res)

Merci pour votre aide, ainsi que votre patience!

Cordialement.

[Fifan31]

Bonjour,

Comme l'avait indiqué wiztricks, dès que tu mets res à false, ce n'est plus la peine de parcourir le reste de ta chaîne. A ce moment là, tu peux faire un 'break' pour sortir prématurément de la boucle

[Sve@r]

Code :

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

1
2
3
4
 
    res = True
    if (s == ""):
        res = True

Code :

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

1
2
3
4
 A = "A"
    C = "C"
    T = "T"
    G = "G"

Donner un nom à une variable en fonction de sa valeur, ce n'est pas ce qu'il y a des plus pérenne. Généralement, on utilise plutôt un nom lié à son rôle (nbCar, nbLig, chaineADN, etc). Parce qu'une variable, c'est avant tout quelque chose qui peut varier (d'où le nom). Que se passera-t-il le jour où "A" devient (par exemple) "F" ? Tu écriras alors A="F" ou bien tu supprimeras ta variable A pour la remplacer par une nouvelle variable F quitte à repasser ensuite tout ton code ???

Code :

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

1
2
3
    carac = A, C, T, G
    for c in s:
        if c not in carac:

Mouais. Moi j'aurais écrit if c not in "ACGT" mais bon, pourquoi faire simple...?

Code python :

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

1
2
3
4
5
def isADN(s):
    for c in s:
        if c not in "ACGT": return False
    return True
# isADN()

Et si on veut se la jouer "python à donf": isADN=lambda s: all(c in "ACGT" for c in s)...