Беспорядок в итерациях Python

Как вы можете видеть ниже, я пытаюсь создать программу, которая берет файл с именем named.txt, разбивает его на список с помощью .split('/n'), а затем добавляет пароль для каждого пользователя в списке. По какой-то причине for look обрабатывает список, как если бы это была строка, поэтому вместо того, чтобы рассматривать его как:

Боб Чарли Рон

Он разбивает его так:

Б о б

С час .... Таким образом, давая каждой букве пароль вместо самого списка.

Что я делаю неправильно? Есть ли способ проверить, что на самом деле дается цикл for?

Я играл с ним, чтобы увидеть, была ли ему дана строка, но список выглядит как список при печати, так почему он будет обрабатываться как строка в цикле for?

def importNames():

    try:
        file = open("names.txt", "r")
        contents = file.read()
        print(contents)
        file.close()

        return contents

    except:
        print("The file was not found.")
        print("Make sure you have a file called \"names.txt\" in the working directory (where you ran this program)")

        exit()

def createPasswordList(nameList):
    print("They are being given the standard password of \"p@ssw0rd.\"")
    #nameList = ["Pickles", "Bob's", 'wow zers john'] #This is for testing purposes. It works just fine, but the original nameList breaks down badly. 
    passList = []

    #for x, value in enumerate(nameList):
    for i in nameList:
        print(i)
        passList.append("p@ssw0rd")

    print(passList)
    return passList

def createUsers(nameList, passwordList): #takes in a list of usernames, then adds creates them. 
    for i in nameList:
        print("This will do something soon.")
        #encPass = crypt.crypt(passwordList[i],"22") #22 is a salt number, use crypt per useradd manual
        #os.system("useradd -p " + encPass + " " + nameList[i]) #useradd -p encryptedpass username

def convertNamesToList(originalList):
    print("Converting names.") #Use newline as a delimiter
    newList = originalList.split('\n')
    print(newList)
    return newList

def convertNameToUsernames(originalNames):
    print("Creating usernames")

print("This program is used to make some users.")
print("If you import a file, have it in the working directory called \"names.txt\".")

print("Would you like to import names or add them manually? (n/m)")
answer = input()

if (answer is 'm' or answer is 'M'):
    createUser()
    print("The user gets created. Good job.")

else:
    originalNames = importNames() #this will return a string of the file.
    convertNamesToList(originalNames) #convert the string to sets of names. Use newlines as delimiters. 
    #convert the names to usernames

    print("Do the users have passwords? (y/n)")
    #answer = input()
    answer = 'n' ###FORCE###
    if (answer is 'n' or answer is 'N'):
        passwordList = createPasswordList(originalNames)


'convertNamesToList(originalList)' не называйте его списком, если это строка. Есть и другие опечатки, но эта сбила меня с толку более чем на 10 секунд. И некоторые функции не используются/не нужны ради заданного вопроса. Пожалуйста, отредактируйте вопрос, чтобы он содержал только соответствующий код вопроса.

BrainDead 07.04.2019 01:07

Не спамить неактуальными тегами

Marcin Orlowski 07.04.2019 01:16
Почему в Python есть оператор "pass"?
Почему в Python есть оператор "pass"?
Оператор pass в Python - это простая концепция, которую могут быстро освоить даже новички без опыта программирования.
Некоторые методы, о которых вы не знали, что они существуют в Python
Некоторые методы, о которых вы не знали, что они существуют в Python
Python - самый известный и самый простой в изучении язык в наши дни. Имея широкий спектр применения в области машинного обучения, Data Science,...
Основы Python Часть I
Основы Python Часть I
Вы когда-нибудь задумывались, почему в программах на Python вы видите приведенный ниже код?
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
Алиса и Боб имеют неориентированный граф из n узлов и трех типов ребер:
Оптимизация кода с помощью тернарного оператора Python
Оптимизация кода с помощью тернарного оператора Python
И последнее, что мы хотели бы показать вам, прежде чем двигаться дальше, это
Советы по эффективной веб-разработке с помощью Python
Советы по эффективной веб-разработке с помощью Python
Как веб-разработчик, Python может стать мощным инструментом для создания эффективных и масштабируемых веб-приложений.
0
2
39
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Ответ принят как подходящий

Вы никогда ничего не делаете с возвращаемым значением из convertNamesToList. Вы его нигде не храните. Затем вы передаете строку originalNames (вы даже указываете это в комментарии) в createPasswordList.

Я думаю, вы хотите что-то вроде:

originalNames = importNames() #this will return a string of the file.
listOfNames = convertNamesToList(originalNames) #convert the string to sets of names. Use newlines as delimiters. 
#convert the names to usernames

print("Do the users have passwords? (y/n)")
#answer = input()
answer = 'n' ###FORCE###
if (answer is 'n' or answer is 'N'):
    passwordList = createPasswordList(listOfNames)

Похоже, что ваш nameList на самом деле является строкой. Предполагая, что у вас есть одно имя в строке файла names.txt, попробуйте следующее:

def importNames():
    try:
        file = open("names.txt", "r")
        contents=[]
        for line in file.read():
            contents.append(line)
        print(contents)
        file.close()
        return contents

Затем вам не нужно преобразовывать его в список позже.

Другие вопросы по теме