Python — проблемы с генератором случайных имен детей — (дублирование ввода, вызов переменных)

Я смотрел на это весь день и не могу понять, почему ввод пола повторяется, несмотря на то, что он вызывается только один раз. Это не часть цикла, который я тоже вижу.

Я пробовал добавлять переменные, чтобы действовать как счетчик, и пытался использовать оператор if, чтобы запускать ввод только в том случае, если переменная счетчика меньше 1, но не могу понять это.

Редактировать: благодаря отличным отзывам здесь я узнал, что get_full_name вызывает дублирование ввода пола в get_first_name, но теперь я сталкиваюсь с проблемами при попытке вывести случайно сгенерированные имя и отчество.

Я решил установить переменные setFirst, setMiddle и setLast как глобальные, но потом получаю ошибку NameError. Я также пытался создать новую функцию для их отображения, но это тоже не сработало. Я попытался добавить «я». (без кавычек) либо непосредственно в функции(), либо в одном из отступов под ней.

Сначала я покажу ошибку, а затем полный код.

Ошибка: трассировка (последний последний вызов):
Файл "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 о том, что он не определен. Что я делаю не так?

Я добавил это прямо под «случайным импортом», но теперь я получаю несколько мошеннических отображений «Нет», что наводит меня на мысль, что где-то в коде есть утечка. Мысли?

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

person cydonia22    schedule 11.10.2015    source источник
comment
Что вы имеете в виду, говоря, что гендерный ввод повторяется? У нас нет этих файлов, и мы не можем видеть, что происходит. Юу придется рассказать нам.   -  person saulspatz    schedule 12.10.2015
comment
Хорошая мысль, извините. Вот пример вывода: Имя... Введите 1 для мужчины, 2 для женщины или 3, чтобы удивиться! 2 Фиона Кристин Джонс Имя... Введите 1 для мужчины, 2 для женщины или 3, чтобы удивиться!   -  person cydonia22    schedule 12.10.2015
comment
get_full_name появляется дважды в main   -  person Peter Wood    schedule 12.10.2015
comment
Кажется, вы звоните get_full_name() дважды, один раз для печати и один раз для записи. Это означает, что get_first_name вызывается дважды, каждый раз запрашивая пол. Может быть, вы хотите сохранить имя в переменной и распечатать/записать то же значение?   -  person zehnpaard    schedule 12.10.2015
comment
Я попытался запустить ваш код (интерпретатор 3.5), я получил KeyError при доступе к FILES dict из-за того, как вы обрабатываете ввод пола. Вы должны создать тип int из введенного значения str, определяющего ввод, считанный из консоли, следующим образом: determine = input() determine = int(determine)   -  person Szilárd Németh    schedule 12.10.2015
comment
Интересно, спасибо Петр. Комментирование одной из ссылок get_full_name действительно устраняет повторяющиеся входные данные, но затем мне нужно найти способ отобразить случайно сгенерированное имя в консоли и вывести то же имя в плоский файл, требуя ввода только один раз. Какие-либо предложения?   -  person cydonia22    schedule 12.10.2015
comment
Я думаю, что сохранение результата get_full_name() в локальную переменную и использование этой переменной в write() - это правильный путь.   -  person Szilárd Németh    schedule 12.10.2015
comment
Здесь отличное сообщество! Спасибо за все быстрые ответы и предложения. Я попробую локальный переменный маршрут. Это моя первая настоящая попытка освоить Python, ранее я пробовал C++ и SQL много лет назад. Пытаюсь научить себя по двум причинам: создать этот генератор случайных имен для моей жены и подготовить себя к будущему классу, который я, вероятно, буду посещать в следующем году.   -  person cydonia22    schedule 12.10.2015
comment
На мой взгляд, Python — хороший выбор для изучения программирования или обучения :)   -  person Szilárd Németh    schedule 12.10.2015
comment
@SzilárdNémeth - есть мысли о том, как устранить ошибку NameError? Я просматривал другие темы и пытался найти несколько вещей в Google, но каждый продолжал ошибаться.   -  person cydonia22    schedule 12.10.2015
comment
Не могли бы вы уточнить, какую NameError вы имеете в виду? Спасибо!   -  person Szilárd Németh    schedule 12.10.2015
comment
Трассировка (последний последний вызов): файл init.py, строка 100, в ‹module› main(setFirst, setMiddle, setLast) NameError: глобальное имя 'setFirst' не определено   -  person cydonia22    schedule 12.10.2015
comment
Другие потоки, похоже, упоминают, что добавление себя в функцию или каким-либо другим способом, но, похоже, это не устраняет ошибку. Может я не правильно его использую? См. выше...   -  person cydonia22    schedule 12.10.2015


Ответы (2)


Вы звоните 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')
person Burhan Khalid    schedule 11.10.2015
comment
Спасибо, Бурхан - получаю синтаксическую ошибку: файл init.py, строка 36, если name_type в 'first', 'middle': ^ SyntaxError: недопустимый синтаксис - person cydonia22; 12.10.2015
comment
Спасибо за исправление, но теперь TraceBack выдает другие ошибки: 1 — неверный выбор, попробуйте еще раз Traceback (последний вызов последний): файл init.py, строка 51, в ‹module › имя = get_full_name() Файл init.py, строка 42, в get_full_name пол = get_gender() Файл init.py, строка 20, в get_gender возвращает get_gender_input() NameError : глобальное имя get_gender_input не определено - person cydonia22; 12.10.2015
comment
Измените его на get_gender(), так как я немного изменил имена методов. - person Burhan Khalid; 12.10.2015
comment
Он будет работать без ошибок, однако теперь он не будет принимать никаких входных данных, и мне нужно убить его, чтобы выйти. Мысли? Выберите пол: 1 для мужчин, 2 для женщин или 3 для удивления 1 1 недопустимый выбор, попробуйте еще раз Выберите пол: 1 для мужчин, 2 для женщин или 3 для удивления 2 2 не является допустимым выбором , попробуйте еще раз Выберите пол: 1 для мужчин, 2 для женщин или 3 для удивления 3 3 недопустимый выбор, попробуйте еще раз - person cydonia22; 12.10.2015
comment
Вы используете Python2, поэтому вам следует использовать raw_input, а не input. - person Burhan Khalid; 12.10.2015
comment
У меня установлен Python 3.5, извините, это моя первая попытка Python, что заставляет вас думать, что это Python 2? - person cydonia22; 12.10.2015

Вероятно, get_full_name вызывается дважды, и это приводит к тому, что другие функции, на которые ссылается get_full_name, также вызываются дважды. К сожалению, это означает двойной ввод и сбивает пользователя с толку.

person Gustavo6046    schedule 11.10.2015
comment
Я пытаюсь найти способ сохранить вывод в отдельную переменную в main(), чтобы я мог сохранить его в файл и отобразить пользователю. Мы обнаружили, что, включив эту функцию в вывод в файл, она затем снова вызвала get_first_name, что вызвало дополнительный ввод. - person cydonia22; 12.10.2015