Class object does not update empty list

Question

I'm trying to solve this problem on my own. The problem asks to write a class for employee information, then ask user to input that information, and finally print the entered information.

I want to use two for loops, one for getting information and one for printing the information. Unfortunately, the printing for loop does not working.

class Employee:
    def __init__(self, name, id_num, department, job):
        self.__name = name
        self.__id_num = id_num
        self.__department = department
        self.__job = job

    # setters
    def set_name(self,name):
        self.__name = name
    def set_id(self,id_num):
        self.__id_num = id_num
    def set_department(self,department):
        self.__department = department
    def set_job(self,job):
        self.__job = job

    # getters
    def get_name(self):
        return self.__name
    def get_id(self):
        return self.__id_num
    def get_department(self):
        return self.__department
    def get_job(self):
        return self.__job

def main():

    employee_list = []

    for i in range(2):
        name = input('What is the name of the employee? ')
        id_num = float(input('What is the ID number of the employee? '))
        department = input('What is the department where the employee works? ')
        job = input('What is the job title of the empoyee? ')

        personnel = Employee(name,id_num,department,job)

        employee_list.append(personnel)
    return employee_list


    for item in employee_list:
        print(item.get_name())
        print(item.get_id())
        print(item.get_department())
        print(item.get_job())
        print()


main()

As an aside, your style is not very pythonic here. It looks like you are writing "java in python". Don't write getters and setters, they are not necessary and only add unnecessary clutter. Don't use double underscores, use a single underscore to signal that this attribute is not part of the public interface. Double-underscore name-mangling is to prevent name collisions in subclasses. — juanpa.arrivillaga
– juanpa.arrivillaga, Commented Jul 7, 2019 at 1:07
@juanpa.arrivillaga Thanks for the style comments. I'm self-learning from the book "starting out with python". On page 449 it says to hide attributes in Python we use double underscores. I didn't know single underscore works too. — abenol
– abenol, Commented Jul 7, 2019 at 1:19

Reblochon Masque · Accepted Answer · 2019-07-07 02:27:25Z

2

You need to remove the following line in your main() function:

return employee_list

It is causing your main to stop running without ever reaching the printing for loop.

edited Jul 7, 2019 at 2:27

Reblochon Masque

37k10 gold badges60 silver badges85 bronze badges

answered Jul 7, 2019 at 1:05

Akaisteph7

6,9222 gold badges39 silver badges53 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Class object does not update empty list

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related