【初心者歓迎】C/C++室 Ver.100【環境依存OK】 [無断転載禁止]©2ch.net

**デフォルトの名無しさん** · 2016/11/06(日) 22:58:02.60

エスケープシーケンスやWin32APIなどの環境依存なものでもOK。
ただしその場合、質問者は必ず環境を書きましょう。

◆ソースのインデントについて
半角空白やTABでのインデントはスレに貼ると無くなります。
そのため、アップローダーに上げるのも手ですが直接貼る場合は、
全角空白かに置換すると見栄えだけはよくなります。

【アップローダー】(質問が長い時はココ使うと便利)
http://codepad.org/ (コンパイルもできるし出力結果も得られる[]privateをチェック)
http://ideone.com/ (時間帯によってはcodepadが重い事があるのでここも利用)

前スレ
【初心者歓迎】C/C++室 Ver.99【環境依存OK】
http://echo.2ch.net/test/read.cgi/tech/1469177649/

**デフォルトの名無しさん** · 2017/03/26(日) 08:58:38.45

ここはお前の日記を垂れる場所じゃない
「サポートによるとコンパイラのバグだった」で済むのに
アレやってコレやってを書くな低能

**デフォルトの名無しさん** · 2017/03/27(月) 10:03:34.57

事後報告あると参考になっていいんだけどな
今回は本人以外に状況がわからないから事後報告されてもｗ

**デフォルトの名無しさん** · 2017/03/30(木) 03:06:01.35

http://codepad.org/KMQH3TsZ

ポインタをメンバに持たせた場合のget/setの書き方が分かりません
getの方は、ポインタを返すときに返した先で値を変えられないようにconstを付けたのですが、
sizePlus10のような関数を呼びたい場合に、const_castを付けて呼ぶことになってしまいました
値の変更をするのでsetを通してやるのが良いと思うのですが、何か良い書き方ありますか？

また、setの方は、一括で設定させることはできるのですが、
data->id、data->sizeの値を個別に設定したい場合、何か良い書き方はありますか？
コメントの部分のようにsetメンバ関数を何個も書く方法しか思いつきません

**デフォルトの名無しさん** · 2017/03/30(木) 07:50:43.79

getDataの結果を修正したいのだから
杓子定規にgetDataの戻りにconst付けるのを止めればいいのではないか

そう言えばC++17(ドラフト)もstring::dataの戻りが非constに修正されたような

**デフォルトの名無しさん** · 2017/03/30(木) 08:08:59.00

memcpy ｗ

**デフォルトの名無しさん** · 2017/03/30(木) 10:48:22.67

>>525
一般論として形でただ覚えるのではなくて意味を理解しよう
理解があやしいから設計の一貫性がなく混乱してるようにみえる
・Dataをポインタで所有するのはなぜ
・ゲッター／セッターを使ってアクセスするのはなぜ
・ゲッターで返す値にconstをつけるのはなぜ
・sizePlus10がData*を受けとるのはなぜ

MyClassの設計方針にあわせるならsizePlus10をDataではなくMyClassを受けとるようにして
void sizePlus10(MyClass & mc)
{
Data newdata = { mc.getData()->id, mc.getData()->size + 10 };
mc.setData(newdata);
}
あるいは別の関数を作って
void MyClassSizePlus10(MyClass & mc)
{
Data newdata = *mc.getData();
sizePLus10(&newdata);
mc.setData(newdata);
}

これが変だとか非効率だとか思うなら
MyClassの設計が要求にあってないのでそっちを変更する

**デフォルトの名無しさん** · 2017/03/30(木) 20:11:34.52

>>528
おまえは質問の意図を理解しよう

>>525
そのsetgetでいいんじゃない？ただそのgetは俺だったらreferにするな
ついでにsetget両方使えるようにdata* operator->()みたいなの実装かな。というかむしろそっちでアクセスする方向にするな
あとはattachdetach作っておいてm_dataを置換できるにようにしちゃうな
その手のスマートポインタは至るとこで使われてるからこんなとこで聞かずにどっかのライブラリの参考にしたら？

メンバのコピーはpragma pack1でstructしておいてmemcpy

**デフォルトの名無しさん** · 2017/03/30(木) 23:27:09.74

>>529
>メンバのコピーはpragma pack1でstructしておいてmemcpy

それをすると何がいいことあるのか？

**デフォルトの名無しさん** · 2017/03/30(木) 23:45:25.72

こういえクソみたいな方法を自慢気に語り出す奴が居るところかC++の最もクソな所だよね

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/30(木) 23:51:55.55

デフォルトコピーコンストラクターとか構造体代入とか知らないんだろう。

**デフォルトの名無しさん** · 2017/03/31(金) 00:14:17.76

>>532
>構造体代入とか知らないんだろう
ほうほう、それは何ですか？

**デフォルトの名無しさん** · 2017/03/31(金) 01:59:49.12

やべえなおい糞コードの臭いがプンプンしやがるぜ

**デフォルトの名無しさん** · 2017/03/31(金) 02:09:29.70

>>526-529
ありがとうございます

今まではとりあえずget/setを書いてgetにはconstを付けとけば良いみたいな認識でしたが、
想定されるクラスの使われ方に合わせて適宜変更するべきでした

MyClassのようなget/setの分け方をすると、
sizePlus10のような関数やsetで不便になってしまうので、
constを付けないget/set共用のものにしてそれを通してアクセスさせるのが適切でした
( getData()でアクセスする場合、Data * const getData() const { /**/ } みたいな )
とても参考になりました

**デフォルトの名無しさん** · 2017/03/31(金) 11:52:48.85

>>531
くそじゃない方法をおしえて

**デフォルトの名無しさん** · 2017/03/31(金) 12:09:56.69

C#じゃないからないでしょ

**デフォルトの名無しさん** · 2017/03/31(金) 13:26:56.75

>>536
コピーは=使え
というだけの話

**デフォルトの名無しさん** · 2017/03/31(金) 13:38:11.94

>>529
> メンバのコピーはpragma pack1でstructしておいてmemcpy
これ俺も理由知りたい

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 17:24:06.64

typedef struct MY_STRUCT
{
int ch;
} MY_STRUCT;

MY_STRUCT a,b;
a.ch = 0;
b = a; //代入できる

**デフォルトの名無しさん** · 2017/03/31(金) 19:17:35.19

>>529
> おまえは質問の意図を理解しよう

なんか無駄に上から目線だなぁ
って思ったら

> メンバのコピーはpragma pack1でstructしておいてmemcpy

単なる老害だったでござる

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 19:19:25.80

構造体の代入ができなかったのは大昔のことだ。

◆QZaw55cn4c · 2017/03/31(金) 19:35:02.95

>>540
それ，K&R1 から堕落したよね

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 19:40:25.86

構造体定義を#include <pshpack1.h>と#include <poppack.h>で囲むのは、スピードよりもメモリー効率を優先するときにする。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 19:44:56.93

あるいは構造体のレイアウトを分かりやすくするときにパッキングを指定する。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 19:46:22.60

>>543
堕落ではない。進歩だ。

**デフォルトの名無しさん** · 2017/03/31(金) 19:49:13.63

> スピードよりもメモリー効率を優先するときにする
20年くらい前まではそういう老人もいたね
今はメモリレイアウトを固定したい時にしか使わないけど

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/03/31(金) 19:58:28.84

#pragma 直接指定は礼儀が悪い、昔のやり方だ。

**デフォルトの名無しさん** · 2017/04/01(土) 02:25:44.35

メモリ効率だけの問題なら大抵の場合は順番入れ替えるだけで問題ないしな

◆QZaw55cn4c · 2017/04/01(土) 07:47:47.79

>>546
サイズが無限にのびるかもしれないものをコピーするなんてだめだよ
昔はね，構造体はポインタを介してしかつかえなかったんだよ

**デフォルトの名無しさん** · 2017/04/01(土) 08:31:33.23

そうそう
スタックに自動変数とかも言語道断
全部staticじゃなきゃね

◆QZaw55cn4c · 2017/04/01(土) 13:26:49.84

>>551
スタックに自動変数はあたりまえだよ‥static おじさん，じゃないつもりなんだが‥
構造体はポインタで扱うべきもので，実体をコピーするのはなるたけ避けたいね
C++ では　void f(const C &arg) って，可能ならば const と参照の縛りをできるだけいれるよね，余計なコンストラクタを走らせないために

K&R1 は教育的な配慮に優れた実装だと思うんだ‥

**デフォルトの名無しさん** · 2017/04/02(日) 14:13:23.32

SOCKADDRが一言

◆QZaw55cn4c · 2017/04/02(日) 17:06:04.34

>>553
手元のソースをみると
clientSocket = accept(serverSocket, (LPSOCKADDR)&clientSockAddr, &addrLen);
とか
if (connect(destSocket, (LPSOCKADDR)&destSockAddr, sizeof(destSockAddr)) == SOCKET_ERROR) {
とか，構造体のポインタを渡しているね
由緒正しき Berkeley socket では，構造体をコピーするなんて教育的ではない書き方は許していないよ

**デフォルトの名無しさん** · 2017/04/02(日) 17:31:42.61

つまり>>553はK&Rおじさんにも劣る存在

**デフォルトの名無しさん** · 2017/04/02(日) 18:21:59.15

K&R1 おじさん，と
K&R2 おじさん
は区別したまえ，若造

**デフォルトの名無しさん** · 2017/04/02(日) 19:26:19.28

どっちも時代遅れの老人でしょ
K&Rなんて昔存在したという噂でしか知らない
万葉集とか古事記みたいなもの？

**デフォルトの名無しさん** · 2017/04/02(日) 20:22:15.39

func1(a, b)
　　int a;
　　long b;
{
}

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

> &destSockAddr,
> sizeof(destSockAddr)

由緒正しきBerkeley Socketも昔はこういう書き方したな
コイツは進歩が20年前で止まってるのか
今この書き方する奴は死刑でいい

**デフォルトの名無しさん** · 2017/04/02(日) 23:11:21.74

便乗で申し訳ないが、値渡し、参照渡し、ポインタ渡し、の話で
ポインタ渡しといっても、C++の場合はナマポは嫌われて、スマポを使いましょう
ってことになっていると思うんだけど、これはどう渡すのが普通？
スマポを値渡しするのか、参照渡しするのか
はたまたスマポを剥いで、ナマポで渡すのか
個人的に統一しきれないっつーか
所有権やら生存期間やらリソース管理やらが関係してくるなら
スマポで渡すI/Fじゃないとまずいだろうし
逆にそういうのが関係ないなら、スタック上の自動変数を渡したいとかも考えると
ナマポで渡すI/Fのほうが都合が良かったり、どーなんだろ
一度ナマポに戻すと二度とshared_ptrに変換できないって制限がね
言語側でなんとかせずに、ライブラリで解決しようとした弊害かな
ちょうどよいバカよけなんかも知らんが

**デフォルトの名無しさん** · 2017/04/02(日) 23:13:25.39

参照一択

**デフォルトの名無しさん** · 2017/04/02(日) 23:24:57.32

>スマポを値渡しするのか、参照渡しするのか
>はたまたスマポを剥いで、ナマポで渡すのか
スマポの値渡しと生保は使い分けが必要だが、スマポの参照渡しだけはアホの所業。
とスコットメイヤーが言っていたような

**デフォルトの名無しさん** · 2017/04/02(日) 23:43:46.05

>>560
shared_ptrはその名の通り所有権を共有する場合のみ使うもの
で、そういう場合は値渡しでもいいよ。中でmoveすれば
所有権を共有しないなら、nullptrを許容するならナマポそうじゃないなら参照
あんまり無いけど所有権を渡した先に移動させるならunique_ptr

**デフォルトの名無しさん** · 2017/04/02(日) 23:58:32.77

所有権渡したくないならunique_ptrで持っている呼び出し元がgetでナマポを渡すかdereferenceして参照渡すかだな

**デフォルトの名無しさん** · 2017/04/03(月) 00:00:55.90

ナマポで渡してもshared_ptrを再取得する方法はあるだろ
まあ前提がいるが

**デフォルトの名無しさん** · 2017/04/03(月) 00:34:15.74

うむむ、なるほど
ポインタらしき物にこれだけ多くの種類があってI/Fで縛ってしまうのは
一見面倒そうで、最近の流行というか静的型言語的には
型安全と見るべきなんだろうかしら
安易なGC言語より良いかもしれん、単純に情報量は多い
また一つC++が好きになった
うっかり循環参照したら怖いとか
コンパイラのバージョン違いでSTLやmallocの実装に
差異が有ることを前提に組まなきゃならないとか、玉にキズもあるけれど

◆QZaw55cn4c · 2017/04/03(月) 06:55:53.48

>>559
C/C++ で生書きするなら今もおなじだよ

**デフォルトの名無しさん** · 2017/04/03(月) 07:40:50.83

型に対してじゃないのに括弧つけるの？

**デフォルトの名無しさん** · 2017/04/03(月) 08:48:24.23

そこはインデントや{}の位置が人によって違ったりするアレだろ

◆QZaw55cn4c · 2017/04/03(月) 19:32:41.84

>>557
K&R2 は C89 準拠だよ‥C89 が時代遅れだというけれども，いろんなことは C89 のときからやってきていて，C99 で追認された，というだけだよ

**デフォルトの名無しさん** · 2017/04/03(月) 20:29:22.04

>>570
あいわかった
お前は今後「K&R1おじさん」(略して「K&Rおじさん」)な
どうせ定数defineや関数マクロまみれなんでそ

◆QZaw55cn4c · 2017/04/03(月) 21:53:46.44

>>571
略しないほうがいいよ‥
K&R2 おじさん＝C89 はまっとうなCer, K&R1 おじさんは時代遅れ，この二つは全然違うよ

**571** · 2017/04/03(月) 22:02:42.26

>>572
つまりお主(K&R1おじさん)は
K&R2おじさんが自分と一緒くたにされることを
申し訳ないと感じてるのだな！？

(>543)
> それ，K&R1 から堕落したよね

(>550)
> サイズが無限にのびるかもしれないものをコピーするなんてだめだよ

(>552)
> K&R1 は教育的な配慮に優れた実装だと思うんだ‥

**デフォルトの名無しさん** · 2017/04/03(月) 22:14:40.64

voidを返す関数には感心したな

生成されるコードは大して変わらないかも知れんけど

**デフォルトの名無しさん** · 2017/04/04(火) 01:00:11.73

関数プロトタイプを除けば、K&R1で十分かもしれないな
voidを返す関数? int でいいんだよ

**デフォルトの名無しさん** · 2017/04/04(火) 15:11:19.04

戻り値をどうするかはcallerの勝手だからな

◆QZaw55cn4c · 2017/04/04(火) 19:47:09.26

>>575
確かに関数プロトタイプは有用だね，これがないと結構ミスるかもしれないね
void を返す関数？int でいいんだよ

**デフォルトの名無しさん** · 2017/04/04(火) 20:27:50.00

int funca01(...);
int funca02(...);
int funca03(...);

**デフォルトの名無しさん** · 2017/04/04(火) 23:08:17.42

老人は意味とか概念を理解しようとしないから嫌
「bool? intでいいだろ」
「forループ？本質はgotoと同じ」
「enumなんてintでいいだろ」
ホントもう存在が害悪

**デフォルトの名無しさん** · 2017/04/05(水) 00:33:27.19

>>550
何が「昔はね」だよｗ
Qちゃんその当事Cなんて触ってなかったろ？
アンタ最近（たかだか数年？）プログラミングで遊び出しただけの人でしょ

**デフォルトの名無しさん** · 2017/04/05(水) 02:38:54.31

>>579
「static const intなんてenumでいいだろ」
こうですね

**デフォルトの名無しさん** · 2017/04/05(水) 04:30:52.52

奴らは#defineだ

**デフォルトの名無しさん** · 2017/04/05(水) 08:14:48.69

マジックナンバー直書きでしょ

**デフォルトの名無しさん** · 2017/04/05(水) 08:19:06.54

K&Rだと～
c89だと～
過去の異物がまだ現役だと信じて話してけるのがうざい

**デフォルトの名無しさん** · 2017/04/05(水) 09:06:55.68

それ言うならC自体が。
いまさらC++と互換性のないCを選ぶ理由がない。

**デフォルトの名無しさん** · 2017/04/05(水) 10:35:04.67

>>585
C++と互換性のないCってどんなの？

**デフォルトの名無しさん** · 2017/04/05(水) 19:40:11.14

ベターC的な所全般だな
範囲forとか2進リテラルとかスレッドライブラリとか参照とか
Cにはnullptrすら無いし細かいところでは条件演算子が右辺値とか
ようするに別言語
オーバーロードに対するtgmathとかもう糞の臭いしかしない

◆QZaw55cn4c · 2017/04/05(水) 19:44:22.68

>>585
たしかに，C99 の数々の改悪は C++ との互換性を考慮していない悪手の連発，ほとほと困ってしまう

>>584
C++ とほどほどの交換性のある C89 が最良だよ

◆QZaw55cn4c · 2017/04/05(水) 19:46:32.21

>>580
「K&R1 は構造体は基本的にポインタでしか扱えない」制限なんて，使ってもいなかったらわからないだろう？

**580** · 2017/04/05(水) 20:02:06.85

え？Qちゃんてサンデープログラマじゃないの？
職業プログラマでもないし
プログラミング暦せいぜい10年未満だと思ってた

◆QZaw55cn4c · 2017/04/05(水) 20:07:28.87

>>579
＞「forループ？本質はgotoと同じ」
そんなことはいわないとおもうよ，K&R1 の時代から for はあるしね
きがつくと for(;;) { } と break だけで書いてたりするのはあるかもね，while 使いにくいとか，do while なんかマクロの中でしかつかわない，とかさ

enum?
#define でいいとおもうよ‥

◆QZaw55cn4c · 2017/04/05(水) 20:08:45.69

>>590
サンデーだよ
でも長いサンデーなんだ‥

**デフォルトの名無しさん** · 2017/04/05(水) 21:26:45.75

ニートじゃ分からんことも多いのでは

◆QZaw55cn4c · 2017/04/05(水) 22:05:58.72

たしかにそのとおりだが，K&R2 = C89 こそ必要十分，ということはわかるよ

**デフォルトの名無しさん** · 2017/04/05(水) 22:10:26.67

そういうこと言ってるとそのうち技術についていけなくなるよ

**デフォルトの名無しさん** · 2017/04/05(水) 22:22:23.34

C99やC11って、新しい技術とかそういうんじゃないじゃん。

**デフォルトの名無しさん** · 2017/04/05(水) 22:40:44.77

規格

**デフォルトの名無しさん** · 2017/04/06(木) 10:47:58.81

企画ものは好きじゃないからな

**デフォルトの名無しさん** · 2017/04/06(木) 16:35:26.64

ころころ変わるんだから最新もクソも無いわけだが

pakomamaはガチらしいな

◆QZaw55cn4c · 2017/04/06(木) 20:02:44.92

>>595
別についていけなくてもいいよ，その殆どは自分には要らないものだから
それはそうと，これは！とおもう新しい技術って最近はどんなものがあるの？

**デフォルトの名無しさん** · 2017/04/06(木) 20:16:13.94

まず、C#のasyncだのawaitだのはC++にも欲しいところ
というか、取り入れる方向で議論していたような
どのみちQZには使いこなせない機能だから知らなくてもよいけど
知れば発狂することだろう

◆QZaw55cn4c · 2017/04/06(木) 20:21:15.74

まずC#をやらないとね‥でも Java に走ってしまった‥Java に同じようなのはないの？

**デフォルトの名無しさん** · 2017/04/06(木) 22:33:11.66

>>601
coroutineの事なら根本的な所が違うから勘違いしないようにな

**デフォルトの名無しさん** · 2017/04/07(金) 20:08:34.35

馬鹿の特長：目的／機能の違いと実現するための実装の違いの区別がつかない

**デフォルトの名無しさん** · 2017/04/08(土) 12:56:12.29

C#のawaitはGUIのスレッドを意識出来るので有用だが
C++のawaitはただ裏で走らせるだけの機能的には劣化版な気がしてならない

**デフォルトの名無しさん** · 2017/04/08(土) 13:14:20.88

>>605
c++で標準的なgui使うとすると何になるの？

**デフォルトの名無しさん** · 2017/04/08(土) 13:49:52.49

>>605
そりゃ C++ と C# のポリシーの違いだろ

**デフォルトの名無しさん** · 2017/04/09(日) 22:32:13.26

C使ってる人で変数宣言を途中に書く人いる？
やめた方がいい？

**デフォルトの名無しさん** · 2017/04/09(日) 23:00:21.55

むしろそうして欲しいかな
勿論ブロックスコープは意識してる前提で

**デフォルトの名無しさん** · 2017/04/14(金) 09:11:56.63

初期のCはブロック先頭しかコンパイルが通らない
コンストラクタも型推論もないのに途中に置くメリットが少ない
初期化漏れというモンスターの隠れ家になりやすい

**デフォルトの名無しさん** · 2017/04/14(金) 10:33:55.73

一応Cで書くときは関数ブロック直後に書いてるけど、それはそれでforで使うiとかを使い回す事になるから、C++みたくプログラマの良識を信じて(ブロックを意識して変数宣言すると信じて)、好きな場所で宣言出来た方が正解なんだろうな。

**デフォルトの名無しさん** · 2017/04/14(金) 10:35:17.63

出来た方がと言うか、今は出来るんだから、した方が正解に訂正。

**デフォルトの名無しさん** · 2017/04/14(金) 12:59:25.70

出来ることと乱用していいことはイコールでない
一瞬だけの一時変数を直前で宣言するのならありだけど
関数全体で使うなら頭で宣言すべきだし、そもそも中間で宣言するような長い関数は
短くなるよう見直すべき

**デフォルトの名無しさん** · 2017/04/14(金) 13:04:43.28

魔法の言葉
ケースバイケース

**デフォルトの名無しさん** · 2017/04/14(金) 19:32:18.15

スコープは可能な限り狭く

おじいちゃんの遺言

**デフォルトの名無しさん** · 2017/04/14(金) 19:38:55.23

初期化漏れはブロック先頭でしかかけない方が起こしやすいだろう

長い範囲で使う変数でも、変数ははじめて使うところで宣言した方が分かりやすいしバグも減る。

**デフォルトの名無しさん** · 2017/04/14(金) 20:04:35.98

初期化忘れ防止はどこで宣言したってあんま関係ないというか
大差ないだろうな実際
必要になった時点で宣言するってことでよいと思う
さもないと上手くいかない場合もあるし
いちいち使いもしない「ダミーの値」で初期化するのは馬鹿らしいし
余計にバグりそう

**デフォルトの名無しさん** · 2017/04/14(金) 20:13:31.25

auto result = func();
まぁこういう風にしたいわな

int result = 0;
・・・
result = func();
↑その「0」って初期化値は何なんですか、何を意味した「0」なんですか
では0の代わりに1で初期化したらどうなるんですか
この初期化値の意味は一体何なんですか
ってことになるしな
結局「0」とか「1」とかの初期化値はダミーでありプログラム上なんの意味もなかったりするし
逆に初期化値に意味がある場合もあるから余計にややこしくなる
bool flag = false;
if( func() )
{
　　flag = true;
}
これらを区別したいから必要になった個所で宣言に一票

**デフォルトの名無しさん** · 2017/04/14(金) 21:45:43.28

老人の「昔は良かった」を見て顔真っ赤にして論破しようとマジレス
みっともない

**デフォルトの名無しさん** · 2017/04/14(金) 23:47:26.34

必要なものを必要なときに宣言する
それだけの話

**デフォルトの名無しさん** · 2017/04/15(土) 09:16:54.52

>>618
>この初期化値の意味は一体何なんですかってことになるしな

勝手に無駄な初期化しておいてどれだけ頭が悪いんだぜ
ぜんぜややこしくない
お前が馬鹿なだけ

**デフォルトの名無しさん** · 2017/04/15(土) 09:26:22.77

もともとが
初期化忘れを防ぐためには関数の先頭で宣言～
への反論だったからだぜ

**デフォルトの名無しさん** · 2017/04/15(土) 09:41:32.48

>622
日本語もわからないチンパンは無理して
コンピュータ使わなくても
祖国でオナニーしてればいいんだぜ