『富士の樹海』に入ります
- nanamisnow
- 1808
- 0
- 0
- 0
あの人のソースはコメントがちゃんとしていないのが一番痛い。わからないのにSTL使ってんじゃねーよ!とかもあるけど、変数の説明と実際が違ってるとかもう何を信じていいのか。
2011-05-20 10:24:31仕様もどこにも上がっていない上に、バグりまくってるから、何が正解がわからないという。仕様の洗い出しからしないといけないという辛さ。何これ怖い。設計書かないならソースの頭に設計方針ぐらいは書いておくべきだと、痛切に思った。人の振り見てなんとやら。
2011-05-20 10:28:03大枠の方針はきまった。とはいえ、ソースコードレベルではやることはまだまだあったりする。とりあえず変数、関数の説明を修正する。その変数や関数でやろうとしていることがわからない限り、修正なんぞできぬ。
2011-05-22 18:23:47ゲームはGUI周りの仕様書はほぼ書かないようで。うちだけかもしれないけど、インターフェース丸ごとぽいすてとかふつーらしい。今プロジェクトでも大幅にデザインから変わった。当時GUI担当だっただけにあれは泣けた。とはいえ、今のほうがいいと私も思うので、まあ、それはそれ。
2011-05-23 18:58:35コメントが書いてないのと、内容の間違っている内容がコメントとして振ってあるのでは、まだ書いてないほうがまし。とはいえ、基本全てにコメントを書くのは必須。だから、ソースの更新とともにコメントの更新もちゃんとしないといけない。当たり前のことだが・・・。
2011-05-23 19:05:49『富士の樹海』はソースレベルで見出したけれど、まずは全体把握からしないと。流れがつかめるかがポイント。一つじゃなくて、3つくらいの機能が関係してるだけに、見渡すのは骨が折れる。
2011-05-24 23:02:52むむ・・・同じ意味の変数を違うクラスに持っている。保存して利用しているだけ。速度を考慮して?そもそもがどちらもシングルトンなのだから、元の関数で呼べばいいだけじゃ・・・。毎回計算したくなくて計算結果を保存したいなら元のクラスにするべきだろうに。
2011-05-27 16:04:15おそらく、機能の切り分けがちゃんとできていないんだろうな。極力依存度は下げる。依存関係のあるものは、その関係を明確に定義してやらねば。
2011-05-27 16:06:35まさかのwww 意味がわからない変数があると思ったら、使ってないwwwしかも複数www ここにもつ意味がわからないって思って調べたらこれだよww
2011-05-27 18:26:29『富士の樹海』手ごわすぎ。掘れば掘るほどに地雷が出てくる。隠れた条件が多すぎる。なんでこんなことしてるん?ってのがあっても誰もわからない。これは厳しい。
2011-05-30 23:27:32社長とソース組んだ人が直接やりとりして、その条件やらなんやらをどこにも記述してないし、引継ぎ一応しようとしたんだけど、本人がわからないとか言うし。自分で組んだソースが半年とか経てば言われてもなかなかわからないってのはあるかもだけど、だからこそコメント振っとけよと。
2011-05-30 23:30:04『富士の樹海』は順調に掘り進んでおります。光明がみえてきました。しかし、半分くらいは原型をとどめなさそうwwってわけで、今日は離脱!
2011-05-31 22:19:56まー業務でやってれば色んな人がいて、色んなソースがあって。綺麗なソースだけ見てたら、直すスキルは身につかなかったりするからなー。 なんでもかんでもいいものだけ見てればいいかってとそうでもない。
2011-06-02 13:54:55~モードとか変数の説明には書いてあって、セット関数使用してるところで、~モードにするとか書いてある。その「~モード」が何を意味するか、どういう動きをするのか一切どこにも書いてない。ソースからそれを解読するという作業をやっている。そんな変数が沢山ある。なんなんだ・・いったい・・。
2011-06-02 16:17:11