C++相談室 part154

**デフォルトの名無しさん** · 2021/01/08(金) 17:54:00.55

※前スレ
C++相談室 part153
https://mevius.5ch.net/test/read.cgi/tech/1602339500/

テンプレここまで

**デフォルトの名無しさん** · 2021/02/07(日) 17:59:39.88

sansyo::_stdSanshoをどうしても参照にしたいならこうするしか、
class sansho {
private:
std::vector<short> _stdSansyo;
public:
sansho(std::vector<short>& src) : _stdSansyo(src) { }
void setSansyo(std::vector<short>& sansyo);
};
void sansyo::setSansyo(std::vector<short>& sansyo) {
_stdSansyo[5] = 500;
}
int main() {
std::vector<short> moto;
sansyo cSansyo(moto); //ここで参照渡し
for (int i = 0; i < 10; i++) {
short tmp = i * 2;
moto.push_back(tmp);
}
cSansyo.setSansyo(moto); // moto[5]に500が入りまくり
for (int i = 0; i < 10; i++) {
std::cout << "値=" << moto[i] << "\n"; //値が変わった！（予定
}
};

**デフォルトの名無しさん** · 2021/02/07(日) 18:00:52.99

>>598
だったら何だとぬかす気だ？
> 存在しないものを見てニヤニヤしてる変態こわい
> 文章を字面通りにしか解釈できない人？

要するに話になんない奴ってことだろ
言ったことにも裏の意味にもコメントされたくない
じゃあ相手しねえでやるから黙ってろゴミ

**デフォルトの名無しさん** · 2021/02/07(日) 18:02:35.24

訂正orz
誤： private: std::vector<short> _stdSansyo;
正： private: std::vector<short>& _stdSansyo;

もしcSansyo.setSansyo(moto)の呼び出し時に参照を渡したいんじゃああ！
という向きにはsansyo::_stdSansyoはstd::vector<short>* _stdSanshoにして
ポインタを持つようにすべき

**デフォルトの名無しさん** · 2021/02/07(日) 18:10:05.07

ちゅかもっと大きな一般原則としてつぎのどっちかにすべき
(1) std::vector<short>の実体の所有権をmain()（で定義いているmoto）に固定してcSanshoクラスにmotoのアドレスを記憶させない
(2) std::vector<short>の実体の所有権をcSanshoクラスのインスタンス（_stdSansyo;）に固定してmain()でmotoを定義するのをやめる
※ 個人の感想です

つまり出題が悪い。出し直し

**はちみつ餃子** ◆8X2XSCHEME · 2021/02/07(日) 18:10:05.98

>>602
元の考え方を変えずに動作させるにはその方法は十分に妥当だと思うよ。

ただ、もっと複雑なプログラムになったときにうっかりデータよりポインタのほうが
長生きすることになっても発見しづらいデザインになっている。

可能なら全体のデザインを見直すべきという話で、
いっそ所有権を渡してしまったほうが間違いにくいかもねという意味で右辺値参照や swap の話題が出ているので、
>>599 をベースにして右辺値参照や swap を使う例に書き換えるのは難しい。
別物になってしまう。

**デフォルトの名無しさん** · 2021/02/07(日) 18:11:12.01

>>607
自分以外の複数の人を同一人物だと思っちゃう人？

**デフォルトの名無しさん** · 2021/02/07(日) 18:14:03.86

>>611
ネットの総意とか言い出しちゃう人？

**デフォルトの名無しさん** · 2021/02/07(日) 18:16:20.19

検証不能な事物を錦の御旗にするような人間が
ソフトウェアのテスト推しなのは
大いなる矛盾である氏ね

**デフォルトの名無しさん** · 2021/02/07(日) 18:18:25.00

>>611
あの流れで
> 文章を字面通りにしか解釈できない人？
なんてぬかすやつは複数ID自演厨と見なされて当然だ
疑われたくなければ口の利き方に気をつけな

疑いは晴れてない
この後の発言にも気をつけるんだな

**デフォルトの名無しさん** · 2021/02/07(日) 18:24:44.17

>>566
なにこれ

**デフォルトの名無しさん** · 2021/02/07(日) 18:29:05.24

参照をフィールドに保持するのは、一部のパーサーくらいでは？

しかも、状態を関数に切り分けないとデバッグが辛いので、仕方なくそうするだけで、バッド何とかの類だし。

**デフォルトの名無しさん** · 2021/02/07(日) 18:31:34.10

むかし5chで誰かが、プログラムを書くときは必ずテストしてるはずなんだって言ってましたが。
それを単体テストとして書いておけばずっと使えて便利だよと。

その書き込みを見て世界中の人がテストフレームワークを書き始めたんですよ。

**デフォルトの名無しさん** · 2021/02/07(日) 19:06:50.71

>>616
想像力が足りなすぎ
>>617
想像力が豊かすぎ

**デフォルトの名無しさん** · 2021/02/07(日) 20:53:42.20

バッドエンディングですね。

**デフォルトの名無しさん** · 2021/02/07(日) 21:03:56.50

グローバル変数使うなとは言わないけどコメントもないのはキツい
grep掛けて検索しても訳分からん
ポインタで飛び火してるときなんてもう…

**デフォルトの名無しさん** · 2021/02/07(日) 22:09:55.11

演算子オーバーロードの厄介さはキーワード検索で拾いにくいこと。
反復子もそうだが。ま、言い出したらきりないが。

**仮トリップ** ◆4DF7.T7u4M · 2021/02/07(日) 22:57:26.65

>>561を書いたのは私なんだけどなぁ...
面倒くさそうな人がいるし、一般論だけ言って立ち去ろって思っただけ

**仮トリップ** ◆4DF7.T7u4M · 2021/02/07(日) 23:02:22.83

ちなみに、>>561以降は何も述べてない。
私は一般論を語っただけだから、他の人が私と似た考えでツッコミを入れただけに過ぎない。

殺虫剤のパラドックス？それがどうした？
私はテストコードを書いて不具合を激的に抑えているが？

殺虫剤のパラドックスを説明したところで、不具合を限りなくゼロに近づける試みが無駄であることの証明にはならない。

**蟻人間** ◆T6xkBnTXz7B0 · 2021/02/07(日) 23:09:32.25

>>622-623
自我の芽生え、おめでとう！
では、かえるの歌を歌います。

【...JASRAC権利関係の為、自主検閲...】

♪パパーン
(全員でクラッカーを鳴らす)

**デフォルトの名無しさん** · 2021/02/07(日) 23:20:12.91

殺虫剤のパラドックスなんてものは存在しない。
ただの退行テスト不足だ。

**デフォルトの名無しさん** · 2021/02/08(月) 06:24:22.18

殺虫剤のパラドックスって言いたかっただけやろw
そもそもテストで発覚したバグを修正したら同じテストで摘出できないのは当たり前
摘出できたら単なる修正漏れだしw
JSTQBの関係者か信奉者が言い出したんだろうけどあまり意味のない用語だと思う

**デフォルトの名無しさん** · 2021/02/08(月) 06:42:41.05

蒸し返さなくていいから……

**デフォルトの名無しさん** · 2021/02/08(月) 07:03:25.17

虫返さないですと！？

**デフォルトの名無しさん** · 2021/02/08(月) 11:05:43.31

>>610
そもそも>>600は的外れだと思うんだが
メンバの方を参照かポインタで持つ、で終わりだろ
3行目以降完全に蛇足

**デフォルトの名無しさん** · 2021/02/08(月) 11:20:50.36

>>627,628
そういうのは無視しろ

**デフォルトの名無しさん** · 2021/02/08(月) 12:00:56.57

スルーの法則

**デフォルトの名無しさん** · 2021/02/08(月) 12:43:41.70

>>602
>>599
誰か
永続的に使用するなら shared_ptrで受け取れ。今のままだとsetSansyoを呼び出す側が所有権手放していいのかわからなくて困るだろ。
て指摘した？
今や生ポインタなんて性能優先のときに内部的に使用するもんで、インターフェイスで使用するもんじゃない。

あと、参照も関数内だけで使用する引数に使うもんで、永続的に所有する引数に使うもんじゃない。

**デフォルトの名無しさん** · 2021/02/08(月) 14:04:18.90

コンストラクタでなら参照型のメンバ変数に保存できる。ほとんどの人はやらないけど。

**デフォルトの名無しさん** · 2021/02/08(月) 18:54:29.18

>>633
できるのと実際に実装するのは別の話だわな。
所有権の無いオブジェクトの参照を保存するなんて狂気の沙汰だ。

**デフォルトの名無しさん** · 2021/02/08(月) 19:12:34.60

アイキャンユーキャン。

**デフォルトの名無しさん** · 2021/02/08(月) 20:51:28.21

>>633
クラス間に親子関係のようなものがあって、子の生存期間が常に親の生存期間内にあるような場合は、親への参照をメンバ変数とすることは普通にあると思うけど。例えば>>484とか

**デフォルトの名無しさん** · 2021/02/08(月) 21:15:43.14

shared_ptr(またはスマポ)至上主義の変なやつ以前から居るんだよな
相手にするな

**デフォルトの名無しさん** · 2021/02/08(月) 21:37:27.20

ナマポ使ったほうが良い場面なんて皆無でしょう

**デフォルトの名無しさん** · 2021/02/08(月) 21:48:31.75

出たwナマポ完全否定奴w

**デフォルトの名無しさん** · 2021/02/08(月) 21:58:57.06

参照型のメンバ変数は参照オブジェクトの生存期間を保証できないからweak_ptrで保存しておいて使う時だけshared_ptrを取得するのがC++的な解決なんじゃないの。

**デフォルトの名無しさん** · 2021/02/08(月) 22:08:09.48

スマポにしたら所有権の問題を考えなくてよい
というわけではないからスマポ使えというのは妥当ではない
逆に所有権に矛盾が無ければインスタンスは生成元が与えた参照を持って良い

ていうかスコープを抜けたら自動的に解放される系のクラスを書いたら
エラー処理上エラー通知先としての生成元オブジェクトの参照保持はほとんど不可避

**デフォルトの名無しさん** · 2021/02/08(月) 22:11:14.29

コピー禁止のshared_ptr欲しい

**デフォルトの名無しさん** · 2021/02/08(月) 22:25:23.27

すべてスマポで書くスタイルも悪くないと思うけど、メイヤーズ神もツリー構造で子が親のポインタを持つときはナマポで十分と書いている

**デフォルトの名無しさん** · 2021/02/08(月) 22:27:43.60

木のノードは子へのポインタじゃなくて子のノードIDを持て😡

**デフォルトの名無しさん** · 2021/02/08(月) 22:48:15.38

自分よりポインタの方が寿命が長いことが保証されているなら確かに生ポで問題ないが
それが成り立つ状況ってなかなかないよな。

**デフォルトの名無しさん** · 2021/02/08(月) 23:02:19.81

参照先の実体がまだ生存しているかどうか知るにはweak_ptr::expired()を使うしかないのが現状でしょ。

**デフォルトの名無しさん** · 2021/02/08(月) 23:07:56.19

Chromiumはstd::unique_ptrを全面的に使ってるけど、ポインタを使う設計そのものが古いような気がする。

**デフォルトの名無しさん** · 2021/02/08(月) 23:13:28.50

まあcコードを全く使わないってのならいいんでないの。
ただc++のポータビリティーは君が思ってるより低いけどね。

**デフォルトの名無しさん** · 2021/02/08(月) 23:23:15.47

露骨すぎるマウント取りたがりダッセーｗｗｗ

**デフォルトの名無しさん** · 2021/02/09(火) 01:58:29.04

マウントじゃなくてポインタでは？

**デフォルトの名無しさん** · 2021/02/09(火) 02:46:23.52

C++に特化したAPIなんてどこのOSでも提供されないから、結局、Cの配列と互換性のあるstd::vectorやstd::arrayを使わざるを得なくなる。

**デフォルトの名無しさん** · 2021/02/09(火) 04:01:45.22

>>644
木もコンテナで良いですよね。
所有権がハッキリしてて。

**デフォルトの名無しさん** · 2021/02/09(火) 07:34:27.86

>>651
data()を必ず使う手合いか？

**デフォルトの名無しさん** · 2021/02/09(火) 07:39:10.06

木は木全体を収める領域ごと一気に解放する場合もあるから別に

**デフォルトの名無しさん** · 2021/02/09(火) 17:49:12.66

>Cの配列と互換性のある

doubt

**デフォルトの名無しさん** · 2021/02/09(火) 17:53:30.91

ダウトとデビューって似てるよね

**デフォルトの名無しさん** · 2021/02/09(火) 18:35:55.16

親が子のポインタを所有する木は、親を消すとネストして子孫のデストラクタを呼ぶので、スタックが枯渇します。

したがって、子から順番に消さなくてはなりません。

**デフォルトの名無しさん** · 2021/02/09(火) 18:56:53.48

その様はまるで摩天楼がドミノ倒しのように連鎖倒壊するようでもあり、DOMINOというピザにもなっています・

**デフォルトの名無しさん** · 2021/02/09(火) 19:07:16.84

なんと後味の悪い話

**デフォルトの名無しさん** · 2021/02/09(火) 20:39:36.84

意味が分からない

**デフォルトの名無しさん** · 2021/02/09(火) 21:07:48.14

>>657
それで枯渇するようじゃ木の探索すらできない。

**デフォルトの名無しさん** · 2021/02/09(火) 21:13:02.23

探索はループでもできるんで、たとえば数万個の要素を持つツリーを子から消せというのは真理なのかも

**デフォルトの名無しさん** · 2021/02/09(火) 21:27:40.55

消すのは独立して実行できるので別スレッドに送れば実質コスト0

**デフォルトの名無しさん** · 2021/02/09(火) 21:38:09.75

>>662
DFAとPDAではやれることの範囲の違いがちげう

つかこの場合は安易にスマポとかコンテナを使うから破棄ごときにトラバースの手間がかかるんである

**デフォルトの名無しさん** · 2021/02/09(火) 21:44:42.72

それもそうであるな

**デフォルトの名無しさん** · 2021/02/09(火) 21:50:17.60

ちゅか垂直探索なら木をメモリに持っておく必要が無い
水平探索ならメモリ上に世代1、世代2、....と木を育てていく（らしい）が
兄弟ノードを全部見終えたのでいざ従妹ノードに移ろうとするときに
親ノードへの参照は欲しい木がするがするとshared_ptrなら即循環参照になり、
子の情報で親（子孫がいっぱいぶら下がっている）を生成できるわけもないから、
ウィークポインタの出番でもない

**デフォルトの名無しさん** · 2021/02/10(水) 01:37:49.58

子から消さなかったばかりに。
2000人を乗せた航空機が洋上で消えた。
ってなるかも？

**デフォルトの名無しさん** · 2021/02/10(水) 01:48:13.89

A380でさえ853席だが2000人なんてどうやって載せるんだろう

**デフォルトの名無しさん** · 2021/02/10(水) 01:51:41.17

く、空中で衝突するとか…

**デフォルトの名無しさん** · 2021/02/10(水) 01:53:08.99

インドでは屋根の上にも乗ります。

**デフォルトの名無しさん** · 2021/02/10(水) 01:55:11.55

https://trafficnews.jp/post/80031

証拠写真みつけてきた。

**デフォルトの名無しさん** · 2021/02/10(水) 01:55:17.84

GTA5のMOD作成者みたいな会話だな

**デフォルトの名無しさん** · 2021/02/10(水) 02:01:19.54

>>653
昔はdata()がなかった。(遠い目)

**デフォルトの名無しさん** · 2021/02/10(水) 06:40:17.61

アントノフの貨物室にすし詰めでどんくらい乗れるかな

**デフォルトの名無しさん** · 2021/02/10(水) 06:40:27.16

東京: TK
京都: KT
大阪: OSK
淡路: AWG

**デフォルトの名無しさん** · 2021/02/10(水) 07:59:22.76

航空機や原発みたいなクリティカルなシステムでは
全部固定長の配列で書いてあるんじゃないの

**デフォルトの名無しさん** · 2021/02/10(水) 08:06:21.27

釣れますか。
ヒープのほうが少し安全です。

**デフォルトの名無しさん** · 2021/02/10(水) 10:25:11.13

>>671
どこが証拠やねん!
せいぜり100人やないかヽ(`Д´)ﾉ

**デフォルトの名無しさん** · 2021/02/10(水) 14:32:04.65

すみません、std::for_each を使っていて continue したくなりましたが、サポートしてないです
よね? これって:
1. for_each は continue する必要がないような処理に限ってに使うべき。
2. ループの中で大きい if ブロックを作って空の処理にすればよい。
3. その他

**デフォルトの名無しさん** · 2021/02/10(水) 15:13:45.18

returnすれば？
for_each(begin(foo), end(foo), []{ if(bar) return; });

**デフォルトの名無しさん** · 2021/02/10(水) 16:06:00.49

return は continue ではなく break じゃないの？
質問者の2.大きなif分で実質何もしないでいいと思う

**デフォルトの名無しさん** · 2021/02/10(水) 16:20:11.24

for_eachをbreakするにはthrowかlongjmpがいるぞ

**デフォルトの名無しさん** · 2021/02/10(水) 16:35:39.62

きったねぇジャンプだ

**デフォルトの名無しさん** · 2021/02/10(水) 16:51:31.94

あ、ラムダ式になってたのか

**はちみつ餃子** ◆8X2XSCHEME · 2021/02/10(水) 17:51:08.81

>>679
for 文で書いた方が短く書けそうだけど、
あえて for_each を使いたい理由が何かあったりするの？

**デフォルトの名無しさん** · 2021/02/10(水) 18:41:41.17

Perlなら問題無くできる（continue→next、だが
C++規格委員会がfor_eachのcontinueを許可しないのは最後の一線なのかもしれん…

**デフォルトの名無しさん** · 2021/02/10(水) 19:51:57.06

何の一線よ

**デフォルトの名無しさん** · 2021/02/10(水) 20:18:41.39

いやreturnでいいだろ…
許可しないってなんのことだよ

**デフォルトの名無しさん** · 2021/02/10(水) 21:21:18.68

vectorで一億件ほどで、飛ばしたいのが100～1000件程度なんだろう

**デフォルトの名無しさん** · 2021/02/10(水) 21:51:25.56

>>679
for_eachをrange based forに変更は出来ないの？

**デフォルトの名無しさん** · 2021/02/10(水) 21:59:01.21

>>690
おまえのスキルのためにか？

**デフォルトの名無しさん** · 2021/02/10(水) 22:17:23.69

range based forなら普通にcontinueできるぞ

**デフォルトの名無しさん** · 2021/02/10(水) 22:18:26.21

>>689
飛ばす手段があったとして、飛ばすかどうかどこでどう判断するの？

**デフォルトの名無しさん** · 2021/02/11(木) 04:16:42.30

early returnは正義

**デフォルトの名無しさん** · 2021/02/11(木) 11:22:44.35

>>685
テンプレのときとか

**デフォルトの名無しさん** · 2021/02/11(木) 17:53:02.68

679がなぜfor_eachを使うのかを無視するのは
他人の領分を侵すお節介だ

for_eachの使い方をアドバイスできんやつは
しゃしゃり出てくるな

**デフォルトの名無しさん** · 2021/02/11(木) 18:00:13.22

よくよく話を聞いてみたらまるで異なる解決策が見つかるなんてよくあることじゃん。そんな経験ないの？
何をしたかったか確認するのは重要

**デフォルトの名無しさん** · 2021/02/11(木) 18:15:41.21

流れをよく読んでみろよ
for_eachをロクに使ってないやつなのモロバレだろ
質問者が尋ねていないことを答えたいから協力しろなんてぬかすのは
回答者の資格ねえんだよ

**デフォルトの名無しさん** · 2021/02/11(木) 18:18:25.32

自分が何か尋ねているときに
質問内容に付き合ってやれる懐のねえやつは
うぜえだけだろが

**デフォルトの名無しさん** · 2021/02/11(木) 18:23:57.31

質問内容において自分より下なやつに教えを請いたいかよ

**デフォルトの名無しさん** · 2021/02/11(木) 18:34:11.92

for_eachを使う場合の答えはとっくに出てるのに何言ってんの？

**デフォルトの名無しさん** · 2021/02/11(木) 19:17:08.97

3のその他で答えただけ。

**デフォルトの名無しさん** · 2021/02/11(木) 20:01:29.92

他人の領分なんて侵してナンボ

**デフォルトの名無しさん** · 2021/02/11(木) 20:03:18.37

>>702
はあ？returnではいけない理由は？

**デフォルトの名無しさん** · 2021/02/11(木) 20:19:00.69

>>679はループしたいだけだろ
std::for_eachを何が何でも使わなければならない特殊な事情があるなら仕方ないけどそんなこと言ってないし