Почему не работает решение для задачи с Codewars?

Question

[SushkaCraftHabr]

Решил я порешать задачки с Codewars, начал с простеньких и дошёл до задачи "Well of Ideas - Easy Version".
Я написал такое решение:

def well(x):
    
    n = 0
    
    for j in x:
        for i in j:
            if str(i).lower() == 'good':
                n += 1
    if n == 1 or n == 2:
        return 'Publish!'
    
    elif n > 2:
        return 'I smell a series!'
    
    return 'Fail!'

Но ни один тест не проходит, почему?

Comments

[serhiops]

задание мы сами должны искать?

[SushkaCraftHabr]

serhiops, прошу прощения. Вот задание-

For every good kata idea there seem to be quite a few bad ones!

In this kata you need to check the provided array (x) for good ideas 'good' and bad ideas 'bad'. If there are one or two good ideas, return 'Publish!', if there are more than 2 return 'I smell a series!'. If there are no good ideas, as is often the case, return 'Fail!'.

ссылка на него - https://www.codewars.com/kata/57f222ce69e09c3630000212

[Vladimir Tolstov]

serhiops, https://www.codewars.com/kata/57f222ce69e09c3630000212/

Answer 1

[Vladimir Tolstov]

def well(x):
    good = x.count('good')
    
    if good == 0:
        return 'Fail!'
    
    elif good == 1 or good == 2:
        return 'Publish!'
    
    elif good > 2:
        return 'I smell a series!'

У меня сработало такое решение

Comments

[SushkaCraftHabr]

Спасибо, а то я уже думал, что придется решать задачу как-нибудь через итераторы и бесконечные циклы...

[Vladimir Tolstov]

2. а зачем считать bad?

Не подумал на счет этого....
Действительно не нужно.

Answer 2

[serhiops]

Делюсь своими тремя примерами, скорость выполнения указана в коментариях к каждой функции

def well(x):
    #  0.05ms
    a = [i for i in x if i == 'good']
    if 3 > len(a) > 0:
        return 'Publish!'
    elif len(a) > 2:
        return 'I smell a series!'
    return 'Fail!'

from collections import Counter
def well2(x):
    # 0.19ms
    c = Counter(x)
    if 3 > c['good'] > 0:
        return 'Publish!'
    elif c['good'] > 2:
        return 'I smell a series!'
    return 'Fail!'

def well3(x):
    # 0.11 ms
    countOfGoodIdeas = 0
    for i in x:
        if i == 'good':
            countOfGoodIdeas+=1
    if 3 > countOfGoodIdeas > 0:
        return 'Publish!'
    elif countOfGoodIdeas > 2:
        return 'I smell a series!'
    return 'Fail!'