Python - trouver des chiffres dans une chaîne de caractères

def get_digits(str1):
    c = ""

    for i in str1:
        if i.isdigit():
            c += i
            return c

Ci-dessus est le code que j'ai utilisé et le problème c'est qu'il ne renvoie uniquement le premier chiffre de chaînes de caractères. Pour cela, je dois garder les deux pour la boucle et de l'instruction return. Quelqu'un sait comment résoudre ce problème?

Grâce.

Est-ce devoirs? Si oui, veuillez étiqueter comme tel.
J'ai ajouté un test sur ma réponse.

OriginalL'auteur Kevvv | 2012-08-17

python

9

Votre retrait est un peu complètement foireuse (indentation en Python est très important). Mieux:
```
def get_digits(str1):
    c = ""
    for i in str1:
        if i.isdigit():
            c += i
    return c
```
Plus courte et la plus rapide solution à l'aide de générateur d'expressions:
```
''.join(c for c in my_string if c.isdigit())
```
C'est un générateur d'expression, pas une compréhension de liste, mais +1 pour la manière.
oh, vous êtes 🙂
est-il un meilleur endroit que le PEP pour la documentation sur le générateur d'expressions?
J'ai édité votre réponse à la lien vers la partie du Tutoriel Python, il y a aussi le langue de référence.
génial, merci!

OriginalL'auteur Benjamin Wohlwend

Comme les autres ont dit, vous avez un problème sémantique sur votre indentation, mais vous n'avez pas à écrire une telle fonction pour le faire, plus pythonic façon de le faire est:

def get_digits(text):
    return filter(str.isdigit, text)

Sur l'interprète:

>>> filter(str.isdigit, "lol123")
'123'

Quelques conseils

Toujours tester des choses en vous-même quand les gens s'affiche "plus vite" méthodes:

from timeit import Timer

def get_digits1(text):
    c = ""
    for i in text:
        if i.isdigit():
            c += i
    return c

def get_digits2(text):
    return filter(str.isdigit, text)

def get_digits3(text):
    return ''.join(c for c in text if c.isdigit())

if __name__ == '__main__':
    count = 5000000
    t = Timer("get_digits1('abcdef123456789ghijklmnopq123456789')", "from __main__ import get_digits1")
    print t.timeit(number=count)

    t = Timer("get_digits2('abcdef123456789ghijklmnopq123456789')", "from __main__ import get_digits2")
    print t.timeit(number=count)

    t = Timer("get_digits3('abcdef123456789ghijklmnopq123456789')", "from __main__ import get_digits3")
    print t.timeit(number=count)



~# python tit.py
19.990989106  # Your original solution
16.7035926379 # My solution
24.8638381019 # Accepted solution

OriginalL'auteur Tarantula

2

c'est parce que votre return déclaration est à l'intérieur de la boucle for, on en revient donc après la première vraie if condition et s'arrête.
```
  def get_digits(str1):
      c = ""
      for i in str1:
        if i.isdigit():
            c += i
      return c
```
OriginalL'auteur Ashwini Chaudhary
1

Bien sûr, il ne retourne que le premier chiffre, vous indiquez explicitement Python de retour dès que vous avez un chiffre.

Modifier l'indentation de la return déclaration et cela devrait fonctionner:
```
def get_digits(str1):
    c = ""

    for i in str1:
        if i.isdigit():
            c += i

    # Note the indentation here
    return c
```
OriginalL'auteur Some programmer dude

il y a un problème d'indentation qui revient quand on trouve le premier chiffre, comme avec indentation courante, il est intepreted comme une déclaration à l'intérieur de if déclaration,
il doit être parallèle à la for déclaration
pour être considéré en dehors de for déclaration.

def get_digits(str1):
    c = ""

    for i in str1:
        if i.isdigit():
            c += i

    return c

digits = get_digits("abd1m4m3m22mmmbb4")
print(digits)

Un des accolades équivalent de votre code incorrect est :

def get_digits(str1){
    c = ""

    for i in str1 {
        if i.isdigit(){
            c += i
            return c    # Notice the error here
        }
    }
}

Et lorsque le code est corrigé pour déplacer l'instruction de retour dans l'alignement avec for, l'équivalent est :

def get_digits(str1){
        c = ""

        for i in str1 {
            if i.isdigit(){
                c += i               
            }
        }
        return c    # Correct as required
    }

OriginalL'auteur DhruvPathak

1

Votre code était presque ok, sauf la return déclaration devait être déplacé vers le niveau de votre foren boucle.
```
def get_digits(str1):
    c = ""
    for i in str1:
        if i.isdigit():
            c += i
    return c   ## <--- moved to correct level
```
donc, maintenant:
```
get_digits('this35ad77asd5')
```
rendements:
```
'35775'
```
Explication:

Précédemment, votre fonction est de retour seulement le premier chiffre parce que quand il a trouvé l'une de l' if déclaration a été exécuté, comme ce fut le return (vous causer de retour de la fonction qui veut dire que vous n'avez pas à continuer la recherche par le biais de la chaîne).

Espaces/indentation vraiment questions en Python comme vous pouvez le voir (à la différence de beaucoup d'autres langues).

OriginalL'auteur Levon

Vous devez vous connecter pour publier un commentaire.