Path: csiph.com!usenet.pasdenom.info!weretis.net!feeder4.news.weretis.net!eternal-september.org!feeder.eternal-september.org!mx04.eternal-september.org!.POSTED!not-for-mail From: markspace <-@.> Newsgroups: comp.lang.java.programmer Subject: Re: JOutlookBar Date: Fri, 27 Apr 2012 12:33:00 -0700 Organization: A noiseless patient Spider Lines: 39 Message-ID: References: <33049662.264.1335479370702.JavaMail.geo-discussion-forums@pbcnk4> <15997725.898.1335482389670.JavaMail.geo-discussion-forums@pbbps5> <7588566.806.1335496661608.JavaMail.geo-discussion-forums@pbqv7> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Injection-Date: Fri, 27 Apr 2012 19:33:03 +0000 (UTC) Injection-Info: mx04.eternal-september.org; posting-host="Nhfo9ulyZqrLDUaAyycsPw"; logging-data="1157"; mail-complaints-to="abuse@eternal-september.org"; posting-account="U2FsdGVkX1+RJM+8UYnRSWSvU2YpwDbTC7GU7PVAdRY=" User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20120327 Thunderbird/11.0.1 In-Reply-To: Cancel-Lock: sha1:y7695hlvGXC4O/TTPEE787yQYaU= Xref: csiph.com comp.lang.java.programmer:13949 On 4/27/2012 10:11 AM, Daniel Pitts wrote: > On 4/26/12 8:17 PM, Peter Cheung wrote: >> I already set it to 1ms, but it still runs slowly. >> >> So the animation is not smooth. > You're timer simply calls draw. Also, 1ms is really too fast, and will > cause more harm than good. You should aim for closer to 30hz->60hz. Try > around 33ms timer. I happen to know a little about animation (graphics is a personal hobby of mine). The gold standard in animation is 25 frames per second, or 40 ms in between frames. However, as a practical matter, much lower frames rates are acceptable. Some cheaper animation, like some of the early so-called "japanimation," used as little as 4 or 6 frames per second in parts of their work. I'd start at 250 ms. 1 ms will clearly overwhelm any normal desktop system with too much work. Get 250 ms working, then see if it can be improved. > Ideally, the "work" for your animation should be done in the timer > handler too (so that your state changes on the timer, not just a redraw). This. Peter, what I've looked at in your code base is pretty bad. Most of it is really ugly generated code. Nothing has comments. There's no test harness that I saw. You need to slow down and write some code by hand, try to understand what is really going on. One good class is better than 100 cruddy classes, and it is certainly my impression that the latter is what you have.