高慢と偏見（4） 嵐の金曜日：Press Enter■：エンジニアライフ
ふつーのプログラマです。主に企業内Webシステムの要件定義から保守まで何でもやってる、ふつーのプログラマです。 「56行めからの関数だけどね、selectPartsList()ってやつね。なんでこれ、privateになってるの？」 「すみません、意味がよく分かりませんが……」富永さんは疲れたように言った。 メソッドを「関数」と呼んでいることは置いておくとして、確かに意味がよく分からない。 該当のメソッドは、画面の初期化処理であるinitialize()から呼ばれていた。ある条件に基づいて、コンボボックスで使用する部品の一覧を部品マスタから取得している。部品マスタはこのシステムのほとんどの機能で使用する主要マスタの1つなので、さまざまなパターンが共通ロジックとして作成されているのだが、この画面ではどのパターンにも当てはまらないため独自に実装したようだ。 こういう処理は、共通ロジックとして切り
高慢と偏見（3）コードレビューは踊る：Press Enter■：エンジニアライフ
ふつーのプログラマです。主に企業内Webシステムの要件定義から保守まで何でもやってる、ふつーのプログラマです。 次の日から、三浦マネージャによる新体制がスタートした。 もちろん、急に何かが変わったわけではない。これが開発初期だったら、それまでの作業をなかったことにしてゼロからスタートという事態もありえただろうけど、機能数でいえば3割ほど実装済みのものを捨てることは、さすがの三浦マネージャもできなかったようだ。 それでも三浦マネージャの「方針」は早速実施され、私たちの開発に負担を強いることになった。 まず、コードレビュー。 最初、三浦マネージャは週2回、各3時間で実行するつもりだったらしい。しかも全員参加で。 これには平良さんが難色を示し、かなり強く異議を唱えた。開発初期ならともかく、ここまで開発が進んだ状態で、しかもスケジュールが押している時に、週6時間も実装とは別に時間を取られるのはバカ
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
