Итерация по моему списку не возвращает значение, которое я ожидал - PullRequest
Новая
       5

Итерация по моему списку не возвращает значение, которое я ожидал

0 голосов
/ 23 февраля 2020

Я читаю через CSV и сохраняю одно из значений в списке. Когда я пытаюсь сравнить значения, оказывается, что список повторяется по одному элементу. Если я распечатаю список, я могу увидеть все содержимое списка. Однако, если я печатаю после a для l oop (for i in list:), я получу только первый элемент, сколько раз в списке есть элементы, но всегда одно и то же значение. 

def getMail(fname, lname):
    firstname = fname.split()
    lastname = lname.split()

    email = firstname[0].strip() + '.' + lastname[0].strip() + '@test.com'
    e_mails=[]

    with open('test-input.txt','r') as AD:
        rows = csv.DictReader(AD)

        for ad in rows:
            e_mails.append(ad['UserLogonName'])




    for i in e_mails:
        print(f'{i} and {email}')

         '''
             This is what i get when I print i and email. i expecxted i to be 
             an item on the list is stored.
             carlos.jimenez@test.com and Apolinar.Vargas-Morales@test.com
             carlos.jimenez@test.com and Antonio.Ochoa@test.com
             carlos.jimenez@test.com and Steven.Diaz@test.com
             carlos.jimenez@test.com and Jorge.Medrano@test.com
             carlos.jimenez@test.com and Daniel.Cerano@test.com
         '''


        print(e_mails)
        '''
           this is the content of my list from CSV
           ['carlos.jimenez@test.com', 'Carri.Dix@test.com', 'Alena.Azizi@test.com', 'Berna.Lagonaso@test.com', 'Christophe.deCarbonnieres@test.com', 'daniel.min@test.com', 'David.Li@test.com', 'John.Mitchell@test.com', 'Naveen.Sisodia@test.com', 'nadia.shakir@test.com', 'Sumeet.Sharma@smurfitkappa.com', 'benedicte.bar@test.com', 'Patricia.Aguirre@test.com']
        '''

        if email.lower() == i.lower():
            # print(f'{i} and {email}')

            print(len(firstname))

            if firstname ==2:
                email = firstname[0]+'.'+firstname[0][0:1]+'.'+lastname[0]+'@test.com'
                return email.lower()

            elif firstname ==1 :

                email = firstname[0]+'.'+firstname[0][0:1]+'.'+lastname[0]+'@test.com'
                return email.lower()
        else: 
            return email.lower() 




with open('users.csv') as csv_file:
    rows = csv.DictReader(csv_file)

    with open('users-COI2-Multi.csv', 'w', newline='') as output:
        header = ['FirstName','Initials','LastName','DisplayName','Description','Office','TelePhone','UserLogonName','SamAccountName','JobTitle','Department','Manager','Mobile','faxNumber','Notes','Assistant','employeeID','ex1','ex2','ex3','ex15','Office365License','ExpiresInDays','EmailToUSer','AddToGroup']

        output_file = csv.DictWriter(output, fieldnames=header, delimiter=';')
        output_file.writeheader()

        for data in rows:
            employeeId = data['Associate ID']
            fName = data['First Name']
            lName = data['Last Name']
            Location = data['Location']
            Department = data['Department']
            Manager = data['Manager Name']
            JobTitle = data['Title']

            context = {
                'FirstName' : fName,
                'Initials' : getInitials(fName, lName),
                'LastName' : lName,
                'DisplayName' : getDisplayName(fName, lName),
                'Description' : 'LMS Account for: '+getDisplayName(fName, lName),
                'Office': getOffice(Location).strip(),
                'TelePhone' : '+1 XXX XXX XXX',
                'UserLogonName' : getMail(fName, lName),
                'SamAccountName' : getSamA(fName, lName),
                'JobTitle' : JobTitle,
                'Department' : Department,
                'Manager' : Manager,
                'Mobile' : '',
                'faxNumber' : '',
                'Notes' : '',
                'Assistant' : '',
                'employeeID' : employeeId,
                'ex1' : 'End User',
                'ex2' : 'NoMailbox',
                'ex3' : getSiteCode(Location),
                'ex15' : getSKID(Location),
                'Office365License' : '',
                'ExpiresInDays' : '',
                'EmailToUSer' : 'victor.u.reyes@test.com',
                'AddToGroup' : '',
            }
            output_file.writerow(context)

1 Ответ

1 голос
/ 23 февраля 2020

Виновным является следующий оператор возврата: 

return email.lower()

в конце функции getmail(). Это будет применяться в 99% случаев (всякий раз, когда if email.lower() == i.lower(): не True), поэтому поиск заканчивается слишком рано.

Я не понимаю точных бизнес-правил приложения, которое вы собираетесь использовать разработать, но я предполагаю, что функция должна быть 

def getMail(fname, lname):
    [...]

    for i in e_mails:
        [...]

        if email.lower() == i.lower():
            [...]
                               # removed else here
     return email.lower()      # outdent here

Другой вариант будет 

        elif firstname ==1 :

            email = firstname[0]+'.'+firstname[0][0:1]+'.'+lastname[0]+'@test.com'
            return email.lower()
        else:                      # indent here
            return email.lower()   # indent here

Также обратите внимание: два случая if firstname ==2: и if firstname ==1: выглядят одинаково для меня , Я предполагаю, что это не предназначено.

...