久久久久久久999_99精品久久精品一区二区爱城_成人欧美一区二区三区在线播放_国产精品日本一区二区不卡视频_国产午夜视频_欧美精品在线观看免费

標題: 解決一個C語言程序分號“;”引起的bug [打印本頁]

作者: f556 時間: 2022-9-13 23:45
標題: 解決一個C語言程序分號“;”引起的bug
曾想也標題黨一次，寫成《一個C程序分號“;”引起的血案》，筆者原本厭惡標題黨，厭惡那些“xxxx了”、“XXXX震驚，不看后悔”、“XXXX，看這個就夠了”。也就不在此惡心各位同好。
C程序的分號，是任何寫過C程序的人都明白的，很重要，用的很多。但此文將要說明的是分號不正確的使用，導致一個隱藏的Bug。

一、發現錯誤
家用的太陽能熱水器控制，執行電加熱后，本來有一個LED燈閃爍，提示電加熱是開啟的。原來用了兩年多，一直正常，前幾天發現是啟動電加熱后，閃一次LED就熄滅。全套控制、接線都是自己一點點搞起來的，所以要解決這個故障，有了優勢，但更有了苦命累積。一個同事二貨在別墅頂上裝了太陽能熱水器，為了陰天有熱水，另外加了一個電熱水器，還要考慮閥門切換，考慮電熱水器防雨、考慮啟動或關閉總電源等等，反正一切由安裝的說了算，不用費腦子。想想我布的多根網線、多點測溫、兩組太能能占據掉的位置，自己在斜的瓦屋頂上熔水管、焊接線，那么多辛苦都過去了，借肋中秋節放假而且一直下雨在家的時間，有必要把LED不指示的問題解決。

二、查找問題
記得LED指示燈與電加熱分別各用一個IO口，也必須如此，才可以閃爍，不然加熱時變成繼電器反復通斷。LED用9012PNP管驅動，且平時一直會微閃，這個一開始就如此，胡亂微閃就閃吧，大概分析了一下，如果用NPN管不會微閃，PNP如果調偏置電阻也可以使全滅，涉及小小的三極管復雜的功能，在此不跑主題，也不細說。
未連接系統，隨意看了一下程序，程序應該沒有動過，上月寫兩個發明專利時，為了測試幾個數據，重新下載過一套改版的程序到Flash，后又恢復了。原因可能是在什么時候動過程序，導致出現Bug。程序看不出明顯的錯誤，必須聯機調試，監視一下變量。
真是累不動。還好SBW下載線制作過一根專用的，直接插上就OK，不用其它排列順序的下載線換線位。搜索全部工程文件“電加熱” “電熱”，找到以下語句：
#define EHeat_off P7OUT |=  BIT0; EHisON=0  //電加熱off
#define EHeat_on P7OUT &=~ BIT0; EHisON=1  //電加熱on
聚焦到EHeat_off語句，再搜EHeat_off，只有3處有過：
1、             BASICTIMER_VECTOR定時器中斷250ms一次，快結束時啟動一次測溫，在測溫程序中，結果出來后，用了以下：
  /*-----------停止電加熱-----------*/
if(UperT > EleHeatTempe*10)  EHeat_off;    //達到設置溫度（45.0---450），關閉電加熱。
2、             第二處是狀態機控制的菜單程序中，當Cancel時，關閉電加熱：
if(Menu_sel==3) EHeat_off;
另外一處就是define處的定義了。

三、解決問題
程序高手估計已經發現問題了，筆者愚鈍。啟動電加熱后，中斷并監視P7口，P70確實是0，剛好這兩周，家里主空開被我拆了調整空開位準備在家門口接充電樁，用鉗表卡16平總進戶線電流5A多，關閉電加熱時0.3A。證明加熱正常。僅僅是指示燈出故障。再搜索全部工程，除了3處定義，只在__interrupt void BT_ISR(void)中出現過：
if(  EHisON) P4OUT ^=  BIT2;    //電加熱指示燈閃
else  if((P4IN & BIT2) ==0 ) P4OUT |=  BIT2;
程序前強加一個EHisON=1; 指示燈會閃。說明PNP管、LED均正常。EHisON是一個標志電加熱是打開狀態的變量，原意是開關電熱時，一并切換此標志是開或關。監視此變量總是為0。
回到if(UperT > EleHeatTempe*10)  EHeat_off;終于發現問題。這個語句相當于：
if(UperT > EleHeatTempe*10)  P7OUT |=  BIT0; EHisON=0;
很明顯，if執行了“P7OUT |=  BIT0;”，而“EHisON=0;”不論if如何都會執行，解決辦法自然是加{}。當然還有另一個解決辦法，不用加{}，看大家的興趣，我在后邊回復中說明。

我一般寫define時，習慣是最后不加“;”而在執行代碼中加，這樣看上去比較統一。此次定義是P7OUT |=  BIT0和EHisON=0兩個語句，故在中間順手加了一個“;”。又一個習慣是簡單的if喜歡一行寫完，不加{}。甚至長一點的if都喜歡一行寫完，以讓小小的屏幕好容納上萬行代碼。各種習慣、巧合，導至這個Bug正常運行了2年時間。今天回頭找最老版的程序查看，這個錯誤一直在，不明白為什么原來可以正常閃爍，難道編譯器根據option設置修正過此錯誤？如果有明白的大神，請不吝賜教，先謝謝了。主控是MSP430F4152，但IDE我不說了（IXX6），涉及其它不必要的麻煩。
另外，我不喜歡那些上傳整套程序的，一般也無心看，除非是自己手上有的小模塊，而又沒有玩過的，針對此模塊的驅動可以看看。也更喜歡關注思路、方法、原理。所以也不上傳整套程序，畢竟各種場合的應用不一樣、解決方案各不同，拙作也就不傷大家眼了。

作者: wangnengjie 時間: 2022-9-14 12:56
學C語言的經典坑了。

作者: heicad 時間: 2022-9-15 20:53
對初學者很有用,當年好像遇到過這情況

作者: wujian122656 時間: 2022-9-27 16:29
弄了個開發板，準備開始學單片機啦，加油

作者: imxuheng 時間: 2022-10-9 09:30
別提了，我也有你提到的編程壞習慣，語句寫成一行，又不加{}，開始時死活找不出bug，以后真的要注意了

作者: DoneDone 時間: 2022-10-11 07:20
后面你一定會養成if的內容加花括號的習慣的

作者: zzzzz8840 時間: 2022-10-16 17:31
{}太重要了，被坑過

作者: nanjingcxy2008 時間: 2022-10-28 10:10
受教了，給大家也提了個醒，以后會注意這個問題，養成良好的編程習慣。

作者: ggbob 時間: 2022-11-7 15:30
確實細節很重要

作者: Longan.Wang 時間: 2022-11-11 14:29
這個根本就不是"；"號的號題！！！！
在定義多條語句宏的時候，至少需要用{}把語句框起來！
#define EHeat_off {P7OUT |=  BIT0; EHisON=0;}  //電加熱off
#define EHeat_on {P7OUT &=~ BIT0; EHisON=1;}  //電加熱on
如果想更加保險一點應用下面格式寫更好
//電加熱off
#define EHeat_off do {P7OUT |=  BIT0; EHisON=0;} while(0);
//電加熱on
#define EHeat_on do {P7OUT &=~ BIT0; EHisON=1;} while(0);
為什么這樣寫，網上有大佬分析過。

作者: Angle145 時間: 2022-11-11 23:58
使用宏替換多行代碼時，在調用養成加大括號后再調用的習慣

作者: Angle145 時間: 2022-11-11 23:59

Longan.Wang 發表于 2022-11-11 14:29
這個根本就不是"；"號的號題！！！！
在定義多條語句宏的時候，至少需要用{}把語句框起來！
#define EHea ...

對的，用do來概括多行宏，根本不是分號問題，給你一個贊

作者: f556 時間: 2022-11-12 01:19

Longan.Wang 發表于 2022-11-11 14:29
這個根本就不是"；"號的號題！！！！
在定義多條語句宏的時候，至少需要用{}把語句框起來！
#define EHea ...

感謝回復，我的解決辦法就是這樣:
#define EHeat_off {P7OUT |=  BIT0; EHisON=0;}  //電加熱off
#define EHeat_on {P7OUT &=~ BIT0; EHisON=1;}  //電加熱on
用do while應該有它的好處，大佬畢竟是大佬。
基于現在屏幕大部分時候是橫向的，特別是對于我用筆記本來說。更喜歡把一件事寫在一行，好象看著更簡潔，好比：
unsigned char aa,bb,cc;
至于"定義多條語句宏的時候，至少需要用{}把語句框起來"，這有點絕對了。
實際上：
#define EHeat_off P7OUT |= BIT0 ,EHisON=0    //電加熱off
#define EHeat_on P7OUT &=~ BIT0, EHisON=1  //電加熱on
上邊語句完全可以實現功能，且最后是沒有分號的，以方便按習慣在調用時后邊再加分號，如：
if(...)  EHeat_off;
上邊實際上是“,”的用法，好比在寫if時為節約行數，if后同時要做的事一并寫了，不用{}。
如：if(a>b) c=a,a=b,b=c; //交換順序
畢竟是業余玩，不是按代碼行數拿薪水，我個人更喜歡上邊這種寫法，當然，壞處是出現了我本主題的毛病，另外也不排除在上邊if中隨手把“,”寫成分號導致隱藏的錯誤。

作者: wufa1986 時間: 2022-11-12 08:31
一般不是數值的話，我都是搞成函數的形式，而不是#define

作者: SURUBU1976 時間: 2022-11-15 15:56

Longan.Wang 發表于 2022-11-11 14:29
這個根本就不是"；"號的號題！！！！
在定義多條語句宏的時候，至少需要用{}把語句框起來！
#define EHea ...

為什么這樣寫，網上有大佬分析過請給連接想學下。謝謝

作者: 程卓 時間: 2022-11-27 20:37
B站上有c 語言學習視頻

作者: 明記冷氣 時間: 2023-1-11 22:17
語句短我一般加,號不用{}

作者: f556 時間: 2023-2-15 22:44

明記冷氣發表于 2023-1-11 22:17
語句短我一般加,號不用{}

謝謝，還是找到一個知音了。

作者: 快樂眼淚 時間: 2023-4-11 20:20
以后學C語要細心了！

作者: Afterwards_1 時間: 2023-4-24 21:00
答主講的非常仔細

作者: Ubuntu312 時間: 2024-8-3 13:42

好多次了，IFELSE后跟一條語句，不加{}；后來客戶提要求，加了之后出BUG，頭疼了半個多鐘頭

作者: zhangdong533 時間: 2025-4-4 01:33
哈哈，很有意思啊

作者: zyftank 時間: 2025-5-22 15:11
那個分號可以用逗號替代，就不用加花括號了

歡迎光臨 (http://m.zg4o1577.cn/bbs/)