C++相談室 part154

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

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

テンプレここまで

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

マスクも手洗いもコロナを100%防げるわけじゃないから
マスク付ける必要も手を洗う必要もないね

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

テストツールいらねえなんて言ってねえぜ
テストツール万能論はアホの考えだと言っている

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

で、身に覚えのあるやつが返事してるから
ニヤニヤ笑ってるのさ

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

「テストツール万能論」を主張したレスなんて一つも見えないんだけど
存在しないものを見てニヤニヤしてる変態こわい

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

それを>>587がぬかすとは笑止な

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

構い過ぎじゃね？

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

>>591
まさかと思うけど>>587がマジで言ってるとか思ってないよね？w

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

ああ・・・そういうことね・・・すまん触るべきじゃなかった

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

すまんで済んだらユニットテストは要らんわ！

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

野菜食えで馬鹿みたいにブチギレるホリエと一緒やな

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

>>593
奴は「必要ないね」とぬかしたんだ
別に武漢熱に話をすり替えようとすり替えまいとそこは関係ない

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

>>597
文章を字面通りにしか解釈できない人？

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

vectorを参照渡しするときの方法について教えてくください
class sansyo
{
private:
std::vector<short> _stdSansyo;
public:
void setSansyo(std::vector<short>& sansyo);
}
;void sansyo::setSansyo(std::vector<short>& sansyo)
{
_stdSansyo = sansyo;
_stdSansyo[5] = 500;
}
int main()
{
std::vector<short> moto;
sansyo cSansyo;
for (int i = 0; i < 10; i++) {
short tmp = i * 2;
moto.push_back(tmp);
}
cSansyo.setSansyo(moto);//ここで参照渡しのはず
for (int i = 0; i < 10; i++) {
std::cout << "値=" << moto[i] << "\n"; //値が変わっていない
}
}
こんなふうに、参照渡しにしてみたのですがどうも参照渡しになっていないでそのまま新しいvectorが作られちゃったぽいのですが
これだとメモリを倍使っちゃってるし、時間も掛かってますよね
どういうふうにしたら、喚ばれた側の関数で参照渡しに出来るのでしょう

**蟻人間** ◆T6xkBnTXz7B0 · 2021/02/07(日) 16:37:08.22

>>599
右辺値参照かstd::swap

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

>>600
サンプル貼ってよ

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

std::swapも右辺値参照も全然上手く行きそうになかっのたで

宣言をポインタにして
private:
std::vector<short> *_stdSansyo;

void sansyo::setSansyo(std::vector<short>& sansyo)
{
_stdSansyo = &sansyo;
sansyo[6] = 100;
_stdSansyo->data()[5] = 50;
}

関数をこんなふうに書き換えたら、ちゃんと参照だけで動いたのでもういいや

けど右辺値参照とstd::swapの例も出してくれたらこれから色々いたスカリマス

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

>>600
ほれほれ
貼れんのか

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

>>599
参照渡し自体はできてるはず。
ただ参照渡ししたvectorをクラスのメンバに代入して、代入されたクラスメンバの方を更新しているからvectorを渡した元の方にはまったく影響が無いというだけ。

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

>>598
もうそう言うことにしないと自我が保てないんだろw

**デフォルトの名無しさん** · 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を許可しないのは最後の一線なのかもしれん…