slipstick:
Try posting your code inline using the </> code tags and post COMPLETE programs. More people will read them that way.I can't make any sense of your second version. You have a function rpmCalculation() which you never use and then you use a variable which never seems to be updated also called rpmCalculation. It's like you're going out of your way to confuse things.
Steve
oops sorry! I have post out the code
so do you mean I should put this function "rpmCalculation()" in the loop?
Void loop() {
rpmCalculation()
}