Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!news.glorb.com!news-out.readnews.com!transit4.readnews.com!news-out.news.tds.net!newsreading01.news.tds.net!53ab2750!not-for-mail
From: "Daniel Pitts" <daniel.pitts@1:261/38.remove-s5i-this>
Subject: Re: How can you make idle processors pick up java work?
Message-ID: <50197007.55903.calajapr@time.synchro.net>
X-Comment-To: qwertmonkey
Newsgroups: comp.lang.java.programmer
In-Reply-To: <50197006.55897.calajapr@time.synchro.net>
References: <50197006.55897.calajapr@time.synchro.net>
X-FTN-AREA: COMP.LANG.JAVA.PROGRAMMER
X-FTN-MSGID: 1:261/38 9d296a27
X-FTN-REPLY: 1:261/38 5237cbaf
Content-Type: text/plain; charset=IBM437
Content-Transfer-Encoding: 8bit
X-Gateway: time.synchro.net [Synchronet 3.16a-Win32 NewsLink 1.98]
Lines: 56
Date: Wed, 01 Aug 2012 18:09:37 GMT
NNTP-Posting-Host: 69.21.70.65
X-Complaints-To: news@tds.net
X-Trace: newsreading01.news.tds.net 1343844577 69.21.70.65 (Wed, 01 Aug 2012 13:09:37 CDT)
NNTP-Posting-Date: Wed, 01 Aug 2012 13:09:37 CDT
Organization: tds.net
Xref: csiph.com comp.lang.java.programmer:16937

  To: qwertmonkey
From: Daniel Pitts <newsgroup.nospam@virtualinfinity.net>

On 7/31/12 7:46 PM, qwertmonkey@syberianoutpost.ru wrote:
>   just in case someone ponders over the same. At the end of the day there is
>   no speed improvement whatsoever compared to doing the char reading into a
>   buffer and parsing out the sentences yourself
> ~
>   the code below does the same thing
> ~
>      BfR = Files.newBufferedReader(IFlPth, Charset.forName("UTF-8"));
> // __
>      aSx = BfR.readLine();
>      while(aSx != null){
>       iSL = aSx.length();
>       for(int k = 0; (k < iSL); ++k){
>        iKdPnt = aSx.codePointAt(k);
>        ++lTtlKdPnts;
>       }
>       ++lLns;
>       iTtlRdByts += iSL;
>       aSx = BfR.readLine();
>      }// (iRdByts > -1)
> // __
>      BfR.close();
> ~
>   lbrtchx
>

You're code uses such terrible naming conventions that it is impossible to read 
and understand. "BfR" should be something like "reader" or "bufferedReader".  
aSx? Should that be "line"?

k? who uses "k" as a for loop index? Unless you're doing a three-level deep 
Dynamic Programming algorithm.

It also appears that you're trying to use some sort of Hungarian notation, and 
doing it contra the original intent:  See
<http://www.joelonsoftware.com/articles/Wrong.html>

BfR.close() should be in a finally block.

You also aren't declaring any variable here, so we have no idea what types they 
are.  You should declare variables as closely to the first use as is possible.

Also, it looks like you're trying to micro-optimize this code which reads one 
line at a time.  What is the point of doing iKdPnt = aSx.codePointAt(k), when 
you don't actually use that variable in the for loop?

What is it exactly that you're trying to do? I mean, what's your end goal, not 
what the means you've chosen for that end.

--- BBBS/Li6 v4.10 Dada-1
 * Origin: Prism bbs (1:261/38)
--- Synchronet 3.16a-Win32 NewsLink 1.98
Time Warp of the Future BBS - telnet://time.synchro.net:24