関数を作り呼び出したい。

Question

#include <stdio.h>

int hms2sec(int h,int m,int s)
{
int x;
x=h,m,s;
return x;
}
int main(){
  int n;
  n= hms2sec(2,3,4);
  printf("%c時間%c分%c秒\n",h,m,s);
}

 clang-7 -pthread -lm -o main main.c
main.c:6:5: warning: expression result
      unused [-Wunused-value]
x=h,m,s;
    ^
main.c:6:7: warning: expression result
      unused [-Wunused-value]
x=h,m,s;
      ^
main.c:12:33: error: use of undeclared
      identifier 'h'
  printf("%c時間%c分%c秒\n",h,m,s);
                            ^
main.c:12:35: error: use of undeclared
      identifier 'm'
  printf("%c時間%c分%c秒\n",h,m,s);
                              ^
main.c:12:37: error: use of undeclared
      identifier 's'
  printf("%c時間%c分%c秒\n",h,m,s);
                                ^
2 warnings and 3 errors generated.
compiler exit status 1
 

hms2secという関数を作りたいと思っています。この関数は「ｈ時間m分s秒」を意味しており、h,m,sを引数に、戻り値を秒数に変換したものにしたいですが、自分が組んだものではこのような警告とエラーが出てしまいました。どのように改善すれば上記に当てはまる結果を出すことができるのでしょうか。お返事お待ちしております。(警告の内容がいまいち理解できていないので、そちらの解説もできるならばお願いしたいです。）

Comments

[miyabi-sun]

> タグ名: 関数型プログラミング
これは「オブジェクト指向プログラミング」のようなプログラミングスタイルに関する用語です。
C言語は「手続き型プログラミング言語」と呼ばれるプログラミング言語であり、
「関数型プログラミング言語」とは対局の存在の位置づけです。

編集機能でタグを追加・削除出来るので、削除しておいてくださいね。

Answer 1

int hms2sec(int h,int m,int s) {
  int x;
  x=h,m,s;
  return x;
}

↑コレ、なにしたいんです?

h*3600 + m *60 + s を計算して返すんじゃないんですか?
そんな式、この関数のどこにもありません。

Answer 2

main.c:6:5: warning: expression result
      unused [-Wunused-value]
x=h,m,s;
    ^
main.c:6:7: warning: expression result
      unused [-Wunused-value]
x=h,m,s;
      ^

コンマ演算子でm,sがunused-value、つまり使われていません。
時分秒を秒数に変換したいのならその計算式を書いてください。

main.c:12:33: error: use of undeclared
      identifier 'h'
  printf("%c時間%c分%c秒\n",h,m,s);
                            ^
main.c:12:35: error: use of undeclared
      identifier 'm'
  printf("%c時間%c分%c秒\n",h,m,s);
                              ^
main.c:12:37: error: use of undeclared
      identifier 's'
  printf("%c時間%c分%c秒\n",h,m,s);
                                ^

main関数内でh,m,sは宣言されていません。