Skip to content

Add scala#5

Merged
showwin merged 6 commits intoshowwin:masterfrom
Goryudyuma:add_scala
Jun 10, 2018
Merged

Add scala#5
showwin merged 6 commits intoshowwin:masterfrom
Goryudyuma:add_scala

Conversation

@Goryudyuma
Copy link
Collaborator

Scalaで書きました!
実装はRubyを参考に、フロントはGoのテンプレートを参考にScala用に書き直しました。

メモ:
Workloadが1と2のときは正の点数が(ギリギリ)出ますが、3以上にするとエラーが増え、負の点数になってしまいます・・・。
実装上のミスではなく、ただこの実装が遅すぎるだけだと思いますが・・・。
Rubyによる初期実装に寄せるためにScalaの流儀じゃなさそうなことを何箇所かやってるせいだと思います。(例外によるhaltなど)

感想:
Rubyの普通の変数が、Scalaで言うOption型とかになっちゃったりするところとか、なかなか辛かったです。
マージされればまたブログ書きます✌

@showwin
Copy link
Owner

showwin commented Jun 10, 2018

Scala実装の追加ありがとうございます!!😆
ざっと確認しましたが、問題なさそうです。LGTM 👍
READMEの変更もありがとうございます !

Workloadが1と2のときは正の点数が(ギリギリ)出ますが、3以上にするとエラーが増え、負の点数になってしまいます・・・。

お〜、Scalaの方が点数出そうなのに意外ですね。

Rubyによる初期実装に寄せるためにScalaの流儀じゃなさそうなことを何箇所かやってるせいだと思います。(例外によるhaltなど)
Rubyの普通の変数が、Scalaで言うOption型とかになっちゃったりするところとか、なかなか辛かったです。

参考実装のRubyとGoもかなり差があるし、Rubyに寄せなくてもScalaっぽく書いてもらっても良いですよ。
Rubyっぽく書いている部分が初期スコアに影響しているのであれば、また修正PR出してもらえると嬉しいです!
(ぼくも時間見つけてやってみますが)

@showwin showwin merged commit ceaab93 into showwin:master Jun 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments