Python - 随机婴儿名称生成器问题 - （重复输入，调用变量）

Question

我整个下午都在看这个，不明白为什么性别输入会重复出现，尽管似乎只被调用了一次。它也不是我能看到的循环的一部分。

我尝试添加变量以充当计数器，并尝试使用 if 语句仅在计数器变量小于 1 时运行输入，但无法弄清楚。

编辑：感谢这里的大量反馈，我发现 get_full_name 导致 get_first_name 中的重复性别输入 - 但现在我在尝试输出随机生成的名字和中间名时遇到了问题。

我想将 setFirst、setMiddle 和 setLast 变量设置为全局变量，但随后出现 NameError。我还尝试创建一个新函数来显示它们，但这也不起作用。我尝试添加“自我”。 （不带引号）直接在 function() 或它下面的缩进之一。

我会先显示错误，然后显示完整代码。

错误： 回溯（最近一次通话最后一次）：
文件“init.py”，第 100 行，在 主要（）
文件“init.py”，第 92 行，在 main
打印 displayName(setFirst, setMiddle, setLast)
NameError：未定义全局名称“setFirst”

尝试将 setFirst、setMiddle 和 setLast 连接到另一个变量以获得全名时，我也会遇到名称错误。

代码如下：

from os.path import abspath, join, dirname
import random

full_path = lambda filename: abspath(join(dirname(__file__), filename))


FILES = {
    'first:male': full_path('dist.male.first'),
    'first:female': full_path('dist.female.first'),
    'last': full_path('dist.all.last'),
}


def get_name(filename):
    selected = random.random() * 90
    with open(filename) as name_file:
        for line in name_file:
            name, _, cummulative, _ = line.split()
            if float(cummulative) > selected:
                return name


def get_first_name(gender=None):
    global determine
    global setFirst

    print ("First name... Enter 1 for Male, 2 for Female or 3 to be surprised! ")
    determine = input()

    if determine == 1:
        gender = 'male'
    if determine == 2:
        gender = 'female'
    if determine == 3:
        print ("You want to be surprised!")
        gender = random.choice(('male', 'female'))

    return get_name(FILES['first:%s' % gender]).capitalize()
    setFirst = get_first_name()
    print setFirst + " "

def get_middle_name(gender=None):
    global setMiddle

    if determine == 1:
        gender = 'male'
    if determine == 2:
        gender = 'female'
    if determine == 3:
        gender = random.choice(('male', 'female'))

    return get_name(FILES['first:%s' % gender]).capitalize()
    setMiddle = get_middle_name()
    print setMiddle + " "

def get_last_name():
    global setLast

    #We will implicitly pass a Last Name until other issues are fixed
    return “Smith”

    setLast = get_last_name()
    print setLast

def get_full_name(gender=None):
    return u"%s %s %s" % (get_first_name(gender), get_middle_name(gender), get_last_name())

#def displayName(setFirst, setMiddle, setLast):
#    print setFirst + " " + setMiddle + " " + setLast

def main():
    #print u"%s %s %s" % (setFirst, setMiddle, setLast)
    #print displayName(setFirst, setMiddle, setLast)

    f = open('output', 'a') #append output to filename output
    f.write(get_full_name() + '\n') #and add a line break after each run
    f.close()

if __name__ == "__main__":
    main()

即使我尝试将变量传递给 main()，例如：

def main(setFirst, setMiddle, setLast):

它仍然给出关于未定义的 NameError。我做错了什么？

我在“导入随机”下添加了这个，但现在我得到了一些流氓“无”显示 - 这让我相信某处的代码存在泄漏。想法？

setFirst = None
setMiddle = None
setLast = None

这是我创建的用于跟踪它的函数： def displayName(setFirst, setMiddle, setLast):

if setFirst == None:
    print ("Random Baby Name Generator")

else:
    print setFirst
    print setMiddle
    print setLast

if setMiddle == None:
    print ("Double check the middle name variable.")

if setLast == None:
    print ("Double check the last name variable.")

Answer 1

您调用了两次get_full_name()，需要保存结果：

def main():
    full_name = get_full_name()
    print(full_name)
    f = open('output', 'a') #append output to filename output
    f.write(full_name + '\n') #and add a line break after each run
    f.close()

您也有一些缩进问题，而且您对全局变量的使用有点低效。理想情况下，函数应该执行一项且仅一项任务；这使它们更容易调试。

试试这个不同版本的代码：

from os.path import abspath, join, dirname
import random

full_path = lambda filename: abspath(join(dirname(__file__), filename))


FILES = {
    'first:male': full_path('dist.male.first'),
    'first:female': full_path('dist.female.first'),
    'last': full_path('dist.all.last'),
}

GENDER_MAP = {'1': 'male', '2': 'female'}

def get_gender():
    result = input('Select a gender: 1 for Male, 2 for Female or 3 to be surprised')
    if result not in ('1', '2', '3'):
       print('{} is not a valid choice, please try again'.format(result))
       return get_gender()
    if result == '3':
       return random.choice(('1', '2'))
    return result

def get_name(filename):
    selected = random.random() * 90
    with open(filename) as name_file:
        for line in name_file:
            name, _, cummulative, _ = line.split()
            if float(cummulative) > selected:
                return name

def get_name_from_file(name_type='first', gender='male'):
    if name_type in ('first','middle',):
        name = get_name(FILES['{}:{}'.format(name_type, gender)]).capitalize()
    else:
        name = get_name(FILES['last']).capitalize()
    return name

def get_full_name():
    gender = get_gender()
    gender_file = GENDER_MAP.get(gender, '')
    first_name = get_name_from_file('first', gender_file)
    middle_name = get_name_from_file('middle', gender_file)
    last_name = get_name_from_file('last')

    return '{} {} {}'.format(first_name, middle_name, last_name)

if __name__ == '__main__':
    name = get_full_name()
    print(full_name)
    with open('output', 'a') as f:
       f.write('{}\n'.format(full_name))
    print('Done')

Answer 2

get_full_name 可能被调用了两次，这导致get_full_name 引用的其他函数也被调用了两次。不幸的是，这意味着双重输入和混淆用户。