h,m,sについてのコンパイルエラーを直したい。

Question

hms2secという関数自分で作り、この関数を使ったプログラムをしたいと考えています。ifを使ってh,m,sが0<=h<60,0<=m<60,0<=s<60でない時、－１となるようにしたいです。（0<h<60,0<m<60,0<s<60の時は、秒数を表す。）一応組んではみたのですが、h,m,sが宣言されていないと出てしまいました。どのように改善すれば下記のようにコンパイルの結果が出るのでしょうか。お答えお待ちしております。

#include <stdio.h>
int hms2sec(int h,int m,int s){
int x;
x=h*3600+m*60+s;

return x;
}

int main(){

 if(0<=h<60,0<=m<60,0<=s<60){
  int n;
  n= hms2sec(0,0,10);
  printf("%d秒\n",n);
  int l;
  l=hms2sec(2,33,51);
  printf("%d秒\n",l);
  int o;
  o=hms2sec(24,0,0);
  printf("%d秒\n",o);
  int p;
  p=hms2sec(-1,23,45);
  printf("%d秒\n",p);
  int q;
  q=hms2sec(23,45,67);
  printf("%d秒\n");
  int r;
  r=hms2sec(1,-3,0);
  printf("%d秒\n");
 }
 else
 printf("-1秒\n");
 
 
}

コンパイルの結果↓

 clang-7 -pthread -lm -o main main.c
main.c:11:8: error: use of undeclared identifier 'h'
 if(0<=h<=60,0<=m<=60,0<=s<=60){
       ^
main.c:11:17: error: use of undeclared identifier 'm'
 if(0<=h<=60,0<=m<=60,0<=s<=60){
                ^
main.c:11:26: error: use of undeclared identifier 's'
 if(0<=h<=60,0<=m<=60,0<=s<=60){
                         ^
compiler exit status 1
main.c:26:12: warning: more '%' conversions than data
      arguments [-Wformat]
  printf("%d秒\n");
          ~^
main.c:29:12: warning: more '%' conversions than data
      arguments [-Wformat]
  printf("%d秒\n");
          ~^
2 warnings and 3 errors generated.

理想のコンパイル結果↓

10秒
9231秒
86400秒
-1秒
-1秒
-1秒

Answer 1

h,m,sの変数が宣言されているのは、hms2sec関数の中であり、main関数内には居ません。
よって「定義されていない」とエラーが出るのです。
また、hms2secの機能として範囲外のデータが入力された場合は-1を返すのであれば、if文もhms2secの中に必要です。

if文に複数条件を設定する際は、論理演算子（&&,||）が必要です。
","でつなげただけの場合は、一番最後の条件だけ評価され、それ以外の判定結果は捨てられます。
S < x < L の形の条件式は正しく評価されません。
演算子の優先順位により(S<X)<Lとなり{true|false}<Lの評価をすることになります。

これらをふまえると

int hms2sec(int h,int m,int s){
  if(( 0<=h && h<60 ) && ( 0<=m && m<60 ) && ( 0<=s && s<60 )){
    int x;
    x=h*3600+m*60+s;
    return x;
  }else{
    return -1;
  }
}

int main(){
  int n;
  n= hms2sec(0,0,10);
  printf("%d秒\n",n);   //10秒

  int r;
  r=hms2sec(1,-3,0);
  printf("%d秒\n",r);  //-1秒
}

Answer 2

• 範囲チェックはhms2secで行なう。
• 0 <= h <= 60 みたいな比較はC言語ではできないので、分けて書いて&&で接続する。以下のコードでは範囲外のチェックをしているので||で接続している
• ,演算子では末尾の条件式のみ使われることになるので、hに関する条件、mに関する条件、sに関する条件を&&や||で接続する。以下のコードでは範囲外のチェックをしているので||で接続している

#include <stdio.h>

int hms2sec(int h, int m, int s){
    if (h < 0 || h >= 60 ||
        m < 0 || m >= 60 ||
        s < 0 || s >= 60) {
        return -1;
    }
    return h * 3600 + m * 60 + s;
}

int main(void)
{
    printf("%d秒\n", hms2sec(0, 0, 10));
    printf("%d秒\n", hms2sec(2, 33, 51));
    printf("%d秒\n", hms2sec(24, 0, 0));
    printf("%d秒\n", hms2sec(-1, 23, 45));
    printf("%d秒\n", hms2sec(23, 45, 67));
    printf("%d秒\n", hms2sec(1, -3, 0));

    return 0;
}