Discussion :

[lucie34]

Bonjour,

Je cherche la position d'une lettre dans un mot. Par ex. "a" est la position 1 du mot "maison".
Je vous remercie pour votre aide.

Mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
def find1st(c,s):
	i = 0
	lettre = s[0]
	for elt in range(len(s)):
		if s[elt] == s[i]:
			lettre = s[i]
			return i
		i = i + 1
	return None
 
print(find1st("i", "maison"))

[Julien N]

Salut.

L'utilisation d'une boucle for est une contrainte?

Si oui, parcourez les lettres. Si la lettre actuelle est celle recherchée sortez de la fonction. Sinon pousuivez jusqu'à plus de lettres et retournez None. Vous y êtes presque:

Code :

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

1
2
3
4
5
def look_for_letter(s, letter):
    for i, car in enumerate(s):
        if car == letter:
            return i
    return None

Ici j'utilise enumerate() qui permet de pourcourir un itérable (une chaîne de caractère dans le cas présent) en renvoyant deux éléments: un indice, et l'élément. Vraiment pratique.

Mais bon, si la boucle for n'est pas obligatoire, vous pouvez directement demander l'index de la lettre:

Code :

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

'maison'.index('a')

J

[lucie34]

J'ai testé votre code, mais ca n'a pas trouvé une lettre cherchée.

Oui, je dois utiliser for pour trouver la position d'une lettre dans un mot.

[lg_53]

La fonction de Julien fonctionne très bien. Soyez attentif et regarder d'un peu plus près les paramètres de la fonction de Julien, et les paramètres de votre fonction (en particulier l'ordre).

[wiztricks]

Je cherche la position d'une lettre dans un mot. Par ex. "a" est la position 1 du mot "maison".
Je vous remercie pour votre aide.

Mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
def find1st(c,s):
	i = 0
	lettre = s[0]
	for elt in range(len(s)):
		if s[elt] == s[i]:
			lettre = s[i]
			return i
		i = i + 1
	return None
 
print(find1st("i", "maison"))

Si dans votre code, la lettre cherchée est passée via le paramètre "c", il n'y a rien dans la boucle qui compare ce caractère là aux différents caractères de la chaîne. De plus, à quoi sert la variable "i" puisque celle qui est donnée par votre boucle s'appelle "elt"?
Votre code ne peut pas fonctionner, à vous de comprendre pourquoi en ajoutant des "print" pour visualiser l'état de vos variables à chaque itération: ce boulot là (comprendre pourquoi le code que vous avez écrit ne fonctionne pas en relisant le code en vous posant des questions sur ce qu'il fait ou pas) fait partie de ce que vous devez apprendre tout seul.

- W

[lucie34]

Merci, j'ai modifié le parametre, ca fonctionne.

La fonction de Julien fonctionne très bien. Soyez attentif et regarder d'un peu plus près les paramètres de la fonction de Julien, et les paramètres de votre fonction (en particulier l'ordre).

[Pyramidev]

Bonjour,

Mais bon, si la boucle for n'est pas obligatoire, vous pouvez directement demander l'index de la lettre:

Code :

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

'maison'.index('a')

Attention, 'maison'.index('e') ne retourne pas None, mais lance une exception ValueError, donc la méthode index n'est pas équivalente à look_for_letter.
'maison'.find('e'), lui, retourne -1.
Si on veut retourner None quand la lettre n'est pas trouvée, alors on peut écrire :

Code :

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

1
2
3
def look_for_letter(s, letter):
	x = s.find(letter)
	return x if (x != -1) else None

[tomsawyer1311]

Bonjour,

La fonction de Julien est bien, mais, si le mot a plusieurs fois la lettre qu'on cherche, ça ne retourne que la première occurrence. Si tu les veux toutes, tu peux déclarer une liste vide dans sa fonction en amont de la boucle puis agrémenter cette liste des indices de chaque occurrence :

Code python3 :

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

1
2
3
4
5
6
7
8
9
def look_for_letter(s, letter):
    occurrences = []
    for i, car in enumerate(s):
        if car == letter:
            occurrences.append(i)
    if occurrences:
        return occurrences
    else:
        return None

[wiztricks]

ça ne retourne que la première occurrence

çà c'est par défaut.

.find et .index savent aussi chercher une occurrence de à partir de et jusqu'à.

notez que côté interface, si votre fonction retourne la liste des index qui... la liste vide s'exprime par [] et non par None. Ça ne change rien côté valeur booléenne car bool([]) - et tout objet de taille nulle - est aussi False mais çà aide à écrire for index in look_for_letter sans avoir à tester la condition.

- W

[tomsawyer1311]

Merci Wiztricks je viens de piger en testant. Ma fonction, si la lettre n'est pas dans le mot m'indique une erreur comme quoi None n'est pas itérable. J'ai supprimé la condition et retourne la liste après le for. Ainsi, je n'ai pas d'erreur dans une expression telle

Code python 3 :

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

1
2
for occurrence in look_for_letter("maisons", "g"):
    print(occurrence)

[wiztricks]

J'ai supprimé la condition et retourne la liste après le for. Ainsi, je n'ai pas d'erreur dans une expression telle [CODE=python 3]for occurrence in look_for_letter("maisons", "g")

Voilà... et maintenant que vous avez "standardisé" le retour, vous pouvez simplifier le code:

Code :

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

look_for_letter = lambda s, letter: [ i for i, ch in enumerate(s) if ch == letter ]

- W

[tomsawyer1311]

Merci Wiztricks.

Je ne connais pas bien le mot clé lambda. Qu'apporte cette écriture par rapport à une définition de fonction, mise à part moins de lignes de code ?

[wiztricks]

Je ne connais pas bien le mot clé lambda. Qu'apporte cette écriture par rapport à une définition de fonction, mise à part moins de lignes de code ?

C'est pas le mot clef lambda qui est important mais la compréhension de liste: [ i for i, ch in enumerate(s) if ch == letter ] qui revient écrire classiquement (avec une fonction normale):

Code :

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

1
2
def look_for_letter(s, letter):
        return [ i for i, ch in enumerate(s) if ch == letter ]

Et si on peut le faire, c'est parce qu'on retourne une liste dans tous les cas (éventuellement une liste vide).
Et de là au "lambda":

Code :

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

look_for_letter = lambda s, letter: [ i for i, ch in enumerate(s) if ch == letter ]

Qu'est ce qu'apporte tout çà?
Un programme, c'est un tas de fonctionnalités à agencer. Qui dit "tas", dit plein de lignes de code où il va falloir discerner l'essentiel, le flux général. Et tout ce qui peut être réduit ainsi se lira bien plus facilement car très "carré".
Lorsqu'on débute çà n'a pas grande importance: c'est vous qui avez écrit le code, et quand il y a rarement plus de quelques centaines de lignes. Mais si vous devez lire/comprendre le code écrit pas d'autres pour trouver la cause d'un "bug", vous verrez qu'il est bien plus agréable de travailler sur/lire un code concis.

- W