Skip to content

アビリティの特定項目のランキングを取得するメソッド#25

Open
mitsuru793 wants to merge 5 commits into
masterfrom
add_method_fetch_specify_ability
Open

アビリティの特定項目のランキングを取得するメソッド#25
mitsuru793 wants to merge 5 commits into
masterfrom
add_method_fetch_specify_ability

Conversation

@mitsuru793
Copy link
Copy Markdown
Collaborator

#17 に対応

同じアビリティ名でも、アビリティレベルが違い場合は別ハッシュになります。レベル別全アビリティから取得することになります。

@mitsuru793
Copy link
Copy Markdown
Collaborator Author

マージのご検討、よろしくお願い致します。

@Imfloppy
Copy link
Copy Markdown
Owner

👍
コンフリクトの解決を行ったらマージでいいと思います!
私がやってしまおうか迷ったのですが、強制プッシュすることになってしまうと思い一時保留にしてます。

なんとなく気になった箇所を

https://github.com/matthew70/vainglory/blob/2222de796289c9ba8c01971bc2c20325719c729a/lib/vainglory.rb#L63-L101

好みの部分もあるかとは思うのですが、上記の部分のコードが少し長い気がします。
例えば↓こんな感じにしてしまうのはどうでしょう?

def ability(status_name, order = :desc)
  create_ability_list
  sort_ability_list
  return ability_list
end

private
def create_ability_list()
...

@Imfloppy Imfloppy assigned mitsuru793 and unassigned Imfloppy Sep 14, 2015
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