Python - 随机婴儿名字生成器问题 - (重复输入，调用变量)

Question

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

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

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

我想将 setFirst、setMiddle 和 setLast 变量设置为全局变量，但后来我得到了 NameError。我还尝试创建一个新功能来显示它们，但这也不起作用。我尝试添加“ self ”。 (不带引号)直接在 function() 中或它下面的缩进之一。

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

错误: 回溯(最近调用最后):
文件“init.py”，第 100 行，位于 主要()
主
中的文件“init.py”，第 92 行 打印显示名称(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')