if() else if() VS if() else if() in PWM control The Next CEO of Stack OverflowPWM Motor Speed ControlPIC32 PWM minimal examplePWM Control with MOSFET switchMotor control using PWMPWM 1HZ PIC18F14K50PWM Control vs Variable Voltage Control - Calculating Duty CycleSolenoid control over PWMDimming multiple LED panels without PWM to LEDs; modified DMX control or something else?PWM voltage control that can handle 20APWM Power Control

Why does the UK parliament need a vote on the political declaration?

Non-deterministic sum of floats

Is micro rebar a better way to reinforce concrete than rebar?

Make solar eclipses exceedingly rare, but still have new moons

Extending anchors in TikZ

Why does standard notation not preserve intervals (visually)

Anatomically Correct Strange Women In Ponds Distributing Swords

Limits on contract work without pre-agreed price/contract (UK)

How did the Bene Gesserit know how to make a Kwisatz Haderach?

Are there any unintended negative consequences to allowing PCs to gain multiple levels at once in a short milestone-XP game?

How do scammers retract money, while you can’t?

What is ( CFMCC ) on ILS approach chart?

Solving Integral Equation by Converting to Differential Equations

If the heap is initialized for security, then why is the stack uninitialized?

Written every which way

Sending manuscript to multiple publishers

Different harmonic changes implied by a simple descending scale

Why has the US not been more assertive in confronting Russia in recent years?

What's the best way to handle refactoring a big file?

Interfacing a button to MCU (and PC) with 50m long cable

Preparing Indesign booklet with .psd graphics for print

Is it my responsibility to learn a new technology in my own time my employer wants to implement?

How to transpose the 1st and -1th levels of arbitrarily nested array?

Is it possible to search for a directory/file combination?



if() else if() VS if() else if() in PWM control



The Next CEO of Stack OverflowPWM Motor Speed ControlPIC32 PWM minimal examplePWM Control with MOSFET switchMotor control using PWMPWM 1HZ PIC18F14K50PWM Control vs Variable Voltage Control - Calculating Duty CycleSolenoid control over PWMDimming multiple LED panels without PWM to LEDs; modified DMX control or something else?PWM voltage control that can handle 20APWM Power Control










0












$begingroup$


I'm using timer0 and a gpio pin to generate a 300Hz PWM signal on an ATmega2560.



The details if you care: F_CPU=16M, timer clock prescaler /256, count_max=208



The frequency changes depending on if I use an "if() else if()" or an "if() else if()".
The first generates 300Hz and the second generates 244Hz. Below is the code that generates the correct frequency. If I delete the curly brackets after the first two else's, I get 244Hz.



void pwm_on(uint8_t duty)
if(TCNT0 <= duty)= (1<<7); //turn on PB7
else
if(TCNT0 >= duty) //portion of counter inactive
PORTB &= ~(1<<7);//turn off PB7
else
if(TCNT0 >= COUNT_MAX) //if counter full
TCNT0=0;//reset counter
else
//pwm_on


Why am I getting such a significant frequency change with code that doesn't change the functionality?



Also calling this function is the only thing I do in Main after initializing.










share|improve this question











$endgroup$







  • 1




    $begingroup$
    I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
    $endgroup$
    – brhans
    yesterday










  • $begingroup$
    @brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 1




    $begingroup$
    It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
    $endgroup$
    – brhans
    yesterday






  • 4




    $begingroup$
    Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
    $endgroup$
    – Attie
    yesterday










  • $begingroup$
    @brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
    $endgroup$
    – TammerTheHammer
    yesterday















0












$begingroup$


I'm using timer0 and a gpio pin to generate a 300Hz PWM signal on an ATmega2560.



The details if you care: F_CPU=16M, timer clock prescaler /256, count_max=208



The frequency changes depending on if I use an "if() else if()" or an "if() else if()".
The first generates 300Hz and the second generates 244Hz. Below is the code that generates the correct frequency. If I delete the curly brackets after the first two else's, I get 244Hz.



void pwm_on(uint8_t duty)
if(TCNT0 <= duty)= (1<<7); //turn on PB7
else
if(TCNT0 >= duty) //portion of counter inactive
PORTB &= ~(1<<7);//turn off PB7
else
if(TCNT0 >= COUNT_MAX) //if counter full
TCNT0=0;//reset counter
else
//pwm_on


Why am I getting such a significant frequency change with code that doesn't change the functionality?



Also calling this function is the only thing I do in Main after initializing.










share|improve this question











$endgroup$







  • 1




    $begingroup$
    I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
    $endgroup$
    – brhans
    yesterday










  • $begingroup$
    @brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 1




    $begingroup$
    It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
    $endgroup$
    – brhans
    yesterday






  • 4




    $begingroup$
    Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
    $endgroup$
    – Attie
    yesterday










  • $begingroup$
    @brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
    $endgroup$
    – TammerTheHammer
    yesterday













0












0








0





$begingroup$


I'm using timer0 and a gpio pin to generate a 300Hz PWM signal on an ATmega2560.



The details if you care: F_CPU=16M, timer clock prescaler /256, count_max=208



The frequency changes depending on if I use an "if() else if()" or an "if() else if()".
The first generates 300Hz and the second generates 244Hz. Below is the code that generates the correct frequency. If I delete the curly brackets after the first two else's, I get 244Hz.



void pwm_on(uint8_t duty)
if(TCNT0 <= duty)= (1<<7); //turn on PB7
else
if(TCNT0 >= duty) //portion of counter inactive
PORTB &= ~(1<<7);//turn off PB7
else
if(TCNT0 >= COUNT_MAX) //if counter full
TCNT0=0;//reset counter
else
//pwm_on


Why am I getting such a significant frequency change with code that doesn't change the functionality?



Also calling this function is the only thing I do in Main after initializing.










share|improve this question











$endgroup$




I'm using timer0 and a gpio pin to generate a 300Hz PWM signal on an ATmega2560.



The details if you care: F_CPU=16M, timer clock prescaler /256, count_max=208



The frequency changes depending on if I use an "if() else if()" or an "if() else if()".
The first generates 300Hz and the second generates 244Hz. Below is the code that generates the correct frequency. If I delete the curly brackets after the first two else's, I get 244Hz.



void pwm_on(uint8_t duty)
if(TCNT0 <= duty)= (1<<7); //turn on PB7
else
if(TCNT0 >= duty) //portion of counter inactive
PORTB &= ~(1<<7);//turn off PB7
else
if(TCNT0 >= COUNT_MAX) //if counter full
TCNT0=0;//reset counter
else
//pwm_on


Why am I getting such a significant frequency change with code that doesn't change the functionality?



Also calling this function is the only thing I do in Main after initializing.







pwm timer firmware






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited yesterday







TammerTheHammer

















asked yesterday









TammerTheHammerTammerTheHammer

729




729







  • 1




    $begingroup$
    I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
    $endgroup$
    – brhans
    yesterday










  • $begingroup$
    @brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 1




    $begingroup$
    It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
    $endgroup$
    – brhans
    yesterday






  • 4




    $begingroup$
    Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
    $endgroup$
    – Attie
    yesterday










  • $begingroup$
    @brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
    $endgroup$
    – TammerTheHammer
    yesterday












  • 1




    $begingroup$
    I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
    $endgroup$
    – brhans
    yesterday










  • $begingroup$
    @brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 1




    $begingroup$
    It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
    $endgroup$
    – brhans
    yesterday






  • 4




    $begingroup$
    Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
    $endgroup$
    – Attie
    yesterday










  • $begingroup$
    @brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
    $endgroup$
    – TammerTheHammer
    yesterday







1




1




$begingroup$
I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
$endgroup$
– brhans
yesterday




$begingroup$
I'm voting to close this question as off-topic because it's a C syntax understanding problem and has nothing specifically to do with the hardware the code is running on.
$endgroup$
– brhans
yesterday












$begingroup$
@brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
$endgroup$
– TammerTheHammer
yesterday




$begingroup$
@brhans I edited the title of the question to specify it is for PWM control since that seems to be the largest point of contention for moving this question to stackoverflow. I am not a computer programmer and this is not a computer programming question. It is about how the syntax is affecting register level hardware timer0, NOT why doesn't this syntax work.
$endgroup$
– TammerTheHammer
yesterday




1




1




$begingroup$
It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
$endgroup$
– brhans
yesterday




$begingroup$
It doesn't really matter what the end application is - your problem is that you're not understanding standard C syntax. You would have this same problem if you were writing similarly structured code to run on a PC. The fact that you're writing firmware for a micro is only relevant in how this problem is presenting itself - it's just the symptom.
$endgroup$
– brhans
yesterday




4




4




$begingroup$
Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
$endgroup$
– Attie
yesterday




$begingroup$
Why aren't you using the perhipheral to generate the PWM for you? Which would allow you to dedicate zero CPU time to this task.
$endgroup$
– Attie
yesterday












$begingroup$
@brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
$endgroup$
– TammerTheHammer
yesterday




$begingroup$
@brhans I have only been asking about the symptom, that being the change in frequency. My understanding shouldn't matter since the question has always been about the symptom.
$endgroup$
– TammerTheHammer
yesterday










2 Answers
2






active

oldest

votes


















5












$begingroup$

This is more a code syntax question. Could be offtopic, might be moved to stackoverflow.



if() else if() and if() else if() are different.

With correct indentation and brackets the issue is immediately visible:



if() else if()



if(condition)
statement
else
statement

if(condition)
statement



And



if() else if()



if(condition)
statement
else
if(condition)
statement







share|improve this answer









$endgroup$












  • $begingroup$
    i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 7




    $begingroup$
    @TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
    $endgroup$
    – Hasan alattar
    yesterday






  • 2




    $begingroup$
    @TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
    $endgroup$
    – Agent_L
    yesterday



















2












$begingroup$

The reason why my frequency goes down and the period goes up with the "if() else if()" is because TCNT0 >= duty is always true when TCNT0 >= COUNT_MAX so the last statement that resets the counter at COUNT_MAX never runs so my counter overflows instead of resetting at a count that gets me a frequency of 300Hz.



I figured it out pretty quickly after posting, sorry if I am misusing this site, I'm posting a question on meta to see what the collective thinks so I can learn and use this resource as best as possible.






share|improve this answer











$endgroup$








  • 4




    $begingroup$
    Posting of your own solution to a problem is encouraged.
    $endgroup$
    – AndrejaKo
    yesterday










  • $begingroup$
    if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
    $endgroup$
    – Hasan alattar
    yesterday











  • $begingroup$
    @Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 2




    $begingroup$
    Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
    $endgroup$
    – user694733
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
    $endgroup$
    – r_ahlskog
    yesterday











Your Answer





StackExchange.ifUsing("editor", function ()
return StackExchange.using("mathjaxEditing", function ()
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
);
);
, "mathjax-editing");

StackExchange.ifUsing("editor", function ()
return StackExchange.using("schematics", function ()
StackExchange.schematics.init();
);
, "cicuitlab");

StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "135"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);













draft saved

draft discarded


















StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2felectronics.stackexchange.com%2fquestions%2f429453%2fif-else-if-vs-if-else-if-in-pwm-control%23new-answer', 'question_page');

);

Post as a guest















Required, but never shown

























2 Answers
2






active

oldest

votes








2 Answers
2






active

oldest

votes









active

oldest

votes






active

oldest

votes









5












$begingroup$

This is more a code syntax question. Could be offtopic, might be moved to stackoverflow.



if() else if() and if() else if() are different.

With correct indentation and brackets the issue is immediately visible:



if() else if()



if(condition)
statement
else
statement

if(condition)
statement



And



if() else if()



if(condition)
statement
else
if(condition)
statement







share|improve this answer









$endgroup$












  • $begingroup$
    i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 7




    $begingroup$
    @TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
    $endgroup$
    – Hasan alattar
    yesterday






  • 2




    $begingroup$
    @TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
    $endgroup$
    – Agent_L
    yesterday
















5












$begingroup$

This is more a code syntax question. Could be offtopic, might be moved to stackoverflow.



if() else if() and if() else if() are different.

With correct indentation and brackets the issue is immediately visible:



if() else if()



if(condition)
statement
else
statement

if(condition)
statement



And



if() else if()



if(condition)
statement
else
if(condition)
statement







share|improve this answer









$endgroup$












  • $begingroup$
    i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 7




    $begingroup$
    @TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
    $endgroup$
    – Hasan alattar
    yesterday






  • 2




    $begingroup$
    @TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
    $endgroup$
    – Agent_L
    yesterday














5












5








5





$begingroup$

This is more a code syntax question. Could be offtopic, might be moved to stackoverflow.



if() else if() and if() else if() are different.

With correct indentation and brackets the issue is immediately visible:



if() else if()



if(condition)
statement
else
statement

if(condition)
statement



And



if() else if()



if(condition)
statement
else
if(condition)
statement







share|improve this answer









$endgroup$



This is more a code syntax question. Could be offtopic, might be moved to stackoverflow.



if() else if() and if() else if() are different.

With correct indentation and brackets the issue is immediately visible:



if() else if()



if(condition)
statement
else
statement

if(condition)
statement



And



if() else if()



if(condition)
statement
else
if(condition)
statement








share|improve this answer












share|improve this answer



share|improve this answer










answered yesterday









Jeroen3Jeroen3

11.6k1748




11.6k1748











  • $begingroup$
    i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 7




    $begingroup$
    @TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
    $endgroup$
    – Hasan alattar
    yesterday






  • 2




    $begingroup$
    @TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
    $endgroup$
    – Agent_L
    yesterday

















  • $begingroup$
    i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 7




    $begingroup$
    @TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
    $endgroup$
    – Hasan alattar
    yesterday






  • 2




    $begingroup$
    @TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
    $endgroup$
    – Marko Buršič
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
    $endgroup$
    – Agent_L
    yesterday
















$begingroup$
i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
$endgroup$
– TammerTheHammer
yesterday




$begingroup$
i'm asking about the change in frequency (hardware) caused by the change in syntax (code) which is why I asked it here.
$endgroup$
– TammerTheHammer
yesterday




7




7




$begingroup$
@TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
$endgroup$
– Marko Buršič
yesterday




$begingroup$
@TammerTheHammer Of course, but you should learn to use if-else-elseif statements to solve the puzzle. It's not the hardware, but your code.
$endgroup$
– Marko Buršič
yesterday




1




1




$begingroup$
electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
$endgroup$
– Hasan alattar
yesterday




$begingroup$
electronics.stackexchange.com/help/on-topic / @TammerTheHammer you asked in the right stack. this has been discussed many times before and some people try to redirect others to stackOverflow while actually you should ask here for softwares related to programming microcontrollers
$endgroup$
– Hasan alattar
yesterday




2




2




$begingroup$
@TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
$endgroup$
– Marko Buršič
yesterday




$begingroup$
@TammerTheHammer Well, looking at the question tittle, people would think you are asking how to use if then statements.
$endgroup$
– Marko Buršič
yesterday




1




1




$begingroup$
@TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
$endgroup$
– Agent_L
yesterday





$begingroup$
@TammerTheHammer You haven't asked how to use if-else, but that's only because you don't realize it's the root of your problem. code that doesn't change the functionality - here it is.
$endgroup$
– Agent_L
yesterday














2












$begingroup$

The reason why my frequency goes down and the period goes up with the "if() else if()" is because TCNT0 >= duty is always true when TCNT0 >= COUNT_MAX so the last statement that resets the counter at COUNT_MAX never runs so my counter overflows instead of resetting at a count that gets me a frequency of 300Hz.



I figured it out pretty quickly after posting, sorry if I am misusing this site, I'm posting a question on meta to see what the collective thinks so I can learn and use this resource as best as possible.






share|improve this answer











$endgroup$








  • 4




    $begingroup$
    Posting of your own solution to a problem is encouraged.
    $endgroup$
    – AndrejaKo
    yesterday










  • $begingroup$
    if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
    $endgroup$
    – Hasan alattar
    yesterday











  • $begingroup$
    @Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 2




    $begingroup$
    Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
    $endgroup$
    – user694733
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
    $endgroup$
    – r_ahlskog
    yesterday















2












$begingroup$

The reason why my frequency goes down and the period goes up with the "if() else if()" is because TCNT0 >= duty is always true when TCNT0 >= COUNT_MAX so the last statement that resets the counter at COUNT_MAX never runs so my counter overflows instead of resetting at a count that gets me a frequency of 300Hz.



I figured it out pretty quickly after posting, sorry if I am misusing this site, I'm posting a question on meta to see what the collective thinks so I can learn and use this resource as best as possible.






share|improve this answer











$endgroup$








  • 4




    $begingroup$
    Posting of your own solution to a problem is encouraged.
    $endgroup$
    – AndrejaKo
    yesterday










  • $begingroup$
    if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
    $endgroup$
    – Hasan alattar
    yesterday











  • $begingroup$
    @Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 2




    $begingroup$
    Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
    $endgroup$
    – user694733
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
    $endgroup$
    – r_ahlskog
    yesterday













2












2








2





$begingroup$

The reason why my frequency goes down and the period goes up with the "if() else if()" is because TCNT0 >= duty is always true when TCNT0 >= COUNT_MAX so the last statement that resets the counter at COUNT_MAX never runs so my counter overflows instead of resetting at a count that gets me a frequency of 300Hz.



I figured it out pretty quickly after posting, sorry if I am misusing this site, I'm posting a question on meta to see what the collective thinks so I can learn and use this resource as best as possible.






share|improve this answer











$endgroup$



The reason why my frequency goes down and the period goes up with the "if() else if()" is because TCNT0 >= duty is always true when TCNT0 >= COUNT_MAX so the last statement that resets the counter at COUNT_MAX never runs so my counter overflows instead of resetting at a count that gets me a frequency of 300Hz.



I figured it out pretty quickly after posting, sorry if I am misusing this site, I'm posting a question on meta to see what the collective thinks so I can learn and use this resource as best as possible.







share|improve this answer














share|improve this answer



share|improve this answer








edited yesterday

























answered yesterday









TammerTheHammerTammerTheHammer

729




729







  • 4




    $begingroup$
    Posting of your own solution to a problem is encouraged.
    $endgroup$
    – AndrejaKo
    yesterday










  • $begingroup$
    if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
    $endgroup$
    – Hasan alattar
    yesterday











  • $begingroup$
    @Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 2




    $begingroup$
    Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
    $endgroup$
    – user694733
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
    $endgroup$
    – r_ahlskog
    yesterday












  • 4




    $begingroup$
    Posting of your own solution to a problem is encouraged.
    $endgroup$
    – AndrejaKo
    yesterday










  • $begingroup$
    if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
    $endgroup$
    – Hasan alattar
    yesterday











  • $begingroup$
    @Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
    $endgroup$
    – TammerTheHammer
    yesterday






  • 2




    $begingroup$
    Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
    $endgroup$
    – user694733
    yesterday






  • 1




    $begingroup$
    @TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
    $endgroup$
    – r_ahlskog
    yesterday







4




4




$begingroup$
Posting of your own solution to a problem is encouraged.
$endgroup$
– AndrejaKo
yesterday




$begingroup$
Posting of your own solution to a problem is encouraged.
$endgroup$
– AndrejaKo
yesterday












$begingroup$
if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
$endgroup$
– Hasan alattar
yesterday





$begingroup$
if you compile without optimaziation i think you will get same frequency but much lower than 244Hz anyway. i find it wierd that you right else . check this out : cpp.sh/3rcrt
$endgroup$
– Hasan alattar
yesterday













$begingroup$
@Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
$endgroup$
– TammerTheHammer
yesterday




$begingroup$
@Hasanalattar I'll need to research more on compiling before I'll understand this, but why I use else is because i learned misra-c standard in school and it was drilled into my brain that all if() statements must have its accompanying else.
$endgroup$
– TammerTheHammer
yesterday




2




2




$begingroup$
Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
$endgroup$
– user694733
yesterday




$begingroup$
Sidenote: To my understanding misra requires else only at the end of if ... else if constructs, and not on plain if statements.
$endgroup$
– user694733
yesterday




1




1




$begingroup$
@TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
$endgroup$
– r_ahlskog
yesterday




$begingroup$
@TammerTheHammer I tried looking for that MISRA rule, and as far as I can tell there is no such rule. There is one for chained if .. else if ... else if ... which sort of make sense but for a simple if there is no need for an empty else
$endgroup$
– r_ahlskog
yesterday

















draft saved

draft discarded
















































Thanks for contributing an answer to Electrical Engineering Stack Exchange!


  • Please be sure to answer the question. Provide details and share your research!

But avoid


  • Asking for help, clarification, or responding to other answers.

  • Making statements based on opinion; back them up with references or personal experience.

Use MathJax to format equations. MathJax reference.


To learn more, see our tips on writing great answers.




draft saved


draft discarded














StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2felectronics.stackexchange.com%2fquestions%2f429453%2fif-else-if-vs-if-else-if-in-pwm-control%23new-answer', 'question_page');

);

Post as a guest















Required, but never shown





















































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown

































Required, but never shown














Required, but never shown












Required, but never shown







Required, but never shown







Popular posts from this blog

រឿង រ៉ូមេអូ និង ហ្ស៊ុយលីយេ សង្ខេបរឿង តួអង្គ បញ្ជីណែនាំ

Crop image to path created in TikZ? Announcing the arrival of Valued Associate #679: Cesar Manara Planned maintenance scheduled April 17/18, 2019 at 00:00UTC (8:00pm US/Eastern)Crop an inserted image?TikZ pictures does not appear in posterImage behind and beyond crop marks?Tikz picture as large as possible on A4 PageTransparency vs image compression dilemmaHow to crop background from image automatically?Image does not cropTikzexternal capturing crop marks when externalizing pgfplots?How to include image path that contains a dollar signCrop image with left size given

Romeo and Juliet ContentsCharactersSynopsisSourcesDate and textThemes and motifsCriticism and interpretationLegacyScene by sceneSee alsoNotes and referencesSourcesExternal linksNavigation menu"Consumer Price Index (estimate) 1800–"10.2307/28710160037-3222287101610.1093/res/II.5.31910.2307/45967845967810.2307/2869925286992510.1525/jams.1982.35.3.03a00050"Dada Masilo: South African dancer who breaks the rules"10.1093/res/os-XV.57.1610.2307/28680942868094"Sweet Sorrow: Mann-Korman's Romeo and Juliet Closes Sept. 5 at MN's Ordway"the original10.2307/45957745957710.1017/CCOL0521570476.009"Ram Leela box office collections hit massive Rs 100 crore, pulverises prediction"Archived"Broadway Revival of Romeo and Juliet, Starring Orlando Bloom and Condola Rashad, Will Close Dec. 8"Archived10.1075/jhp.7.1.04hon"Wherefore art thou, Romeo? To make us laugh at Navy Pier"the original10.1093/gmo/9781561592630.article.O006772"Ram-leela Review Roundup: Critics Hail Film as Best Adaptation of Romeo and Juliet"Archived10.2307/31946310047-77293194631"Romeo and Juliet get Twitter treatment""Juliet's Nurse by Lois Leveen""Romeo and Juliet: Orlando Bloom's Broadway Debut Released in Theaters for Valentine's Day"Archived"Romeo and Juliet Has No Balcony"10.1093/gmo/9781561592630.article.O00778110.2307/2867423286742310.1076/enst.82.2.115.959510.1080/00138380601042675"A plague o' both your houses: error in GCSE exam paper forces apology""Juliet of the Five O'Clock Shadow, and Other Wonders"10.2307/33912430027-4321339124310.2307/28487440038-7134284874410.2307/29123140149-661129123144728341M"Weekender Guide: Shakespeare on The Drive""balcony"UK public library membership"romeo"UK public library membership10.1017/CCOL9780521844291"Post-Zionist Critique on Israel and the Palestinians Part III: Popular Culture"10.2307/25379071533-86140377-919X2537907"Capulets and Montagues: UK exam board admit mixing names up in Romeo and Juliet paper"Istoria Novellamente Ritrovata di Due Nobili Amanti2027/mdp.390150822329610820-750X"GCSE exam error: Board accidentally rewrites Shakespeare"10.2307/29176390149-66112917639"Exam board apologises after error in English GCSE paper which confused characters in Shakespeare's Romeo and Juliet""From Mariotto and Ganozza to Romeo and Guilietta: Metamorphoses of a Renaissance Tale"10.2307/37323537323510.2307/2867455286745510.2307/28678912867891"10 Questions for Taylor Swift"10.2307/28680922868092"Haymarket Theatre""The Zeffirelli Way: Revealing Talk by Florentine Director""Michael Smuin: 1938-2007 / Prolific dance director had showy career"The Life and Art of Edwin BoothRomeo and JulietRomeo and JulietRomeo and JulietRomeo and JulietEasy Read Romeo and JulietRomeo and Julieteeecb12003684p(data)4099369-3n8211610759dbe00d-a9e2-41a3-b2c1-977dd692899302814385X313670221313670221