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/22(日) 21:54:34.58

値渡しや戻り値で戻せることと、
std::string内部での（ヒープ）メモリ管理の詳細は関係なくね？？

引数や戻り値としてスタックに構造体を積めるようになった時点で値渡しや戻り値で戻せることとはほぼ自明
（よほど変なコピコンを定義してコピーをわざと不首尾に終わらせない限りそれはできる

2018/04/22(日) 23:24:42.33

>>213
まあそうだけれど無駄なコピーはなるべく避けましょうということで

2018/04/22(日) 23:50:24.01

今時のコンパイラは、そこらの人間よりよっぽど高度な最適化をするから、素直で単純なコードを書くと速くなる。
効率化するつもりで余計なことをすると、コンパイラの最適化を妨げて、かえって効率が落ちたりする。

本当に重要な部分は実測しながら試行錯誤、それ以外はできるだけ素直なコードを書く、ってあたりが基本。

2018/04/23(月) 01:28:01.46

stringを値渡しせずにconst参照等々で渡すことはc++の基本なので、素直で単純なコードの部類に入ると思うが。

2018/04/23(月) 01:32:17.81

って、あたりが基本(｀･ω･´)ｷﾘｯ

道化かw

2018/04/23(月) 01:37:49.36

先生！OpenMPへの展開も自動でやってくれるんですか？

2018/04/23(月) 01:46:26.60

値渡しを素直なコードとか言う人って他人の書いたコード読まないんだろうな
宿題以外で何か書いたことあるのか

2018/04/23(月) 02:04:07.25

const type &

2018/04/23(月) 04:33:47.47

const参照渡しが基本だったのって C++11 以前の世界の話だよね…
幸い、近頃はそんなコードをあまり読まずに済んでるけど。

余計なことをしない、ってのが大事なんだよ。
参照でなきゃいけない理由がなければ、参照を使うべきじゃないんだ。

2018/04/23(月) 06:33:47.30

どこの世界に生きてる人？

2018/04/23(月) 07:20:47.54

どんな作品を作っているのかなぁ？

2018/04/23(月) 08:32:33.12

初心者です。
今書いてるプログラムをコンパイルして実行すると、エラーが出たり出なかったり謎の挙動を起こします

最初はエラーが出て、その後数回、コードを一切変えずコンパイルしなおして実行すると急にエラーが出ず上手く行ったりします。

これって何が原因なのでしょうか…
こういう事ってよくあるのですか？

2018/04/23(月) 08:34:14.27

キチガイ発言はツイッターかブログでどうぞ

2018/04/23(月) 08:35:11.51

>>224
よくあることです
デバッグしましょう

2018/04/23(月) 10:17:10.24

>>224
そのエラーとやら、もしかして警告か？
エラーは直すまで何度コンパイルしても絶対に通らない
警告はコンパイルが完了しオブジェクトファイルができる

そのコンパイルとやら、もしかしてビルドか？　それともmakeか？
ビルドやmakeはオブジェクトファイルとソースファイルの日付を比較して
ソースファイルのほうが古いとコンパイルを省く
だから警告が出てからもう一度ビルドしても同じ警告は出ない

おそらく、こういうことだと見受ける

2018/04/23(月) 10:24:23.61

変数を初期化してないんだろ

2018/04/23(月) 12:27:19.92

>>227
日本語読めないのかよ...

> 実行すると、エラーが出たり出なかったり謎の挙動を起こします

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は無い)