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/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って書くこともある

2018/03/14(水) 18:58:33.90

>>636
ビットフィールド使えば？

2018/03/14(水) 19:09:24.20

シフト使うな、ビットフィールドで書け
なんて言われると、俺は相手にもよるが逆らいそう

2018/03/14(水) 19:37:46.29

>>637
ビットフィールドは割り付け順序が処理系定義だったりしたからあまり使ったことがない

2018/03/14(水) 21:26:40.49

たまには2進リテラルちゃんのことも思い出してあげてください

2018/03/15(木) 06:36:14.64

議論の本筋(1日の秒数として即値で86400と書くことの是非)とはさらに離れるけど、
ビットシフトを使う定数は常にカッコで囲んでくれ。

res = a + 1<<4 + b;
res = a +(1<<4)+ b;

**デフォルトの名無しさん** · 2018/03/15(木) 07:15:43.22

>>641
3つ以上の項があると静的解析ツールで怒られるから
res = (a +(1<<4))+ b;
と書く

2018/03/15(木) 08:58:15.76

>>641
下のつもりで上書いたらただのバグじゃん

2018/03/15(木) 09:19:10.68

算術と混用するときは結合順位に注意せよ、というだけのことを
いちいち必ずだ！ルールだ！と金切り声で吠えついてくるやつはウザい

2018/03/15(木) 09:55:31.41

>>642
可読性悪くしてかえって品質低下招きそうな指摘だな

2018/03/15(木) 13:29:47.33

vectorをつなげる際、copyが良いとか、insertを使えとか色々書いてあるのですが、どれが良いのでしょうか？

2018/03/15(木) 13:54:37.80

あらかじめ領域を広げておく必要があるけどinsertの方がチェックが入らない分速い

2018/03/15(木) 15:23:48.90

reserveしてinsertだな

2018/03/15(木) 15:36:42.69

insertの直前にreserveって意味あるのか？

2018/03/15(木) 16:32:16.09

reserveは終端のポインタが更新されないからresizeじゃないとだめ

2018/03/15(木) 16:57:46.96

reserveしてback_inserterでcopy

2018/03/15(木) 17:05:27.14

>>648
insertはサイズチェックしてるでしょ。
>>649
iteratorを2つ受け取るタイプのinsertなら意味あるんじゃね？
>>650
resizeは不味いだろ。

2018/03/15(木) 17:09:09.32

>>652
アンカしくった。
サイズチェックの指摘は>>647向け

2018/03/15(木) 17:21:30.23

ごめんinsertとcopy逆だった

2018/03/15(木) 17:28:15.72

>>652
insertの中で追加数数えてreserveと同じことしてるでしょ
まさかループで一個ずつ挿入なんて実装はあるまい

2018/03/15(木) 19:10:49.64

>>655
失礼。挿入位置のiteratorもあるから、3つ受け取るタイプの誤り。

2018/03/15(木) 22:07:51.82

cpprefjpの任意の式によるSFINAEの頁でdecltype(a + b, bool())と記載されているコードがあるんだけれど、
このa+b, bool()はラムダ式？

該当頁
https://cpprefjp.github.io/lang/cpp11/range_based_for.html

2018/03/15(木) 22:09:23.09

アドレス間違えた
https://cpprefjp.github.io/lang/cpp11/sfinae_expressions.html

2018/03/15(木) 22:15:55.91

ラムダ式が[で始まらないことってあるのか？

2018/03/15(木) 22:41:44.53

>>657
カンマ演算子とdecltypeでコンパイル時に評価される
decltypeの中身を左から順に評価し、結果は一番右の型が帰る
この場合はa+bの型が評価出来ればbool、できなければSFINAE

2018/03/15(木) 22:41:47.80

>>657
>a + b
a + bが可能かどうか調べる

>,
次の式を評価する

>bool()
bool型の初期値を返す

つまりdecltype(a + b, bool())はa + bでSFINAEをしつつカンマ演算子でdecltypeにbool型の値を与えて返り値の型をbool型にしている

2018/03/15(木) 22:57:18.20

decltypeで,も使えたんですね。一番右の型が帰るということはdecltype(a+b,a-b,a*b,a/b,bool())とかしてもよさそう。。。

自分ではその機能を説明してるページを見つけれませんでした。
ありがとうございます。

2018/03/15(木) 23:50:51.42

>>660-661
それで良いのか。いいこと知った。
いちいち enable_if をしとったわ。

2018/03/16(金) 07:54:12.89

decltypeって実行時の型を得れる？

2018/03/16(金) 09:23:42.20

>>656
a.reserve(a.size()+b.size());
a.insert(a.end(),b.begin(),b.end());
こんな感じのことをやるのかと思ったんだけど、これ意味ないでしょ？

2018/03/16(金) 10:04:08.28

>>664
declとはdeclareの略であることを考えると

2018/03/16(金) 10:17:31.56

a.reserve(a.size()+b.size());
copy(b.begin(),b.end(),back_inserter(a));

2018/03/16(金) 15:36:10.69

>>665
random iteratorの場合の特殊化実装がありました。ごめんなさい

2018/03/16(金) 18:08:23.55

>>662
decltype comma trickでググれ

2018/03/16(金) 20:56:30.80

まあoperator,()が定義されてるとまずいんだけどな

2018/03/19(月) 18:07:03.75

インスタンスにdeleteかけたとき、メンバ変数が破壊されるのとデストラクタが実行されるのはどっちが先？
今書いてるコードがデストラクタでメンバ変数アクセスすると落ちることがあるから、破壊されるのが先なんだろうか？

2018/03/19(月) 18:15:15.06

>>671
原則としては構築の逆順って覚えておくと間違えない。