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/21(土) 06:06:21.46

>>191
VS2017では警告されるぞ（おそらくVS2015でも）

cl 191.cpp /c /W4
Microsoft(R) C/C++ Optimizing Compiler Version 19.13.26129 for x86
Copyright (C) Microsoft Corporation. All rights reserved.

191.cpp
191.cpp(7): warning C4213: 非標準の拡張機能が使用されています: 左辺値でキャストしました。

後はわかるな？

2018/04/21(土) 06:22:18.02

ポインタや参照のconst外しは合法だが
なぜ合法なのか（合法にせざるを得なかったのか）というとmemory aliasing状況下での最適化の限界に関する
ラクダ本ばりに長い議論を要する
悪寒、
プログラミング言語C++か何かでは古いライブラリのconst付け忘れにコード修正無しで対応できるＹＯ！みたいな簡単な説明しかないが

しかし左辺値のキャストというのはどうなの？

2018/04/21(土) 08:02:29.99

static_cast<double>(this->val)自体は合法でしょ
this->valと同じ値のdouble一時オブジェクトができるだけ

おかしいのはそれに代入してること

2018/04/21(土) 11:34:19.85

int x;
(float&)x = 1;

2018/04/21(土) 11:34:55.76

途中で書き込まれてしまった

int x;
(float&)x = 1; //well-formed
(float )x = 1; //ill-formed

2018/04/22(日) 17:24:24.80

std:stringを引数や戻り値で使う場合、値渡しでも大丈夫ですか？

2018/04/22(日) 17:29:11.38

大丈夫

2018/04/22(日) 18:29:08.58

stringは内部に参照カウンタを保持した共有リファレンサを持っているから&&使用せずに直で渡しても特に副作用もなく何の問題もないよ

2018/04/22(日) 18:53:47.99

今どきのstringはヒープじゃなくスタックを使ったりもするね
鬼のように最適化されている

2018/04/22(日) 19:26:19.72

>>203
おじいちゃんそれ大昔の話ですよ

2018/04/22(日) 19:52:27.54

>>205
貴様は過去を否定するのか？
温故知新という言葉を知らないのか？
青二才

2018/04/22(日) 20:00:35.45

ありがとうございます
勝手にデストラクタでdeleteされるとか色々考えていましたがそこは先人たちが考慮して作られているのですね

2018/04/22(日) 20:19:16.41

いやいや老害に騙されちゃだめですよ。
参照カウントなstringは大昔に禁止されているので、ちゃんと参照や右辺値渡等を使いましょう

2018/04/22(日) 20:23:51.32

>>208
何言ってんだおまえ
頭悪すぎるにもほどがあるだろ

2018/04/22(日) 20:58:25.32

COWなstringは効率が悪くてマルチスレッドとの相性が最悪なのでとっくの昔に打ち捨てられました
規格のCOW実装許容するための記述はC++11でバッサリ捨てられました
今はCOWは規格違反です

2018/04/22(日) 21:39:50.50

なん…だと！？

**デフォルトの名無しさん** · 2018/04/22(日) 21:51:37.63

おじいちゃんｗ

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
コード読めないの？