Hi Adam,
Here are some comments:
FITCalcv1 class
-------------------
1. Class name, FITCalcv1? v1 I pressume is version number, how about when you'll have v2? Will you copy/paste all that code to other source file and will name v2? Don't. Exclude such things from class names. You could have it in comments form if you want it. Lets assume you ended up with FITCalc. What does it tell you? To me not much honestly. Calc I could guess what it means - calculator. What about FIT? Looking to program code, can't decypher. Try to think about the better name.
2. Variable names. reading1. Why 1? user_input, it doesn't follow
Java variables naming convention. That should read as userInput. Would suggest to improve it. Next - lr, cr. Try to remove everything what is after equals sign, what this variable itself lr or cr tells you? Nothing. Variable name should tell you exactly what it represents right away by looking to it, without any effort looking further into the code. I'd name them - lastReading, currentCreading. In Fact, you have them already, but for some reason you decided to used them for a very short period of time. Are you aware that you can read integers, decimal numbers by using methods nextInt(); nextDouble()?
3. Why you have some spaces before and after parentheses?
why you don't have them here then?
and here
here
why those spaces appeared here again? In fact after 'd' there isn't one
here again
Can you see the inconsistency in your formatting? If you think it isn't important, you're mistaken. If you get chance, read the book "Clean Code" by Robert C. Marting. It might will change the way you think about code consistency, about which many people think it isn't important. In fact, it is important, and equally than any other issues in code. I'd avoid spaces before and after parentheses when we talk about paremeters.
4. Can you notice how you duplicate your code within FITCalcv1 class? Lines 10 - 13 and lines 15 - 18. Consider having method which accepts prompt message and returns user input. Might move such method to a Utility class. Look in this forum for Utility class,
you should find some examples.
MeterReader class
--------------------
1. Shouldn't MeterReader have a constructor so you could pass lastReading, currentReading and modelNumber to it and create MeterReader object? So later by using internal data you could use it to calculate difference, display information and any other related information.
Well, enough probably for now, try to improve your code and lets see what others will advice you.