|
|
Featured White Papers
- Why choose an Open Source BPM?
- 10 FAQs on Business Process Management
- Best Practices for Getting Started with BPM
- Free Your Devs From The Pain Of Java Redeploys
- Increase Velocity 40% by Using JRebel
- What Developers Want: The End of Application Redeploys
- Developer Productivity Report: Java Tools, Tech, Devs, and Data
- Coding with JRebel: Java Forever Changed
- Utilizing Fast Testing to Transform Java Development
- Do You Really Get Classloaders?
- Pragmatic Continuous Delivery with Jenkins, Nexus, and LiveRebel
- Your Next Java Web App: Less XML, No Long Restarts, Fewer Hassles
- Coding with JRebel, Java Forever Changed
- 2012 Developer Productivity Report: Java Tools, Tech, Devs, and Data
If else if else if else!!!!!
Submitted by garnaout on Tue, 09/20/2011 - 15:15.
I am parsing a string where the number of words is always changing. Since I have a minimum number of 1 word and max of 12 I wrote this function:
private void initializeParts(Scanner scanner) {
if ( scanner.hasNext() ) // replace with a function/switch case for a better coding practice
{
part1 = scanner.next();
if(scanner.hasNext())
{
part2 = scanner.next();
}
if(scanner.hasNext())
{
part3 = scanner.next();
}
if(scanner.hasNext())
{
part4 = scanner.next();
}
if(scanner.hasNext())
{
part5 = scanner.next();
}
if(scanner.hasNext())
{
part6 = scanner.next();
}
if(scanner.hasNext())
{
part7 = scanner.next();
}
if(scanner.hasNext())
{
part8 = scanner.next();
}
if(scanner.hasNext())
{
part9 = scanner.next();
}
if(scanner.hasNext())
{
part10 = scanner.next();
}
if(scanner.hasNext())
{
part11 = scanner.next();
}
if(scanner.hasNext())
{
part12 = scanner.next();
}
}
}However I know this is a very bad code practice. Any recommendations?