「女子大生とペアプロするだけの簡単なお仕事」でSSSをとるまでに考えたこと
今回で二回目になる「POH!」
女子大生とペアプロするだけの簡単なお仕事です!|paizaオンラインハッカソンVol.2
前回*1は「合理的かつきれいに書けば通る」ってレベルだったのですが、
少なくとも私の(わりと雑な)プログラミングレベルだと、
今回は相当突き詰めないと難しかったのです
- コード:https://gist.github.com/parrot-studio/10815136
- 修正履歴:https://gist.github.com/parrot-studio/10815136/revisions
せっかく苦労して解いたので、自分の中でも一度整理しておきたいので、
思考の過程を思い出せる範囲でメモしてみます...φ(・ω・`)
まずは仕様通りに書く
問題を大雑把に整理すると、「あるサイズのオブジェクトが、
既存のオブジェクトと衝突しない座標の数を求めよ」ということです
言い換えれば、「オブジェクトを各座標ごとに動かしながら、
オブジェクトの範囲に何か障害物があるか?」ということになります
これを馬鹿正直に実装したのがこちら(´・ω・)っ
- https://gist.github.com/parrot-studio/10815136/7bd496e09c6b758977e53104e82d0a4a8045e7a5
- http://paiza.jp/poh/paizen/result/abc3618a53b7b7da07400343b7a02dae
あらかじめ各座標に「オブジェクトがあるか?」を入れておき、
座標を動かしながら、オブジェクトの範囲を探索する、という方法です
これでもcase4までをパスしています
全部埋まっている行は調べるだけ無駄だよね・・・?
サイトにある「入力例1」を見てもらえばわかるのですが、
4行目から7行目は「1」で埋まっている、つまり探索するだけ無駄です
それをふまえて実装したのがこちらですが・・・
- https://gist.github.com/parrot-studio/10815136/130e41388289d6bf82cd61d7b8745af885c77c0a
- http://paiza.jp/poh/paizen/result/4f222df289dfbc2d4e60b75e6dc7f06d
・・・大差はありません(´-ω-)
データ構造に情報を畳み込もう
最初のやり方の問題は、各座標における探索回数が、
「縦×横」、つまりは「O(n^2)」になっていることです
これを「座標個×オブジェクト数」繰り返すとなると、
相当に無駄が含まれます
そこで、あらかじめ「横にいくつ空いているか?」を計算しておく、手を思いつきました
元データ 1000 1101 1001 計算後データ 0321 0010 0210
こうすると、ある座標を見ただけで、
そこに横幅いくつのオブジェクトが入るかすぐわかるので、
縦方向の探索だけで衝突判定できますヽ(`・ω・´)ノ
この構造を起点に、いろいろな計算回数の無駄を省いたのがこちら
- https://gist.github.com/parrot-studio/10815136/a7aa0a0c5b5cc72b5d324729c63585ea64a435eb
- paizen2.rb
- http://paiza.jp/poh/paizen/result/6d279a6289bb5c20a3cdd040e17b53a0
これでcase5を突破しました
縦も考えてみる・・・?
元のデータが「横軸のデータを縦に向かって読む」という構造だったため、
どうしても横ばかり考えてしまいましたが、
状況によっては「縦軸の方が効率がよい」場合があります
縦に向かって探索する場合、「横2×縦100」のオブジェクトに対し、
最悪100回の衝突判定が必要ですが、
これを横に向かって探索すれば2回で終わります
なので、「縦にいくつ空いているか?」のデータも計算しておき、
「縦か横で計算回数が少なくなる方」で判定していけば、
計算量を減らせるのではと推測しました
- https://gist.github.com/parrot-studio/10815136/a1162fd4eee10faa7f83d4bf09b9a410459566c0
- paizen3.rb
- http://paiza.jp/poh/paizen/result/dca478c337ea383a0e9bbcb02259a3a5
まあ、あまり効果がなかったんですけどねΣ(・ω・ノ)ノ
盤面じゃなく、チェックするオブジェクトの問題じゃない・・・?
ここまでやっていまいちな成果しか得られなかったので、
別なところにボトルネックがあると推測しました
例えば、盤面のサイズが普通であったとしても、
チェックする対象が1万件あり、しかもそれが「2x2」の同じサイズなんて場合、
いちど計算すればあとの9999件は同じ値を出せばいいのです
つまり、「メモ化」しろと(`・ω・´)
ver2とver3に対し、それぞれ「結果を記録」するロジックを入れたところ、
たいして変わらなかったので、複雑化していたver3を破棄し、
ver2をメモ化したものをベースに進めることに
- https://gist.github.com/parrot-studio/10815136/c909a2f9a2a0153846efe41df004b6d4dc18f726
- わかりづらいですが、「paizen2.rb」が対象です
- http://paiza.jp/poh/paizen/result/3dbb079041abc8a3f68cd8d9b740f051
最終的にはメモ化というより「存在するパターンだけ調べる」に変わり、
ついでにコードも整理してみたのですが、やっぱりいまいちですね(´-ω-)
まだチェックするオブジェクトに無駄があるよね・・・?
盤面より大きなオブジェクトは排除するにしても、
もっと細かいレベルでオブジェクトを排除できないかと考えました
縦と横の空きデータを作る時点で、
「縦横それぞれこのサイズまでしか入らない」というのはわかるので、
それを元に弾いてしまえばいいのではと
- https://gist.github.com/parrot-studio/10815136/0d23637c8aa3042f906e23b38a85947380546556
- paizen4.rb
- http://paiza.jp/poh/paizen/result/4fd6144322ba720a5503a5c718c46e75
ずっと壁になっていた、case6をついに突破しましたヽ(`・ω・´)ノ
さらに、「ある横幅を基準に考えて、縦幅を小さい順に探査したときに、
ある高さで答えが0ならば、それ以降の縦幅を計算するのは無駄」と気づきました
つまり、「横2×縦3」のオブジェクトが入らない状況で、
「横2×縦4」のオブジェクトは絶対に入りません
そこを削ればいいのではと
- https://gist.github.com/parrot-studio/10815136/21af47337f2bbce839a56ebffc61b287ba7cfa96
- paizen4.rb
- http://paiza.jp/poh/paizen/result/19295f1c27e78ba08a8a98326d898616
case6突破時点の11sから8sに縮まりました( ゚д゚)o彡゚
そして深淵へ・・・
ここからはひたすら細かいチューニングを繰り返しました
オブジェクトの生成を抑えてみたり、
最終的には足し算の回数まで気にしてみたりΣ(゚Д゚)ガーン
- https://gist.github.com/parrot-studio/10815136/47c8e2a4cce5624d1c8c1d37a4c154e5a1288efe
- paizen5.rb
- http://paiza.jp/poh/paizen/result/da003df380f521e46f8d1ae449edb06d
case6で4sまで縮まったものの、ここで完全に行き詰まりました・・・(´-ω-)
ちゃんと調べてみる
ここでいったん立ち止まり、わかっている問題を整理してみました
- ループが多い
- もっと単純な探査ができないか?
- つまり、「計算する次元」を落とせないか?
- データ構造が複雑
- 構造が複雑だから、探査に時間がかかる
「衝突」という言葉を使っていますが、
ここまでくるとまさに、ゲームの当たり判定の話だと思いまして、
こちらの本の衝突判定をじっくり読んでみました
- 作者: 平山尚(株式会社セガ)
- 出版社/メーカー: 秀和システム
- 発売日: 2008/11/15
- メディア: 単行本
- 購入: 112人 クリック: 3,473回
- この商品を含むブログ (193件) を見る
2次元を1次元ベクトルに落とし込む方法とか、
いろいろ書いてありまして、これをふまえて実装してみたのですが、
case4以降でエラーになってしまいました(´・ω・`)
1行の判定を1回の計算で済ませる
「効率の良い衝突判定」を調べていくうちに、ふと思い当たるものがありました
「bit演算でええやん」、とΣ(゚Д゚)ガーン
b:盤面のある行 o:オブジェクトのある行 b:1001 o:0011 b & o = not 0 => 衝突 b:1001 o:0110 b & o = 0 => OK b:1001 o:1100 b & o = not 0 => 衝突
このように、例えば「幅2」を表すbit列をずらしながら、
その列のbitパターンとand演算を行い、
「0」になるか否かという、実にシンプルな判定になります
データの構築もただのArrayになりますし、
判定ラインも横に対してのみ考えれば良くなります
さらに、横のパターンだけ「オブジェクト列に存在する値」にしつつ、
縦の探索方法を変えました
「何回連続して衝突しなかったか?」=「そこに入る最大の縦サイズ」であり、
それより小さいものは全て入るはずです
コードの動きを具体的に書いてみますと・・・
盤面 1000 1001 1101 0001 探査するパターン 0110 1行目 1000 & 0110 => 0 counter += 1 h[1] = 1 2行目 1001 & 0110 => 0 counter += 1 h[1] = 2 h[2] = 1 3行目 1101 & 0110 => not 0 counter = 0 h[1] = 2 h[2] = 1 4行目 0001 & 0110 => 0 counter += 1 h[1] = 3 h[2] = 1 => 横2×縦1は3個、横2×縦2は1個入る
普段bit演算は使わないというか、個人的に苦手なので避けていたので、
これに思い当たるまでにだいぶ遠回りしましたね・・・(´-ω-)
- https://gist.github.com/parrot-studio/10815136/3650f82ab6fe261f32d4483061dc1ec9d222d52a
- http://paiza.jp/poh/paizen/result/82967b508c0f18216f02892f36adb69d
そんなわけで、無事に全てのTest caseを突破しました∠( ゚д゚)/
一応まとめ
ご覧の通り、最終的なコードはver5までと比較にならないほどシンプルです
やはり、効率の良いコードはシンプルに落ち着くのだと思います *2
for "https://paiza.jp/poh/paizen"
普段、ここまでチューニングを要求されるシステムは担当してませんし、
どちらかといえば可読性を優先してコードを書いているつもりですが、
たまにはこういうコードも面白いですね
次回も期待しています(`・ω・´)ノ
追記
どうも判定には「S」どころか「SS」や「SSS」もあるそうなのですが、
個人的には「S」だけ取れれば十分です(((((( ;゚Д゚)))))
さらに追記
ちょっといじったら「SSS」に・・・(; д ) ゚ ゚
- https://gist.github.com/parrot-studio/10815136/9b15a29f4f21b6ca5441f0d7d0f470eb6bd792c9
- http://paiza.jp/poh/paizen/result/0ba0a0c1b970fb72525886d3abc42e15
カウントアップのところって、
カウンタがリセットされるときで十分だよな・・・と思っただけなのですが
つまり、カウンタが3まで行ってリセットされるとすると、
縦1は+3、縦2は+2、縦3は+1・・・となっているはずです
そのように数行書き換えただけなのですが、
それでカウントアップの計算回数が劇的に変わったと思われます
*1: 前回のコードと結果はこちら(´・ω・)っ https://gist.github.com/parrot-studio/7763295
*2: とはいえ、それは「結果」なのであって、最初から効率の良いコードが書ければいいのですが、普通はそこまで簡単ではありません。だからこそ、思考の過程を残しておくことで、「次回」に生かせるのではないかと。(Gitのような)「コード管理」についても、「履歴」という「思考の過程」を残せることが、重要なメリットだと思っています