1. Comment your code!

Ah, yes, of course. Sorry about that. I'll comment it right away.

2. Are you using the MVC pattern? It's not really clear from the
code, but it looks somewhat like it

nope, I don't use pattern. I just want to make the HistoryList and
HistoryItem 'talk' to each other. I don't know anything about pattern
(I've just started reading Headfirst DP book).

3. What's "hl.as"? It's an ill chosen file/class name.

Oops, my bad ... It shouldn't be there! It's the external .as that I
included in the first (procedural) version of my app.

Thanks,
--
Anggie Bratadinata
Web|Graphic|Flash
Jl. Raya Langsep 21
Malang - East Java
I N D O N E S I A
http://design.ibshastautama.com



Per Bolmstedt wrote:

Anggie Bratadinata wrote:

This is the simple app I created a few days ago, about which I'd
like to have your expert advice on how to improve it or maybe how
to code it in a much better way.

1. Comment your code! It looks structured enough, so adding comments
should be easy. If you use FDT, using Javadoc is straightforward: http://java.sun.com/j2se/javadoc/writingdoccomments/

2. Are you using the MVC pattern? It's not really clear from the
code, but it looks somewhat like it. If you aren't you should; if you
are, maybe you could make it more clear, by changing class names,
adding an interface or two, or adding comments.

3. What's "hl.as"? It's an ill chosen file/class name.

-- Per

_______________________________________________
Flashcoders@chattyfig.figleaf.com
To change your subscription options or search the archive:
http://chattyfig.figleaf.com/mailman/listinfo/flashcoders

Brought to you by Fig Leaf Software
Premier Authorized Adobe Consulting and Training
http://www.figleaf.com
http://training.figleaf.com

Reply via email to