C++相談室 part133

2017/11/24(金) 16:52:50.43

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

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

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

このスレもよろしくね。
【初心者歓迎】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

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

2018/01/11(木) 07:23:14.65

>>863
おお...気付かんかった
C++なのにCで書いてあるからそもそもだめだったか...
fread/fwriteでまず考えてみます
FileCopyみたいなの欲しい

2018/01/11(木) 07:33:45.62

freadを考えてみたけどファイルサイズ不明な場合一度に読み込むバッファサイズとかどうなるん？

2018/01/11(木) 08:03:11.10

各マシンでの最適なバッファサイズを逐一計算して求めるのがC++

2018/01/11(木) 08:10:51.08

>>865
環境わからんからなんとも言えんが10KB程度とっときゃいいんじゃね？

2018/01/11(木) 09:08:45.65

>>861
＞関数仮引数を動的検査している
解説お願いします
＞終了ステートが負論理
ダメですか？
＞system_errorを使っていない
ダメですか？
＞C++なのにfopen
ダメですか？

2018/01/11(木) 09:58:35.85

>>860
fopen の引数は “rb” “wb” にしておくのが一般的
もし実用品として使うならなるべく大きな単位、
最低でも1Mバイト程度はまとめて読み書きすると良いよ。100Mでもいい。
細かい単位で読んで書いてを行うとHDDで大きなファイルを処理する場合とてつもなく遅くなる。

あとそういう書き方をしたいならそれは c++ ではなく
c のスレ(あるのか知らんが)で聞く内容だと思う

2018/01/11(木) 10:03:28.58

>>868
不勉強を恥じろボケ

#include <filesystem>
#include <iostream>

auto copy(char const* src, char const* dst)
{
try
{
return std::filesystem::copy_file(src, dst);
}
catch(std::exception& err)
{
std::cerr << err.what();
}
return false;
}

2018/01/11(木) 10:19:39.83

filesystem がまだ使えない環境もあるかもしれんので小ネタを披露しておくと、
rdbuf を使う方法があるんや。

ifstream in(src);
ostream out(dst);
out << in.rdbuf();

みたいな要領な。エラー処理は適宜うまいことやってくれ。

2018/01/11(木) 11:19:16.78

パフォーマンス厨だが
もし実用するなら setvbuf の rdbuf 版みたいなのがあるからそれも呼ぶべきと一言

**デフォルトの名無しさん** · 2018/01/11(木) 12:15:07.96

system("cp -f a.txt b.txt") でいいよね(´・ω・｀)

2018/01/11(木) 13:09:33.65

パフォーマンス厨なら
リードとライトは別スレッドにするとか
読み書きサイズをクラスタサイズの倍数にするとか

2018/01/11(木) 13:20:11.53

物理的に別のデバイスならリード/ライトは別スレッドで回したほうが速いだろうが、同じデバイスだとシングルスレッドでも大差ないだろうな
あとはバッファサイズをできるだけ大きく取って、IOの発行数を抑えるのが王道かな

2018/01/11(木) 14:07:04.69

>>860
fgetc、fputc？笑っちゃうくらいのダメダメ
俺の部下が万が一こんなコード書いてたらもう二度とそいつには仕事回さない

2018/01/11(木) 14:43:56.63

プログラミングなんてどっかからコード引っ張って書けば一切手を汚すことなくできるのに何偉そうに言ってんだか...
英語読めないからって海外の文献を参考にせず自分で作ろうとする日本人の無駄な努力には呆れるよ
アメリカ「アホだから」
韓国「レベル低いな」
インド「アジアの恥さらし」って思われてそう

2018/01/11(木) 14:48:19.11

必死こいてコード書いてるやつ見ると情けない
世界65億人130カ国！
お前の書いてるコードがまさか地球上にないものだと思ってんの？
プログラマーの端くれが、そりゃトップで活躍するプログラマーなら常に新しいものを求めてるから生き残るけど...
それ未満は10年後にはAI、またはコード共有化が進んで消えた職業になっているだろうね
悲しいけど現実見てくれ

2018/01/11(木) 15:17:43.47

構ってちゃんが必死で連投してるな
著作権だの互換性だのと縁のない幸せ者だな

2018/01/11(木) 15:32:14.08

>>876
バカにするなら模範コードぐらい書いてやれよ。

2018/01/11(木) 16:16:00.19

テストしてないけど
const size_t BUF_SZ=8192;
char buf[BUF_SZ];
while(!feof(fi)){
size_t n=fread(buf,sizeof char,BUF_SZ,fi);
if(n&&n!=fwrite(buf,sizeof char,n,fo)) {perror("ディスクエラー");return -1;}
}
>>873
WindowsならSHFileOperationかな？
windowsでsystem叩くのは文字数制限とか""のくくり方とか微妙に怪しくてなんか嫌なんよね

◆QZaw55cn4c (ﾜｯﾁｮｲ 4a60-cexj) · 2018/01/11(木) 16:40:41.56

お題：http://mevius.2ch.net/test/read.cgi/tech/1480579110/981
回答
http://mevius.2ch.net/test/read.cgi/tech/1480579110/997
http://mevius.2ch.net/test/read.cgi/tech/1434079972/30
https://ideone.com/LV4UdE
while (bytesDataNum > 0) {
int n;
fin.read((char *)&buffer, std::min(N, bytesDataNum));
if (fin.fail())
return 0;
n = fin.gcount();
fout.write((char *)&buffer, n);
if (fout.fail())
return 0;
bytesDataNum -= n;
}
　return 1;
}
C++fstream/read/write を使うのだったらこんなところか
>>881 の n = fread() が BUF_SZ に満たなくてもエラーが立ってしまうのが思案のしどころだね、あらかじめ読み込みバイト数を知っておかなければならない‥

2018/01/11(木) 17:11:02.74

せやね。エラーチェックは難しいのう。
const size_t BUF_SZ=8192;
char buf[BUF_SZ];
while(!feof(fi)){
size_t n=fread(buf,sizeof char,BUF_SZ,fi);
if(!n) {perror("ディスク読み込みエラー");return -1;}
if(n!=fwrite(buf,sizeof char,n,fo)) {perror("ディスク書き込みエラー");return -1;}
}

2018/01/11(木) 17:32:08.08

>>880
さてそんなこと言われてもプラットフォームすら指定されていないし
例えばWindowsなら↓一発で済む話だよな
https://msdn.microsoft.com/ja-jp/library/cc429185.aspx
ファイルコピーごときに自前でコードなんか書いてられるかよ

2018/01/11(木) 17:55:46.00

ISO/IEC14882の現行規格を差し置いて何を今さら。。。
Windowsなら純正コンパイラが対応しているだろうが

2018/01/11(木) 17:57:34.42

ニートが部下がどうのこうの言ってて笑える

2018/01/11(木) 18:03:32.29

>>886
お前の境遇はよく分かったから勝手に笑っとけ

2018/01/11(木) 19:35:14.53

>>885 >>884
C++17はともかく、boost::filesystemあたりが妥当だろうな。

2018/01/11(木) 19:49:16.17

boost依存は出来る限り避けたいなあ

**デフォルトの名無しさん** · 2018/01/11(木) 21:10:50.25

またこの荒らしか
Pythonのスレにも居たな

**デフォルトの名無しさん** · 2018/01/11(木) 21:11:35.20

>>877
>>878
宛てな

2018/01/11(木) 21:40:25.98

とりあえずだ、fcloseの戻り値判定してないのも誰かつっこんでやれよ。。。
普段気にするところじゃないけど、わりと重要だよな？書き込み系のclose処理

2018/01/11(木) 22:01:49.89

バッファサイズ8kだと少し大きなファイルコピーすると結構遅くなるよ
書き込み側はOSのキャッシュが効くからそうはならんだろうけど
メモリはいっぱいあるんだし動的確保でもして一気に128Mくらい行くべし

>>889
簡単な処理はあまり外部依存しないコードにしたいよね

2018/01/11(木) 22:07:24.01

関数のconst参照引数って
コピーが重い方だと意味あるけど
intとかsize_tとかだと全く意味ないの?

**デフォルトの名無しさん** · 2018/01/11(木) 22:18:41.87

>>894
どっちが軽いか考えてみればいいじゃん
(1) 常に8バイトの数値(アドレス)を引数として渡すためにコピーして、呼び出された関数側でそのアドレスから値をコピーして、処理に使う
(2) 1～8バイトの数値を引数として渡すためにコピーして、呼び出された関数側でその値を処理に使う

2018/01/11(木) 22:39:48.54

>>895
あーなるほど
どこかの入門サイトで参照は変数の名前の置き換えって見たから
コンパイル時に置き換わるもんだと思ってた
中身はアドレスなのね

2018/01/11(木) 23:12:05.45

>>896
内実としては参照はデリファレンスが自動化されたポインタと思ってもさしつかえないと思う。
ABI レベルで見ればポインタと互換性が有りさえするアーキテクチャも有る。 (言語仕様で保証しているわけではない。)

2018/01/12(金) 07:31:11.05

はちみつ餃子はc++とscheme以外に何が使えるの？

2018/01/12(金) 07:51:53.51

そんなこと知ってどうするんや

2018/01/12(金) 07:54:27.14

好奇心ですよ好奇心

2018/01/12(金) 07:54:44.69

マウント取りに行くんだろ

2018/01/12(金) 07:56:18.96

そんなに悪いかよ
じゃあ結構です

**デフォルトの名無しさん** · 2018/01/12(金) 08:14:19.27

>>899
恥ずかしガンなって

2018/01/12(金) 22:48:03.44

メンバがint2つくらいだとコピー渡しにするか参照渡しにするかちょっと悩む
int4つくらいまでならキャッシュの関係でコピーのほうが早いとか聞いたことあるけど本当なのかね

2018/01/12(金) 22:50:47.10

FF15ではベクトルクラスはコピーにしたら劇的に早くなったと言っていた

2018/01/13(土) 01:03:55.02

ゲーム機は尖ったアーキテクチャだったりするから……。
やってみないとわからん。

2018/01/13(土) 11:47:10.72

いやゲーム屋なら当たり前の話だよ
IntelのCPUだってSSEあるし、__m128は出来るだけ値渡しした方がいいはず
(もちろんインライン展開された場合は除く

2018/01/13(土) 12:22:15.14

>>905
ソース教えて欲しい

2018/01/13(土) 12:24:13.94

>>908
http://jp.gamesindustry.biz/article/1609/16090501/

2018/01/13(土) 12:34:35.23

>>909
ありがとうございます

2018/01/13(土) 13:06:59.87

インライン展開されたら値渡しだろうが参照渡しだろうが（あんま）関係無いんじゃ…

※ 参照と不十分なインライン展開が重なったときmemory aliasingによる最適化の阻害の危険性がちょっと増すぐらい

2018/01/13(土) 21:20:30.95

クラスの継承について質問です。

スーパークラス(Super)を継承したサブクラス(Sub)内にて
スーパークラスの関数func()をoverrideしています。

このサブクラスのfunc()の中で、
スーパークラスのfunc()をそのまま呼び出したいときは、
このようにする方法がありますよね。
void Sub::func(){
____Super::func();
____/*その他の処理*/
}

しかし、これはサブクラス側の自由であり、
スーパークラスのfuncを実行しないこともできます。
そこで、サブクラスのすべてのfunc()で
スーパークラスのfunc()が必ず処理されるように強制したいです。
これは可能でしょうか？
文字通りではなく、擬似的な手段で構いません。

よろしくお願いします。

2018/01/13(土) 21:22:05.60

↑誤字修正致します。
☓そこで、サブクラスのすべてのfunc()で
○そこで、すべてのサブクラスのfunc()で
申し訳ございません。

2018/01/13(土) 21:27:01.54

スーパークラスとかサブクラスとか言ってるやつにイラッとするのが C++er

2018/01/13(土) 21:35:37.12

メソッドチェーンというものを自作すればコントロール可能だが、ややこしい上にオーバーヘッドがある。

2018/01/13(土) 21:42:39.62

ごめん、間違った。
メソッドチェーンじゃなくてC#デリゲート。

2018/01/13(土) 21:45:31.67

>>914
親と子の方が一般的ですか…？
>>915
ありがとうございます！
メソッドチェーン自体は知っていましたが、
それを活用するという考えはありませんでした。
これをヒントに色々試してみます。

2018/01/13(土) 21:49:30.82

>>916
入れ違いになってしまった。
デリゲートでしたか！

2018/01/13(土) 22:24:25.09

NVIでぐぐれや

2018/01/13(土) 22:56:11.51

Java厨絶滅成就（・人・）ﾅﾑﾅﾑ

2018/01/13(土) 23:27:00.62

>>917
スーパーというのはある集合を内包する、より大きな集合。サブというのはある集合の一部の集合。
このとき、オブジェクト指向におけるクラスというのはインスタンスの集合とみなして、
サブクラスはスーパークラスの一部の集合とする考え方によってスーパー/サブと言ってる。

しかし、あるクラスをクラスが持つ機能 (メソッド) の集合と考えるとスーパー/サブが逆転する。
これはわかり難くて混乱の元。

という教訓に基づき基底クラス (base class) と派生クラス (derived class) という用語を C++ では採用した。

ちなみに C++ にはサブオブジェクトという用語はあって、
これはあるオブジェクトのデータメンバや基底のオブジェクトのこと。

**デフォルトの名無しさん** · 2018/01/14(日) 00:30:48.99

>>921
> ちなみに C++ にはサブオブジェクトという用語はあって、
> これはあるオブジェクトのデータメンバや基底のオブジェクトのこと。

へー、スーパークラス＝基底クラスで、基底のオブジェクト＝サブオブジェクトなのか

これがスーパー/サブの逆転ってやつか

確かにわかりづらいね

2018/01/14(日) 00:33:39.99

スーパーとかダセェ

2018/01/14(日) 01:04:12.68

VC++だと派生クラスBarから基底クラスFooのメソッドfuncを

　__super::func()

という風に

　__super

キ──ワ──ドが使えるんじゃ！

2018/01/14(日) 05:55:30.36

privateなvirtual funcImpl()をカスタマイズポイントとして派生クラスに自由にオーバーライドさせて
基底クラスのpublicなfunc()で>>912で言うところのSuper::func()相当の処理してからfuncImpl()を呼ぶ
(func()自体はnon-virtual)
ってのがまあ常套手段だと思うけどそれだと困る？

2018/01/14(日) 06:37:11.95

>>924
多重継承してるとあかんけどな

2018/01/14(日) 10:01:41.75

>>922
多分ちげーんだろーな
C++のサブオブジェクトはサブリミナルとかのサブだろ
オブジェクトに潜んでる何かだろ
つい最近にもあった潜性／顕性の潜性の方だろ

無理に対義語作るならジス(this)・オブジェクトかメイン・オブジェクトじゃねえの

2018/01/14(日) 12:55:55.66

サブセットのサブだろ
あるオブジェクトの一部分がサブオブジェクト
対義語はやっぱりスーパーだよ

2018/01/14(日) 13:03:04.48

部分オブジェクトの反対語をスーパーオブジェクトとは言わん

2018/01/14(日) 13:14:31.73

でもサブセットの反対はスーパーセットだろ

2018/01/14(日) 13:32:41.21

subset, subhuman などに使われる前置詞 sub だろ
語義は「不完全な」だ

**デフォルトの名無しさん** · 2018/01/14(日) 14:17:12.73

じゃあ対義語はconcreteか

2018/01/14(日) 15:03:36.89

集合は要素の重複を認めないのでスーパーオブジェクトとかサブオブジェクトとか言う方が異端なのでは…

2018/01/14(日) 15:06:32.87

英語圏の方々の言語感覚は正直ﾜｶﾗﾝこともあるが
「具体的な」（Concrete）と「完全な」（Complete）は多分使い分けられれているのではないか

2018/01/14(日) 15:11:45.36

>>926
クラスcがクラスaの単一継承なら
{ クラスcに属するオブジェクト } ⊂ { クラスaに属するオブジェクト } ・・・ (1)
なのでa::func()はcから見て__super::func()で正しいが、

クラスcがクラスaとbの間の子なら、(1)および
{ クラスcに属するオブジェクト } ⊂ { クラスbに属するオブジェクト } ・・・ (2)
はどっちも成立しないから__superキーワードが使えないのはある意味当然で整合的と言える、

2018/01/14(日) 15:30:00.21

ｽﾏﾝ訂正
クラスcがクラスaとbの間の子（で公開継承）なら、(1)と(2)が両方成立するんだったorz
（cのインスタンスはaのインスタンスとしてもbのインスタンスとしても扱える

ついでに補足で、>>933は「包含関係の意味でスーパーとかサブとか言うなら」異端という意味

2018/01/14(日) 21:03:41.51

>>931
incompleteとは違うね

◆QZaw55cn4c (ﾜｯﾁｮｲ 4a60-cexj) · 2018/01/14(日) 23:25:18.74

std::set にて std::set<int> に特化して実装しようとしていますが、
ここで private メンバーである iterator positionToInsert(int n) を記述しようとして、はまっています

iterator posisionToInsert(int n) ：
n を挿入するべき内部テーブルの位置（イテレータ）を返す
n がすでに set にあったら nullptr を返す
としたかったんですが

・nullptr を iterator に代入できない
・そもそもコンテナ.end() が nullptr だ（無理にキャストしてみて分かった）…①

返り値を複数にするしかないのかな…
std:set.insert() も pair を返すみたいですし
①は規格で決まっているのでしょうか？処理系依存でしょうか？
https://teratail.com/questions/25576

2018/01/15(月) 00:07:45.76

iteratorはポインタちゃうで

2018/01/15(月) 00:52:09.62

変なメンバ関数つくるよりfind使っておけよトーシロー
理解できないなら余計な機能追加はするな
余計にわからなくしてプログラムを目茶苦茶にするな

2018/01/15(月) 00:54:40.48

findじゃないなlower_boundで位置は把握できるだろこのハゲー

2018/01/15(月) 01:03:25.78

std::setは挿入する位置を気にしないはずだが。設計思想が間違っている。

2018/01/15(月) 08:06:37.84

おれば後ろより前に挿入する方が好き

◆QZaw55cn4c (ﾜｯﾁｮｲ 4a60-cexj) · 2018/01/15(月) 10:02:53.84

>>940
これから作るその find() の内部で iterator を返す private な関数を作ろうとしているのです
二分探索で find() するのも、二分探索で挿入先を探すのも、やることは一緒ですよね
lower_bound() も、これから記述するのです

>>942
std::set は挿入する要素が互いに比較できることを必要としています
挿入先の位置を求めるメンバ関数は、あくまでも private としています

2018/01/15(月) 11:18:44.64

>>944
まずは iterator じゃないものを iterator で返すという発想をやめて
pair でも参照型引数でもいいから結果と成否は別に返すのが良いと思う

2018/01/15(月) 14:02:37.88

>>938
> end()がnullptr
たんに指し示した先のデータが0だったってだけじゃないの？
双方向イテレータとして使用可能なのでnullptrだと上手くいかないと思うよ。

◆QZaw55cn4c (ﾜｯﾁｮｲ 4a60-cexj) · 2018/01/15(月) 18:11:46.06

>>939-942,945-946
コメントありがとうございます

>>945
はい、結果と成否をわけることにしました
http://mevius.2ch.net/test/read.cgi/tech/1434079972/33

2018/01/15(月) 22:43:32.66

class Foo{
　void function(){
　　hoge.exe();
　}
private:
　Hoge hoge;
}

と

class Foo{
　void function(Hoge& hoge){
　　hoge.exe();
　}
}

どっちが最適化かかりやすい?

2018/01/15(月) 22:51:21.17

ほぼ変わらないと思うけど
スタックに積まない分前者かなと思う

2018/01/15(月) 23:37:09.62

ただのラッパなら自由関数にしれ

2018/01/16(火) 00:21:59.76

>>948
情報処理における罪の多くは最適化の名のもとでなされる。
そこはメンテナンス性の高い方を選ぶべき。

これから何千行のコードを書こうとしてるのか知らないけど、いちいちそんなことに頭使ってたら禿げるよ。

2018/01/16(火) 06:29:10.56

データメンバへのアクセスはthis経由なので
ポインタにオフセットを書けて逆参照という動作になる
左辺値参照の場合はオフセットなしで逆参照
いずれにせよ仮引数を受け取るという点は同じ動作だ

最適化という観点からはデータメンバはrestrictが付いているようなもので
若干有利ではないかな

ただしデータメンバの場合は全ての非静的メンバ関数が関与する可能性があり
仮引数の場合は当該関数のみというアクセス範囲の違いがあるので
最適化する以前の、基本的な設計で選ぶべきだろう

2018/01/16(火) 06:49:48.40

messageboxで変数は表示できますか。

2018/01/16(火) 07:53:30.35

できるけど、モーダルでもモードレスでもかなりうぜえぞ
template <typename T>
void display(T const& variable)
{
stringstream ss;
ss << variable;
MessageBox(NULL, ss.str().data(), typeid(variable).name(), MB_OK);
}

2018/01/16(火) 17:35:22.56

質問させてください。VS2008で仮想継承使っていたら以下の警告が出たんですが、警告が出ないようにするにはどうすればいいでしょうか？

warning C4250: 'CHoge' : 2 つ以上のメンバが同じ名前を持っています。'CTest1::CTest1::Draw' から継承します。
----------------------
class ITest1 {
public: virtual void Draw() = 0;
};

class CTest1 : virtual public ITest1 {
public: virtual void Draw() { printf("Draw\n"); }
};

class ITest2 : virtual public ITest1 {
public: virtual void Func() = 0;
};

class CHoge : public ITest2, public CTest1 {
public: virtual void Func() {}
};

2018/01/16(火) 18:00:29.71

VS2017ではデフォでは何も言ってこず
/W4にするとC4250が出る

warningだろ？ errorは直すしかないが
warningは内容を読んで理解したうえでどうするかを判断するものだ
必ず消そうと思うな、キリねえぞ

意味を理解しないからdisられたと思っちまうんだよ

2018/01/16(火) 18:02:50.15

どうしても「消すこと」が絶対ならこれやっとけ
#pragma warning(disable : 4250)

**デフォルトの名無しさん** · 2018/01/16(火) 18:08:17.81

>>955
なんでITest2を仮想継承してないの？

2018/01/16(火) 18:11:37.58

この場合どっちを継承するとかの問題もないし
普通にこっちにも virtual つければいいだけだと思うが vc だと違うのか？

>>955
＞class CHoge : public ITest2, public CTest1 {
＞public: virtual void Func() {}
＞};

こう

class CHoge : virtual public ITest2, virtual public CTest1 {
public: virtual void Func() {}
};

2018/01/16(火) 18:30:21.88

おまえら質問者に教えて貰うあべこべになりそうだなｗ

2018/01/16(火) 18:55:59.62

iTest2のDraw()が零でCTest1のDraw()が定義されているからそりゃ衝突する罠

2018/01/16(火) 20:35:10.19

>>956
VS2017では警告出ないのですか。試して頂いてありがとうございます。
自分なりに意味を考えたところ、
無視して大丈夫そうという結論に至りました。
#pragma で消す事にします。
>>958
１つにしたいのはITest1::Drawだけなので、
ITest1のみ仮想継承しました。
>>959
試してみましたが、同じ警告が出てしまいました。

>>961
衝突した結果、有効な方が選択されるので特に問題なさそうな気がしますね。

2018/01/17(水) 12:08:14.47

boost::spirit::qiで
boost::spirit::qi::rule<std::string::iterator, std::string()> hogehoge;
としてとき std::string() のようにかっこ付でテンプレート引数に与えていますが
このかっこ付で型名をあたえる方法はどのように活用できますか？（一般論として