This week's book giveaway is in the OO, Patterns, UML and Refactoring forum. We're giving away four copies of Refactoring for Software Design Smells: Managing Technical Debt and have Girish Suryanarayana, Ganesh Samarthyam & Tushar Sharma on-line! See this thread for details.
Having your constructor take the dimensions of your array requires you to know what is inside your file. However, since the program is going to open the file and read these dimensions anyway, just let the program figure it out. All you need to know then is the filepath to the file and your program will do the rest. Thus I modified the code to take advantage of this. Also, as Aly had said using the split method from the String class is much simpler.
Then you can call your printTable(), smallest() and largest() methods. Here's how I called it :
and the output:
The first four lines are from the println() on line 28. I was using that to debug and forgot to remove it. That should be removed