"Calvin Spealman" <[EMAIL PROTECTED]> writes: > It is hard to determine what is wrong with your code without you > telling anyone why it is you believe something is wrong with it. Did > you get an exception? Did it simply not do what it was expected to do? > There seems to be some apparent indenting problems, but maybe that is > just from pushing it through the e-mail. I see some general stylistic > problems as well, but without know what you are actually asking, I > won't know what questions to answer. "What is wrong with my code?" is > a container of many many smaller questions, and you need to focus your > questions better. > > On 21 Dec 2006 09:16:58 +1100, Pyenos <[EMAIL PROTECTED]> wrote: > > import cPickle, shelve > > > > could someone tell me what things are wrong with my code? > > > > class progress: > > > > PROGRESS_TABLE_ACTIONS=["new","remove","modify"] > > DEFAULT_PROGRESS_DATA_FILE="progress_data" > > PROGRESS_OUTCOMES=["pass", "fail"] > > > > > > def unpickleProgressTable(pickled_progress_data_file): > > > > return unpickled_progress_table > > > > def pickleProgressTable(progress_table_to_pickle): > > > > return pickled_progress_data_file > > > > # Of course, you get progress_table is unpickled progress table. > > def progressTable(progress_table, action, task, > > pid=len(progress_table), outcome=PROGRESS_OUTCOMES[1]): > > pid_column_list=progress_table[0] > > task_column_list=progress_table[1] > > outcome_column_list=progress_table[2] > > > > # But a task must also come with an outcome! > > def newEntry(new_task, new_outcome): > > new_pid=len(task_column_list) > > > > pid_column_list.extend(new_pid) > > task_column_list.extend(new_task) > > outcome_column_list.extend(new_outcome) > > > > def removeEntry(pid_to_remove, task_to_remove): > > > > if > > pid_column_list.index(pid_to_remove)==task_column_list.index(task_to_remove): > > # Must remove all columns for that task > > index_for_removal=pid_column_list.index(pid_to_remove) > > > > pid_column_list.remove(index_for_removal) > > task_column_list.remove(index_for_removal) > > outcome_column_list.remove(index_for_removal) > > > > # Default action is to modify to pass > > def modifyEntry(pid_to_modify, > > outcome_to_modify=PROGRESS_OUTCOMES[0]): > > index_for_modifying=pid_column_list.index(pid_to_modify) > > > > # Modify the outcome > > outcome_column_list[index_for_modifying]=outcome_to_modify > > -- > > http://mail.python.org/mailman/listinfo/python-list > > > > > -- > Read my blog! I depend on your acceptance of my opinion! I am interesting! > http://ironfroggy-code.blogspot.com/
it says that progress_table is not defined. i don't know why. -- http://mail.python.org/mailman/listinfo/python-list