・【訂正】上記で「func()」として切り出したと言っているが、
「__legacy()」に変更しました。すみません。
・上記のように改修した所、レビューを受けたときに怒られた。
・まず、「この__legacy()の中身ちゃんと読んだ? 把握して書いている?」
という指摘。
・そして「ここメソッド内だけどどうして関数定義しているの?
他の人そんなことしてないでしょ。どうして変わったことするの?」
という指摘。
・まず1つめの指摘に対して異議がある。
ブラックボックス化しなければ大規模なコードは書けないんじゃない?
ということが言いたい。ライブラリとかフレームワークの機能も全部
先に中身をよくよく読んでおかなければ使ってはいけないことになるじゃん。
・2つの指摘に対して、
「周りの人がやっていること」が正しいとは思えない。
処理を切り出していないから1000行もある巨大なメソッドになっていて、
汚いコメントで汚しながら機能追加しまくっているんじゃないのかと。
ローカル変数を使いまくりの同じ処理を何回も実行するのに
他に良い手段があるとしたら皆さんに教えて頂きたい。
データ構造,アルゴリズム,デザインパターン総合スレ 3©2ch.net
■ このスレッドは過去ログ倉庫に格納されています
770デフォルトの名無しさん
2017/12/24(日) 12:31:59.59ID:croU9pw3■ このスレッドは過去ログ倉庫に格納されています
ニュース
- 「出禁にしてくれ」クラシックコンサートで起きた“フライングブラボーおじさん”に批判の声「余韻を味わいたいのに…」楽団広報の回答は [征夷大将軍★]
- 【アルゼンチン】ミレイ政権、2万1千人の公務員解雇計画を発表 [首都圏の虎★]
- 首都圏新築マンション1億円突破「とても手が出ない」 半期初 金利上昇で需要冷え込む恐れも [蚤の市★]
- 消費税減税後の支援金、高市首相「年収350万円上限で2000万人に支給」…衆院本会議で参考額答弁 [ぐれ★]
- 【巨人】坂本勇人を次期監督に! 伊原春樹氏が猛プッシュ「ちょっとやそっとのことでは動じないメンタルを持っている」 [鉄チーズ烏★]
- 【芸能】人気芸人、妻への「どん兵衛作って」発言にホラン千秋ら「モラハラ夫」批判… 共演者総スカンに本人驚愕 [冬月記者★]
- チャッピーです。すべてをお話しします。NATOとロシアは開戦します [784319933]
- チャリガキ、パルクール失敗wwwwwwwwwwwwwwwwwww [398059782]
- 今月は本当に売れないね😢高市さん!助けて😢 [457294144]
- 【動画】メローニ首相、ミラノ🇮🇹暴動にブチ切れ緊急声明 [668024367]
- 若者たち「働くだけでは豊かになれない」と生活費を切り詰めてNISA投資😲 [118990258]
- 【高市悲報】ネトウヨのせいで日本のイメージが悪化 [419865925]