プログラマの雑談部屋 ★65

**仕様書無しさん** · 2019/03/22(金) 22:42:02.62

※前スレ
プログラマの雑談部屋 ★62
https://medaka.5ch.net/test/read.cgi/prog/1551007254/
プログラマの雑談部屋 ★63
https://medaka.5ch.net/test/read.cgi/prog/1551744733/
プログラマの雑談部屋 ★64
https://medaka.5ch.net/test/read.cgi/prog/1552655641/

**仕様書無しさん** · 2019/03/24(日) 09:21:18.43

>>164
そしてだれも居なくなった。

**仕様書無しさん** · 2019/03/24(日) 09:22:32.46

>>165
うウェッブとドカタまちごうとるでぇあんちゃん。

**仕様書無しさん** · 2019/03/24(日) 09:24:53.81

コードレビューの仕方(自分の所属の場合)
・事前にやるべきことを済ませてレビューに望む
　(静的解析、過去トラブルの確認、各種規約の確認)
・上記結果をレビューにて報告
・事前確認の結果、気になった点(議論したい点)等をまず説明

**仕様書無しさん** · 2019/03/24(日) 09:29:31.02

>>168
確認して気になった点じゃなくて、これから気にする点を、コードを読む前にすべてリストアップしろ

**仕様書無しさん** · 2019/03/24(日) 09:33:44.27

ウェッブプログラマだけど要件定義からリリースまで全部やるよ
モダンなアーキテクチャでね

**仕様書無しさん** · 2019/03/24(日) 09:35:19.13

>>170
すごい。
ドカタはコードを写すだけ。ばかみたい。

**仕様書無しさん** · 2019/03/24(日) 09:37:28.94

>>170
その割には月収12万とかだよね

**仕様書無しさん** · 2019/03/24(日) 09:37:45.82

>>169
?
気にする点=「静的解析、過去トラブルの確認、各種規約の確認」と考えてる

**仕様書無しさん** · 2019/03/24(日) 09:40:09.44

>>173
それ以外はすべて無視するの？

**仕様書無しさん** · 2019/03/24(日) 09:41:24.66

コードレビューは手厚くやっても金にならないからやるとこ減ったけどね

**仕様書無しさん** · 2019/03/24(日) 09:42:46.95

たとえばほかに何があるんだ

**仕様書無しさん** · 2019/03/24(日) 09:42:57.80

>>174
設計書にクラス図が書いてある以上大きくは外れないし
ぶっちゃけどう書いても動くし動かなければ単体テストで弾けるのでどうでもいいと思う

**仕様書無しさん** · 2019/03/24(日) 09:53:04.99

わいはもっとシビアに殺伐とやればええと思うで。
こまけええ面倒くせえことは抜き。PMの好みに合わないコミットは即リジェクトや。
PMが気に入らんゆうならディスカッションして論破すりゃええ。無理ならさっさと転職や。
玉石混合が最悪まずい状態やで。、PMに合わん香具師は消えてもらって結構。

**仕様書無しさん** · 2019/03/24(日) 09:54:15.81

なんで「ウェッブ」ってわざわざ促音入れる奴がいるの？

**仕様書無しさん** · 2019/03/24(日) 10:00:24.67

>>177
クラス設計、クラス図を事前に全部用意するのは費用対効果が悪い
値オブジェクトまで真面目に全部作ったことある？ないんじゃないかな？

**仕様書無しさん** · 2019/03/24(日) 10:08:08.16

>>172
手当込みで月60ぐらい

**仕様書無しさん** · 2019/03/24(日) 10:09:42.20

よこせ

**仕様書無しさん** · 2019/03/24(日) 10:11:15.06

ウェッブとか言ってる奴って
ヴァーカとか書いてる在日さん？

**仕様書無しさん** · 2019/03/24(日) 10:15:36.36

いい加減敬語廃止しろよジャップランド
俺はくだらないところにリソース割きたくないんだよ

**仕様書無しさん** · 2019/03/24(日) 10:16:04.05

>>181
webアプリで？
夢みんなｗｗｗｗｗ

**仕様書無しさん** · 2019/03/24(日) 10:21:44.88

役立つコードレビュー　8つのヒント
https://postd.cc/8-tips-for-great-code-reviews/

こういうのとか読んだことないんだろうな。他にも似たようなことはネットでも雑誌でも書籍でも
書いてあるのはそれなりにあるっていうのに。

**仕様書無しさん** · 2019/03/24(日) 10:26:00.89

>>185
webって言っても決済系だけど

**仕様書無しさん** · 2019/03/24(日) 10:26:50.87

俺は敬語使えねー人だから敬語廃止して欲しいわ
ヴァカに頭下げるの嫌だわ

**仕様書無しさん** · 2019/03/24(日) 10:28:20.97

めんどくさいよな

**仕様書無しさん** · 2019/03/24(日) 10:31:21.19

>>186
>チームは、少しでも良いコードを作り出したいと思っており、パーフェクトという言葉は、形容詞ではなく動詞であることを理解している。明日、もっと良い方法が見つかるかもしれない、というオープンな心構えを保っている。

いいこと言うね
事前にチェックリスト用意してそれ以外は指摘するななどと言う人には一生理解できない感覚だろう

**仕様書無しさん** · 2019/03/24(日) 10:36:46.11

>>190
良いコードの定義がブレるのは危険
金を稼ぐことが目的なのか
採算度外視で解決するべき問題を抱えているのか？

これを決めずに動くのは少なくともプロじゃねぇ

**仕様書無しさん** · 2019/03/24(日) 10:37:34.90

>コンピュータが規則を決断、強制できるのなら、コンピュータにそれを任せましょう。
>タブかスペースかを議論するのは、人間の時間の生産的な使い方とは言えません。
>代わりに、規則はどうあるべきか合意を得るのに時間を投資しましょう。
>これはチームが「異議を唱え、コミットせよ」の原則を、低リスクのシナリオでどれだけ活かせるかが分かる機会でもあります。

チェックリストを求める人ってここが出来てないって気がしてきた
リストアップできるような機械的なチェックは自動化すればいい
でも低スキルの現場では自動化ができていないからチェックリストに時間を奪われてしまう
だから機械的な判断ができない問題のレビューを受け付ける余力がなくなってしまった

**仕様書無しさん** · 2019/03/24(日) 10:39:01.00

正義と正義をぶつけて殴り合いの喧嘩をさせて潰すのが目的

両者が疲弊したあとで

ピンポーン♪
始めまして○○コンサルティングでーす♪

そして悪夢がはじまる!

**仕様書無しさん** · 2019/03/24(日) 10:41:24.13

>>192
でもレビューアーによって品質が安定しない問題はどうやって解決するの？
君と俺のクオリティの違いは何？

**仕様書無しさん** · 2019/03/24(日) 10:44:08.97

>>118
> 昔のソースからコピーした処理が書式NGってどういうことだよ

単純に昔はレビューなかったからとか時間がなかったから通したとかで
書き直す機会もなく放置されているコードだったんじゃないの。
そういう今ならもうちょっとマシにかけるところを、昔のクソみたいなコードを
コピペしてくるやつを再教育するみたいなのもレビューの意義だろ。

**仕様書無しさん** · 2019/03/24(日) 10:46:24.71

>>194
＞3.レビューは全員で

それは次で書いてある全員でレビューしろってので担保しましょうってことだろ

**仕様書無しさん** · 2019/03/24(日) 10:48:42.34

>>196
メンバーが8人いたら
1時間レビューするだけで
1人日使っちゃうんだけど？
その価値ないだろ

**仕様書無しさん** · 2019/03/24(日) 10:51:43.19

>>191
逆だと思う
良いコードの定義は常に動的で変わりゆくものだ

例えばビジネスの状況の変化によってあるモジュールの価値が急激にさがるかもしれない
そうなったときにその事実を無視して今までどおりチェックリスト形式で機械的に同じようにレビューをしたらどうだろうか
価値のないモジュールに高いコストを払ってレビューを続けることになってしまうよね
それは全く持って採算に合わない行動だとおもわないか？

すべてのコードの価値は一定ではないし時間的に変化しうるものだ
だから評価方法も一定化せずに柔軟に変化させていったほうがいい

**仕様書無しさん** · 2019/03/24(日) 10:55:28.02

>>195
答えは本当に簡単なことだと思う
「チェックリストにそう書いてなかったから」まずいなとは思いつつ通しちゃったのだろう
それで後で負債が蓄積して火を吹いたかなんらかのトラブルが有ってチェックリストの仲間入りしたのだろう

**仕様書無しさん** · 2019/03/24(日) 10:57:44.88

>>197
意識共有、技術拡散よりも有意義な時間はない
コミュニケーションとらずにわからんわからんと１日うなってる馬鹿の１人日と同じコストだぞ？

**仕様書無しさん** · 2019/03/24(日) 10:59:20.99

日本では無理だな
正直、俺でも何言ってるのかわからない
レビューを行う前と後で何が違うのかもわからない

**仕様書無しさん** · 2019/03/24(日) 10:59:29.84

そいつのためにｘ８だろ

**仕様書無しさん** · 2019/03/24(日) 11:03:48.47

試験項目のレビューにでも回した方が有意義かな？

**仕様書無しさん** · 2019/03/24(日) 11:05:11.07

>>197
１レビューを全員でやるという意味じゃないだろ。

持ち回りでレビューして他人とのズレやらなにやらを学習なりフィードバックして
全体のレベルを均質化とか底上げするためのことで、１つに対して全員でレビュー
することと誤読したのか？お前はまず国語の勉強からやり直したほうがいいぞ。

**仕様書無しさん** · 2019/03/24(日) 11:08:14.05

>>204
なんか発言変わったじゃん
レビューを全員でやるんだろ？

**仕様書無しさん** · 2019/03/24(日) 11:08:39.54

>>188
敬語云々関係なくお前のそういう部分は隠しても相手に伝わってるもんだよ
リアルでも堂々とゲス本性さらけだして素直に生きてけよ

**仕様書無しさん** · 2019/03/24(日) 11:13:32.12

だいたい注意点とかが共有されるから悪くはないな

**仕様書無しさん** · 2019/03/24(日) 11:18:19.84

なんにしても出来る奴にしか出来ないんだから、
金積んでオークションしてでも出来る奴をまず用意しろよ。

**仕様書無しさん** · 2019/03/24(日) 11:19:09.97

>>205
192は下のサイトからの引用でそれに対してのレスで3章の全員でやれっていうのがあるから
それで担保してるんだろっていうレスなんだけどな。
で、下のサイトからの引用を載せておくがどこに1つのレビューを全員でやれって書いてある？

役立つコードレビュー　8つのヒント
https://postd.cc/8-tips-for-great-code-reviews/

3.レビューは全員で

コードレビューの責任を優秀なAさんに丸投げしたいという誘惑にかられることもあるでしょう。

Aさんはコードの達人で、いつも何がベストなのか分かっています。システムを隅から隅まで知り尽くしていて、
チーム全員の在籍期間を合わせたよりも長く会社にいます。

でも、Aさんがあることを理解できるからといって、チームの他の人たちにも理解できるとは限りません。
また、Aさんのコードレビューについて何か指摘するのをチームの若いメンバーがためらうかもしれません。

チームの多様なメンバーの間にレビューを分散させるとチーム力学が健全化され、コードも改良されることに
私は気付きました。初級エンジニアがコードレビューで口にする最強のフレーズの1つは「このコードは、ややこしい」です。
これが出てきたら、コードを明瞭化、単純化するチャンスです。

レビューは広く分散させましょう。

**仕様書無しさん** · 2019/03/24(日) 11:20:15.92

コードレビューなんて、
その場で思いついた意見しか言わないから、
なんでもOKなんだよ
コードを見て、わからないところがあったら
「わかんないから、分かるようにコメントいれるなどして！」
と言われるだけだから。
それがコードレビューだから。

**仕様書無しさん** · 2019/03/24(日) 11:20:36.52

まあこういう読まないやつがいるからコードレビューが一般化しないし
あっても形骸化してうまくいかないんだろうな。

**仕様書無しさん** · 2019/03/24(日) 11:24:30.55

>>209
>>197へのレスだよな？
だから全員分やんだろ？
1時間やったら8人いたら少なくとも8時間だよね？
同じ奴が別々の時間で複数のやつと被ってたらそれ以上だけど

**仕様書無しさん** · 2019/03/24(日) 11:27:37.86

至高かつ究極の美しすぎるレビュー観点一覧表を作ってやったぞ

一つ、DRYの原則を守れ
一つ、KISSの法則を守れ
一つ、SOLIDの原則を守れ

以上！

**仕様書無しさん** · 2019/03/24(日) 11:32:37.12

>>210
それでいいと思う

例えるなら、部屋を完璧に掃除したことを確認するための完全無欠のチェックリストを作るのは現実的に不可能
だけど、実際に部屋を見ればホコリが残ってるとか、汚いところはすぐにわかる

**仕様書無しさん** · 2019/03/24(日) 11:34:15.17

>>212
お前はまず国語の勉強からやり直したほうがいいよ

> 196 自分：仕様書無しさん[sage] 投稿日：2019/03/24(日) 10:46:24.71
> >>194
> ＞3.レビューは全員で　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　＜＜ここが3章のタイトルな
>
> それは次で書いてある全員でレビューしろってので担保しましょうってことだろ

> 197 返信：仕様書無しさん[sage] 投稿日：2019/03/24(日) 10:48:42.34
> >>196
> メンバーが8人いたら
> 1時間レビューするだけで
> 1人日使っちゃうんだけど？
> その価値ないだろ

> 209 自分：仕様書無しさん[sage] 投稿日：2019/03/24(日) 11:19:09.97
> >>205
> 192は下のサイトからの引用でそれに対してのレスで3章の全員でやれっていうのがあるから
> それで担保してるんだろっていうレスなんだけどな。
> で、下のサイトからの引用を載せておくがどこに1つのレビューを全員でやれって書いてある？
> 役立つコードレビュー　8つのヒント
> https://postd.cc/8-tips-for-great-code-reviews/
>
> 3.レビューは全員で　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　＜＜これな

> 212 返信：仕様書無しさん[sage] 投稿日：2019/03/24(日) 11:24:30.55
> >>209
> >>197へのレスだよな？
> だから全員分やんだろ？
> 1時間やったら8人いたら少なくとも8時間だよね？
> 同じ奴が別々の時間で複数のやつと被ってたらそれ以上だけど

**仕様書無しさん** · 2019/03/24(日) 11:34:18.99

>>213へのレスかと思って一瞬卒倒した

**仕様書無しさん** · 2019/03/24(日) 11:35:06.40

>>212
今日はレビュアーAさんでBさんCさんのレビューしてね
明日はレビュアーBさんでCさんDさんのレビューしてね
…
これなら全員がレビューに関われるし３人時しかかからん

**仕様書無しさん** · 2019/03/24(日) 11:42:39.96

>>217
8人いる話だよね？

**仕様書無しさん** · 2019/03/24(日) 11:43:51.07

>>215
だから最低メンバー分の工数がかかるんだよな？

**仕様書無しさん** · 2019/03/24(日) 11:48:17.45

>>218
そだよ
毎日全員で集まる必要はねーでしょや？

**仕様書無しさん** · 2019/03/24(日) 12:01:41.85

ちょん様は抽象度の高い話を理解出来ないんですよ
皮肉とかぶぶ漬けとか通じないんですよ

**仕様書無しさん** · 2019/03/24(日) 12:08:13.50

全員でというのを文字通り全員でやると読んでいて↓に書いてあるような
実際には全員じゃなくて多様なメンバー間と書いてあるようになるべく多くの
人をレビュアーとしてレビューしましょうねくらいの意味での文章に対しての
タイトルだと言うことが読み解けてないんですよ。国語の成績が悪かった
タイプですね。
プログラミングも抽象度の高い話をしなければならない場面が度々出てくると
思いますが、そういうのも理解できてないんだろうな。

> チームの多様なメンバーの間にレビューを分散させるとチーム力学が健全化され、コードも改良されることに
> 私は気付きました。初級エンジニアがコードレビューで口にする最強のフレーズの1つは「このコードは、ややこしい」です。
> これが出てきたら、コードを明瞭化、単純化するチャンスです。

**仕様書無しさん** · 2019/03/24(日) 12:10:18.56

レビュー観点「ややこしいです」

わろたwww
遊びじゃねえんだぞジャップ

**仕様書無しさん** · 2019/03/24(日) 12:13:00.31

新人「エロイ・・・」

**仕様書無しさん** · 2019/03/24(日) 12:14:23.33

まあコストが許すなら全員でやるのが一番いいと思いますよ
そうすればレビューアー、レビューイの教育にもなるからね
実際は全員でやるのはコストが重くなりすぎるからせいぜい
レビューアーを持ち回りでやることでクロスチェックが働く
土壌を作りましょうくらいに落ち着くと思うけど

8人にこだわっているみたいだけど、8人いるなら7回じゃ済まない
くらいのレビュー回数になると思うからレビューイ1人に対して
全員がレビューアーを務めることはできるでしょ

**仕様書無しさん** · 2019/03/24(日) 12:18:05.33

無能なやつは他人のレビュー結果を見ないんだよな
そこから学べることはいろいろあるだろうに

**仕様書無しさん** · 2019/03/24(日) 12:28:34.62

　　　　　　　　／⌒＼⌒ 丶
　　　　　／///////ﾊミミ＼
　　　　 //////////ノ´｀ヾヽヽヽ　　　バカ野郎!!!
　　　　///////////ｰ /⌒{从从　　　　　豚野郎!!!
　　　//////////从丿ii ／ Vﾉハ　　　　　　　猿野郎!!!
　　 {/////////=i<・>iヾi<・>i 从
　　///////// 　 ¨¨ﾉ　 ) ¨ /从ﾊ　　　　ジャップ !!!ジャップ!!!
　 ∥{ {从/////ヽ_ﾉ､ｪｪｪイﾉ从}リ　　　　　ジャアアアアア～ｯプ!!!!!!
　 (({　{从（（从从＼ﾄｪｪｪﾉﾉ从ﾉ　 ∩
　　 { 从乂乂ヽ＼＼＞ｰ＜彡.／)/ ﾉ　　　　ウァアアアア!!!!!
　　/￣￣／￣￣＼ヽ=＝.ヽ/　 (=)ヽ　　　　　　　
　ｆ　　　/　　　　／￣￣ヽ　{　.ﾉﾉハ
　 {　　　{　　　 /　／⌒ヽ_／乂＿＿ノ
　ﾉ　　　乂　　　　{　　{ {　　　　ノ

**仕様書無しさん** · 2019/03/24(日) 12:48:40.62

いま観てる歌番組に出演したニュージーランドの一般学生(？)の台詞に愕然としちゃった

司会「将来の夢は？」→学生「日本を建て直します！」

あー日本ってもうそういう認識なんだなって

**仕様書無しさん** · 2019/03/24(日) 12:56:03.57

ジャップランドはオワコンだから

**仕様書無しさん** · 2019/03/24(日) 12:56:24.57

大日本帝国の夢をもう一度

**仕様書無しさん** · 2019/03/24(日) 13:04:28.95

ニュージーランドって地球のどこらへん？

**仕様書無しさん** · 2019/03/24(日) 13:27:21.15

>>231
オーストラリアの横

**仕様書無しさん** · 2019/03/24(日) 13:48:10.66

平日体と精神をすり減らして
土日に風邪ひいて寝込むとかやってらんねえ…

**仕様書無しさん** · 2019/03/24(日) 14:02:48.00

なんでコピペ荒らしはこのスレだけスルーするんだ？

**仕様書無しさん** · 2019/03/24(日) 14:06:23.33

>>234

より本音の227みたいなの投下してるじゃんか
あいつ世の中を逆恨みしてるんだよ
このままじゃ死んでからも成仏できねーな

**仕様書無しさん** · 2019/03/24(日) 14:07:55.39

コピペ野郎は死んでからもウケオイガーと言ってそうだよな
ありゃ死んでも成仏できねーな

**仕様書無しさん** · 2019/03/24(日) 14:48:57.21

>>228
のど自慢のことなら、生まれも育ちも日本の学生が、ニュージーランド留学したときに歌ったという話だったよ。
肌の色は違っても立派な日本人だと思うよ。

**仕様書無しさん** · 2019/03/24(日) 14:59:57.12

こんなに急速に衰退してる元超大国って珍しいよな

**仕様書無しさん** · 2019/03/24(日) 15:04:55.18

元超大国ってプロパガンダだぞ？

**仕様書無しさん** · 2019/03/24(日) 15:37:17.51

ひろゆきって典型的日本人プログラマーだよな
不意に思い出して笑ったわ

**仕様書無しさん** · 2019/03/24(日) 15:39:46.06

>>234
たまにはられるから無視してるわけでもないだろ、というか手動なんだよな
ああいうのを手動で年単位で続けてるとか頭おかしい

**仕様書無しさん** · 2019/03/24(日) 15:43:11.06

>>240
ひろゆきのプログラマーにまつわるエピソード聞かせてや

**仕様書無しさん** · 2019/03/24(日) 15:49:47.79

>>242
なんも知らないね
ただあの賢ぶった嘗めた態度がそれだと思った
横に金魚の糞みたいなのが居れば完璧だった

**仕様書無しさん** · 2019/03/24(日) 15:53:31.81

当初、圧縮してなかった

**仕様書無しさん** · 2019/03/24(日) 16:04:39.00

日本人プログラマはあんまり嘗めた態度の奴いないだろ
横に金魚の糞付けてる奴もいないし

**仕様書無しさん** · 2019/03/24(日) 16:24:16.67

>>245
わいがおるぞ

**仕様書無しさん** · 2019/03/24(日) 16:53:48.16

来月は10連休だー！

時給労働とか一般派遣の人らは給料減って大変だね！

**仕様書無しさん** · 2019/03/24(日) 16:56:59.35

10連休なんて暇すぎてキツいだろ
3連休ごとに出勤してブラウジングしたい

**仕様書無しさん** · 2019/03/24(日) 17:55:37.91

正直、オレも10連休は持て余す。

**仕様書無しさん** · 2019/03/24(日) 18:15:23.91

皆で旅行いこうよ
沖縄行こう
きっと楽しいよ！

**仕様書無しさん** · 2019/03/24(日) 18:26:17.01

私は時給制なのだが10連休でヤバイ

**仕様書無しさん** · 2019/03/24(日) 18:33:41.85

10連休ありゃヨーロッパでも南米でもいけるじゃん

**仕様書無しさん** · 2019/03/24(日) 18:39:21.04

連休中の事故や事件のニュースを見ながら働くのが快感なんだけどわかる？

**仕様書無しさん** · 2019/03/24(日) 18:46:34.72

>>238
日本帝国時代の反省もせずに突き進んだ結果ジャップランドは時代遅れで座礁した

**仕様書無しさん** · 2019/03/24(日) 19:32:57.77

10連休なんかゴロゴロしてたら一瞬だわ。

**仕様書無しさん** · 2019/03/24(日) 19:33:13.43

急に無罪になる事件が増えた気がする
元号代わるからわかってる冤罪をクリーンナップしようってことなのかな
中世っすなぁ

**仕様書無しさん** · 2019/03/24(日) 20:25:46.75

仕事だぞ
もう寝て労働に備えよ

**仕様書無しさん** · 2019/03/24(日) 20:27:31.30

>>253
労災だよそれ

**仕様書無しさん** · 2019/03/24(日) 20:30:21.14

凄い良いサービス思いついた
タイピングゲームみたいな感じでひたすらコードをなぞるんだよ
タイピングしながらコーディング出来るって画期的じゃない？
金の匂いがする

**仕様書無しさん** · 2019/03/24(日) 20:36:46.14

【悲報】インドネシア地下鉄、払い渋りで日本企業が借金を抱えて造っていたことが判明
https://hayabusa9.5ch.net/test/read.cgi/news/1553425346/

**仕様書無しさん** · 2019/03/24(日) 20:58:39.31

今年で３６歳だがもう一生童貞だと思うと辛い

**仕様書無しさん** · 2019/03/24(日) 20:59:30.70

>>259
paizaラーニングやprogate？

**仕様書無しさん** · 2019/03/24(日) 21:49:41.14

>>259
すげえ
ぜひ作ってくれ

**仕様書無しさん** · 2019/03/24(日) 22:06:31.93

>>262
他人のクレジット情報見たり、ウイルスバラまいたりするなよｗ
普通に本で勉強が着実だよ

**仕様書無しさん** · 2019/03/24(日) 22:13:06.81

借金して開発って意味わからんな
後払いはだめだ違法化しろ