プログラムの関数化

質問

皆様に助けていただき、なんとかコーディング出来たのですが、関数化したいと思います。アドバイスお願いします。

int overflow_check1();
int overflow_check2();
int overflow_check3();
int overflow_check4();
int lop; // 左のオペランドに入力する値
int rop; // 右のオペランドに入力する値
int result; // 計算結果
int mod; // 除算の余り
int max = INT_MAX; // 表現可能な最大値
int min = INT_MIN; // 表現可能な最小値
char *check; // 変換不能な文字を格納
char op; // 入力する演算子
int main ( int argc, char *argv[] )
{
/* 引数の個数チェック */
if ( argc != 4 ) {
printf ( "usage : %s lop op rop\n", argv[0] );
exit ( 0 );
}
/* 引数を取得して整数型に変換 */
lop = atoi ( argv[1] );
rop = atoi ( argv[3] );
/* 引数の取得 */
op = ( argv[2][0] );
/* startが整数であるか文字であるかのチェック */
lop = strtol ( argv[1], &check, 10 );
if ( errno != ERANGE ) {
if ( *check != '\0' ) {
printf ( "error : 整数を入力して下さい。\n" );
exit ( -1 );
}
/* startに範囲を超えた値を入力した場合 */
} else {
printf( "error : 整数型の範囲を超えました。\n" );
exit( -1 );
}
/* endが整数であるか文字であるかのチェック */
rop = strtol ( argv[3], &check, 10 );
if ( errno != ERANGE ) {
if ( *check != '\0' ) {
printf ( "error : 整数を入力して下さい。\n" );
exit ( -1 );
}
/* endに範囲を超えた値を入力した場合 */
} else {
printf( "error : 整数型の範囲を超えました。\n" );
exit( -1 );
}
/* argv[2]が2文字以上の場合 */
if ( strlen ( argv[2] ) >= 2 ) {
printf ( "error : 演算子を正しく入力して下さい。\n" );
exit ( -1 );
}
/* argv[2]に入力された演算子が'+'の場合 */
switch ( op ) {
case '+':
overflow_check1();
result = lop + rop;
break;
/* argv[2]に入力された演算子が'-'の場合 */
case '-' :
overflow_check2();
result = lop - rop;
break;
/* argv[2]に入力された演算子が'*'の場合 */
case '*':
overflow_check3();
result = lop * rop;
break;
/* argv[2]に入力された演算子が'/'の場合 */
case '/' :
/* 0による除算 */
if ( rop == 0 ) {
printf ( "error : 0による除算は禁止です。\n" );
exit ( -1 );
}
overflow_check4();
result = lop % rop; // 剰余を求める
break;
default:
printf ( "error : '+' '-' '*' '/'のいずれかの演算子を入力して下さい。\n" );
exit ( -1 );
}
/* 割り算の結果として余りが出なかった場合 */
if ( result == 0 ) {
result = lop / rop;
printf ( "%d %c %d = %d", lop, op, rop, result );
exit ( 0 );
/* 割り算の結果として余りが出た場合 */
} else {
result = lop / rop;
mod = lop % rop;
printf ( "%d %c %d = %d余り%d", lop, op, rop, result, mod );
exit ( 0 );
}
/* 数値1 演算子 数値2 = 演算結果の形式で出力する */
printf ( "%d %c %d = %d", lop, op, rop, result );
exit ( 0 );
}
int overflow_check1 ()
{
/* オーバーフローのチェック */
if ( lop > 0 && rop > 0 ) {
if ( lop > max - rop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( 0 );
}
}
/* オーバーフローのチェック */
if ( lop < 0 && rop < 0 ) {
if ( lop < min - rop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( 0 );
}
}
return 0;
}
int overflow_check2 ()
{
/* オーバーフローのチェック */
if ( lop > 0 && rop < 0 ) {
if ( lop > max + rop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( 0 );
}
}
/* オーバーフローのチェック */
if ( lop < 0 && rop > 0 ) {
if ( lop < min + rop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( 0 );
}
}
return 0;
}
int overflow_check3 ()
{
/* オーバーフローのチェック */
if ( lop > max / rop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( -1 );
}
/* オーバーフローのチェック */
if ( rop < max / lop ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( 0 );
}
return 0;
}
int overflow_check4 ()
{
/* オーバーフローのチェック */
if ( ( lop == min ) && ( rop == -1 ) ) {
printf ( "error : オーバーフローが起こります。\n" );
exit ( -1 );
}
return 0;
}

回答No.2

ibe_tattsu · ベストアンサー

お返事ありがとうございました。

先ほどは、プログラムをよく検証せずに方法のみで返事をさせていただいたのですが、
よくよく見ると、このプログラムは正常に動くのでしょうか？

実際に引数を入力して動かしてみると、
#a.out 3 - 1
3 - 1 = 3 0
となりました。正しい結果じゃないですね＾＾；

上手く動作しないということ以外でなければ、プログラムを提示するときは、動作を一応確認しておいてください。
今回の質問者さんの目的は関数化ですし。

それとこの質問から見始めた方もいると思うので、プログラムの仕様なども記載しておくべきと思います。

＞ちなみに少しだけ関数化させてみたのですが、あまり意味のない関数化だと思い、全く自信がありません。
経験がなければ、自信が無いのは普通です。経験と共にきれいなプログラムが書けるようになってきますし、
最初はメイン関数のみになるというのも普通のことです。
今回、関数化されているところは私は適切だと思いますし、
まずはできるだけ自信を持って、自分なりにやってみてください。

お礼

一応オーバーフローも考慮して、ちゃんと動くようになりましたよ。
開発環境はvc2005です。ファイル名 オペランド 演算子 オペランド
と入力して実行すればちゃんと動きますよ！

<<今回、関数化されている部分は適切
ありごとうございます。少しずつ関数になれていきたいと思います。

補足

失礼しました。確認したところ確かにバグがありました。

case '/' :

/* 0による除算 */
if ( rop == 0 ) {
printf ( "error : 0による除算は禁止です。\n" );
exit ( -1 );
}
overflow_check4();
result = lop % rop; // 剰余を求める

/* 割り算の結果として余りが出なかった場合 */
if ( result == 0 ) {
result = lop / rop;
printf ( "%d %c %d = %d", lop, op, rop, result );
exit ( 0 );

/* 割り算の結果として余りが出た場合 */
} else {
result = lop / rop;
mod = lop % rop;
printf ( "%d %c %d = %d余り%d", lop, op, rop, result, mod );
exit ( 0 );
}
break;

default:
printf ( "error : '+' '-' '*' '/'のいずれかの演算子を入力して下さい。\n" );
exit ( 0 );
}

/* 数値1 演算子 数値2 = 演算結果の形式で出力する */
printf ( "%d %c %d = %d", lop, op, rop, result );
exit ( 0 );
}

ちなみに掛け算のオーバーフローのコードが間違っている気が
するのですが、これで正しいのでしょうか？