Why am I getting a TypeError the second time I run a class method?

New to Python and programming in general. Currently learning about classes and objects and really struggling. In this particular example, I have an Animal parent class and a EggLayers child class. The EggLayers class has a method to calculate a particular instance's egg production for the week and then append the value to a list. I have another method that calculates the chicken's lifetime production. The idea being, each week's totals go into a list from running the first method and then the second method sums the list to return a lifetime total for the instance. Both run fine the first time around, however, when I try to run the first method a second time I get a TypeError ('int' object is not callable).

I've tried int() in various places in the code. Also don't think I'm overriding the int() function anywhere. Apparently these are the two most common reasons for this error. Wouldn't think it would run the first time if I had either of the above issues.

class EggLayers(Animal):
    """A class for egg laying animals"""
    type = "chicken"

    def __init__(self, name, age, color, tag):
        super().__init__(name, age, color, tag)

    def eggs_week(self):    # to calculate eggs/week and then append to list
        self.eggs_list = []
        self.eggs_day = int(input("Eggs today?  "))
        self.eggs_day = self.eggs_day
        self.eggs_week = self.eggs_day * 7
        self.eggs_list.append(self.eggs_week)
        print(f"{self.name} laid {self.eggs_week} this week")
        return self.eggs_week

    def eggs_lifetime(self):  # to sum the values in the list above
        self.eggs_lifetime = sum(self.eggs_list)
        print(f"Liftetime production for {self.name} is {self.eggs_lifetime}")
        return self.eggs_lifetime

chicken1 = EggLayers('chicken1', 3, 'white', '002')

chicken1.eggs_week()
chicken1.eggs_lifetime()
chicken1.eggs_week() # Only throws an error the second time

It's because in your eggs_week function, you have

        self.eggs_week = self.eggs_day * 7

After that, chicken1.eggs_week is no longer a function. It's an integer.

You don't really need to include a prefix on your function names, since it's already an eggs-related class. Also, most of the values you compute do not need to be saved as state in the object, so they do not need self..

More like this:

class EggLayers(Animal):
    """A class for egg laying animals"""
    type = "chicken"

    def __init__(self, name, age, color, tag):
        super().__init__(name, age, color, tag)
        self.eggs_list = []

    def per_week(self):    # to calculate eggs/week and then append to list
        eggs_day = int(input("Eggs today?  "))
        eggs_week = eggs_day * 7
        self.eggs_list.append(eggs_week)
        print(f"{self.name} laid {eggs_week} this week")
        return eggs_week

    def lifetime(self):  # to sum the values in the list above
        eggs_lifetime = sum(self.eggs_list)
        print(f"Liftetime production for {self.name} is {eggs_lifetime}")
        return eggs_lifetime

Personally, I think it is bad practice to do I/O in a class method like this. The class should hold functionality directly related to the object. It should not care where the information comes from. For example, right now, you couldn't easily use this to read the data from a file or a database.

I would have a loop outside the class asking for the input, then have a class method that allows me to pass that information in, like add_week. However, I haven't made that change here.