(Python 2.7) Use A List As An Argument In A Function?
Solution 1:
There are a handful of problems here:
- You're trying to iterate over
fizz_count
. Butfizz_count
is your function.x
is your passed-in argument. So it should befor x in x:
(but see #3). - You're accepting one argument with
*x
. The*
causesx
to be a tuple of all arguments. If you only pass one, a list, then the list isx[0]
and items of the list arex[0][0]
,x[0][1]
and so on. Easier to just acceptx
. - You're using your argument,
x
, as the placeholder for items in your list when you iterate over it, which means after the loop,x
no longer refers to the passed-in list, but to the last item of it. This would actually work in this case because you don't usex
afterward, but for clarity it's better to use a different variable name. - Some of your variable names could be more descriptive.
Putting these together we get something like this:
def fizz_count(sequence):
count = 0
for item in sequence:
if item == "fizz":
count += 1
return count
I assume you're taking the long way 'round for learning porpoises, which don't swim so fast. A better way to write this might be:
def fizz_count(sequence):
return sum(item == "fizz" for item in sequence)
But in fact list
has a count()
method, as does tuple
, so if you know for sure that your argument is a list or tuple (and not some other kind of sequence), you can just do:
def fizz_count(sequence):
return sequence.count("fizz")
In fact, that's so simple, you hardly need to write a function for it!
Solution 2:
when you pass *x
to a function, then x
is a list. Do either
def function(x):
# x is a variable
...
function('foo') # pass a single variable
funciton(['foo', 'bar']) # pass a list, explicitly
or
def function(*args):
# args is a list of unspecified size
...
function('foo') # x is list of 1 element
function('foo', 'bar') # x is list with two elements
Solution 3:
Your function isn't taking a list as an argument. *x
expands to consume your passed arguments, so your function is expecting to be called like this:
f(1, 2, 3)
Not like this:
f([1, 2, 3])
Notice the lack of a list object in your first example. Get rid of the *
, as you don't need it:
# Write your function below!
def fizz_count(lst):
count = 0
for elem in lst:
if elem == "fizz":
count += 1
return count
You can also just use list.count
:
# Write your function below!
def fizz_count(lst):
return lst.count('fizz')
Solution 4:
It must be a typo. You're trying to iterate over the function name.
Solution 5:
try this:
def fizz_count(x):
counter = 0
for element in x:
if element == "fizz":
counter += 1
return counter
Post a Comment for "(Python 2.7) Use A List As An Argument In A Function?"