Thread: how to make this 10 lines more elegant?

hi, this code help me to find some "missing value" in message sent from a slider in my user interface. (iphone app)
This code works fine, but i this isn't very elegance...someone has some advices?

thanks

Code:

/* i receive "actualValue" from slider on my interface
 * this interface doesn't send every value, but the values sents depends on users
 *  speed, but i want to receive update on every value that is multiple of STEP.
 *  so, when i receive an update, i "miss" some value.
 */
 //if acutualValue is multiple of step
if (! (actualValue % step)) {
			//and difference between previsukyevalue > step
			if ( abs(actualValue - previusKeyValue) > step) {
				int i;
				if (actualValue > previusKeyValue) {
					/* i'm looking for values between previuskeyvalye + 1 and actualvalue that are multiple of step.
					 * i'm going from low value to higth
					 */
					for (i = previusKeyValue + 1; i < actualValue ; i++) {
						if (!(i % step)) {
							print("you skipped %d",i);
						}
					}
				}
				else {
					/* the same as previus code, but whene i go from higt value to low value.
					for (i = previusKeyValue - 1; i > actualValue ; i--) {
						if (!(i % step)) {
							print("you skipped %d",i);
						}
					}
					
				}
			}
			print("you found %d",actualValue);
			previusKeyValue = actualValue;
		}

So, is this a slider, or a guessing game, or what? This will miss some multiples, if that's what you're asking (i.e. if step is 5 and you go from 78 to 82, this code won't ever notice that you missed 80). I think what you want is this:

Code:

int orig = previusKeyValue/step; //I can't believe you're making me misspell previous
int now = actualValue/step;
if (orig != now) {
    int low = orig < now ? orig : now; //this is cheap but so what
    int high = orig < now ? now : orig;
    for (int i = low+1; i <= high; i++) {  //this is unusual but matches the situation
        printf("You skipped %d", i*step);
    }
}

I'm skipping the bit where actualValue % step == 0 because you are too.