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

时间:2015-10-11 21:21:49

标签: python python-3.x random dynamically-generated python-3.5

我整个下午一直在关注这个问题,并且无法弄清楚为什么性别输入会重复,尽管只是被称为一次。它也不是我能看到的循环的一部分。

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

编辑:由于这里的反馈很好,我发现get_full_name导致get_first_name中的重复性别输入 - 但是现在我在尝试输出随机生成的第一个&时遇到了问题。中间名。

我想把setFirst,setMiddle和setLast变量设置为全局变量,但后来我得到一个NameError。我也尝试创建一个新功能来显示它们,但那也没有用。我尝试添加“自我”。 (不带引号)直接在function()或其下的一个缩进中。

我将首先显示错误,然后显示完整的代码。

错误:     追溯(最近的呼叫最后):
      文件“ init .py”,第100行,在         main()
      在主要文件中输入“ init .py”,第92行         print 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。我做错了什么?

我在“import random”下添加了这个权利,但是现在我得到了一些流氓“无”显示 - 这让我相信某处某处有代码泄露。想法?

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.")

2 个答案:

答案 0 :(得分: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')

答案 1 :(得分:0)

可能会调用

get_full_name两次,这导致get_full_name引用的其他函数被调用两次。不幸的是,这意味着双重输入并使用户感到困惑。