 <p>I think your approach is fine, it gets the job done.</p> <p>I'm not too sure what you're asking by saying how should you loop the two output statements, followed by if it should be in the main method. From what I understand, and by looking at your code, running this input loop is perfectly fine from the main class. The do-while is fine although I'd move the first 'introductory' output outside the loop so you don't see it every time the loop reiterates. Also, I notice you're not actually calling/instantiating the <code>Store</code> class in your main method, there's no data being added to the <code>Store</code> class for when it iterates through the accounts ArrayList.</p> <p>As far as the answer that stated a more "modern" approach, I think the for loop you used is fine. I think the person was referring to the for-each loop. It doesn't really matter how you loop through it with the little amount of data you have. </p> <p>There's some error in the logic for that loop. The <code>getHighestCustomerTotal()</code> is referencing an empty accounts ArrayList. You declared an ArrayList within the <code>Store</code> class and tried to loop through it but it's empty unless you called the <code>addAccount()</code> method from your main method at some point, so you'd need some error checking on that.</p>