解決済みの質問
質問:No.4335072
暇なときに回答ください
お気に入り投稿に追加する (0人が追加しました)
回答数3
閲覧数98
スタックオーバーフロー
はじめまして!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が押されていても処理は実行しません。

以上です。宜しくお願い致します(@_@;;)
投稿日時 - 2008-09-17 09:47:47
質問者が選んだベストアンサー
回答:No.1
これだけではスタックオーバーフローするようには見えませんが…
再帰呼び出しをしているワケでも、どでかいローカル変数を確保しているようにも見えませんし。

>1.初期化として、一度Stopを実行する。Stopは初期画面の表示と、初期画面でそれぞれSWが押された時にとぶ関数の先頭アドレスを確保する。

>if(SwData & STOP_SW_ON)
>{
>Stop();
のコードの関連が微妙だとは思います。

ついでに…
if() ~ else if() ~ else if() ~ else if()
と続けていくのならば、
switch(){
case :~
}
が使えないか検討してはどうでしょう?

# この程度ならコンパイラの最適化で何とかなりそうだとも思いますが。
投稿日時 - 2008-09-17 23:56:10
この回答を支持する
(現在0人が支持しています)
お礼
ご回答ありがとうございます!
自身でもデバック環境でスタックの状態を見てみたのですが、これといって異常はありませんでした(^^;)
もう一度検討してみますね!!
初歩的な質問にお答え頂き、ありがとうございました☆
投稿日時 - 2008-09-18 08:35:26
この質問は役に立ちましたか?
12人が「このQ&Aが役に立った」と投票しています
ベストアンサー以外の回答
回答:No.3
このコーディング部分だけでは(極端に小さなスタックサイズでなければ)スタックオーバーフローになるとは思えません。

問題があるとすれば4の関数の方ですね。
投稿日時 - 2008-10-01 01:55:40
この回答を支持する
(現在0人が支持しています)
回答:No.2
単なる用語の間違いかと思いますが
「確保する」
というのがあれば、「開放する」
がなければそのうちメモリーがなくなりますね。
投稿日時 - 2008-09-18 13:20:36
この回答を支持する
(現在0人が支持しています)
もっと聞いてみる
関連するQ&Aはこちら
始めまして。サンフランシスコ近郊の街、バークレーに語学留学中の者です。7月28日にBerkeley Greek Theatreで行われる、MANU CHAOのライブのチケットをお持ちの方で1枚でも譲っ...
sw
最近、パソコンをシャットダウンすると、少ししてから(10秒程)、 swというウィンドウが出てプログラムの終了と書かれています。そのウインドウは1秒~10秒程で消え、その後すぐに、赤い×のついたウインド...
「複合機の制御SWおよびサービス連携SW開発」 SWとは? ...
この他の関連するQ&Aをキーワードで探す
開発のサブカテゴリ
[PR] おすすめの注目情報
カテゴリ:その他(開発)
RSS
-PR-
PR
-PR-