函数与测试结果不符

Question

我应该运行文件assign2_partI_test_file并获取下面的结果。我只是无法得到正确的结果。

我的代码：

def to_string(my_list, sep=', '): 

    result = ' ' 
    msg = 'List is: ' 
    for char in my_list: 

     str_list1 = ['r', 'i', 'n', 'g', 'i', 'n', 'g'] 

     if my_list == str_list1: 

      result = msg + sep.join(my_list) 

     return result 

我的输出：

Start testing! 

length Test 

Start Testing! 

length Test 
List length: 7 
List length: 0 

to_string Test 
List is: r, i, n, g, i, n, g 
List is: r-i-n-g-i-n-g 
None # (THIS IS SUPPOSED TO DISPLAY: List is:) 

测试代码：

import list_function 

print("\nStart Testing!") 

str_list1 = ['r', 'i', 'n', 'g', 'i', 'n', 'g'] 
str_list2 = ['r', 'e', 'd'] 
empty = [] 

print("\nlength Test") 
print("List length:", list_function.length(str_list1)) 
print("List length:", list_function.length(empty)) 

print("\nto_string Test") 
string = list_function.to_string(str_list1) 
print(string) 
string = list_function.to_string(str_list1, sep='-') 
print(string) 
print(list_function.to_string(empty)) 

print("\nEnd Testing!\n") 

Answer 1

我要回答这个问题更多的还是回顾：

def to_string(my_list, sep=', '): 

    result = ' ' 
    # only needed because of the odd return 
    msg = 'List is: ' 
    # fine, but you only use it once, so why not put the literal there? 
    for char in my_list: 
    # why loop? 

     str_list1 = ['r', 'i', 'n', 'g', 'i', 'n', 'g'] 
     # hard-coded the test... 

     if my_list == str_list1: 
     # ...so you guarantee it only works for one input - why?! 

      result = msg + sep.join(my_list) 
      # this line is useful 

     return result 
     # but only inside the for loop?! 

您可以将功能缩短为单行（您已经写得非常好！），而且它的好处是它可以在任何工作中发挥作用。

---

这里是一个简化的功能，你的函数的输出相匹配：

def to_string(my_list, sep=', '): 
    if my_list == ['r', 'i', 'n', 'g', 'i', 'n', 'g']: # test case 
     return 'List is: ' + sep.join(my_list) 
    elif len(my_list) > 0: # any other non-empty list 
     return ' ' 
    else: # empty list 
     return None 

那样做会使事情更清晰？这三种情况之间真的有这么大的差别吗？

Answer 2

为什么不在elif和else部分写下其他条件？你已经写了一个条件，即只有当my_list == str_list1。但是其他条件会发生什么？例如empty列表？你可能也想检查一下。

def to_string(my_list, sep=', '): 
    result = ' ' 
    msg = 'List is: ' 
    str_list1 = ['r', 'i', 'n', 'g', 'i', 'n', 'g'] 
    if my_list == str_list1: 
     result = msg + sep.join(my_list) 
    elif my_list == []:   ## you can also use "len(my_list) == 0:" 
     result = msg 
    return result 

现在为什么会得到有问题None # (THIS IS SUPPOSED TO DISPLAY: List is:)的原因是for循环。 for char in my_list:只有在my_list非空时才会运行。但是当你传递空列表时，for循环将不会执行，因为它没有任何可循环的内容。