Project

General

Profile

Feature #691

スキルの無効化

Added by yusuke kokubo almost 10 years ago. Updated almost 10 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
Target version:
Start date:
03/25/2011
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)

Description

ポイントが0になったスキルは無効化する。

物理削除は行わずに論理的にenableプロパティをfalseにする。

PointdownControllerを書き換えるのと、
getSkillsも対応する


Subtasks

Feature #698: 過去のスキル表示UI終了(Closed)啓介 大橋03/25/2011

Actions
#1

Updated by yusuke kokubo almost 10 years ago

  • Status changed from 新規(New) to 解決(Resolved)

7bddf6019e8d36c5558e7993e834bae2000eed27

とりあえず実装してみた。
ArrayListをフィルタリングする処理がとてもかっこ悪い…。

#2

Updated by 啓介 大橋 almost 10 years ago

フィルタリングするところ(if(rel.getPoint() == null) continue;)ってinMemoryFilterじゃまずいんでしたっけ??

#3

Updated by yusuke kokubo almost 10 years ago

サーバ側で処理するべきかクライアント側で処理するべきかで迷ってたんですけど、
サーバ側でやった方が良いかもしれませんね…。

#4

Updated by 啓介 大橋 almost 10 years ago

ちょっと全体の仕組みを見ておきま~す!

#5

Updated by yusuke kokubo almost 10 years ago

すみません。やっぱり後で直しておきます。クライアント側で色々難しいことをやらせない方が良いのかも。
元々は、

enable=trueなやつもfalseなやつも取得しておいて、
表示するときはtrueなやつだけで、「過去のスキル」みたいなリンクをクリックすると
falseなやつがグニュって表示されるイメージでした。

でもよくよく考えたら過去のスキルをクリックしたタイミングでサーバアクセスすればすむ話なんですよね。

#6

Updated by 啓介 大橋 almost 10 years ago

なるほど!
過去のスキルはのちのちDatastoreから消すのかな?って思ってました。

確かに最初は取ってこないで、リンク押下時に取ってきたほうが優しいですね!

#7

Updated by yusuke kokubo almost 10 years ago

とりあえずenableなスキルを取得するAPIとdisableなスキルを取得するAPIをわけるようにします。
勝手なお願いですが、過去のスキルを取得するUIを作ってもらってもよいですか?

#8

Updated by 啓介 大橋 almost 10 years ago

勝手なお願いですが、過去のスキルを取得するUIを作ってもらってもよいですか?

了解です!
別チケットで管理させてくださいw

#9

Updated by yusuke kokubo almost 10 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)

APIはわけました。
このチケットはとりあえず終了にしておきます。

Also available in: Atom PDF