为什么自动分级器给我关于 CodeHS 8.4.9: Owls Part 2 的错误？

Question

Here's the assignment:

This program is an extension of your previous 'Owls' program. In addition to just reporting how many words contained the word owl, you should report the indices at which the words occurred! Here's what an example run of your program might look like:

Enter some text: Owls are so cool! I think snowy owls might be my favorite. Or maybe spotted owls.
There were 3 words that contained "owl".
They occurred at indices: [0, 7, 15]

As you can see from the output, you'll have to use another list to store the indices where you found the words containing “owl”. The enumerate function might also come in handy!

Here's my code now:

def owl_count(text):
owl_lower = text.lower()
owl_split = owl_lower.split()
count = 0
index = 0
sec_in = []
owl = "owl"
for i in range(len(owl_split)):
    if owl in owl_split[index]:
        count = count + 1
        sec_in.append(index)
    index = index + 1
print("There were " + str(count) + " words that contained owl.")
return "They occurred at indices: " + str(sec_in)

text = "I really like owls. Did you know that an owl's eyes are more than twice as big as the eyes of other birds of comparable weight? And that when an owl partially closes its eyes during the day, it is just blocking out light? Sometimes I wish I could be an owl."
print(owl_count(text))

When I run the code it's perfectly fine. But when it has to go through the autograder it says I'm doing things wrong. And this input is what's getting me the least errors. Here are some of the ones I used before if it helps:

Owlets are baby owls. Baby peafowls are called peachicks.

Owls are so cool! I think snowy owls might be my favorite. Or maybe spotted owls.

I think owls are pretty cool

Here's the link to the code I used to help me.

First autograder picture

Second autograder picture

Third autograder picture

Answer 1

As mentioned in the comments, your code has a lot of redundant variables - this is a tidied up version of your code - it should work exactly the same as yours :

I think the biggest issue is that your code prints the count line, and then returns the indices line. If the autograder simply executes your function and ignores the return value it will ignore the indices. Note that the example code you cite prints both lines - and doesn't return anything. This is corrected in the version below.

Note that this code uses enumerate - it is a very good function to get into the habit of using if you need the contents of a list and you need to track the index in the list too.

def owl_count(text):
    owl_lower = text.lower()
    owl_split = owl_lower.split()
    sec_in = []
    owl = "owl"
    for index, word in enumerate(owl_split):
        if owl in word:
           sec_in.append(index)
    print("There were " + str(len(sec_in)) + " words that contained \"owl\".")
    print("They occurred at indices: " + str(sec_in))

text = "I really like owls. Did you know that an owl's eyes are more than twice as big as the eyes of other birds of comparable weight? And that when an owl partially closes its eyes during the day, it is just blocking out light? Sometimes I wish I could be an owl."
owl_count(text)

There is a more efficient method to solve the problem, without a lot of variables that are only used to create others - and you have a for loop which could/should be a comprehension - so an even better version would be :

def owl_count(text):
    sec_in = [index for index, word in enumerate(text.lower().split())
                      if 'owl' in word]
    print("There were " + str(len(sec_in)) + " words that contained \"owl".")
    print("They occurred at indices: " + str(sec_in))

Update - 02/11/2020 - 14:12 - The expected result from the autograder expects quotes around the word 'owl' in the first output message. The above code has been updated to include those quotes.