C++相談室 part135

2018/03/31(土) 20:20:06.25

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

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

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

このスレもよろしくね。
【初心者歓迎】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/04/23(月) 13:44:26.09

>>229
日本語でしゃべってくれ

2018/04/23(月) 13:58:46.97

実行してるということはコンパイルは通ってるということだと思う
つまりコンパイルの問題ではなく、コンパイルエラーではないだろうという推測

2018/04/23(月) 13:58:59.41

>>230
「実行すると」

ビルド時のコンパイラなどからのエラーではなく、
プログラムを実行したときの挙動についての質問のようですよ

2018/04/23(月) 14:45:37.00

初心者つってるから
どこまでがコンパイルで
どこからが実行なのかの
区別からあやしいと見ている

実行時エラーにしても
stopダイアログなのか
結果がおかしいのか
あの質問では言ってない

2018/04/23(月) 14:53:14.79

すげーみっともない奴だな

2018/04/23(月) 14:56:19.11

嘘は言っていない

2018/04/23(月) 15:52:04.86

言論の自由を行使し過ぎw

2018/04/23(月) 16:27:46.06

コンパイルして(リンクを)実行すると

だったりして

2018/04/23(月) 17:14:12.91

初心者がいちいちobjからリンクしてるのか

2018/04/23(月) 17:15:44.11

しつけえな
ここはプログラム技術板
技術的な内容が皆無のレスでひっ絡んでくるなカス

2018/04/23(月) 17:30:01.10

>>224
何を実行するのか目的語をはっきりさせろ
プロ意識が欠けている

2018/04/23(月) 17:32:39.31

あ、初心者だったのか…

2018/04/23(月) 17:34:51.03

>>224
ソースを公開することは可能ですか？
https://ideone.com

2018/04/23(月) 21:12:48.39

pthread を使ってループ処理をマルチスレッド化したのですが、シングルスレッドと同じスピードしか出ませんでした。
pthread でこういう事やっても無駄なんでしょうか？

2018/04/23(月) 22:04:36.67

流石に情報が少なすぎて
このままだと下手糞としか言いようがない。
ソースコード貼り付けれ

2018/04/23(月) 22:09:13.69

シングルコアのPCかも

2018/04/23(月) 22:17:20.63

pthread使うとひとつの処理が倍速にでもなると思ったのだろうか

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sdba-gnn3) · 2018/04/23(月) 22:21:51.35

不完全な質問はスルーすっと

2018/04/23(月) 22:42:58.69

舌足らずですみません。コードはこんな感じです。
threadFunctionは単なる加算値、joinFunctionは集計処理です。
コアは物理２論理４です。
template< class ArgType >
void Reduce( std::vector< ArgType >& threadArgs, void* (*threadFunction)(void*), void (*joinFunction)(std::vector< ArgType >&) )
{
const size_t threadCount = threadArgs.size();
threads.resize( threadCount );
std::vector< void* > voidPtrArgs = CastArgsToVoidPtrs( threadArgs );
for ( int threadIndex = 0; threadIndex < threadCount; ++threadIndex )
{
sched_param schedParam;
schedParam.sched_priority = sched_get_priority_max( SCHED_FIFO );

pthread_attr_t threadAttribute;
pthread_attr_init( & threadAttribute );
pthread_attr_setschedpolicy( & threadAttribute, schedPolicy );
pthread_attr_setinheritsched( & threadAttribute, PTHREAD_EXPLICIT_SCHED );

pthread_t& thread = threads[ threadIndex ];
pthread_setschedparam( thread, schedPolicy, & schedParam );
pthread_create( & thread, & threadAttribute, threadFunction, voidPtrArgs[ threadIndex ] );
}

for ( int threadIndex = 0; threadIndex < threadCount; ++threadIndex )
{
pthread_t thread = threads[ threadIndex ];
pthread_join( thread, NULL );
}
joinFunction( threadArgs );
}

2018/04/23(月) 22:45:44.01

>>243
この世界では、何かやって思い描いてたようにならなかった場合
まず自分の能力不足を疑うのが鉄則

2018/04/23(月) 22:51:38.20

>>248
アハハハハ！ジョークのつもりかなんか？
そうじゃないならjoinの動きを勉強しろ

2018/04/23(月) 22:53:42.39

釣り針デカいな

2018/04/23(月) 23:14:39.42

いまだに関数ポインタ使ってるのか。野蛮人。
std::functionってスレッドセーフじゃないの？

2018/04/23(月) 23:18:02.49

いまだ関数ポインタが使いこなせないんだけどやばいかな？

2018/04/23(月) 23:20:18.71

考えたらjoinの問題じゃないか
もし「単なる加算処理」が１スレッドでメモリ帯域使い潰していたらマルチスレッドにしてもどうしようもないのは明らかだよ

2018/04/23(月) 23:21:31.95

>>253
未だに関数ポインタなんて使ってるほうがやばい

2018/04/23(月) 23:30:05.62

画像処理で合成処理をパラメータでもらう場合があるんだが
その場合内部処理と対応させるために関数ポインタは使うが
そういうのもダメ？

2018/04/23(月) 23:35:26.41

ダメってわけじゃないけどさあｗ
C++ならもっと柔軟性のあるやりかたが幾らでもあるってこと

2018/04/23(月) 23:36:21.76

virtual関数もラムダ式も関数ポインタ

2018/04/23(月) 23:40:33.42

std::byteが邪魔すぎるんですけどg++で無効にするオプションってありますかね？

2018/04/23(月) 23:50:59.76

>>257
あんた昨日のおじいちゃんだろ。
functionの時代もとっくに終わってるぞ。
autoとlambdaで関数ポインタを使うべき。

2018/04/23(月) 23:51:42.53

>>258
ライブラリ制作者でもなければ関数ポインタなんぞ触らんよ。
よほどCとの兼ね合いが無いと。

2018/04/24(火) 00:06:39.75

>>248
threadcountいくつになってるの？

2018/04/24(火) 00:06:56.48

忘れがちなことだが std::function は実行時の型を扱う。
画像処理などのようにヘビーな繰返しがあるような場面では関数ポインタを使った場合との間に深刻な速度差が生じることもなくはない。

2018/04/24(火) 00:16:24.07

>>248
関数名からして、一度のreduce処理量は大したことなくて、何度も繰り返し呼んでない？
thread処理に必要な処理量が相対的に無視できなくなってるんじゃね？
スレッドは4本に制限して、各スレッドが処理する量を増やすかスレッドプール式にしては？

**デフォルトの名無しさん** · 2018/04/24(火) 00:17:16.38

C++だから関数ポインタ使わないとか頭おかしい

関数ポインタのほうが高速かつシンプルに書けるならそちらを選択すべき

2018/04/24(火) 00:20:31.80

皆富豪ばかりではない

2018/04/24(火) 00:26:04.88

富豪かどうかはおま環だろ
だから自己申告しないヤツが悪い

なんでこっちがエスパーみたいなことしなきゃいけないんだ
わたくしは教えないがあなたがわたしの環境を忖度しろってか？
ヴァカじゃねえの？
アフォに対してちゃあんと「テメーのスペックはいかほどですか」と尋ねろクズ

富豪かどうかはわからない、それを言わない人間がまず間違い、
それを逆手にとって相手をマウンティングするアフォがいるから話が進まない

2018/04/24(火) 00:38:30.68

関数ポインタの構文の方がかっこいいだろ！

2018/04/24(火) 00:47:04.42

どんなスペックだろうと他のソフトがどれだけメモリや処理時間を喰い潰していようと自分の処理はサクッと終わらせたいといつも思う

2018/04/24(火) 01:40:17.33

>>260
へんてこりんなマウンティングするやつだなあ
ちなみに昨日なんて俺書き込んでないからｗ

2018/04/24(火) 03:16:15.83

>>248
調べてみたけどさっぱり判りません。
pthread_joinで各スレッドの終了を待って、その後、集計処理をするというのはごく渡り前の処理に見えるのですが、
何が行けないのでしょうか？　
別の方法でスレッドの終了を待たねばならないのでしょうか？
自分勝手デスミア線が、具体的に問題点、改善点を指摘して下さいm(_ _)m。

2018/04/24(火) 03:31:10.06

いいから「単なる加算処理」全部見せろよこの包茎野郎

2018/04/24(火) 03:56:33.85

threadFunctionが、
void* CalcBasicStatics( void* threadArg ) {
　　　　BasicStaticsThreadArg* arg = reinterpret_cast< BasicStaticsThreadArg* >( threadArg );
double intervalOfX = arg->intervalOfX;
double x = arg->dividedRangeOfX.start;
double sumOfY = 0.0;
double sampleCount = 0;
const sc::Sampler& f = arg->f;
while ( x <= arg->dividedRangeOfX.end ){
double y = f( x );
sumOfY += y;
sampleCount++;
x += intervalOfX;
}
arg->sumOfY = sumOfY;
arg->sampleCount = sampleCount;
return nullptr;
}
joinFunctionが、
void CalcBasicStaticsJoin( std::vector< BasicStaticsThreadArg >& args ) {
double sampleCount = 0.0;
double sumOfY = 0.0;
for ( int i = 0; i < args.size(); ++I ) {
sumOfY += args[ i ].sumOfY;
sampleCount += args[ i ].sampleCount;
}
for ( int i = 0; i < args.size(); ++i ) { // 結果を書き込み
BasicStaticsThreadArg& arg = args[ i ];
arg.average = sumOfY / sampleCount;
}
}
です。細々すみません。

2018/04/24(火) 05:46:22.80

そのコード見ても
並列度もスレッドあたりのサンプル数も
1サンプルあたりのコスト(f)もわからないので
まるで意味がない

2または4並列で、1スレッドあたり1～10Mサンプルくらい処理するようにすれば
速くなるか少なくともスレッドを使わない場合より遅くならないと思う

同じ速度ということはメモリ帯域が律速なのかもね

2018/04/24(火) 05:46:49.67

同じ速度
↓
スレッドを使わない場合と同じ速度

2018/04/24(火) 05:55:18.81

fでメモリのどっかから数値を読んでいるんだと思うけど、
これがなるべく連続したアクセス(局所化を謀る)になるようにループを構成できれば速くなるかもしれない。

この辺りはググれば色々参考になるページがあると思うが
いまググッたらそれらしいページがあったので書いておく
http://myoga.web.fc2.com/prog/cpp/opti02.htm

仕様として外からfが与えられるなら無理な話かもしれない。

もちろん interval が 1 で f(x) が { return v[x]; } のような最適なケースよりは速くならないので
その辺りは無駄な努力をしないよう測っておきましょう。

2018/04/24(火) 06:54:08.05

fの中身が公開出来るなら公開して
複数あるならそのうちの1個でいいから

あと、
1回のthreadFunctionで何個くらいfを計算する？

2018/04/24(火) 09:24:50.43

>>250
joinについて勉強しろとか偉そうに言ってたのは何だったの

2018/04/24(火) 18:12:35.67

>>278
こりゃ「多分」joinの問題じゃないなと判断して>>254を書いたんだが
これくらいのコンテキスト読めないとマルチスレッドは無理だよ

2018/04/24(火) 18:15:41.14

fやら具体的なargsの内容やら処理時間測定のやりかたも記述されてないし
んなもの誰も答えられるかよ

2018/04/24(火) 18:32:28.83

>>279
違う違う
なんで明らかに間違えてる事を偉そうに述べられるのかと聞いてんだけど
読解力もスキルもないのか

2018/04/24(火) 18:41:27.90

>>281
本当に「明らかに」だと思ってるの？バカですか？

2018/04/24(火) 18:45:24.94

サンプルレベルのJoinの使い方をみて
「アハハハハ！ジョークのつもりかなんか？そうじゃないならjoinの動きを勉強しろ」
は流石に笑ってしまう

2018/04/24(火) 18:47:53.38

勝手に笑ってればｗ

2018/04/24(火) 18:49:15.43

cin で、個数の決まっていない整数たちを読み込みたいのですが、どうすればいいでしょうか？

整数たちの個数 n が分かっていれば、以下のように読み込めばいいですが。。。

vector<int> v;
int i;
for (int i = 0; i < n; ++i) {
cin >> i
v.push_back(i)
}

2018/04/24(火) 18:59:54.20

>>282
明らかにだろどう見ても。
何をどう勘違いしたのか説明してくれよ。

2018/04/24(火) 19:02:18.64

>>285
他に入力がないなら cin.eof() で入力の終了を検出できるよ。

cin のブール演算を使ってもいいけど、どうにも慣れなくてね(個人の見解)。

2018/04/24(火) 19:15:13.81

>>287

ありがとうございました。

別の質問なのですが、一般的に、vectorの使用頻度というのはどれくらいでしょうか？
配列でやれることもすべて vector を使ってやるという人は多いでしょうか？
それとも、効率などを考えて配列で極力済ませるという人が多いでしょうか？
もちろん、ケースバイケースでしょうけれども、そのあたりの常識がないので、大体
どんな感じなのかが知りたいです。

2018/04/24(火) 19:17:35.43

自分としては、効率など細かいことは考えずに、vectorを使って問題ない
場面ではvectorを使うという風にしたいのですが。。。

vectorを使っても速度などの点で問題ない場合、一般的なプログラマーなら
どうするのかが知りたいです。

2018/04/24(火) 19:20:20.45

vector<int> v;
int n;
cin >> n;
int t;
for (int i = 0; i < n; ++i) {
cin >> t;
v.push_back(t);
}

int *p;
int n;
cin >> n;
p = new int[n];
for (int i = 0; i < n; ++i) {
cin >> p[i];
}

どちらにするのが普通なのかの常識がありません。

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sdba-gnn3) · 2018/04/24(火) 19:30:42.86

>>290
生のnew/deleteは、なるべく使わないのがいい。delete忘れ、例外などでバグの元や維持コストになる。

2018/04/24(火) 19:33:20.39

>>291
ありがとうございました。
ということはできるだけvectorを利用したほうがいいということでしょうか？

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sdba-gnn3) · 2018/04/24(火) 19:36:23.45

>>292
動的にサイズが変わらないなら配列かstd::array。
動的にサイズが変わるならstd::vector。

2018/04/24(火) 19:38:54.99

>>290
上だったらv.reserve(n)しておこう

◆QZaw55cn4c (ﾜｯﾁｮｲ ba60-Mp6C) · 2018/04/24(火) 19:42:04.62

>>289
私は書き始めは std::vector を専ら使っており、後で他のコンテナに換えています

2018/04/24(火) 19:46:25.46

>>293
コード読めないの？

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sdba-gnn3) · 2018/04/24(火) 19:51:37.43

>>296
普通の人間なら、文脈を読み取れるけど、俺はコンピューターに近いんだ。

2018/04/24(火) 20:04:19.79

>>289
実用上の問題が無いことがわかっている範囲内であれば、
深く考えずに vector だけで乗り切るのも悪い選択じゃないと思うよ。

ただ、使い分けることで意図を表現しやすい。

たとえば list を使っていれば要素の挿入や削除が頻繁なデータなんだなって思うし、
array が使われていれば要素の個数が固定なんだなって思う。
速度的に影響がない程度の規模であっても、
それが適しているような操作をこれからするのだという意思表明は人間がプログラムを読むときのヒントになる。

そして、そういうヒントは書いている途中にこそ必要なものなので、 >>295 のように後から整理していくスタイルは個人的には好きじゃないな。

◆QZaw55cn4c (ﾜｯﾁｮｲ ba60-Mp6C) · 2018/04/24(火) 20:22:55.46

>>298
＞それが適しているような操作をこれからするのだ

うーむ、いろいろと考えさせられます
std::vector でなら使えても、std::list では使えない、というのはあるから、最初からそれを考慮しておくのは…よくありますねえ

2018/04/24(火) 20:43:20.11

vector は list に比べた場合、

データがメモリ上隣接して並んでいるので
→そういう引数を要する各種 API にそのまま渡せる
→メモリアクセスが局所的にできてキャッシュが効く

予約領域を拡張する場合にのみアロケータが呼ばれるので追加時のアロケータによるオーバーヘッドが低い

とかの良い特性もあるので要素のコピーが軽くて個数が小さいものはvectorにして損することは少ない

2018/04/24(火) 21:34:54.87

BidirectionalIteratorとRandomAccessIteratorだろ
規格用語で言えば短く済む

2018/04/24(火) 22:35:44.34

std::listはメモリ局所性がないので今どきのマシンだとクソ遅い
積極的に使う理由は基本的にない

2018/04/24(火) 23:33:49.11

積極的にって...
使いどころで使うための物だよ

2018/04/24(火) 23:51:07.95

メリットとデメリットを見極められないとコンテナを使いこなすのは難しい
昔から配列を弄り倒している古参にとってはこんなに便利な物はないと思うがね
90年初頭辺りにタイムスリップして実際に構造を真似てフルスクラッチでテンプレートなんぞなかった世界で組んでみればコンテナの挙動は自ずと理解できると思うが時代がわるかったな
今は何も苦労しなくても容易になんでも手に入る世界だからな
修業が足らんよ青二才

2018/04/24(火) 23:55:55.63

>>303
その使い所がよほど特殊な状況以外にないんだよ

2018/04/24(火) 23:58:59.30

大規模C++ソフトウェアデザインという本を読んでいます。

冗長インクルードガードが紹介されているのですが、効果あるんですかね。

古めの本なのですが、最近のコンパイラだと意味ないですかね

**>>306** (ﾜｯﾁｮｲ e3b3-9jjH) · 2018/04/25(水) 00:13:26.67

http://ir9.jp/prog/ayu/datlog/tech_cpp/1149427282/1149427282_04.html

ここの967以降に同じ質問がありました。質問撤回します。

2018/04/25(水) 00:14:20.28

#pragma once　って規格化されたんだっけ？

2018/04/25(水) 01:03:31.17

されてない
しようしようと20年言われ続けて技術的な問題でできずにいる

2018/04/25(水) 02:07:21.83

243です。
アドバイスを頂き検討したのですが、メモリが散らかっているのが原因と判断しました。
都合により細々とした実装の話は割愛しますが、付き合ってくれた皆さんありがとうございました。

2018/04/25(水) 02:33:10.54

どおりで未だに警告でるわけだ

2018/04/25(水) 06:26:59.61

>>305
その特殊な状況の為にlistが存在する

私の場合は特殊なプログラムを書くことが多いので
使いどころは多いのかもしれない

また、普通の組み込みC言語でも簡易な片方向リストとかを使ったりする (C++じゃないのでlistは無い)

2018/04/25(水) 07:15:49.01

>>309
技術的な問題ってなに？
既に実装してる環境の方が多くね？

2018/04/25(水) 07:30:39.44

今はモジュールの方を標準化しようとしてるんじゃ

2018/04/25(水) 08:47:25.49

偉い人が考えてることはわからん

2018/04/25(水) 11:24:32.26

>>313
プリプロセッサは C++ じゃないからね
C++ 以外の言語と共有しているツールなので
それらと歩調を合わせる必要があるし
プリプロセッサだけ独立の規格にするなら
C++ を含め、諸言語の規格も
プリプロセッサのバージョンとどう付き合うのか
策定せにゃならん

2018/04/25(水) 12:23:57.05

>>316
意味わからん
C/C++の規格にプリプロセッサの仕様もあることを知らんのか？
そもそも他の言語と共有してる環境なんて見たことないし

2018/04/25(水) 15:37:25.53

現行では、 #include ディレクティブは「対象ファイルの内容がそこに書かれているかのようにふるまう」という規則なので、
ヘッダファイル内のプラグマ (#pragma once) の解釈が始まるのはインクルードされた後になる。
もちろん対象範囲がコンパイル単位全体に及んでしまっては #pragma once の意味がないが、
現状の仕様に辻褄を合わせるとそうなる。

実装した処理系がもうあるんだから実装に沿うように規定しなおすってことはできなくは無いんだろうが、
#pragma once を仕様に入れるのに #pragma once の項目を追加すれば済むわけではないってことは理解してくれ。

それと、プリプロセッサの仕様は C/C++ の一部なのは確かだが、
挙動を規定しきれていないのじゃないかということは指摘されている。
https://qiita.com/ruiu/items/4d471216b71ab48d8b74#3%E6%9C%8817%E6%97%A5
うやむやでやってきてるところを整理する必要は有ると思う。

>>317
Haskell (GHC) は C プリプロセッサを使うよ。
汎用的に使いたいなら M4 とかの方がいいとは思うけど、
Cプリプロセッサに慣れている人は多いから……。

2018/04/25(水) 16:38:46.53

また別な話
#includeしようとしているファイルが
過去に#includeしたファイルと同一かどうか
という判定も意外に厄介だね
ハードリンクできるファイルシステムと
そうでないファイルシステムがあったりするし
ハッシュが一致しても衝突かどうかの問題もある

2018/04/25(水) 18:09:25.23

ハードリンク、シンボリックリンクが無いファイルシステムだったとしても、サーチパスの問題も思いつくな。

たとえばカレントディレクトリと、カレントディレクトリ直下の foo ディレクトリからヘッダファイルを探すようになっているとき、

#include "bar.h"

と

#include "foo/bar.h"

は同じファイルを指しているが、表現が異なる。
同一のファイルとして除去すべきだろうか？

2018/04/25(水) 19:34:37.54

#if ****
#pragma once
#endif

とかどうなるの？
複数回のインクルードで条件がちがっていたら？

2018/04/25(水) 20:33:35.69

>>318
> 実装した処理系がもうあるんだから実装に沿うように規定しなおすってことはできなくは無いんだろうが、
規定すればいいだけだろ？
どこに技術的な問題があるんだ？
政治的な問題だと言うならまだしも
>>319-320みたいな話は処理系定義ですむ話
>>321は少し悩ましいがそもそも途中まで読んでから#pragma onceとか言われても面倒だから書くならファイルの最初に書けとかの制限をつければいい

> 挙動を規定しきれていないのじゃないかということは指摘されている。
いやいや、その子ちゃんと規格読めてないだけでしょ w

> Haskell (GHC) は C プリプロセッサを使うよ。
仕様を流用してるだけでしょ？
何かのコンパイラと共有してるわけじゃないと思うが

2018/04/25(水) 20:43:06.43

君よりよっぽど賢い人達が20年間悩み続けて未だに出来てないことを舐めない方がいい

2018/04/25(水) 21:22:23.29

>>323
別に悩み続けてなんていないでしょ w
#pragma onceで事足りてるからわざわざ政治的なことに足突っ込みたくないだけ

2018/04/25(水) 21:47:31.85

新規格の季節になると毎回のように標準化委員会の議題に上がってるんだが？

2018/04/25(水) 22:20:34.77

モジュール方式になったらヘッダファイルが要らなくなる？

2018/04/25(水) 22:28:42.99

>>322
> 処理系定義ですむ話

それはおかしいでしょ
ハードリンクできる処理系からできない処理系に移植したtarボールの
#includeの挙動が未規定なら結局インクルードガードを自前で書くことになる

2018/04/25(水) 22:48:43.95

>>325
でも誰も進めようとしないんだろ w
よくある話

>>327
C/C++ ってそういう言語だしそもそも環境変える予定があるならそんなものに依存したファイル構成にしないでしょ

2018/04/25(水) 22:55:12.61

#pragma once なんて
once upon a time だぜ
今はもうだれも気に留めてやしねぇ

2018/04/25(水) 23:31:56.94

アホの相手は疲れたわ
もう寝る