ToHeart2 ほか AquaPlus/LeafのGPLゲーをいじるスレ

1 名前：名前は開発中のものです。 [2005/12/13(火) 01:33:48 ID:TT+LcNFZ] 弊社製品のムービー再生にxvid.orgのムービー展開ライブラリを使用していた件について。 弊社製品、 『アルルゥとあそぼ!!（ランチャとRoutesミニシナリオのみ）』 『Tears To Tiara』 『鎖−クサリ−』 『ToHeart2 XRATED』 の４商品では、動画再生に www.xvid.org/ において公開されいるXViD再生エンジンを使用しております。 この動画再生エンジンのプログラムはGPL(GNU General Public License)に基づくソフトウェアとして公開されていますが、これを使用した場合、GPLに定められている通り、オファーがあればソースコードを公開する義務があります。 このGPLのソース公開義務に従い、弊社の上記４作品のソースコードを全て公開する事に致しました。現在ソースコード公開に向けて社内のソースコードのまとめを行っておりますが、２年ほど前のタイトルも含まれておりますので、多少時間がかかります事をご容赦下さい。 今後、XViDの開発元に弊社より謝罪を含め連絡をとり、GPLの基準に合致しているかを確認を行った上で、全ソースコードの公開を致しますので公開まで少々お待ち頂けるようお願い致します。 leaf.aquaplus.co.jp/xvid.html 245 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 13:19:28 ID:o9P35CpQ] コメント削除して公開って、GPL的にOKなのか？ 246 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 13:25:28 ID:ZcsNJUgN] >>245 OK。 要はコンパイルが通ることが条件になってるだけ。 247 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 13:37:56 ID:zxS3ZF2L] >>246 それってGPLのどの項に書いてある？ 248 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 13:52:16 ID:kE73PQP3] とりあえずTH2専用SOUND PAK展開TOOL v1.0うPしました。 サウンドは同じコードで展開できた。他はやり方違うので見てるとこです 249 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 13:55:59 ID:2SR90tBk] >>247 コメントを消してもいい　って書いてあるわけじゃなく、 GPL中での「完全なソースコード」の定義が、 「プログラムに対応していて」「関連モジュールやインターフェースも含むもの」だから、 別にコメントを消したところでプログラムとの対応関係は崩れないし、 関連モジュールが減るわけでもない。 250 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:04:04 ID:yETwwrc3] >>248 どこに？ 251 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:06:20 ID:9p4CG6az] Cマガが特集してくれんかなーって思ったんだが…休刊なのな 252 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:10:25 ID:kE73PQP3] >>250 もち元のソースが置いてあったうpろだ。パス無しソース付GPLでw 253 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:21:20 ID:Vo8Ih3D3] DLしたお でも何かソース足らなくね？ readFile.cppが開けないと言うか同根されてないお（；＾ω＾） 254 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:23:01 ID:Vo8Ih3D3] ×同根 ○同梱 恥ずかしいお（；＾ω＾） 255 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:25:18 ID:kE73PQP3] readFile.cppはTH2の奴使っておくれ 256 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:36:15 ID:SUxUnqO9] >>251 ２月号発売したとのメルマガきたけど、休刊なの？ 257 名前：名前は開発中のものです。 [2006/01/18(水) 14:39:42 ID:laXFj5Ga] >>256 今ぐぐってみたら・・・ ttp://www.fx-it.com/kihon/top.html ((((((；ﾟДﾟ))))))ｶﾞｸｶﾞｸﾌﾞﾙﾌﾞﾙ 258 名前：256 mailto:sage [2006/01/18(水) 14:44:47 ID:SUxUnqO9] >>257 ６年近く毎月買ってたのに……。(´･ω･｀) 259 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:56:22 ID:J2GY/YuU] >>236 betterCとして使ってるんじゃね？ 260 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 14:57:11 ID:nOYGrh+r] まあ、最近は完全にウェブの後追いだったからな。雑誌じゃ小回りが効かん。 261 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 15:06:15 ID:g+zxdCud] 最新の情報を追うのは遅くなってたけど、まとまった情報源としての価値は大きいと思うんだけどなぁ。 今でも98､9年ごろの特集が役に立つことが多いし。 3月売り分ってことは4月号で休刊か… 刊行継続されることを祈るしかないな。 262 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 15:13:27 ID:yETwwrc3] どこだか分かんないし 見あたんないからまあいいや。 263 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 15:16:58 ID:Vo8Ih3D3] >>262 >>178 264 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 15:18:11 ID:Vo8Ih3D3] >>179だった 265 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 16:24:51 ID:fFrV7vks] ＞２４８ GNU 一般公衆利用許諾契約書の複製物がないおｗｗｗ 266 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 16:47:08 ID:kE73PQP3] しまった入れ忘れた、証拠隠滅))彡 あと少しで全部解凍出来そうなので そん時忘れずに入れときます 267 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 16:51:50 ID:wB2vkzXb] betterCまでたどり着けてないな。 まぁToHeart2延期の謎は全て解けたが。 ・・・ここって、ソースのレビューしてやるスレ？ 268 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 16:57:43 ID:/WD9qG7O] ここはソースのバグ修正やソースを利用した（当てずっぽうじゃない）ぶっこ抜きツールなどを作るスレ。 269 名前：名前は開発中のものです。 [2006/01/18(水) 17:52:57 ID:MQQtoSlq] 238は社員 270 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 18:20:28 ID:FyGFDL1J] とりあえず、最初にやることはリファクタリングだなｗｗ 271 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 18:30:35 ID:PTTErmDt] まったくだ、混沌としすぎてるなこのソースは。。。 272 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 19:58:21 ID:jJUl8H6R] プロのコードってんで楽しみにしてたんだが…… 力技っていうか……なんていうか…… 273 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 20:15:05 ID:FyGFDL1J] まあ、クローズドのソースなんてどこもこんなもんさ。 274 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 21:05:10 ID:t8gwWK0y] 誰かリファクタリングやってくんね？ てかどっかまとめ作って分散してリファしたいな。 275 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 21:11:36 ID:osFctAmr] あれ意外にソース少ないね、ノベルだからかな 276 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 21:55:04 ID:bY+51Db2] 予想通りの展開だな。ここでうれしそうにC++どうこうとかいってる奴は、 まともなゲームプログラムなんか作ったことのない奴だろうｗ このくらいの規模のプログラムならCライクに作るという選択はありうる。 見たまんまで、クラスライブラリよりも読むのがはるかに楽だからな。 >272 派手な打ち回しなどいらないんだよ、静かに勝ってこそプロ。 277 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:04:22 ID:rGchGrM3] そんな事より、コミパＰのコード公開してくださいよ。 あの酷い読み込みの嵐 278 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:10:14 ID:4jqE4KU9] >>276 何処のエロゲーメーカー所属ですか? 279 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:16:49 ID:zxS3ZF2L] SourceForgeまだー？ 280 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:22:32 ID:lWJMeLuy] 人のソースにケチつけるようなプログラマって見っとも無いと思う。 281 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:30:37 ID:rGchGrM3] 俺サイコーっていう最低プログラマーに事実を教えないのは 人として冷たいと思うよ。 282 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:37:27 ID:4jqE4KU9] げんしけん7巻82ページを思い出した。 283 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:56:09 ID:4W+22aVY] いまどきクラス構造もマトモにかけないプログラマは化石級だよ 284 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 22:59:54 ID:RvKbImaS] お前の認識の甘さが化石級だｗ 285 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:00:22 ID:gEgW1I6k] >見たまんまで、クラスライブラリよりも読むのがはるかに楽だからな。 （　ﾟдﾟ）ﾎﾟｶｰﾝ 286 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:00:28 ID:g35b7xFX] クラス構造ってなんでつか？ 287 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:07:34 ID:iCks23xZ] だから、オマエラの友達っつか知人に一人くらいいるだろ？ コメントアウト付けるプログラマは屑だ！ ソースから全てを読み取れ！ 俺なら出来る！ って言う、奇才。 ちなみに漏れはそんな香具師見た事ありません。(´・ω・｀)ｼｮﾎﾞｰﾝ 288 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:07:46 ID:zxS3ZF2L] >>280 せっかくのGPLだから ケチつけるだけじゃなく直さないとな。 289 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:08:59 ID:4W+22aVY] >>284 確かに認識があまかったようなだ社員さんよ(pgr 290 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:22:25 ID:4jqE4KU9] TH2でゲームプレイヤーに、18禁な名前が付けられないようにするコードを探してみた。 多分この関数で名前を登録している。 Winmain.cpp の、 BOOL CALLBACK NameDialogBoxProc( HWND hwnd, UINT wmes, UINT wparam, LONG lparam ) 読む限り、もしかして18禁な名前をチェックしてない？ わくわくしながら探したのに30分無駄になった… しかし、変数が_(アンダーバー)で始まるのが多いですね。_で始まる変数はC言語ではシステム予約 で使ったらダメだったと記憶しているのだが。(うろ覚え) 291 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:26:10 ID:P2MjgQRi] >290 そんなマナー規約、実際にエラー（名前のバッティング）が発生しなければ何ともないぜ！ それが職業プログラマクオリティ 292 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:27:17 ID:vR0Gn7As] チェックする前に卑猥な名前つけられるかどうか製品版で試してみればいいのに 293 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:27:33 ID:3nmnCkEQ] STLをCライクに使ってどこがどう読みやすくなるのかと小一時間（ｒｙ 294 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:29:44 ID:4W+22aVY] >>290 それ残骸だよ。 _で始まっている変数も残骸多数。 295 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:35:10 ID:osFctAmr] >>290 俺もちょっとうろ覚えなんだが、確か _大文字が予約語なのよ _小文字はOKのはず、GoF本でも使ってたはず Leafのソースは観てないのでどちらかは知らないけど。 296 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:36:06 ID:g35b7xFX] 予約語ではない。あくまで紳士協定。 297 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:47:35 ID:osFctAmr] いや予約語だよ。俺も自信ないので結局めくってきたがｗ uppercaseが大文字の事ね 17.4.3.1.2 - Global names [lib.global.names] -1- Certain sets of names and function signatures are always reserved to the implementation: Each name that contains a double underscore ("__") or begins with an underscore followed by an uppercase letter (lex.key) is reserved to the implementation for any use. Each name that begins with an underscore is reserved to the implementation for use as a name in the global namespace.* [Footnote: Such names are also reserved in namespace ::std (lib.reserved.names). --- end foonote] www.kuzbass.ru:8086/docs/isocpp/lib-intro.html 298 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:48:40 ID:t8gwWK0y] これ、アンダーバー２つじゃね？ 299 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:52:29 ID:osFctAmr] >>298 or begins with 【an】 underscore followed by an uppercase letter こっちス 300 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:54:48 ID:t8gwWK0y] >>299 ほんとだ。すまん。 301 名前：名前は開発中のものです。 mailto:sage [2006/01/18(水) 23:58:06 ID:P2MjgQRi] そのreservedは、 いわゆる予約語（キーワード（if とか int とか））のとは意味が違うんではないか。 302 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:20:07 ID:5Wiz9rJA] JIS X3010(ISO/IEC 9899:1999) 7 ライブラリ 7.1.3予約済識別子 　下線に続き大文字一文字または下線に続きもう一つの下線で始まるすべての識別子は、 　いかなる使用に対しても常に予約済みとする。 　一つの下線で始まるすべての識別子は、通常の名前空間およびタグ名前空間の双方における 　ファイル有効範囲を持つ識別子としての使用に対して、常に予約済みとする。 303 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:20:56 ID:THSyh1jG] 結局ここには初心者にも満たない屑しかいねぇな。 プログラミングそれ自体を勉強して出直してこい屑ども。 304 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:28:14 ID:a9X8kYfE] ゲームなんて動けばいいんだよ。 バグがあったところで、別に人が死ぬわけじゃないんだから。 305 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:32:12 ID:BoxiYWRY] >>302 それ違う。 6.4.1のキーワードの方が予約語。X3014(C++)なら2.11。 識別子として用いられないものを予約語という。 ttp://www.jisc.go.jp/ ここで、X3011とX3014のpdfが見られるので見たい人はどうぞ。 306 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:35:55 ID:+CiPqcdC] 識別子として用いられないものを予約語という 307 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:44:30 ID:7sJ2B52z] >>303 これくらいのソースで給料貰えるのは正直うらやましい 漏れのとこでこんなソース書いたら白い目で見られるわ文句言われまくりだわで大変だ。。。 308 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:56:26 ID:YIdLOiMb] 折角のGPLなんだからエレガントにしてあげてください 309 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 00:57:31 ID:gajhu98D] >>303 口だけなら、誰でも出せるんですよ。(pgr 310 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 01:05:57 ID:wJ5oggz1] 俺も自信ないのでドラフト見てきたら既に書き込まれてるし。 ていうか俺、インクルードガードに３連アンダースコア使ってた。１つや２つが予約されているのなら３つと。 いつどこで仕込まれたのか重い打線けど、規格見る限りこれまずいよな。次から改める。 311 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 01:17:05 ID:LE/6U+/2] Netscape Navigatorのソースよりも綺麗で驚いた 312 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 03:22:07 ID:1g8/eWMn] スクリプト解析して使えるようにして欲しい 313 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 04:43:42 ID:4TdhbyZQ] ttp://www.pro.or.jp/~fuji/mybooks/cdiag/ >第２章　これでもプロ >売りものであるにもかかわらず、超きたない！ >構造的な欠陥の指摘〜引数、ポインタの活用 314 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 07:15:09 ID:eUUjjqw/] TH2専用 PAK展開TOOL v1.0うpしました。今回は全てのPAKに対応します。 PASS:nekoneko 注意点 1)コンパイルする時は必ず同梱のcomp_lzsとreadfileを使用してください。 2)Xvidは1.1で確認しました。2.2では再生しませんでした。 315 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 13:17:57 ID:1FTU/l5f] 2.2の方は間違い、別物でしたm(_ _)m 316 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 13:24:42 ID:nZR9MDty] >>311 それは言うな、悪夢を思い出した。 そもそも、Mosaicから続いからてるんだ。一発屋のエロゲと一緒にするな。 317 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 16:10:51 ID:6mVz+StZ] >293 そもそもこの程度のことにＳＴＬ使うのが間違い。 318 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 16:29:25 ID:19MQyoRS] >>317 楽したもの勝ち コの程度のことに、ライブラリ使わなくてどうするの 319 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 16:30:56 ID:9luY8TRl] どうするのって何言ってんの 320 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 16:38:33 ID:6mVz+StZ] この処理がライブラリ使って楽になるとも思えんが。 STLを使うには、STLの使い方という、C言語の言語仕様の外の知識が 必要になるからな。 しかし、葉鍵板のほうがまだソースを利用しようとしてる奴が 多いというこの逆説。ここじゃ文句つけてるだけだものな。 あ、「弄る」って、利用するって意味じゃなく、ネタにするってことか？ いわゆる関西芸人的な。 321 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 17:04:23 ID:fXIaRsvR] >>320 日本語でおｋ 322 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 17:10:04 ID:wJ5oggz1] このソースを利用しろと言われても困ってしまうな。正直、情熱のあるエロゲオタに譲るよ。 323 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 17:37:22 ID:iYkDFC6/] 実際のところゲームプログラムの参考にするなら吉里吉里の方がよくまとまってるしなぁ… 吸い出しや移植以外に用途が思いつかない 324 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 19:13:13 ID:6mVz+StZ] >323 吉里吉里は吉里吉里で、ちょっと大げさすぎるからなー。 どっちも部品単位で参考にする目はあると思うけど。 325 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 19:26:40 ID:79/azYTu] は？　え？　規模の問題じゃねえよｗ 326 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 20:19:55 ID:6mVz+StZ] 規模というより、吉里吉里は汎用性をもたせようとしすぎ。 あそこまでやる必要ないだろう。オブジェクト指向とか。 327 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 22:27:39 ID:7sJ2B52z] でもスクリプト周りがしっかりしてたらわざわざプログラム変えてあんな妙なことをやらなくてもイイと思うんよ。 328 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 23:21:45 ID:qjrBgih/] 話を聞いてると吉里吉里のソースのほうがおもしろそうなのでそっち行くわ。んじゃ。 329 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 23:32:31 ID:Z5cXhUw2] 別の商品を同じもので統一するわけにはいかんのだろう。 汎用プログラム＝商品ごとに使わない無駄が出る＆バグが全商品に影響 あとそういう仕事を知ってる人なら分かると思うけど もし完全自社内開発でなければ大人の事情もあるわけで。 330 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 23:37:37 ID:j4PTuc+T] 俺は吉里吉里のゲームは例え法人による販売だとしても同人ゲームだと思ってる。 331 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 23:40:32 ID:6mVz+StZ] >327 あれは社内人事絡んでそうじゃね？ メインプログラマが違うんじゃないかと思えた。 TH2のソース見る限り、サポートはこっちのほうが楽だな。 ちょっとだけコンシューマの開発やったことがあるけど、どうせ毎回一作限りで キツいチェック通さなきゃいけないなら、ストレートにハードコーディングして 一回きりのソースとして考えたほうが楽なんだよな。 332 名前：名前は開発中のものです。 mailto:sage [2006/01/19(木) 23:42:49 ID:6mVz+StZ] たとえば、あらゆるタイトル画面を実装するために、 細かい演出処理をスクリプト仕様に組み入れるぐらいなら、 タイトル画面自体をハードコーディングしたほうが楽。 333 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 00:05:12 ID:evtcDsiT] >328 やっとSubversionの使い方が分かったから落としてみた。 >330 お前の同人ゲームの定義が解らん以上意味不明。 334 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 00:12:29 ID:OTKBdpZ5] >>316 いや、少なくとも公開されたソースは3タイトルを含むから、ごちゃごちゃに 至る歴史があったわけで、一発屋ってわけではないと思う。Netscapeのアレは WinMainからしてヘンだったわけでドッコイだけどね。 >>331 うん、オレもソースのハンドリング（という表現はおかしいか？）は いい方だと思った。これなら引き継いでもさほど困らない。 クラス関係が複雑なのかよりも全然わかりやすいし。 335 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 00:28:34 ID:Z8o3taxA] 冗談だろ。ハードコーディングにしても汚いほうだぞ。 336 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 00:36:04 ID:AmUbds5k] まー。とりあえず、コードが汚いとか抜かしてる奴は、 自分のコード晒してからな。 337 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 00:51:26 ID:OTKBdpZ5] そりゃ、人に見せたらなにこれｗっていうレベルだし、これよりも ちゃんと書くことの出来る人はたくさんいるけど。 納期があって、ちゃんと動くパッケージのソースとしては あれくらいでもいいと思うよ。 規約やら美しさどうこう言う人は苦しんだ経験がないんだろう。 まぁそれはそれでうらやましいし、理想は高い方がいいよね。 338 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 01:00:16 ID:LK/T1wXi] 綺麗汚い以前に普通に書いていれば入らないであろう >>233のようなバグが入っているのがおかしい。 339 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 01:22:10 ID:bWwH4+cX] やー、俺がコンシューマの仕事で書いたソースはこれよりはるかに汚いしｗ バグはともかく、ソースの書き方としては綺麗なほうだと思うんだがな。 あと、233はバグじゃないようだぞ。VC++では問題ないんだとか聞いたが。 俺はそもそもSTLとか使わんので分からんが。 340 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 01:24:57 ID:bWwH4+cX] つか、世の中のあらゆるソースがそんな完璧に綺麗なのばかりであれば、 リファクタリングの本なんか売れるわけ無いんだよｗ 341 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 01:46:38 ID:evtcDsiT] >339 デバッグビルドだと思いっきりアサートが掛かる訳だが。 ついでに言えばopenFileNumもArcFile.size()があるから不要。 なぜreserveとpush_backを使って、sizeを使えない？ あと、 arc_file_info::streamInfoはvector使ってるのに arc_file_info::pack_fileはいちいちnew[]で確保している。 しかも削除時はdelete[]じゃないし。 342 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 02:15:05 ID:bWwH4+cX] >341 >デバッグビルドだと思いっきりアサートが掛かる訳だが。 pie.bbspink.com/test/read.cgi/leaf/1135546874/446 を読んだだけなんで、俺はあんまし知らん。すまん。何せ使ってない。 >しかも削除時はdelete[]じゃないし。 これはメモリリークするんだっけ。気づかなかったのかな？ new deleteもほとんど使わんなあ。malloc freeだ。 343 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 02:47:44 ID:LK/T1wXi] ほとんどというか何というか、 よほどシビアな環境じゃなければ普通はどちらかに統一するし クラスを使う以上はnew/deleteにしないと。 344 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 03:30:58 ID:bWwH4+cX] >343 C構造体をmallocで割り当てるソースは良く使うしなあ。 多くのサンプルコードがCで書かれてるし、俺はそっちで統一してるな。 つまりはC++よりbetter Cってことなんだろう。 クラスは使うけど、newで動的には割り当てない。 345 名前：名前は開発中のものです。 mailto:sage [2006/01/20(金) 03:51:38 ID:pqHaw1ZN] >>341 だから>>237なんだろ。 無理してC++を使う必要はないし、Cで統一するのも結構だけど このソースは無理してC++使って破綻してる感じだしな。 うちのとこでもこんなソース書いたら大目玉食らうよ・・・

---

---

[ 続きを読む ] / [ 携帯版 ]

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef