[STM32] save TIM3->CNT to 2 variables for display?

[kgavionics]

Hello
I'm trying to implement a simple menu to modify 2 variables (Volume and Contrast) using a rotary encoder hooked to TIM3 (encoder mode). What I want to achieve is when I press on the rotary switch it toggles between the 2 variables and then rotating the encoder, I can modify their respective values. My problem is I can't find a way to restore the saved counter when I toggle the switch. I tried to use 2 temporary variables, but it when I restore them to the TIM3→CNT doesn't update
Please don't laugh on my code, I'm not a programmer by trade! I hope that someone cleaver in programming help me to resolve this issue!
Thanks in advance

Code: [Select]

{
void rencoder(void)
{
pb_scan();
if(pushb==31)
{
menucounter++;
if(menucounter>2) menucounter=0;
}
switch(menucounter)
case(0):

volume=TIM3->CNT;//how to save this value without affecting the contrast variable

break;


case(1):



contrast=TIM3->CNT;



break;
}

}

[wek]

> when I restore them to the TIM3→CNT

Show that.

JW

[langwadt]

read the code again, look at what it does not what you think it does

you repeatedly write the counter, of course it never counts

[langwadt]

I know that it's not working, that's why I'm asking for help dude!

you write the counter and then read the counter, what could you possibly read other than what you just wrote?

[cv007]

Your button push is when you want to restore the counter-

Code: [Select]

void rencoder(void) {
    pb_scan();
    if( pushb == 31 ){
        switch( ++menucounter ){
            default: menucounter = 0; // if not in a case below, start over at 0
            //fallthrough
            case 0: TIM3->CNT = volume; break; //restore volume value
            case 1: TIM3->CNT = contrast; break; //restore contrast value
        }
    }
    switch( menucounter ){
    case(0):// changing volume value
                volume = TIM3->CNT;
break;

    case(1):// changing contrast value
                contrast = TIM3->CNT;
break;
    }
}

[kgavionics]

Thanks cv007, your code is working like a charm!
You really made my day!
Can you please explain this

Code: [Select]

switch( ++menucounter ) why ++menucounter?

[cv007]

why ++menucounter?

Well, it just replaces the equivalent-

if( pushb == 31 ){
        menucounter++;
        switch( menucounter ){

where the increment takes place before the switch statement uses its value (pre-incremented), and the default case replaces checking for overflow.

[kgavionics]

why ++menucounter?

Well, it just replaces the equivalent-

if( pushb == 31 ){
        menucounter++;
        switch( menucounter ){

where the increment takes place before the switch statement uses its value (pre-incremented), and the default case replaces checking for overflow.

All clear now, Thanks