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

**デフォルトの名無しさん** · 2017/07/18(火) 07:07:27.20

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

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

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

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

**はちみつ餃子** ◆8X2XSCHEME · 2017/09/27(水) 21:51:01.70

>>716
いいえ。たとえば GCC は __builtin_offsetof を持っていて、 offsetof マクロはこれに展開されます。

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

だからoffsetofはコンパイラメーカーが用意したもので
自分たちが自分たちの製品用に用意したんだから動くのは当たり前だろ？
offsetofは動くが(そりゃコンパイラメーカーが用意したんだから当たり前)
だから何なんだ？

nullptrが好きみたいだから
https://cpplover.blogspot.jp/2015/12/blog-post_21.html
↑を読んでみると良い
それもじっくり読んだほうが良い
こういった不慮の事故があるから未定義動作は避けるべきというのは当たり前では?
↑のプログラムが未定義動作で最適化でコンパイラで削除されるのに
offsetofは最適化されずに残るってのは非常に興味深い微妙な部分なんだよ
やってることはヌルポインターの参照で同じことなんだがな
こんなのさじ加減なんで、何が起こるか分からない

**デフォルトの名無しさん** · 2017/09/27(水) 22:44:20.00

>>717
具体例は？

**デフォルトの名無しさん** · 2017/09/27(水) 23:48:58.08

具体例が無いと>>702-704,718の懸念は認められない。
などと思っているならこいつはとんだアホウだ

**デフォルトの名無しさん** · 2017/09/28(木) 06:23:37.76

具体例が出ないことは>>701の証拠となる

**デフォルトの名無しさん** · 2017/09/28(木) 06:28:56.63

つまり>>700の否定だ

**デフォルトの名無しさん** · 2017/09/28(木) 06:46:37.21

そもそも>>717は具体的な例だし>>719の考える具体例ってなに？

**デフォルトの名無しさん** · 2017/09/28(木) 07:37:45.60

元は>>652だな
それに続く>>658の１部だけを抜き出して>>664の書き込み
目的がこの時点でそれて、これ以降アドレスの減算の話になる

ダメな例ってのはアドレスの減算を行って実際にうまく動かない例
オフセット分を戻すことで有効アドレスで使うことになるので、>>701で実際に元に戻らない環境を多く示せばいい

言語規格上保証されてないことは>>670や>>677以外は知っている

**デフォルトの名無しさん** · 2017/09/28(木) 10:53:55.67

それとoffsetofが何の関係があるのか、と聞いているわけだが

**デフォルトの名無しさん** · 2017/09/28(木) 11:32:16.87

offsetofは未定義とされるコードが処理系のライブラリ内で使用されていた例。
「具体例」は単に実際に問題となる例を出せと言っているだけ。
両者は無関係だが>>705が>>715を理解する知能が無かったのが残念。

**デフォルトの名無しさん** · 2017/09/28(木) 11:35:32.70

offsetofと同様の自作マクロがたくさん使われてる
ということを知らないとは

ていうか、順番が逆だ

色々な所で使われてるから標準化された

**デフォルトの名無しさん** · 2017/09/28(木) 11:37:25.53

>>725
offsetofの中身を見れば関連がわかる

**デフォルトの名無しさん** · 2017/09/28(木) 11:46:41.60

きっと>728の使ってるコンパイラでのoffsetofの定義には我々の想像も付かない何かが書いてあるんだよ。
他の人の使ってるコンパイラでも同じことが書いてあるはずだと思ってる理由はわかんないんだけど。
>728はその「offsetofの中身」を明示して話すといいんじゃないかな。知らんけど。

**デフォルトの名無しさん** · 2017/09/28(木) 12:01:16.61

>>727
>色々な所で使われてるから標準化された
お前のパソコンに保存されてる自作マクロには興味ないけど、
offsetofは機能性を規格化しただけで実装は規格化されてないぞ

**デフォルトの名無しさん** · 2017/09/28(木) 12:44:17.37

0番地を構造体のアドレスにキャストして、メンバのアドレスに変換して、それをそのまま整数にキャストする

ってマクロ、昔は一般的に使われてたんだけど
知らないか

>>701の根拠のひとつ

いずれにしろ、
反例がひとつも出て来ないようじゃそんな根拠もいらないね

**デフォルトの名無しさん** · 2017/09/28(木) 12:45:47.97

>>700はトンチンカンで、
>>703も説得力ゼロ

ってことで

**デフォルトの名無しさん** · 2017/09/28(木) 14:59:31.87

ぬるぽ使った自作offsetofがすべての処理系で正常動作するとは限らない
って話じゃないのけ？

**デフォルトの名無しさん** · 2017/09/28(木) 16:36:34.69

ぬるぬるぽっぽっぽ

**デフォルトの名無しさん** · 2017/09/28(木) 18:14:43.67

>>726
offsetofはコンパイラベンダーが用意したのだから動くのは当たり前では
俺らからしたら、未定義動作、処理系依存、実装依存、であっても
コンパイラベンダーからしたら、仕様なのでは？
特にコンパイラベンダーは究極的にはコンパイラの仕様を変更することも
出来る立場なので

**デフォルトの名無しさん** · 2017/09/28(木) 18:44:25.04

>>733
ほとんどの環境でうまくいく
だから良く使われている

そうじゃないっていうなら
うまくいかない例を出して

っていう話

**デフォルトの名無しさん** · 2017/09/28(木) 18:45:14.20

>>735
壊れたレコードプレーヤーか？

**デフォルトの名無しさん** · 2017/09/28(木) 18:47:54.52

壊れたレコードでも何でもいいけど
説明求む

**デフォルトの名無しさん** · 2017/09/28(木) 18:52:01.14

日本語が通じない人に説明は不可能
おれは>>726じゃないし

**はちみつ餃子** ◆8X2XSCHEME · 2017/09/28(木) 18:52:29.72

なんでそんなに仕様に違反したいんだ？
仕様に違反すべきではないってことにそんなに根拠が必要か？

**デフォルトの名無しさん** · 2017/09/28(木) 18:52:32.31

とりあえず落ち着いてスレを読み返しな

**デフォルトの名無しさん** · 2017/09/28(木) 18:54:33.80

>>740
違反したいなんて思ってるのは>>700くらい

メリットがあれば違反することもある
簡単な代替手段があれば違反しない

**デフォルトの名無しさん** · 2017/09/28(木) 18:57:08.19

そもそも、どの環境でも正しく動く保証がある意味のあるコードなんてないから
どのコードもある程度環境を前提としてる

**デフォルトの名無しさん** · 2017/09/28(木) 19:09:22.17

そんな唐突な、そもそも論は、要らないだろ
程度問題だろ
C/C++の仕様的にはNGでも、コンパイラの仕様書にOKって書いてあればOKなんじゃね？
例えば int は 32bit 整数ですって書いてあれば、そういう製品仕様として納得するし
ただ、offsetofをもってしてOKと言えるのかどうか

**デフォルトの名無しさん** · 2017/09/28(木) 19:14:42.61

だからダメな例を出せって
>>699に対して反論したいなら

**デフォルトの名無しさん** · 2017/09/28(木) 19:16:02.07

コンパイラの仕様書に書いてないよ普通
ポインタのビット数が書いてあるだけで

**デフォルトの名無しさん** · 2017/09/28(木) 19:19:48.44

>だからダメな例を出せって
そしたら逆にコンパイラの仕様書でOKって書いてある例を出せ
って話になるでしょ
無ければグレーゾーンになるわけだ
まさに未定義

**デフォルトの名無しさん** · 2017/09/28(木) 19:30:52.46

日本語が通じない人が多いなあ

とおもったら同じ人か

**デフォルトの名無しさん** · 2017/09/28(木) 19:38:01.58

>>745
だから>>717で反例でてるだろ
C++で&演算子を定義されたら簡単に破綻する
https://stackoverflow.com/questions/400116/what-is-the-purpose-and-return-type-of-the-builtin-offsetof-operator/406941#406941

**デフォルトの名無しさん** · 2017/09/28(木) 19:41:37.10

またトンチンカンな例を
話の流れくらい理解してから書け

**デフォルトの名無しさん** · 2017/09/28(木) 19:50:55.33

そりゃ誰もお前の中だけの勝手な話の流れなど知ったことではないだろ

**デフォルトの名無しさん** · 2017/09/28(木) 20:20:35.35

>>750
トンチンカンとしか言えないなら黙ってろよw

**デフォルトの名無しさん** · 2017/09/28(木) 21:45:26.80

単純な話、ベンダー提供のマクロで言語仕様上保証されないような使い方がされていたとして、
そのマクロ自体の動作はベンダーが保証しても同じコードがそのマクロ外で保証されるわけじゃないよね。
実際には動くかもしらんけど。

**デフォルトの名無しさん** · 2017/09/28(木) 22:24:40.88

最近また片山老人が暴れてんの？

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/09/28(木) 23:35:41.44

昔は学生運動が一大イベントだった。鉄砲の音が聞こえたら、学生たちが一斉に走り回る。それは軍隊のように団結力のある集団だった。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/09/29(金) 00:24:14.89

って昔話でもさせるのか？

**はちみつ餃子** ◆8X2XSCHEME · 2017/09/29(金) 00:55:18.83

>>753
「こんな最適化するのか！」って思うようなピンポイントな最適化は有るから、
逆にピンポイントな例外だってありそうよね。

**デフォルトの名無しさん** · 2017/09/29(金) 06:49:45.56

>>753
マクロとそれを展開したソースで動きが異なるってこと？
さすがにそれは考えにくいと思うけど

**デフォルトの名無しさん** · 2017/09/29(金) 10:16:34.25

>>736
すべての環境で動作する保証が無い＝環境依存

**デフォルトの名無しさん** · 2017/09/29(金) 11:32:53.08

お前らが何を話しているのかわからん
相手に何を伝えたいのか
お前らは何を訴えたいのか
何も見えてこない
俺がないならもうやめろ

**デフォルトの名無しさん** · 2017/09/29(金) 12:45:07.45

俺の主張は>699

**デフォルトの名無しさん** · 2017/09/29(金) 12:46:04.43

主張内容に興味無いけど

**デフォルトの名無しさん** · 2017/09/29(金) 21:46:50.94

>>761
またおバカループかよ w

**デフォルトの名無しさん** · 2017/09/29(金) 22:31:02.23

>>761は>>671の『C/C++的』の意味が共有できているといつから思っていた

**デフォルトの名無しさん** · 2017/09/30(土) 01:55:47.47

ちょっと不可解な部分があったから質問させてくれ

string型のデータをサーバに送信したくてOSSの関数に渡しているんだが、
データが一部壊れてしまう
問題のコードは、

string str = "{id:abc, ... , }"
void* payload = (void*) str.c_str();
send(payload, str.length());

これをやると、先頭の5バイトがうまく入らない
いくら考えても全然理由がわからず、意味のないことだと思いつつも以下のコードを書いてみたら何故かうまくいった

string str = "{id:abc, ... , }"
char* tmp = (char*)malloc(str.length() * sizeof(char) + 1);
strcpy(tmp, str.c_str());
payload = (void*)tmp;
send(payload, str.length());

下のコードを使えばうまく動いてるから問題ないんだけど、上のコードがダメな理由がわからずすごく気持ち悪い。
何か上の例だとまずい部分があるのかな？

送信関数は以下
send(void* payload, size_t length);

環境はVC++ 2012
Windows 7 64bit

ちなみに、同じコードを別環境(gcc 4.4.7 Cent OS 32bit)で動かすと、両方うまくいった

**デフォルトの名無しさん** · 2017/09/30(土) 02:12:16.55

>>765
c_str()ってどんな関数？

**デフォルトの名無しさん** · 2017/09/30(土) 02:45:21.56

非同期やbufferedの関数を正しく使えてないんだろ
たしか以前にboost.asioかなんかの同じような質問をきいたことがあったような

**デフォルトの名無しさん** · 2017/09/30(土) 06:20:03.02

ASIOといえば
Audio Stream input output

**デフォルトの名無しさん** · 2017/09/30(土) 07:12:02.93

そのインターフェースで非同期なのに内部で実体のコピーをとってない場合とかだと
実際に送信完了するまで渡した引数の実体が生きてる必要があるね

**デフォルトの名無しさん** · 2017/09/30(土) 09:19:09.47

void *型を使う意図について

**765** · 2017/09/30(土) 10:38:13.12

みんな回答ありがとう。

>>766
string型のメソッドで、文字列をconst char*にするやつ

>>767,769
非同期送信にはなってるんだけど、sendの後に送信結果を受け取る関数を呼んでる。
そのあとでpayloadは解放されるようになってるから、送信完了までは実体は生きてると思う

>>770
OSSの関数がそう定義されてるから変換したんだ

{id:abc. ... , }
のうち先頭5文字の{id:aがおかしくなってるんだよね・・
Wiresharkでみると、バイトコード88ってなってたけどなんでそうなったのかが全然わかんない

**デフォルトの名無しさん** · 2017/09/30(土) 10:53:58.34

>sendの後に送信結果を受け取る関数を呼んでる。
>そのあとでpayloadは解放されるようになってる

ならそのsendのバグか他でのメモリ破壊だろ
そのコード自体には問題はない

**デフォルトの名無しさん** · 2017/09/30(土) 11:13:57.36

>>771
まず
> send(payload, str.length());
を実行する時にpayloadの指す先が正しいか切り分けしなよ

**デフォルトの名無しさん** · 2017/09/30(土) 11:28:17.88

問題点の切り分けは大事だな
元のソースを問題が再現する最小限のところまで切り詰めてみるとか
その過程で問題が起きなくなったら関係ないと思って切り詰めたところが原因だろうし

**デフォルトの名無しさん** · 2017/09/30(土) 13:47:25.00

どうおかしくなってるのか分からない
"{id:abc, ・・・"が"XXXXXbc, ・・・"となる(ascii 88 = 'X')ってことなのか

**デフォルトの名無しさん** · 2017/09/30(土) 23:34:44.40

すいません
メモ帳ソフトのTextTreeみたいに
タブを並べられて、そのタブごとに個別のツリーを格納するには
どうしたらいいんでしょうか？
ttp://www.vector.co.jp/magazine/softnews/080129/images/n08012931b.gif

なんでここで聞くかというと
TextTreeの開発者のコメントに
「wxWidgets」をいじっていたので、これを使って作ってみようと思った、というのがあって
そのwxWidgetsのHPに
wxWidgets is a C++ library～
と表記されていたからです。

普通のメモやテキストエディタは
ツリー＞タブ＞メモという構成ですが
Texttreeのような
タブ＞ツリー＞メモという構成が希望ということです。

またTextTreeは、リッチテキストが扱えるものの
文字が65535バイト以上入力できず、それ以上は入力しても消えてしまうバグを抱えていて
どうしてそのような不具合になるのかも知りたいのですが…

**デフォルトの名無しさん** · 2017/09/30(土) 23:53:50.61

>>771
> そのあとでpayloadは解放されるようになってるから、送信完了までは実体は生きてると思う

str.c_str() 渡した場合は str の寿命や変更で解放されることになる。
他のどこかで解放されるようになってるとしたら、そんなところに渡してはいけないと思う。

どこかで free されることになってるなら、 malloc したポインタを渡さないと
正しく動かないということで >>765 の結果で当然に思う。

「OSSの関数」というなら具体的な関数を明示すればもっと状況がわかるのに。

**デフォルトの名無しさん** · 2017/10/01(日) 00:34:39.70

>>776
wxWidgets固有のことは知らないけど
タブごとにツリーを用意して各タブに対応する子ウィンドウに配置するだけでしょ
もしくはツリーを一つだけ用意してタブ切り替えのたびに中身を書き換えればいいんじゃない

**デフォルトの名無しさん** · 2017/10/01(日) 02:46:34.57

>str.c_str() 渡した場合は str の寿命や変更で解放されることになる。

いやだから、payload==str　は送信結果を受けてから
開放してるって本人は言ってるのでは？

**デフォルトの名無しさん** · 2017/10/01(日) 08:09:58.98

明示的に解放しないstringの方はそこを意識してなかった可能性もあるんじゃね？
非同期ってことは>>765でsendした関数からは一旦抜けてるでしょ？

**デフォルトの名無しさん** · 2017/10/01(日) 09:03:15.51

>>776
1つの方法としては
タブが切り替えられる毎にイベント発生するから
タブAが押されたらダイアログAを表示してそれまでに表示されてたダイアログは非表示にする
ダイアログ毎にツリーとリッチエディットを用意したらいい

あとWin32APIスレの方がいいよ

**デフォルトの名無しさん** · 2017/10/01(日) 09:35:07.78

>string str = "{id:abc, ... , }"
>void* payload = (void*) str.c_str();

>>780はpayloadを明示的に解放すんのかよ

**デフォルトの名無しさん** · 2017/10/01(日) 09:52:09.27

してない可能性を挙げたのに「すんのかよ」って俺に言われてもなw

**デフォルトの名無しさん** · 2017/10/01(日) 10:26:50.12

>>780
>>771に
>> 非同期送信にはなってるんだけど、sendの後に送信結果を受け取る関数を呼んでる。
>> そのあとでpayloadは解放されるようになってるから、送信完了までは実体は生きてると思う
ってあるからその可能性は低いと思う

**デフォルトの名無しさん** · 2017/10/01(日) 10:32:30.15

vc2012だとC++11への準拠はどうなんだろ
c++11以降はc_strも実体を指す保証があるようだけど、
以前は関数内のstatic変数みたいな実装してた気がする・・

**デフォルトの名無しさん** · 2017/10/01(日) 11:40:53.20

>>784
初心者歓迎スレだし可能性は少なくない
GC言語で育ったならポインタをスマポのような何かと勘違いしてるかもしれない
さらにその文章から思い込みだけで検証していないのは明らかだし

**デフォルトの名無しさん** · 2017/10/01(日) 11:49:29.48

可能性を言い出したらなんでもあり
不毛なレスは止めようよ

**デフォルトの名無しさん** · 2017/10/03(火) 17:35:19.79

>>785
>関数内のstatic変数みたいな実装してた気が
んなわきゃない

**デフォルトの名無しさん** · 2017/10/03(火) 18:51:26.51

>>788
どういう実装だった？

**デフォルトの名無しさん** · 2017/10/03(火) 19:50:33.30

shared_ptrを戻すメンバ関数があるんですが
引数の条件に一致する物が無い場合、空というのが分かるshared_ptrを返したいんです。
どう書けばいいですか？

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/10/03(火) 19:52:37.35

ヌルポインター返してgetメソッドで確認。

**デフォルトの名無しさん** · 2017/10/03(火) 19:59:11.57

ありがとうございました！

**はちみつ餃子** ◆8X2XSCHEME · 2017/10/03(火) 22:45:38.84

std::optional を使った方がよくない？

**デフォルトの名無しさん** · 2017/10/04(水) 15:03:07.29

>>788
いちいち古いVC入れ直すのめんどいから覚えてるんなら書いてほしいんだが

あと>>765が解決したのかどうかも知りたい

**デフォルトの名無しさん** · 2017/10/04(水) 21:20:20.65

初心者的質問で申し訳ないですが、libファイルの検索仕方を教えて下さい。
includeファイルはgrepかければ見つかりますが、リンクするとき、静的ライブラリ、libファイルの検索はどうすればいいか分かりません。
とりあえず、怪しい奴を片っ端からリンクしますが、それもどうかと思います。
マニュアルに書いてある訳でもないので皆さまどうされてるのか気になります。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/10/04(水) 21:27:51.47

>>795
bgrep使え。リンク方法が書かれているマニュアルを買え。

**775** · 2017/10/05(木) 07:26:13.70

ありがとうございます。
ネットで調べるのがメインなので、つい、本はケチってしまいました
bgrepは入れられないので-aでなんとかしてみます

**デフォルトの名無しさん** · 2017/10/05(木) 21:03:39.22

>>794
string s1, s2;のc_str()が共用なわけあるかドアホ
VC++6の時からインスタンスのメモリ返すわ

**デフォルトの名無しさん** · 2017/10/05(木) 21:31:37.51

いちいちけなさんと会話も出来んのかゴミクズが
へー、じゃあ寿命は保証されてるわけか、そんなことはなかったはずだが
>>765の問題は寿命の問題だと思うが、ケチつけるなら正確に書いてくれる？
混乱を招くだけのイチャモンとか何の役にも立たんわ

**デフォルトの名無しさん** · 2017/10/05(木) 23:47:16.81

c_strは実態をさすけど変更するなって書いてなかったっけ？

**デフォルトの名無しさん** · 2017/10/06(金) 00:14:51.51

03以前は変更された後にc_str()を使うのも、そもそも内部で連続してる保証も無かったはず
今調べたけどdata(), c_str()の呼び出しでも変更される可能性があるらしい（11は大丈夫

**デフォルトの名無しさん** · 2017/10/07(土) 00:46:39.10

ポインターを覚えたけど、使いどころが思い付かない
誰か教えて……

**デフォルトの名無しさん** · 2017/10/07(土) 02:11:18.12

>>802
一個のサイズが10KBほどある構造体のソートにでも使えよ

**デフォルトの名無しさん** · 2017/10/07(土) 15:35:22.92

それポインタじゃなくて整数でよくない

**デフォルトの名無しさん** · 2017/10/07(土) 16:29:02.76

ポインタと整数じゃ概念が違う
整数をアドレスを指し示す用途で使いますよってのがポインタだし

**デフォルトの名無しさん** · 2017/10/07(土) 17:29:30.12

変数とポインターでメモリをずっと使うか取り除くかの違い？

**デフォルトの名無しさん** · 2017/10/07(土) 17:49:35.28

>>802
mallocの戻り値とかsprintfの引数とかで使えばええねん

**デフォルトの名無しさん** · 2017/10/07(土) 19:03:08.66

みなさんありがとうございます
規模の大きいデータで使うみたいですね
後は他の関数を飛び越えてアクセスしたい時とか

**デフォルトの名無しさん** · 2017/10/08(日) 11:15:35.68

>>805
配列のソートだったらアドレスよりインデックスの方が概念的に適切だろ？

**デフォルトの名無しさん** · 2017/10/10(火) 15:59:13.06

p==p_rgh + rho*gh;
とあるのですが意味がわかりません。
p=p_rgh+rho*gh;であるべきと思うのですが、==と=が使い分けされているので
タイプミスではなさそうです。どなたかよろしくお願いします。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/10/10(火) 16:10:57.64

>>810
「p==p_rgh」は、pとp_rghが等しいとき1 (true)になり、等しくないときゼロ (false)になる。

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/10/10(火) 16:14:41.98

>>810
演算子「==」より演算子「+」の方が結合が強いんだった。
「p==p_rgh + rho*gh」は、pとp_rgh + rho*ghが等しいときtrueになる。

**デフォルトの名無しさん** · 2017/10/10(火) 19:02:14.11

>>810
問題のその式が、if文のカッコの中ではなく、普通に式文としてセミコロンで終わる1行として書かれているなら、ただの誤記の可能性が高いと思うよ。

**デフォルトの名無しさん** · 2017/10/10(火) 19:04:42.99

>>812
ご回答ありがとうございます。が、やはりわかりません。
たとえば、
if (p==p_rgh + rho*gh)
{
…
}
や、q=p==p_rgh + rho*gh;とかならわかるのですが、

…
p==p_rgh + rho*gh;
…
と独立しているのでどんな処理をしているのか・・・

**デフォルトの名無しさん** · 2017/10/10(火) 19:11:07.30

>>814
その本をさらせ、話はそれからだ

**片山博文MZ** ◆T6xkBnTXz7B0 · 2017/10/10(火) 19:12:17.54

>>814
そのように独立していれば何もしない。何もしない文をわざわざ複雑に書くことは意味がないので、代入を間違って書かれたと考えられる。