Zählen von Buchstaben in einem Wort in python-debug -

Als andere erwähnen, die Sie implementieren können, der test mit einem einfachen word.count('e'). Es sei denn, Sie tun dies als eine einfache übung, diese ist weit besser als zu versuchen das Rad neu zu erfinden.

Das problem mit deinem code ist, dass es zählt das Letzte Zeichen zweimal, weil Sie den Test-index -1 am Ende, die in Python gibt das Letzte Zeichen in der Zeichenfolge. Fixieren Sie es, indem Sie while letters >= 0 zu while letters > 0.

Gibt es andere Möglichkeiten, wie Sie können, Räumen Sie Ihren code (vorausgesetzt, dies ist eine übung im lernen):

• Python bietet eine schöne Möglichkeit der Iteration über einer Zeichenfolge mit einem for Schleife. Das ist weit übersichtlicher und einfacher zu Lesen als mit einem while Schleife und die Pflege der eigenen counter-variable. Wie Sie bereits gesehen haben hier, hinzufügen von Komplexität führt zu Fehlern. Halten Sie es einfach.
• Die meisten Sprachen bieten eine += - operator, welcher für ganze zahlen addiert den Betrag einer Variablen. Es ist präziser als count = count + 1.
• Verwenden, einen parameter zu definieren, welche Zeichen man zählen es flexibler zu machen. Definieren Sie ein Standard-argument für die Verwendung von char='e' in der Liste der parameter, wenn Sie einen offensichtlichen Standardwert.
• Wählen Sie einen geeigneten Namen für die Funktion. Der name has_no_e() lässt den Leser denken, der code überprüft, um zu sehen, ob der code hat keine e -, aber was es tatsächlich ist, zählt die vorkommen von e.

Umsetzung dieses alle zusammen, erhalten wir:

def count_letter(word, char='e'):
    count = 0
    for c in word:
        if c == char:
            count += 1
    return count

Einige tests:

>>> count_letter('tee')
2
>>> count_letter('tee', 't')
1
>>> count_letter('tee', 'f')
0
>>> count_letter('wh' + 'e'*100)
100

InformationsquelleAutor marcog

Warum nicht einfach

def has_no_e(word):
    return sum(1 for letter in word if letter=="e")

InformationsquelleAutor Tim Pietzcker

Wird es doppelt, wenn die Endung mit einem e, weil Sie Dekrement letters einmal zu oft (weil Sie loop während letters >= 0 und Sie sollten looping, während letters > 0). Wenn letters null erreicht-Sie überprüfen word[letters-1] == word[-1] das entspricht dem letzten Buchstaben im Wort.

InformationsquelleAutor Klaus Byskov Pedersen

@marcog macht einige ausgezeichnete Punkte;

in der Zwischenzeit können Sie tun, einfache debugging durch einfügen von print-Anweisungen -

def has_no_e(word):
    letters = len(word)
    count = 0
    while letters >= 0:
        ch = word[letters-1]         # what is it looking at?
        if ch == 'e':
            count = count + 1
            print('{0} <-'.format(ch))
        else:
            print('{0}'.format(ch))
        letters = letters - 1
    print count

dann

has_no_e('tease')

gibt

e <-
s
a
e <-
t
e <-
3

aus denen Sie sehen können, dass

1. du gehst durch den string in umgekehrter Reihenfolge
2. es ist richtig zu erkennen e s
3. Sie sind 'umwickeln', um das Ende der Schnur - daher der extra-e, wenn die Zeichenfolge endet in einem

InformationsquelleAutor Hugh Bothwell

Sie nicht haben, um eine while-Schleife. Zeichenketten können verwendet werden for-Schleifen in Python.

def has_no_e(word):
    count = 0
    for letter in word:
        if letter == "e":
            count += 1
    print count

oder etwas einfacher:

def has_no_e(word):
    return sum(1 for letter in word if letter=="e")

InformationsquelleAutor Nope