tm構造体の使い方がイマイチわからない

Question

前提・実現したいこと

日付を扱うプログラムを作っていまして、最初は普通の配列やら構造体を使っていたのですが、tm構造体というのがせっかくあるのだからそれを使ってしまおうと思ったのが運の尽きで、思った通りの動作ができず困っております。「そもそもそのコードのようなことをtm構造体でやるもんじゃない」というのでしたら仕方がないですが、どうにかモノにしたいです・

発生している問題

数値を入力しても、結果が出ません。

ソースコード

#include <stdio.h>
#include <string.h>
#include <ctype.h>
#include <stdlib.h>
#include <time.h>
#include <math.h>

void mjd2data(int mjd, struct tm *data){
	int n, a, b;
	//printf("n=%d",n);
	a=4*n +3 +4*floor( 3/4.0* floor( 4*(n+1)/146067.0 ) );
	b=5*floor(a%1461/4.0 )+2;
	data->tm_year=floor(a/1461.0);
	data->tm_mon =floor(b/153.0)+3;
	data->tm_mday=floor(b%153/5.0)+1;
	if(12 < data->tm_mon ){
		data->tm_year+=1;
		data->tm_mon-=12;
	}
	
}

void main(){
	int mjd;
	struct tm *data;
	char *a;
	scanf("%d",&mjd);
	mjd2data(mjd,data);
	puts("hoge");
	printf("%4d/", data->tm_year);
    printf("%2d/", data->tm_mon);
    printf("%2d ", data->tm_mday);
}

試したこと

*つけたり＆つけたりしてみましたけど、表示されなかったり、表示されてもおかしな値だったりしました。

補足情報（FW/ツールのバージョンなど）

Win10　MinGW

追記

nmoa氏、pepperleaf氏の回答の通り修正しましたらできました。
しかしtoshi17922062氏の指摘の通り、ポインタが、あやふやなところが多いです。
実体やらなんやらがちょっとしっくりこないので修正の理由？がわからないですね。

Comments

[toshi17922062]

>「そもそもそのコードのようなことをtm構造体でやるもんじゃない」
かどうかはともかく、構造体というか、ポインタの理解をもう少し深めたほうが良いと思われます。
>最初は普通の配列やら構造体を使っていたのですが
tm構造体もオリジナルの構造体の定義も、基本は同じだと思います。

ついでながら、処理系によってはintって2byteだったりするので、20100101のように入力したいのならlongかな？と思ってみたり。まあいまどきはintと言えば4byteなんでしょうけど。

[Kazumori102]

修正ユリウス日(MJD)での入力となりますので5桁ほどまでですからlongでなくとも大丈夫だと思います。

[toshi17922062]

なるほど。2byteのintは、-32768 ～ 32767なので、その中で納まるなら問題ないですね。
まあ、いまどきそんな処理系もないかもしれませんし。

[nmoa]

> Kazumori102さん
「実体やらなんやらがしっくりこない」と言われても、こちらはどうしっくりこないのかが分からないので回答しようがないです。
自分で何らかの理解を試みてから、「自分ではこう理解したが、これで合っているのか」という形で新しく質問してはいかがでしょうか。
この質問の主題である「数値を入力しても、結果が出ません。」に対しては回答が出ていると思うので、この質問はクローズすべきです。

[Kazumori102]

例として2012/1/1が55927になるので一応アウトです

[nmoa]

> toshi17922062さん
処理系による型のサイズについて気にするなら、longではなく <stdint.h>の uint32_t を用いれば良いと思うのですがいかがでしょう。

[Kazumori102]

>nmoaさん
了解です。クローズしますね。

[nmoa]

「2012/1/1が55927になる」というのはちょっと気になります。何を入力し、どのような出力を得たのか正確に記述したらなにか分かるかもしれません。
(少なくともscanf("%d",&mjd); に 2012/1/1 と入力したら期待した結果は出そうにないです)

[Kazumori102]

修正ユリウス日の定義上そうなります。
https://ja.wikipedia.org/wiki/%E3%83%A6%E3%83%AA%E3%82%A6%E3%82%B9%E9%80%9A%E6%97%A5
scanfで読み取るのは別途グレゴリオ暦から変換された値ですので直に2012/1/1と打ち込むわけではありません。

[toshi17922062]

一応ポインタについての図解をちょっと検索してみました。参考になれば。

C 言語ポインタ基礎中の基礎の図解 - Diary Blog of Dary
https://temtan.hatenadiary.jp/entry/20101108/1289223558
第３回　アドレスとポインタ変数
http://www.isc.meiji.ac.jp/~re00079/EX2.2011/20110518.html

>nmoaさん
確かに正論です。

[nmoa]

？
よく理解できていないのですが、「2012/1/1が55927になるのでアウト」なんですよね？アウトなのになぜ「修正ユリウス日の定義上そうなる」のですか？定義上そうなる(=期待した動作をしている)のであればセーフですよね。
「scanfで読み取るのは別途グレゴリオ暦から変換された値」と書いていますが、結局値としては何を入力しているのでしょう。それを正確に入力してほしい、と先程のコメントでは書きました。以下のように書いたほうが正確に伝わりやすいと思いませんか？
(例) 入力：〇〇 (yyyy/mm/ddに相当する値) 期待する出力 : □□ 得られた出力 : △△

[toshi17922062]

>2012/1/1が55927になるのでアウト
は、2byteのintではアウトになるということでしょう。
>結局値としては何を入力しているのでしょう
「修正ユリウス日」つまり、上記で言う「55927」を入力するプログラムですね。
つまりこのプログラムは「修正ユリウス日」から「yyyy/mm/dd」を導く、いわば逆変換のプログラムなんだと思います。

[Kazumori102]

この関数は別の何かによってグレゴリオ暦y/m/dから変換された修正ユリウス日nの数字を入力し、
その値を再びグレゴリオ暦に戻す関数です。
例として2012/1/1を変換した55927を関数に入力すると、逆変換され2012/1/1に戻る。
しかし、入力を受け付けるint型の変数が2byteのintだと、32767までしか受け付けられないので55927は範囲外となりアウト。そのためこのような環境の場合、longやらuint32_t やらをつかう。

これでいいですかね？

[Kazumori102]

時差になりましたがtoshiさんの通りです。省かずに2byteだとアウトと言っておけばよかったですね。

[nmoa]

なるほど、よくわかりました。自分のコメントの直後に「2012/1/1が55927だからアウト」と書いてあったので、2byteの範囲に収まらないのでアウト、をプログラムがまだ正しく動いていないのでアウト、と勘違いしていました。すみません。（めちゃくちゃ恥ずかしい勘違いですねこれ）

[Kazumori102]

いえいえ。コメントの連続性にかまけて文章を略した私が悪いんですよ。

Answer 1

main()関数内の

struct tm *data;

struct tm data; (* 不要。実体が確保されない)

で、mjd2data()の呼び出しは、
mjd2data(mjd, &data); (& 必要)

じゃないでしょうか。

あ、main()内の data-> は、data. にする必要ありです。

ざっと気づいたところで。

Answer 2

mjd2dataの方の実装は確認していませんが、struct tm構造体のポインタだけ宣言していて実体がないのが一因かと思います。

void main(){
    int mjd;
    struct tm data; // ポインタではなく実体を作る
    char *a;
    scanf("%d",&mjd);
    mjd2data(mjd,&data); // ポインタを渡す
    puts("hoge");
    printf("%4d/", data.tm_year); // アローではなくドットに
    // 以下略
}

ちなみにtm構造体を使うべきか否かですが、tm構造体は日付の情報を格納するためのint型の集まりにしか過ぎません。そのため、今回の場合は「日付の情報をまとめて扱える」以上のメリットはありません。
(参考 : http://www.bohyoh.com/CandCPP/C/Library/struct%20tm.html)

ただ、time.hに付属する時間を扱う関数はtm構造体に対応するものがあるため、これらを有効活用すればいいのではないでしょうか。
(参考:time.hの関数 http://www.bohyoh.com/CandCPP/C/Library/time_h.html)

Comments

[Kazumori102]

tm構造体の活用としてmain側でstrftimeを使ってみようとしましたが。動作しませんでした。

[y_waiwai]

✕動作しませんでした
○動作させることができませんでした

[nmoa]

（新たに質問を立てたほうがいいとは思いますが）何をやってみて、どのような結果を得ようと期待したけど、実際はこんな結果になって期待通りではなかった、と言うのが不明確です。
なんとなく、tm構造体に入れる数値（特に年、月の値）が間違いやすいのでそこが原因かな、とは思うのですが。どんな値を入れてどう使えばよいか、よく調べてみてください。

[Kazumori102]

y_waiwaiさん＞
細かい言葉使いのご指摘ありがとうございます。しかし経験の浅い自分にはどちらも同意味に思えてしまいます。ご指導願います。
nmoaさん＞
おっしゃる通りですね。