进入 switch case 将每个变量重置为零和一些提示

Posted

技术标签:

【中文标题】进入 switch case 将每个变量重置为零和一些提示【英文标题】:Entering in switch case resets to zero every variable and some tips 【发布时间】:2018-07-07 07:36:31 【问题描述】:

我在考试中编写了这段代码,但它有一些问题(因此我失败了)。

在通过我的板调试时(我有一个 atmel atmega328p Xplained mini),我不知道为什么,但是在进入主开关以及 switch (currentstatus)case checkled:我所有的状态变量都被重置为零。 如果我通过模拟器调试它似乎工作正常。

如果我将代码上传到板子上,它会出现问题,比如它无法识别某些中断。

另外我想问你一件事。我在考试中使用了一个子板,它可以防止访问 PWM 输出的正确引脚,而且我必须驱动特定的输出(我的子板的 LED,例如在这段代码中,我必须驱动 PORTC5 和 PORTC4),所以我必须手动实现 PWM 。我试图实现您可以在代码中看到的解决方案(我使用的是 1 毫秒的计时器)。有没有更简单、更智能的解决方案?

你能帮帮我吗?

#include <avr/io.h>
#include <avr/interrupt.h>

typedef enumcheckled, heatingup, exposure, stop machine;
machine currentstate=checkled;

typedef enumtwenty, fourty, sixty, eighty brightness;
brightness light=twenty;

typedef enumtwoseconds, fourseconds, eightseconds, sixteenseconds exposuretime;
exposuretime time=twoseconds;

volatile uint16_t tick=0;
volatile uint16_t oldtick=0;

volatile uint8_t flagchanged=0;

ISR(PCINT2_vect)

    if((PIND&(1<<PIND2))==0)
    
        if(currentstate==checkled)
            currentstate=heatingup;
    

    if((PIND&(1<<PIND3))==0)
    
        if(currentstate==exposure || currentstate==heatingup)
        
            currentstate=stop;
            PORTC&=~(1<<PORTC4);
            tick=0;
        
    

    if((PIND&(1<<PIND4))==0)
    
        if(currentstate==checkled)
        
            flagchanged=1;

            if(time==sixteenseconds)//rise time
                ;
            else if(time==eightseconds)
                time=sixteenseconds;
            else if(time==fourseconds)
                time=eightseconds;
            else if(time==twoseconds)
                time=fourseconds;
        
    

    if((PIND&(1<<PIND5))==0)
    
        if(currentstate==checkled)
        
            flagchanged=1;

            if(time==sixteenseconds)//decrease time
                time=eightseconds;
            else if(time==eightseconds)
                time=fourseconds;
            else if(time==fourseconds)
                time=twoseconds;
            else if(time==twoseconds)
                ;
        
    

    if((PIND&(1<<PIND6))==0)
    
        if(currentstate==checkled)
        
            flagchanged=1;

            if(light==eighty)//rise brightness
                ;
            else if(light==sixty)
                light=eighty;
            else if(light==fourty)
                light=sixty;
            else if(light==twenty)
                light=fourty;
        
    

    if((PIND&(1<<PIND7))==0)
    
        if(currentstate==checkled)
        
            flagchanged=1;

            if(light==eighty)//decrease brightness
                light=sixty;
            else if(light==sixty)
                light=fourty;
            else if(light==fourty)
                light=twenty;
            else if(light==twenty)
                ;
        
    



ISR(TIMER1_COMPA_vect)

    tick++;


int main(void)

    //DDRs
    DDRD&=~((1<<PIND2)|(1<<PIND3)|(1<<PIND4)|(1<<PIND5)|(1<<PIND6)|(1<<PIND7));//input
    PORTD|=(1<<PORTD2)|(1<<PORTD3)|(1<<PORTD4)|(1<<PORTD5)|(1<<PORTD6)|(1<<PORTD7);//pull-up resistors

    DDRC|=(1<<PORTC0)|(1<<PORTC1)|(1<<PORTC2)|(1<<PORTC3)|(1<<PORTC4)|(1<<PORTC5);//output

    //PCINTERRUPT
    PCICR|=(1<<PCIE2);
    PCMSK2|=(1<<PCINT18)|(1<<PCINT19)|(1<<PCINT20)|(1<<PCINT21)|(1<<PCINT22)|(1<<PCINT23);

    //TIMER
    TCCR0A|=(1<<WGM01);
    TIMSK0|=(1<<OCIE0A);
    OCR0A=15;//((clockfrequency/prescaler)*time)-1 - 1millisecond
    TCCR0B|=(1<<CS02)|(1<<CS00);

    sei();

    volatile uint8_t counter=0;
    volatile uint16_t check=0;

    while(1)
    
        switch (currentstate)//FORM HERE
        
            case checkled://TO HERE every variable is reset to ZERO
                if(flagchanged!=0)
                


                    if(light==twenty)//led intesita
                    
                        PORTC&=~(1<<PORTC0);
                        PORTC&=~(1<<PORTC1);
                    
                    else if(light==fourty)
                    
                        PORTC|=(1<<PORTC0);
                        PORTC&=~(1<<PORTC1);
                    
                    else if(light==sixty)
                    
                        PORTC&=~(1<<PORTC0);
                        PORTC|=(1<<PORTC1);
                    
                    else if(light==eighty)
                    
                        PORTC|=(1<<PORTC0);
                        PORTC|=(1<<PORTC1);
                    


                    if(time==twoseconds)//led time
                    
                        PORTC&=~(1<<PORTC2);
                        PORTC&=~(1<<PORTC3);
                    
                    else if(time==fourseconds)
                    
                        PORTC|=(1<<PORTC2);
                        PORTC&=~(1<<PORTC3);
                    
                    else if(time==eightseconds)
                    
                        PORTC&=~(1<<PORTC2);
                        PORTC|=(1<<PORTC3);
                    
                    else if(time==sixteenseconds)
                    
                        PORTC|=(1<<PORTC2);
                        PORTC|=(1<<PORTC3);
                    

                    flagchanged=0;
                
            break;

            case heatingup:
                if(tick<=3000)
                
                    if(tick!=oldtick)
                    
                        oldtick=tick;

                        if((tick%500)==0)//duty cycle al 50%
                            PORTC^=(1<<PORTC5);

                        if((tick%5)==0)//5% duty cycle
                        

                            if(counter<=94)//per il 95% del periodo sta spento
                            
                                PORTC&=~(1<<PORTC4);
                            
                            else if(counter>=95 && counter<=101)//per il 5% del periodo sta acceso
                            

                                if(counter==101)
                                
                                    PORTC&=~(1<<PORTC4);
                                    counter=0;
                                
                                else
                                PORTC|=(1<<PORTC4);

                            

                            counter++;
                        
                    
                
                else
                
                    currentstate=exposure;
                    counter=0;
                    tick=0;

                    PORTC|=(1<<PORTC5);

                    //checking time of how much the lamp needs to be light up
                    switch (time)
                    
                        case twoseconds:
                            check=2000;
                        break;

                        case fourseconds:
                            check=4000;
                        break;

                        case eightseconds:
                            check=8000;
                        break;

                        case sixteenseconds:
                            check=16000;
                        break;
                    
                
                break;

            case exposure:
                if(tick<=check)
                
                    switch (light)//Checking duty cycle
                    
                        case twenty://20% duty cycle
                            if(tick!=oldtick)
                            
                                if((tick%20)==0)
                                

                                    if(counter<=4)//80% off
                                    
                                        PORTC&=~(1<<PORTC4);
                                    
                                    else if(counter>=5 && counter<=6)//20% on
                                    
                                        if(counter==6)
                                        
                                            PORTC&=~(1<<PORTC4);
                                            counter=0;
                                        
                                        else
                                        PORTC|=(1<<PORTC4);

                                    

                                    counter++;
                                
                            
                        break;

                        case fourty://40% duty cycle
                            if(tick!=oldtick)
                            
                                if((tick%20)==0)
                                

                                    if(counter<=3)//60% off
                                    
                                        PORTC&=~(1<<PORTC4);
                                    
                                    else if(counter>=4 && counter<=6)//40% on
                                    
                                        if(counter==6)
                                        
                                            PORTC&=~(1<<PORTC4);
                                            counter=0;
                                        
                                        else
                                        PORTC|=(1<<PORTC4);

                                    

                                    counter++;
                                
                            
                        break;

                        case sixty://60% duty cycle
                            if(tick!=oldtick)
                            
                                if((tick%20)==0)
                                

                                    if(counter<=2)//40% off
                                    
                                        PORTC&=~(1<<PORTC4);
                                    
                                    else if(counter>=3 && counter<=6)//60% on
                                    
                                        if(counter==6)
                                        
                                            PORTC&=~(1<<PORTC4);
                                            counter=0;
                                        
                                        else
                                        PORTC|=(1<<PORTC4);

                                    

                                    counter++;
                                
                            
                        break;

                        case eighty://80% duty cycle
                            if(tick!=oldtick)
                            
                                if((tick%20)==0)
                                

                                    if(counter<=1)//20% off
                                    
                                        PORTC&=~(1<<PORTC4);
                                    
                                    else if(counter>=2 && counter<=6)//80% on
                                    
                                        if(counter==6)
                                        
                                            PORTC&=~(1<<PORTC4);
                                            counter=0;
                                        
                                        else
                                        PORTC|=(1<<PORTC4);

                                    

                                    counter++;
                                
                            
                        break;

                    
                
                else
                
                    PORTC&=~(1<<PORTC4);
                    PORTC&=~(1<<PORTC5);
                    currentstate=checkled;
                    flagchanged=1;
                

            break;

            case stop://Emergency stop
                if(tick<=2000)
                
                    if(tick!=oldtick)
                    
                        oldtick=tick;
                        if((tick%250)==0)
                        PORTC|=(1<<PORTC5);
                    
                
                else
                
                    PORTC&=~(1<<PORTC5);
                    flagchanged=1;
                    currentstate=checkled;
                
            break;
        
    

【问题讨论】:

欢迎来到 ***.com。请花一些时间阅读the help pages,尤其是名为"What topics can I ask about here?" 和"What types of questions should I avoid asking?" 的部分。也请take the tour 和read about how to ask good questions。最后请学习如何创建Minimal, Complete, and Verifiable Example。 另外,请阅读this question checklist 和idownvotedbecau.se 的所有内容,了解您的问题可能被否决的一些原因。最后请learn how to debug your programs. 您可以通过将所有标识符翻译成英文来使显示的代码更容易理解。 请给我 2 分钟 你能在没有所有中断相关代码的情况下演示问题吗?如果不是,则可能意味着中断正在扰乱您的逻辑。如果是的话,如果您在此处显示无中断代码,您可能会更快获得更好的帮助。它会更好地匹配 MCVE 中的 M。 【参考方案1】:

在主代码中,您启用了定时器 0 的输出比较中断

TIMSK0|=(1<<OCIE0A);

但是您没有此向量的中断处理程序(您有 TIMER1_COMPA_vect - 而是用于计时器 1)。

所以,当定时器中断发生时,它由默认向量处理,它只是将控制重定向到复位向量,这导致应用程序从一开始就启动,包括变量初始化,这会导致你的所有意外行为正在观察。

因此,解决方案很简单:要么禁用 OCIE0A,要么为此中断创建一个处理程序。

【讨论】:

【参考方案2】:

您正在启用 Timer/Counter0 比较匹配 A 中断与下面的行

TIMSK0|=(1&lt;&lt;OCIE0A);

但是,您正在尝试使用下面部分中的 Timer/Counter1 的比较中断 ISR。

ISR(TIMER1_COMPA_vect) tick++;

建议修改:

您可以通过如下修改定时器初始化来启用 Timer/Counter1 比较匹配 A 中断并保持 ISR 不变:

`TIMSK1|=(1

保持Timer/Counter0比较匹配A中断初始化不变,修改ISR如下,在中断时调用Timer/Counter0的ISR。

ISR(TIMER0_COMPA_vect) tick++;

希望这会有所帮助..

【讨论】:

以上是关于进入 switch case 将每个变量重置为零和一些提示的主要内容,如果未能解决你的问题,请参考以下文章

我可以使用带有两个变量的 case/switch 语句吗?

Java switch case语句

switch的用法

变量在“when sample()”语句中自动重置为零

微商进货返利系统代码解析

switch语句怎么用啊 具体啊!