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

2017/12/14(木) 17:04:38.91

メンバの参照を返したらprivateにしている意味が完全に失われるぞ

◆QZaw55cn4c (ﾜｯﾁｮｲ b260-4mDj) · 2017/12/14(木) 17:24:38.06

>>547
上流が const を渡してきたら、下流は逆らえない、ってことでは？

2017/12/14(木) 19:25:05.92

自作クラスのメンバ関数にconst全く付けてないタコが聞きかじりで中途半端に付け始めて
「メソッド呼べない！ムキー！このconstってやつ使えねー」ってパターンが大半だと思ってる

2017/12/14(木) 19:27:31.22

const地獄になるってことはそのコードが間違ってるって事だからな
全部直せ

2017/12/14(木) 20:46:09.64

>>547
本気で言ってるなら、それは経験が足りない

2017/12/14(木) 20:54:50.42

副作用の少ないきれいな設計ならconstだらけになるはずだが

2017/12/14(木) 20:59:02.63

>>553
これ

2017/12/14(木) 21:01:34.68

No.1
1 2 3 4 5
2 3 4 5 6
No.2
1 2 3 4 5
2 2 2 2 2
3 3 3 3 3
No.3
3 4 5 6 7
：
：
No.n
というtxtファイルがあり。No.〇の次の行に整数が記入されている（何行あるかわからない）
整数の左からx1[ i ][ k ]、y1[ i ][ k ]、x2[ i ][ k ]、y2[ i ][ k ]、A[ i ][ k ]の配列にそれぞれ書き込み
もし次の行整数があればi＋１をしまたそれぞれの配列に整数を書き込む
もし次の行に整数がなければ（No.の行）k+1を次の行にいき整数をそれぞれの配列に書き込む。
これをNo.nまで繰り返す。

このプログラムをfopenを使ってプログラムどなたかつくってくれませんかー？

2017/12/14(木) 21:33:16.59

宿題くらい自力でこなせ脳筋

2017/12/14(木) 21:45:38.41

ヒント。
いずあるふぁっていう関数どっかにあるから、先頭にアルファベットがあったらスキップや。
それと宿題スレって落ちたの？

◆QZaw55cn4c (ﾜｯﾁｮｲ b260-4mDj) · 2017/12/14(木) 21:52:05.84

>>555
宿題スレへ
http://mevius.2ch.net/test/read.cgi/tech/1456908378/

2017/12/15(金) 00:18:39.45

const指定って下流が対応してないのに上流からやり始めるのは愚だよね

2017/12/15(金) 09:14:14.38

>>559
どういう意味？

2017/12/15(金) 11:53:03.91

プロパティよりも、deleteするとコンパイルエラーになるポインタが欲しい。

2017/12/15(金) 11:55:37.18

>>561
std::shared_ptr<int> a{std::make_shared<int>()};
delete a; //C2440

2017/12/15(金) 12:49:17.55

>>561
あるじゃん

2017/12/15(金) 15:39:48.05

>>562
int*に代入できたっけ？

2017/12/15(金) 16:14:46.51

>>564
後出しはきたねえよ

2017/12/15(金) 18:44:29.20

スマポも「ポインタ」とは言うけどさぁ

deleteの文脈で言っているんだから、オブジェクトは想定外だわ。

2017/12/15(金) 18:52:20.56

そんなもん何の役にたつのか解らんわ

2017/12/16(土) 06:47:52.76

所有権のないスマポ（？）ならstd::observer_ptrがもうすぐ入るから待っとけ

2017/12/16(土) 06:55:04.26

ところでスマポのコンストラクタの書き方ってどっちがいいの？

class Foo
{
std::shared_ptr<Bar> bar;
Foo(): bar(new Bar(42, 100.5, "xxx")){}
Foo(): bar(std::make_shared<Bar>(42, 100.5, "xxx")){}
};

下の書き方されてるのよく見るけどこれって
「newがあるのにdeleteがない！１１１」って発狂するアホコードチェッカを黙らせる以外にメリットなんかあるの？

2017/12/16(土) 07:12:04.23

↓の備考よめ
https://cpprefjp.github.io/reference/memory/make_shared.html

2017/12/16(土) 07:21:37.50

あーなるほど
new隠しだけのための砂糖だと思ってましたありがとう

ということはunique_ptrの場合はメリットなしでいいんですかね

2017/12/16(土) 07:29:56.22

俺もmake_uniqueはいらねーよなぁって思ってたけど使った方が例外安全なんだとよ

2017/12/16(土) 09:33:48.78

むしろmake_uniqueが11に無くてキレた

2017/12/16(土) 22:08:09.44

point cloud libraryというライブラリを読んでいます。

kdtreeクラスはsearchクラスをパブリック継承しており、
kdtreeクラスの冒頭で、searchクラスのデータメンバやメンバ関数の一部をusingしています。

これはどういう意味があってやっているのか教えて頂けないでしょうか。

2017/12/17(日) 10:02:55.57

IO2D楽しみだなー。
ウニファイドコールシンタックス復活しないかなぁ。

2017/12/17(日) 18:17:59.07

質問です。

#include "classB.h"
class A{
private:
B* b;
public:
void set(B* b){this->b=b;}
B* get(){return b;}
};

クラスAのメンバーにクラスBのポインタとアクセサがあります。
しかし、set()を使わずに、get()を使ってメンバーにアクセスできてしまいます。
例：a->get()=b;

ポインタではなく、複製したインスタンスを返すことも考えたのですが
クラスBのサイズが非常に大きいため、
可能であれば複製はしたくありません。

どうかご助力をお願い致します。

2017/12/17(日) 18:32:55.64

const B *get() const {略}

2017/12/17(日) 18:33:44.11

get()を消す

2017/12/17(日) 18:37:56.02

B* b = nullptr;
B* get(){ assert(b != nullptr); return b;}

こうでもしとけばいいよ

2017/12/17(日) 18:39:29.53

>>576
const B* get() { return b; }

あとこうでは
*(a->get()) = b;

2017/12/17(日) 18:41:17.67

エスパーだがポインタみたいな動作させたいなら operatorｰ>() を定義すりゃいいけど上の説明だとなにがやりたいのか具体的にわからないね

2017/12/17(日) 18:48:28.66

>>578に賛成

2017/12/17(日) 18:56:03.60

>>577-582
ありがとうございます！
なるほどconst、普段しっかり使っていないツケが来ましたorz
ただ、そのままだと代入できなかったので、勉強してきます。
ありがとうございましたm( _ _)m
>>580
すみません、ご指摘の通りです。ありがとうございます！

2017/12/17(日) 19:23:44.84

Bがなんだかわからないけどsetがものすごくまずいように見える

2017/12/17(日) 19:40:09.05

[][] [] [][][][] [][] [] [][][][][
[ ][] [

2017/12/17(日) 20:35:52.39

gtkmmググっても英語含めて１７ページしかないんだわ(´・ω・`)

2017/12/17(日) 20:39:57.73

オブジェクトの内部データへの変更を許すフリーパスを外に与えるようなsetterやgetterはNG、

しかしあまり杓子定規にそれを言い立てるとファイルハンドルとかもgetterで返して良いかどうかﾋﾞﾐｮｰに…
ファイルはオブジェクトの内部データではないお客様という解釈で乗り切る？

2017/12/17(日) 22:46:35.33

いいわけないだろ
教条主義にとらわれていると
そういうことがわからなくなる

2017/12/17(日) 23:31:57.93

フレンドクラスに見せるようにする方が例外扱いが明記される分だけマシ

2017/12/17(日) 23:42:11.90

フレンドはカプセル化を推進する良い子

2017/12/17(日) 23:55:34.47

クラスオブジェクトソリッド

◆QZaw55cn4c (ﾜｯﾁｮｲ b260-E7Ol) · 2017/12/18(月) 00:10:07.85

>>295 >>292
accelerated c++ をぼちぼちやっています。
>>295 にもバグがあり、正しくは
https://ideone.com/sK4aJ2

一行を最後まで読んでしまったのなら、即座にループを抜けないといけない
なんだか最近 while() の使い道がなくなってきたように思えるのです…いつも for(::) { break; } ばかり書くようになってしまった…

2017/12/18(月) 00:12:01.54

停止性を担保するのは良いことだと思うが。
結局無限ループからのブレークしか使ってないなら頭整理してWHILEに置き換えるようにするなぁ。

◆QZaw55cn4c (ﾜｯﾁｮｲ b260-E7Ol) · 2017/12/18(月) 00:20:14.98

>>593
停止性の確認を行った後、余計なことを一切してはいけない、という制約が加わると
途中判定の break 脱出以外に手がない、と考えています
あまり思い出したくないんですが、DOS/Windows の findfirst/findnext なんかも似た類かと

◆QZaw55cn4c (ﾜｯﾁｮｲ b260-E7Ol) · 2017/12/18(月) 00:25:32.54

>>593
ああ、こういうのはアリでしたね
コメントありがとうございます。
https://ideone.com/9r8rZe

2017/12/18(月) 02:19:29.29

アプリケーション上で確保されたヒープ領域を追跡したいです。
operator new/delete と allocator を弄ればできると思い調査していたころ
「プログラミング言語C＋＋第４版」P997 に
「標準ライブラリのすべてのコンテナは、newによってメモリを確保して、deleteによって解放するデフォルトアロケータを（デフォルトで）もっている」
と書かれていました。つまり、new をグローバルでオーバーロードすれば追跡可能です。
実際、手元のコンパイラでは std::vector のメモリ確保と解放を追跡できました。
ここまでの内容は規格で保証されている事柄ですか？
実際のアプリケーションに組込むことを考えています。

2017/12/18(月) 02:36:31.34

メモリの確保形式はさすがに実装依存でしょ。
最近リーヌスがブチ切れたらしいけど。
メモリーオーダーコンシウムとかの話は今やってるところ。

2017/12/18(月) 03:43:10.43

>>596
>アプリケーション上で確保されたヒープ領域を追跡したいです。
つgperftools

2017/12/18(月) 07:43:59.76

>>596
std::allocator<T> (The default allocator) についての [allocator.members] で規定されてる。
> Remarks: the storage is obtained by calling ::operator new, but it is unspecified when or
> how often this function is called.

標準コンテナにアロケータを指定しなければ std::allocator が使われるから、それでいける。
「実際のアプリケーション」に変なものが混ざってなければね。

2017/12/18(月) 07:50:02.22

アロケータ自作してスパイアロケータとか作ればそれっぽいことはできるね。
標準コンテナに限っては。

2017/12/18(月) 09:32:57.28

>>576
ポインタを返さずに、ハンドルクラスを定義してそのオブジェクトを返す。

2017/12/18(月) 12:58:24.39

>>596
グローバルnew/deleteのオーバーロード自体は、規格で許されてるので合法。

ただ、注意深くやらないと吹っ飛ぶ。
とある処理系の標準ライブラリで、ユーザー回避不可能な不具合に当たったこともあった。

2017/12/18(月) 22:55:31.76

596です
規格での確認と該当箇所のコーディングを確認しました
ありがとうございました！

2017/12/19(火) 13:02:14.85

うむ、ご苦労。

2017/12/20(水) 13:42:16.45

sleep_untilで「何秒後」ではなく「何時に」という絶対時刻を指定する方法なんだけどさ
mktimeとfrom_time_tを使う方法しかないかね？　なんかダサくてやなんだけど

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd32-wvVH) · 2017/12/20(水) 21:12:15.08

>>605
標準では、無い。

2017/12/20(水) 22:19:15.39

IDE：Visual Studio Express 2015 for Windows Desktop
浮動小数点モデル設定：Fast Math
コンパイルモード：Release

wWinMain関数内で

wchar_t str[ 16 ];

DirectX::XMMatrixOrthographicLH( 1.0f, 1.0f, 0.0f, 1.0f );

for( int i = 0;; i++ )
{
　float f = i;

　if( f > 0.0f ) break;

　swprintf_s( str, L"%f", f );
}

と書くと、swprintf_sの結果が、期待される0.0ではなく、-0.0になります。
swprintf_sの行と上のif文の行を入れ替えると、0.0になります。

XMMatrixOrthographicLHがなかったり、Debugモードだったり、
Fast MathではなくデフォルトのPreciseだったりすると、
行の入れ替え関係なく0.0。

-0.0になるときは、何が始ま･･･起こってるんです？

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd32-wvVH) · 2017/12/20(水) 23:03:24.90

>>607
アセンブリーを確認せよ。

2017/12/20(水) 23:17:41.47

コンパイラ固有のことなんか聞かれても誰も分からないと思うぞ

2017/12/21(木) 00:36:33.65

>>607
floatの符号部が0だったり1だったりしてるのかと
最適化でswprintfだけが残って、fの値は+0.0f以下のゼロということで-0.0にしてくれてるのかなあ

**607** (ﾜｯﾁｮｲ 0323-of2o) · 2017/12/21(木) 01:46:36.50

>>608
やっぱそれですよね（汗）

>>609
やはりそうですか･･･。
詳しい方なら、何が起こっているかだいたい読めたりしないかなと期待しましたｗ

>>610
たしかに、符号ビットがコロコロしてそうな感じですよね。
ちなみにif文で比較する定数は1.0fとか2.0fでも結果は同じになりますので、
swprintf_sだけが残るためではないですね。

2017/12/21(木) 08:39:02.54

>>606
thx

2017/12/21(木) 14:03:31.02

Gtkmmは4.０も出たのに、詳しい解説ページは２．４のママ・・・(´・ω・`)

でも俺はAPI見ながら頑張るぜ！

2017/12/21(木) 18:40:48.26

今だとC++でGUIやるならQtがデファクトだと思うからそっちの方がいいんじゃない？
GTK使えと命令されてる立場ならすまん

2017/12/21(木) 18:46:54.04

Qtとかでかいしライセンスは高いし謎の独自拡張を使わされるわでかなり使いにくいんだが
そのくせ特別使いやすいわけではない

2017/12/21(木) 18:49:02.51

でもネットや本の情報は多そうじゃない？
まあ用途も聞かずに頭ごなしに勧めるもんではなかったなすまんかった

2017/12/21(木) 19:13:54.15

俺はwxWidgets派

2017/12/21(木) 20:31:35.72

gtk3一択！

2017/12/22(金) 06:33:38.27

io2dまだかー。
ウニファイドコールシンタックス復活せよ～。

2017/12/22(金) 06:57:32.16

基本的な動作について知りたい
msgrcvは、msgsndされた瞬間にキューがたまるから、キューが追加されたと同時に待機状態をやめて動き出す

これであってる？？

あと、msgsndする先やmsgrcvで確認する先はmsggetで動的に確保された番号を知らないといけない
こうであってるかな？

2017/12/22(金) 10:05:15.55

知らんわその関数の実装者に聞いてくれよ

2017/12/22(金) 23:15:32.80

システムコールの話ならLinux板にでも行け
なんかのライブラリの話なら知ったこっちゃないわ

2017/12/23(土) 00:52:53.65

>>620
TRON系のOSかな？

2017/12/23(土) 09:50:06.14

これ実行すると0と2が表示されるので一応は問題ないですか？汚いとは思うけど。

int main(void)
{
vector<int> vec;
vec.push_back(0);
vec.push_back(1);
vec.push_back(2);

for(vector<int>::iterator ite = vec.begin(); ite != vec.end();){
if(*ite == 1){
vec.erase(ite++);
printf("num: %d\n", *ite);
}else{
printf("num: %d\n", *ite);
++ite;
}
}

return 0;
}

2017/12/23(土) 10:06:15.95

>>624
>vec.erase(ite++);
ite = vec.erase(ite); にすべし
あと、次の行のprintfは削除で

2017/12/23(土) 10:16:01.07

おっと、質問は問題あるかどうかだった。
結論から言うと、問題はある。
0～2までではなく0～3までvectorに入れてから実行すれば、
まともに動作していないのは分かるはず。

2017/12/23(土) 11:00:02.97

>>626
やっぱこうしないとダメですね。
ite = vec.erase(ite);
どうもでした。

2017/12/23(土) 11:17:57.62

一応はこれでもいけるってことですかね？削除した時は次の要素になるという理解でいいですか？
for(vector<int>::iterator ite = vec.begin(); ite != vec.end();){
if(*ite == 1){
vec.erase(ite);
}else{
++ite;
}
}

2017/12/23(土) 11:20:42.68

eraceの戻り値が次のイテレータでは

2017/12/23(土) 11:40:33.04

これlistとvectorでも違いました。

listだと++しないと正常に動作せず
for(list<int>::iterator ite = lst.begin(); ite != lst.end();){
lst.erase(ite++);
}
vectorだと++すると上手く動作しない
for(vector<int>::iterator ite = vec.begin(); ite != vec.end();){
vec.erase(ite);
}

でもeraseの戻り値取得するのが一番いいですね。

2017/12/23(土) 11:45:41.24

std::vector::eraseのリファレンスに、こう書いてある。

https://cpprefjp.github.io/reference/vector/erase.html
削除された要素またはそれ以降の要素を指すイテレータや参照は無効になる。

これによると、vec.erase(ite); 実行後にはiteが無効になることになる。

あなたの環境だと、たまたまiteが削除した次の要素になったのかもしれないが、
それがすべての環境で成り立つ保証はない。

2017/12/23(土) 12:08:32.26

>>631
基本的にはvectorは無効になるということですかね。

2017/12/23(土) 12:13:38.89

>>631
そのinvalidationは既に取得してるiteratorに起こるやつでは?

2017/12/23(土) 12:16:58.11

>>631
なんか変な説明だね
vector<int> vec{0, 1, 2};
auto ite = vec.begin();
++ite;
vec.erase(ite);
--ite;
cout << *ite; //これでvec[0]にアクセスできそうに聞こえる
内容的にはこういうことなのに
vec.m_head = realloc(vec.m_head, 2 * sizeof(int));

2017/12/23(土) 12:24:57.38

listだとこれで動くと思っていいのかな
for(list<int>::iterator ite = lst.begin(); ite != lst.end();){
lst.erase(ite++);
}

2017/12/23(土) 13:02:30.52

>>634
eraseした時点で削除された要素のitrも無効だから--できないと思うけど、
別途確保しておいたitrは有効であるかのように読めるから、確かにおかしいね

>>635
どうやら合法。

2017/12/23(土) 14:18:11.55

そもそもこれ何してんの？
表示とともに要素を消したいならFILOなコンテナ使いなよ
std::stackみたいな

2017/12/23(土) 14:52:01.85

>>636
listはOKそうですか。
>>637
いやとくに意味はないです。ただvectorとlistで動きが違ったので気になりました。

2017/12/23(土) 15:08:47.05

アホか
eraseの戻り値が何のためにあると思ってるんだ

2017/12/23(土) 16:15:50.16

>>639
eraseの戻り値使うのが一番いいのは分かったんですが
単にvectorとlistで動作が違うのが気になっただけです。

2017/12/23(土) 16:23:29.49

vector だと要素の再配置が必要になるけど list はそうじゃないって話

2017/12/23(土) 17:51:30.26

https://gabekore.org/vscode-c-windows
↑のサイトを見ながらCの開発環境を作っているのですが④の当たりから分からなくなってしまいました‥‥

説明のユーザー設定というものがそもそも無かったです‥‥ただの設定を開いたらこの画面になったのですが、この右に書いたのは合ってるのでしょうか‥‥？

このあとの⑤からはほんとによく分からなくなってしまいました‥‥この程度が出来ないなら、プログラミングなんて出来ないですよね‥‥(..;)
自分が不甲斐ないです‥‥-_-

https://i.imgur.com/VH9Yt5X.jpg

2017/12/23(土) 18:33:23.06

2017/12/23(土) 19:22:40.52

削除する場合とそうでない場合の処理を分ければＯＫ
forでもwhileでもＯＫ　リストでもＯＫ
std::vector<int> vec{ 0, 1, 2 };
auto it = vec.begin();
while (it != vec.end()) { if (*it == 1) it = vec.erase(it); else it++; }
for (auto n : vec) { printf("vec %d\n", n); }

std::list<int> list{ 0, 1, 2 };
auto it2 = list.begin();
while (it2 != list.end()) { if (*it2 == 1) it2 = list.erase(it2); else it2++; }
for (auto n : list) { printf("list %d\n", n); }

2017/12/23(土) 20:28:00.26

>>640
根本的に勘違いしていると思うけどな。

×　> eraseの戻り値使うのが一番いいのは分かったんですが
○　eraseの戻り値を使わなければならない（そうしないと意味がない）

× > 単にvectorとlistで動作が違うのが気になっただけです。
○ 戻り値を使えば同じソースコードになるように設計されている

何の為にイテレータを使うか分かっておらず、
イテレータ使えば効率が上がると勘違いしているJava鹿程度の知能なんだろ。
Linusが文句言っているのもこういう連中に対してなんだが。

2017/12/23(土) 20:52:04.70

javaとかc#はイテレータで列挙してる最中に要素を削除できない
例外になる

2017/12/23(土) 20:59:27.12

C++ 使っててもLinusに馬鹿にされてるぞ

C原理主義者（OSつくってるひとたち）はC++を馬鹿にして
C++信者はCを馬鹿にする

滑稽だな