[Static Hazard]

I have two classes, called Journal and JournalEntry. The JournalEntry constructor has several parameters that are passed with it is instantiated, like time, date, subject, context. In the Journal class, there is a function called addEntry(self, JournalEntry) that adds an entry to the journal. I'm having trouble getting the addEntry function started...

My problem is that I don't really know how to instantiate the entry...I'm not used to dynamically typed languages like Python.

The actual values are supplied in the JournalEntry class. For example, an entry could be entry1 = journalentry.JournalEntry("1215", "02142013", "My day", "Today was Valentine's Day.").

Class Journal(object):
    def __init__(self):
        journalList = []

    def addEntry(self, entry):
        self.theEntry = entry #is this right?
        self.theEntry = entry(time, date, subject, context) #or is this?
        journalList += entry

The Journal constructor creates an empty list, and then addEntry would add an entry to the list each time. Will this work?

[darek9576]

Well, your init() method is wrong. You need to create an instance variable that will keep track of journal entries so it should be:

self.journalList = []

.

Then, you can create the addEntry() method and simply append it.

def addEntry(self, entry):
     self.journalList.append(entry)

Note that addEntry takes as parameter already instantiated Entry object.

[woooee]

I am assuming that you want to keep each journal entry so each instance is appended to self.journal_list in the code below. It would be simpler to use a list of lists or a dictionary with list to keep track of the data. Also, code is easier for others to read when it conforms to the Python Style Guide

class Journal(object):
    def __init__(self):
        self.journalList = []

    def addEntry(self):
        the_entry = JournalEntryClass(time, date, subject, context) #or is this?
        self.journalList.append(the_entry)

    def print_all_entries(self):
        for instance in self.journalList:
            print instance.time, instance.date, instance.subject, instance.context 

This post has been edited by woooee: 14 February 2013 - 04:11 PM

[darek9576]

woooee, on 14 February 2013 - 04:04 PM, said:

def print_all_entries(self):
for instance in self.journalList():
print instance.time, instance.date, instance.subject, instance.context

self.journalList <-- without ().

And CLASS should be spelt with lower case letters class not Class.

[woooee]

Corrected. Didn't catch the capitalized Class in the original code.