はじめまして!SH-4を使用して操作パネル処理の学習をしている新米です(^^;)
現在、操作パネルのSW操作から、押されたSWによりパネル上に設けた液晶画面に文字を表示するというプログラムをコーディングしています。
そこで以下をコーディングしたところ、「これではSTACKオーバーフローが発生してしまう可能性がある」との指摘を受けました。。。そこで回避(対策?)する方法を考えているのですが、なかなか思い浮かびません。。。なにか良い方法はあるでしょうか?
コーディングは以下の感じです。
*****バッファの宣言*****
void (*pAutoFunc)(); // Auto SW が押された場合に実行する関数へのポインタ
void (*pManuFunc)(); // MANU SW が押された場合に実行する関数へのポインタ
void (*pStartFunc)(); // START SW が押された場合に実行する関数へのポインタ
void (*pDecFunc)(); // DEC SW が押された場合に実行する関数へのポインタ
void (*pIncFunc)(); // INC SW が押された場合に実行する関数へのポインタ
void (*pF2Func)(); // F2 SW が押された場合に実行する関数へのポインタ
void (*pF1Func)(); // F1 SW が押された場合に実行する関数へのポインタ
*****メインループ*****
void MenuMng(void)
{
while(1)
{
// スイッチリード、etc...
Convert();
if(SwData & STOP_SW_ON)
{
Stop();
}
else if(SwData == MANU_SW_ON)
{
(*pManuFunc)();
}
else if(SwData == AUTO_SW_ON)
{
(*pAutoFunc)();
}
else if(SwData == START_SW_ON)
{
(*pStartFunc)();
}
else if(SwData == INC_SW_ON)
{
(*pIncFunc)();
}
else if(SwData == DEC_SW_ON)
{
(*pDecFunc)();
}
else if(SwData == F1_SW_ON)
{
(*pF1Func)();
}
else if(SwData == F2_SW_ON)
{
(*pF2Func)();
}
}
}
処理としては以下を考えています。
1.初期化として、一度Stopを実行する。Stopは初期画面の表示と、初期画面でそれぞれSWが押された時にとぶ関数の先頭アドレスを
確保する。
2.Convert関数内で、SWの状態を読み出し、SwDataに確保する。
3.押されたSWを判定して、確保してある関数の先頭にとんで処理を行う。
4.とんだ関数内pManuFunc、pAutoFunc等に、次にそれぞれSWが押された場合にとぶ関数の先頭アドレスを確保する。
5.処理終了で戻ってきて、2.から5.を繰り返す。
STOP SW が押された場合は、どんな画面になっていても初期画面に戻ります。1度処理を実行したら、SWが全て放されるまで
SWが押されていても処理は実行しません。
以上です。宜しくお願い致します(@_@;;)
No.1ベストアンサー
- 回答日時:
これだけではスタックオーバーフローするようには見えませんが…
再帰呼び出しをしているワケでも、どでかいローカル変数を確保しているようにも見えませんし。
>1.初期化として、一度Stopを実行する。Stopは初期画面の表示と、初期画面でそれぞれSWが押された時にとぶ関数の先頭アドレスを確保する。
と
>if(SwData & STOP_SW_ON)
>{
>Stop();
のコードの関連が微妙だとは思います。
ついでに…
if() ~ else if() ~ else if() ~ else if()
と続けていくのならば、
switch(){
case :~
}
が使えないか検討してはどうでしょう?
# この程度ならコンパイラの最適化で何とかなりそうだとも思いますが。
ご回答ありがとうございます!
自身でもデバック環境でスタックの状態を見てみたのですが、これといって異常はありませんでした(^^;)
もう一度検討してみますね!!
初歩的な質問にお答え頂き、ありがとうございました☆
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# c言語の問題です 3 2023/01/10 16:15
- Windows 95・98 Windows11にバージョン アップしました。 2 2023/07/21 18:36
- スピーカー・コンポ・ステレオ より良いサブウーファーの接続方法を教えて下さい 現行のオーディオシステムは基本的にはCDP→プリ→パ 5 2022/08/24 16:19
- カスタマイズ(車) アンプのハイパス設定 3 2023/05/04 10:32
- 電気・ガス・水道 電気工事に詳しい方 5 2022/07/28 12:47
- プリンタ・スキャナー EPSONのプリンターPM-A890の電源入らず 2 2023/05/27 12:07
- C言語・C++・C# C# で、あるフォルダー内にあるすべてのテキストファイルを別のフォルダーにコピーする。 4 2022/11/21 13:23
- 電気・ガス・水道業 電気図記号について質問です。 画像(照光式押釦SW)のように接点などを破線で結ぶことがありますが、こ 4 2022/06/09 11:24
- CPU・メモリ・マザーボード メモリーを4GBから8GBに変えた方。 13 2022/11/22 16:02
- Visual Basic(VBA) ユーザーフォーム「frm_基本❶」を立ち上げると新規で入力する行数を右下のNoとして表示しています。 1 2023/03/16 19:02
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
Thunderbird 振り分けされなく...
-
vbsファイルによるネットワーク...
-
エクセルで定期的(30分おき)...
-
2010でShiftキーを使っても自動...
-
xcopyバッチを管理者として実行...
-
JP1で月末以外で毎週月曜日のジ...
-
VBAのタイマー
-
WindowsServer2003の、SMB1.0を...
-
bashで15分前と現在のエポック...
-
DIGコマンドでstatusがREFUSED...
-
MODE関数を文字でできる関数は...
-
ポリシーによる自動ログオフ
-
Teratermで、条件分岐させたい
-
ノートンディスクドクターで。。。
-
bat 処理の中止
-
スケジュールタスクが勝手にひ...
-
タスクがSYSTEMアカウントで実...
-
カスペルスキーのオンラインス...
-
ブラウザにIPアドレスを直打ち...
-
Tera Termでのコマンド流し込み...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
Thunderbird 振り分けされなく...
-
エクセルで定期的(30分おき)...
-
2010でShiftキーを使っても自動...
-
xcopyバッチを管理者として実行...
-
JP1で月末以外で毎週月曜日のジ...
-
MODE関数を文字でできる関数は...
-
SSDのTrimって毎日実行していい...
-
VBAのタイマー
-
タスクススケジューラを停止さ...
-
ADのDC移行の手順を教えてください
-
x64とIA64の違い
-
vbsファイルによるネットワーク...
-
タスクがSYSTEMアカウントで実...
-
グループポリシーでログオンス...
-
ログオンユーザーがいない場合...
-
batファイルの実行が1行目のみ...
-
WindowsServer2003の、SMB1.0を...
-
bat 処理の中止
-
ポリシーによる自動ログオフ
-
FLEXlm Log fileを集計するソフト
おすすめ情報