Reply to thread

Message

<blockquote data-quote="NO_MeRcY" data-source="post: 18369146" data-attributes="member: 289641">Options කීපයක් තියනවා මල්ලී මම දෙකක් කියන්නම්[CODE]&nbsp; &nbsp; public static void main(String[] args) { &nbsp; &nbsp; &nbsp; &nbsp; Scanner sc = new Scanner(System.in);&nbsp; &nbsp; &nbsp; &nbsp; int input = 0;&nbsp; &nbsp; &nbsp; &nbsp; while (true) {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; System.out.print(&quot;Enter number:&quot;);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; input = sc.nextInt();&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (input &lt; 900) {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; System.out.print(input);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; break;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; } else {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; System.out.println(&quot;Invalid Input&quot;);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; }[/CODE][CODE]&nbsp; &nbsp; public static void main(String[] args) {&nbsp; &nbsp; &nbsp; &nbsp; Scanner sc = new Scanner(System.in);&nbsp; &nbsp; &nbsp; &nbsp; int input = 0; &nbsp; &nbsp; &nbsp; &nbsp; do {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; System.out.print(&quot;Enter number:&quot;);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; input = sc.nextInt();&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (input &gt; 900) {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; System.out.println(&quot;Invalid Input&quot;);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; } while (input &gt; 900);&nbsp; &nbsp; &nbsp; &nbsp; System.out.println(input); &nbsp; &nbsp; }[/CODE]මේ දෙකෙන් 2nd එක යූස් කරන්න මල්ලී. මොකද while loop එකකට එකක් break; දාන එක java best practice එකක් නෙමේexceptions එහෙම handle කරල&nbsp; code එක හදා ගනින්. Apart from the above &nbsp;මම logic එක නම් කියෙව්වේ නැහැ. පේන විදියට එකත් තව fine tune කරන්න පුලුවන්. &nbsp;උබ දාල තියෙන code එකේ පොඩි වෙනස් කම් ටිකක් කරපන් මෙහෙම&gt; concatenation&nbsp; වලට String literals use කරන්න එපා. StringBuilder&nbsp; යූස් කරන්න.නැත්නම්&nbsp; &nbsp; performance ප්‍රශ්නයක් වෙයි. Modern JRE ඔය වැඩේ අපිට auto කරන දෙනවා ඒත් best practice එකක්&nbsp; විදියට StringBuilder යූස් කරන්න.&gt; [CODE]&nbsp; &nbsp; &nbsp;&nbsp; StringBuilder sb = new StringBuilder();&nbsp; &nbsp; &nbsp; &nbsp; sb.append(denom.values()[i - 1]).append(&quot; &quot;).append(text);&nbsp;&nbsp; [/CODE]&gt; class variables public විදියට යූස් කරන්න එපා (උබ වෙන තැනක access කරන්නෙ නැත්නම්)&gt; try catch දාල&nbsp; exceptions handle කරන්න. &nbsp;&nbsp; E.g. උබගේ මෙන්න මේ line එකෙන් java.lang.ArrayIndexOutOfBoundsException&nbsp; එක පනිනවා&nbsp;&nbsp; [CODE] text = tens.values()[num - 2] + &quot; &quot; + text;[/CODE]&gt; Enum වල elements වලට CAPITAL LETTERS use කරන්න&gt; long Tens&nbsp; should be&nbsp; long tens</blockquote>

[QUOTE="NO_MeRcY, post: 18369146, member: 289641"] [FONT="Arial"][SIZE="4"][B]Options කීපයක් තියනවා මල්ලී මම දෙකක් කියන්නම්[/B][/SIZE][/FONT] [CODE] public static void main(String[] args) { Scanner sc = new Scanner(System.in); int input = 0; while (true) { System.out.print("Enter number:"); input = sc.nextInt(); if (input < 900) { System.out.print(input); break; } else { System.out.println("Invalid Input"); } } } [/CODE] [CODE] public static void main(String[] args) { Scanner sc = new Scanner(System.in); int input = 0; do { System.out.print("Enter number:"); input = sc.nextInt(); if (input > 900) { System.out.println("Invalid Input"); } } while (input > 900); System.out.println(input); } [/CODE] [SIZE="4"][B]මේ දෙකෙන් 2nd එක යූස් කරන්න මල්ලී. මොකද while loop එකකට එකක් break; දාන එක java best practice එකක් නෙමේ [/B][/SIZE] [B][SIZE="4"]exceptions එහෙම handle කරල code එක හදා ගනින්. [/SIZE][/B] Apart from the above මම logic එක නම් කියෙව්වේ නැහැ. පේන විදියට එකත් තව fine tune කරන්න පුලුවන්. උබ දාල තියෙන code එකේ පොඩි වෙනස් කම් ටිකක් කරපන් මෙහෙම > concatenation වලට String literals use කරන්න එපා. StringBuilder යූස් කරන්න.නැත්නම් performance ප්‍රශ්නයක් වෙයි. Modern JRE ඔය වැඩේ අපිට auto කරන දෙනවා ඒත් best practice එකක් විදියට StringBuilder යූස් කරන්න. > [CODE] StringBuilder sb = new StringBuilder(); sb.append(denom.values()[i - 1]).append(" ").append(text); [/CODE] > class variables public විදියට යූස් කරන්න එපා (උබ වෙන තැනක access කරන්නෙ නැත්නම්) > try catch දාල exceptions handle කරන්න. E.g. උබගේ මෙන්න මේ line එකෙන් java.lang.ArrayIndexOutOfBoundsException එක පනිනවා [CODE] text = tens.values()[num - 2] + " " + text;[/CODE] > Enum වල elements වලට CAPITAL LETTERS use කරන්න > long Tens should be long tens [/QUOTE]

Verification

Hath warak paha keeyada? (hatha wadikireema paha)

Search

Latest ads

Reply to thread