C++相談室 part134

2018/01/20(土) 09:05:42.21

次スレを立てる時は本文の1行目に以下を追加して下さい。
!extend:on:vvvvv:1000:512

C++に関する質問やら話題やらはこちらへどうぞ。
ただし質問の前にはFAQに一通り目を通してください。
IDE (VC++など)などの使い方の質問はその開発環境のスレにお願いします。

前スレ
C++相談室 part133
http://mevius.5ch.net/test/read.cgi/tech/1511509970/

このスレもよろしくね。
【初心者歓迎】C/C++室 Ver.102【環境依存OK】
http://mevius.5ch.net/test/read.cgi/tech/1509780815/

■長いソースを貼るときはここへ。■
　http://codepad.org/
　https://ideone.com/

[C++ FAQ]
https://isocpp.org/wiki/faq/
http://www.bohyoh.com/CandCPP/FAQ/ (日本語)

VIPQ2_EXTDAT: default:vvvvv:1000:512:----: EXT was configured

2018/03/11(日) 11:58:27.43

テンプレートで再帰ができたらﾅｱ…('A`)

2018/03/11(日) 12:00:06.04

とわいいつつ、実はC++のテンプレートはチューリング完全になってしまっていた（らしい
ttps://cpplover.blogspot.jp/2013/10/blog-post_20.html

2018/03/11(日) 12:05:49.26

>>536
costexprで階乗書いてみてから言えよ

2018/03/11(日) 16:36:05.55

constexpr unsigned long long factorial(int n){
　unsigned long long r=1;
　for(int i=1;i<=n;++i){r*=i;}
　return r;
}

うん、簡単だね！

2018/03/11(日) 16:41:07.00

毎回毎回コンパイラが計算するのか

2018/03/11(日) 17:05:53.05

たまにソースコードを見て、あ、この数字なんだっけって考えるよりも
10000回コンパイルさせるほうが早いと思うよ

2018/03/11(日) 17:18:40.81

マジックナンバーの本質

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd8a-f9jl) · 2018/03/11(日) 17:51:32.86

>>540
オーバーフローのチェックがないからやり直し。

2018/03/11(日) 18:36:06.03

はーい

constexpr unsigned long long unsafe_factorial(int n){
　unsigned long long r=1;
　for(int i=1;i<=n;++i){r*=i;}
　return r;
}

constexpr int min_factorialable(){
　for(int i=1;;++i){
　　if(unsafe_factorial(i) > unsafe_factorial(i+1)){ return i; }
　}
}

constexpr unsigned long long factorial(int n){
　if (n < 0 || n > min_factorialable()) { return 0; }
　return unsafe_factorial(n);
}

2018/03/11(日) 18:40:21.07

コンパイルエラーになるほうがいいなぁ

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd8a-f9jl) · 2018/03/11(日) 19:45:19.23

static_assert使え。

2018/03/11(日) 20:16:22.23

>>529
実行時計算で答え算出してそれを貼った方が楽じゃね？

2018/03/11(日) 21:15:39.44

>>548
それで事足りるケースならそれでいいよ

2018/03/11(日) 21:19:50.46

>>548
なんでそんなめんどくさくてミスしそうな方法をチョイスするんだよ

**デフォルトの名無しさん** · 2018/03/12(月) 01:21:50.18

>>548
計算＆書き換え手順をマニュアル化して、何十種類の仕向け別にバージョンアップの度に人海戦術で計算＆書き換え＆コードレビュー＆テストするんですね、わかります

2018/03/12(月) 05:00:24.40

min_factorialable() は、階乗を正しく計算できる最大の値を返す
constexprな関数(コンパイル時にint定数に置き換えられることを期待)、
と読み取ったんだけど、なぜにmin_なの?

**デフォルトの名無しさん** · 2018/03/12(月) 05:41:37.75

上限を求めるときに使うのはminだから、かな

知らんけど

2018/03/12(月) 06:20:08.42

最初min_nonfactorialableでnon取った時に直し忘れました

2018/03/13(火) 09:59:46.06

一日の秒数を86400と書いてたらマジックナンバーやめろと言われたんだけど、
プログラムを少しでも書く人間にとってその数字は常識でいいよね？
60*60*24 と書いて「その60とか24は何だ？」と言われないでしょ？
それと同じレベルだと思ってるんだけど。

2018/03/13(火) 10:10:13.70

うるう秒でクラッシュするクソソフトウェアの出来あがり
ニュートン力学と天文学と暦学からやりなおした方がいい

2018/03/13(火) 10:13:40.69

>>555
言われないとわからない奴は言ってもわからないだろうから
お前は一生マジックナンバー使いまくりでいいぞ

2018/03/13(火) 10:38:37.74

そろそろ標準に日付計算のライブラリ欲しいわ

2018/03/13(火) 11:02:13.56

>>558
欲しいね
chronoが中途半端でイラつく

2018/03/13(火) 11:32:32.41

>>556
それだと 60*60*24 と書いても同じだよね。
うるう秒とかは関係ない定数管理してる場合の話ですよもちろん。

2018/03/13(火) 11:37:27.38

>>557
86400がプログラマの常識的にマジックナンバーなのかどうか、って話ですよ

2018/03/13(火) 11:45:44.57

所で皆さんmakefileはどう編集してます
あまり触れていないようなので気になります

2018/03/13(火) 11:45:51.13

2147483647なら俺の常識だが

2018/03/13(火) 11:47:54.17

>>562
趣味では汎用のmakefile作ってそれを使い回してる
仕事ではautotools

2018/03/13(火) 12:00:55.79

他人と共有するコードならなるべく要素を分解して分かりやすく書いた方が好ましい
2の倍数も直接記述するのではなくシフト演算使った方が綺麗

2018/03/13(火) 12:18:10.82

くくるんじゃなくて…

クラス宣言

int main()
{....}

あとは c++ スレで

2018/03/13(火) 12:18:34.80

誤爆しました

2018/03/13(火) 12:51:25.09

常識なんて集団が変われば変わる
そのコードを見せる事を想定した集団の常識かどうかを考えろ

2018/03/13(火) 14:10:31.07

2進数8進数10進数16進数のみではないということ
60進数や24進数の話でしょ時計を語るときは必須の常識

2018/03/13(火) 14:26:54.36

これをn進数って言うんだよ

2018/03/13(火) 14:37:04.66

集団の常識ってことなら、'86400' をマジックナンバーと思ってる人間は少数派っぽいので、
文句はやんわりと無視する方向で行きますわ

2018/03/13(火) 14:53:30.50

というかマジックナンバー言われるのって数字の意味とか集団の常識とかじゃなく書式の問題だよね
計算式の中に定数ポン置きしてる書き方してる奴がチームにいたらそりゃマズいよ、変数とか定数に置き換えろって俺も言う

2018/03/13(火) 14:59:18.81

定数を定数に置き換えるってなんだよ
リテラルは定数なり変数なりにって感じで読み替えといて

2018/03/13(火) 15:34:48.40

>>572
しかしこうはしないだろ？

const int int8_bits = 8;
uint16_t word = (hi << int8_bits) | lo;

こうもしない
const int sec_per_min = 60;
int sec = m * sec_per_min + s;

数値リテラルで書くのが許容される数値というものがある
sec = day * 86400 はどうかというのが>>561
個人的には知らない人でも暗算もせずに初見で合ってるかどうかわかるか、
を基本にすべきだと思う (小さな2の冪乗は除く。2の冪乗知らん奴はシネ)

2018/03/13(火) 15:51:51.20

uint16_t word = (hi << CHAR_BIT) | lo;
これCHAR_BIT != 8な環境を意識するんなら、16って書いてあるのも問題だね

const int micro_per_sec = 1000000;
int us = sec * micro_per_sec;
こう書けって言われたら俺はお断りする

2018/03/13(火) 16:10:54.91

e = 2.71828;
p = 3.1415;

2018/03/13(火) 16:11:31.80

>>575
bit_per_char や shortでもintでもなく int8、uint16_t と明記したのは所与の仕様のつもり。

最近のapiはよくナノ秒返してくるから1000000000 倍
1000.0*1000*1000と分かち書きしたくなる

2018/03/13(火) 16:29:39.63

時刻を自動的に合わせますというコマンドボタンを作って
押すだけで済むのに

2018/03/13(火) 16:33:12.95

>>577
それを言うなら1'000'000'000だろjk

2018/03/13(火) 18:34:24.26

86400って書くと、ideで定義の参照も使えんし
86400で検索したときに
１日の秒数なのか、適当に決めたタイムアウトとかの秒数なのか、ただのIDなのか、連番で出てきただけの数値なのかとか、無意味な数列なのかまざってわからんくなるやろ

2018/03/13(火) 18:56:33.51

#define SEC_PER_DAY (60*60*24)

なんてやる人いないのかな

◆QZaw55cn4c (ﾜｯﾁｮｲ 8a60-hn8E) · 2018/03/13(火) 19:22:14.64

>>581
#define は基本使わない、int const ABC = 12345; なら時々使う

2018/03/13(火) 19:52:43.56

constexpr……

2018/03/13(火) 20:19:53.55

さっさとこう書けるようにしてくれ
今の<chrono>クソすぎんよ

int sec_par_day = static_cast<int>(std::chrono::second{std::chrono::day{{1}});

2018/03/13(火) 20:25:30.79

クソというか機能が圧倒的に足りてない

2018/03/13(火) 20:28:38.82

86400とか1440でわかるけど
24*60*60とか24*60って書けばいいだけだと思う

2018/03/13(火) 20:30:42.63

21世紀も5分の1が過ぎようかっていうこの時代に
何が悲しゅうてたかが時刻を扱うだけでtime_pointの他にtmやらtime_tやらをガチャガチャさせられにゃならんのだよまったく

2018/03/13(火) 22:01:42.63

>>586
お前が正解
他はバカ

**デフォルトの名無しさん** · 2018/03/13(火) 22:08:40.50

86400って何のために使うんだろう

うるう日とかうるう秒とか考慮しなきゃ気持ち悪くね？

2018/03/13(火) 22:17:29.51

予め考慮しようがない

2018/03/13(火) 22:22:44.39

考慮する必要性があるものかどうかが分からないからなんとも
例えば時系列データをその時点から1日分の数を使うというものなら閏秒を考える必要がないし

**デフォルトの名無しさん** · 2018/03/13(火) 22:28:43.21

>>591
1日に1秒満たないかもしれないだろ

2018/03/13(火) 22:35:46.84

int t = 86400; // 24*60*60
これで解決

2018/03/13(火) 23:06:58.39

閏秒の有無等で可変のものなら可変のものなりの書き方するでしょ
計算して変数に入れるとか

問題は不変の数値定数をどう正規化して書くか、正規化する必要はあるか。
24*60*60 派の人と 60*60*24 派の人が両方プロジェクト内にいたらなんか嫌だな

2018/03/13(火) 23:12:45.19

プロジェクトで誰かが決めたらそれに従うよ
派閥作るほどの問題じゃないでしょ

2018/03/13(火) 23:17:41.96

絡むわけじゃないけど
決めたら従うのは当たり前で、それが「決める」ってことだろ。
派閥の話じゃなく混ざってたら気持ち悪いってことが言いたかったんだよ。

2018/03/13(火) 23:20:55.63

あっちこっちに書き散らすものじゃないから混ざってるとかあり得ない
1カ所になるまでリファクタリングしろ

2018/03/13(火) 23:44:10.62

気持ち悪いと思うかどうかは感性の問題だからなあ
例えばある日地球の回転速度が変わって１分が６１秒になった
プログラムを修正しろと言われて、
場所によって書き方が違っていると大変だね
と言う話なら理解出来るけど

2018/03/13(火) 23:51:40.27

constexpr int min = 60;
constexpr int hour = min * 60;
constexpr int day = hour * 24;

2018/03/14(水) 06:39:53.70

hoursは<chrono>にあるだろ
それとoperator""h

2018/03/14(水) 06:41:16.16

operator""dayがねえんだよな<chrono>って

2018/03/14(水) 07:36:00.34

chronoのhoursはstd::chrono::duration<int, std::ratio<3600>>のエイリアスなので
daysはこうすればよい
using days = std::chrono::duration<int, std::ratio<86400>>

2018/03/14(水) 08:15:03.73

>>602
> daysはこうすればよい
> using days = std::chrono::duration<int, std::ratio<86400>>
>>555へ戻る w

2018/03/14(水) 08:42:21.15

閏秒を考慮するのと1日を86400秒とするのは全く関係ないのに閏秒がどうとか言い出した奴が悪いのでは
丁寧に書くなら計算の根拠として60*60*24を見えるところに書いておけばいい

2018/03/14(水) 09:17:22.64

知ってる知識を披露したかっただけだろうから罪は軽い

**デフォルトの名無しさん** · 2018/03/14(水) 10:28:48.31

>>604
全く関係ないは言い過ぎ

2018/03/14(水) 10:52:42.70

ほぼ関係ないな、難癖もいいとこ
実用上は全く関係ない

2018/03/14(水) 11:03:01.46

うるう秒がどうのこうのと言う、問題じゃなく86400秒をそこでしか使わないって根拠が無いんだよね
後でまたその数値が出てくるとしてその度に86400と書くのか、もしその数値を修正する事になったら全部書き直すのか、2回3回使うようになって始めて変数なり定数なりに入れるのか
それなら最初から式中に出てくる数値は変数に格納してから使えって統一した方が話が早い

2018/03/14(水) 11:14:21.74

今時マジックナンバーとか昭和からタイムスリップしてきたのかよ

2018/03/14(水) 11:43:33.36

プログラムにおけるマジックナンバー（英: magic number、魔法の数字）とは、何らかの識別子もしくは定数として用いられる、プログラムのソースコード中に書かれた具体的な数値である。
そのプログラムを書いた時点では製作者は数値の意図を把握しているが、他のプログラマーまたは製作者本人がマジックナンバーの意図を忘れたときに閲覧すると「この数字の意味はわからないが、とにかくプログラムは正しく動く。
まるで魔法の数字だ」という皮肉を含む。
by Wikipedia

2018/03/14(水) 11:50:45.96

「1日の秒数」って定数を86400って書いて怒られたって話じゃないんだ？

2018/03/14(水) 12:06:59.05

>>609
じゃあオマエ、構造化もしてないのか？
昭和時代ほど耳にしなくなっても廃れたのではない話は結構あるぞ

2018/03/14(水) 13:01:38.98

>>611
怒られたという程じゃなくて、
　一日の秒数 = 86400;
と書いたら「マジックナンバーやめろ」と言われたってことさ

2018/03/14(水) 13:18:59.19

わかりきった定数はマジックナンバーではない

2018/03/14(水) 14:08:58.84

>>613
それをヘッダに移動させるだけやん

2018/03/14(水) 14:14:27.78

>>615
そうじゃなくて
60*60*24 て書いて欲しいんだってさー
謎

2018/03/14(水) 14:16:09.81

>>616
まぁ可読性高くて損する事は無いし、なるべく要素は分解して書いた方がいいんじゃないの
何かどうしても86400じゃなきゃ困る理由とかあるなら話は別だろうけど、無いだろ？

2018/03/14(水) 14:34:38.65

>>616
クソどうでもいいな

2018/03/14(水) 15:05:38.89

ITのエンジニアって
こういうくだらないことで
給料貰ってるんだろうなあ

2018/03/14(水) 15:18:02.33

こういうクソみたいなことでもウンチク垂れてテコ入れしろと提案することにより仕事してます感かもし出す為だろうな

2018/03/14(水) 15:45:56.81

>>619
結果のわからない投機的な模索をしているときは
逆にマジックナンバー地獄なコードだよ
他人に渡すコードを清書するときの書き方・考え方とは真逆

2018/03/14(水) 16:27:34.05

俺がわかるからみんなわかるだろうというコードはすげー困るな
例えその場の全員がわかったとしても今後わからない奴が出てきた時の事を考えて
できうるかぎりわかりやすいコードを書くべきだな

2018/03/14(水) 17:02:40.84

そのためのコストにもよる
空想論じみた「わかりやすさ」にまで付き合ってらんないこともある
特に最低限のプロとしてのスキルを欠く者を想定することはしない

ちょっと飛躍するがプログラム言語だってそうなっている
マシンの性能を使いやすさに振るということを無限にはやらない
どのくらいのアホまで付き合ってやるのか性能とのトレードオフで
色んなバランスで棲み分けになっている

2018/03/14(水) 17:03:50.56

まぁ別に自分だけが使う分にはいくら分かりづらく書いてくれても構わんけどな

2018/03/14(水) 17:11:14.36

86400を60*60*24と書くことにはなんのコストも支払ってないから書けばいいな
いちいち話を飛躍させる奴はなんなんだ

2018/03/14(水) 17:20:37.67

人によっては>>599みたいにしろって言うだろうし
結局は人それぞれってことじゃないの

16を2*2*2*2っていちいち書く人はいないでしょ
まぁ1<<4って書けと言われるかもしれんが

2018/03/14(水) 17:21:05.80

学校の試験問題で一日の秒数としてソースに86400を書いたら
マジックナンバーだからと減点された、納得いかない。
みたいな発端じゃないのかなぁ。

2018/03/14(水) 17:22:38.07

それをfindとsedでやる勇気あるか？

たとえば俺なんかは１箇所ずつよく見て有機的に判断しないと怖いし
それの残業代は貰えるのか心配だ

2018/03/14(水) 17:51:03.64

当然その程度のリファクタリングはするべきだしそもそもマジックナンバーをどうにかするべきだし
お前の残業代とかいう飛躍した話はどうでもいい

2018/03/14(水) 18:19:55.18

飛躍じゃねえ実話だよ

ずいぶん昔、社内BBSでそれ系のことを言い出したやつが
偉い人から突っ込まれてた

2018/03/14(水) 18:35:58.25

プロとか言い出してる人がいるけど、それこそ飛躍では？

2018/03/14(水) 18:40:08.58

プログラムの話題から外れてきていると思いませんか？

2018/03/14(水) 18:48:14.13

>>614
だからわかりきってるかどうかは人によって違うって話
プログラムの前に日本語の勉強しろよ w

2018/03/14(水) 18:48:33.76

まぁコスト次第で可読性を犠牲にするケースもあるだろうけど
この場合はメモリ食う訳でも処理遅くなる訳でもソースの構造が変わる訳でもない
86400でも十分分かりやすいかも知れんが60*60*24なら更に分かりやすくなる

どっちが良いかなんて議論の余地無いよね

2018/03/14(水) 18:50:15.11

>>631
別にプロでなくてもいいんだけどさ
おまえさんなりに、同じプロジェクトにいて欲しくない足手まといが
どのくらいから下というのはあるだろ

2018/03/14(水) 18:54:56.26

>>626
> 16を2*2*2*2っていちいち書く人はいないでしょ
> まぁ1<<4って書けと言われるかもしれんが
状況次第だろ
場合分けの条件が４個あって各々２状態をとるとかなら2*2*2*2って書くかもしれない
下から４ビット目のマスクが欲しいなら1<<4って書くこともある