Python Class Function not defined

Question

I'm creating a class with function within, but i keep getting error "name 'direct_report' is not defined"

Basically I'm trying to make an organization chart, creating a list using the direct_report function to add people under each position

class employee:
def __init__(self, name , title, salary):
    self.name = name
    self.title = title
    self.salary = salary
    self.direct_reports_list = direct_report()

def __str__(self):
    #otheremp_list = []
    print(self.title,'-', self.name)
    print('Direct Reports:')
    for emp in self.direct_reports_list:
        print(emp.title,'-', emp.name)
    #    otheremp_list.append(emp.direct_reports_list)
    #print('other employees:')
    #for emp in otheremp_list:
    #    print(emp.title,'-', emp.name)
    #    otheremp_list.append(emp.direct_reports_list)

def direct_report(self,value):
    print(value)
    direct_reports_list = []
    direct_reports_list.append(value)
    print(direct_reports_list)
    return direct_reports_list

ceo = employee("Elon Musk", "CEO",1000000)
devdir = employee("Jeff Bezos","Development Director",500000)
devassoc1 = employee("Beyonce Knowles","Development Associate", 50000)
devassoc2 = employee("Taylor Swift","Development Associate", 50000)
ceo.direct_report(devdir)
ceo.direct_report(devdir2)
devdir.direct_report(devassoc1)
devdir.direct_report(devassoc2)
print(ceo)

The # is my further plan to print the full organization chart, but currently I'm still stuck at the "direct report" parts.

Mohamed Mostafa · Accepted Answer · 2022-02-16 04:57:21Z

1

You need to add one indentation level for the classes methods like that:

class employee:
    def __init__(self, name , title, salary):
        self.name = name
        self.title = title
        self.salary = salary
        self.direct_reports_list = direct_report()

    def __str__(self):
        #otheremp_list = []
        print(self.title,'-', self.name)
        print('Direct Reports:')
        for emp in self.direct_reports_list:
            print(emp.title,'-', emp.name)
        #    otheremp_list.append(emp.direct_reports_list)
        #print('other employees:')
        #for emp in otheremp_list:
        #    print(emp.title,'-', emp.name)
        #    otheremp_list.append(emp.direct_reports_list)

    def direct_report(self,value):
        print(value)
        direct_reports_list = []
        direct_reports_list.append(value)
        print(direct_reports_list)
        return direct_reports_list

ceo = employee("Elon Musk", "CEO",1000000)
devdir = employee("Jeff Bezos","Development Director",500000)
devassoc1 = employee("Beyonce Knowles","Development Associate", 50000)
devassoc2 = employee("Taylor Swift","Development Associate", 50000)
ceo.direct_report(devdir)
ceo.direct_report(devdir2)
devdir.direct_report(devassoc1)
devdir.direct_report(devassoc2)
print(ceo)

answered Feb 16, 2022 at 4:57

Mohamed Mostafa

6201 gold badge6 silver badges12 bronze badges

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

2 Comments

W.tan Over a year ago

Could you explain more what do you mean by adding indentation level? thank you for replying though!

Mohamed Mostafa Over a year ago

@W.tan I mean the tabs you put at the beginning of your function

kibromhft · Accepted Answer · 2022-02-16 05:00:56Z

1

Call the function in this way.

class employee:
    def __init__(self, name , title, salary):
        self.name = name
        self.title = title
        self.salary = salary
        self.direct_reports_list = self.direct_report()

answered Feb 16, 2022 at 5:00

kibromhft

1,0591 gold badge8 silver badges19 bronze badges

1 Comment

W.tan Over a year ago

I got this error now "direct_report() missing 1 required positional argument: 'value'"

NALLAPANENIVENKATESH CHOWDARY · Accepted Answer · 2022-02-16 05:04:42Z

0

Try this code.

class employee:
    def __init__(self, name , title, salary):
        self.name = name
        self.title = title
        self.salary = salary
        self.direct_reports_list = []

    def __str__(self):
        #otheremp_list = []
        print(self.title,'-', self.name)
        print('Direct Reports:')
        for emp in self.direct_reports_list:
            print(emp.title,'-', emp.name)
        #    otheremp_list.append(emp.direct_reports_list)
        #print('other employees:')
        #for emp in otheremp_list:
        #    print(emp.title,'-', emp.name)
        #    otheremp_list.append(emp.direct_reports_list)
        return "Done"

    def direct_report(self,value):
        self.direct_reports_list.append(value)

ceo = employee("Elon Musk", "CEO",1000000)
devdir = employee("Jeff Bezos","Development Director",500000)
devassoc1 = employee("Beyonce Knowles","Development Associate", 50000)
devassoc2 = employee("Taylor Swift","Development Associate", 50000)
ceo.direct_report(devdir)
devdir.direct_report(devassoc1)
devdir.direct_report(devassoc2)
print(ceo)
print(devdir)
print(devassoc1)
print(devassoc2)

answered Feb 16, 2022 at 5:04

NALLAPANENIVENKATESH CHOWDARY

6145 silver badges9 bronze badges

1 Comment

W.tan Over a year ago

This one works well! thank you!!

Will · Accepted Answer · 2022-02-16 05:01:04Z

0

try self.direct_report() instead as you are calling a method within the class

answered Feb 16, 2022 at 5:01

Will

1761 silver badge11 bronze badges

1 Comment

W.tan Over a year ago

I got this error now "direct_report() missing 1 required positional argument: 'value"

Collectives™ on Stack Overflow

Python Class Function not defined

4 Answers 4

2 Comments

1 Comment

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

1 Comment

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related