yasasii-team / Stepippo-iOS

初めての一歩を応援するアプリのiOS版
https://github.com/yasasii-team/Stepippo-iOS.git
21 stars 5 forks source link

達成数をLabeに表示する #205

Closed eyener3 closed 5 years ago

eyener3 commented 5 years ago

達成数をLabeに表示する

fixes #180

(#の後にcloseしたいissueの番号を記述してください)

Summary(要約)

Tested(テストしたこと)

Screen Shot 0001-06-21 at 8 58 54 PM

Screen Shot 0001-06-21 at 9 18 22 PM

eyener3 commented 5 years ago

お疲れ様です! 関数とプロパティにprivateつけられそうですね🙆‍♂️

ありがとうございます^-^! 修正いたします💡

Sab-swiftlin commented 5 years ago

この画面について少し教えて頂きたいです!

・ピンクのプログレスバーは何を表すものでしょうか? ・現在TaskButtonとして表示している部分は、実際には超戦中のタスク(IPPO)が入ると思っていますが、達成したものはここから消えるのでしょうか? ・今回追加した達成数とピンクのプログレスバーが表す値は同じでしょうか?

eyener3 commented 5 years ago

この画面について少し教えて頂きたいです!

・ピンクのプログレスバーは何を表すものでしょうか?

達成数を表すものです^-^💡

・現在TaskButtonとして表示している部分は、実際には超戦中のタスク(IPPO)が入ると思っていますが、達成したものはここから消えるのでしょうか?

達成したものは、設定した期間が終了するまで残ります^-^

・今回追加した達成数とピンクのプログレスバーが表す値は同じでしょうか?

はい、同じになります!💡

eyener3 commented 5 years ago

もっと事前に、決めるべきことを決めていたら..という反省が日々あるので、 使いやすいように変更するのもアリだなと思っています^-^!

Sab-swiftlin commented 5 years ago

@eyener3 ありがとうございます:bow: 達成数というのはこの画面に表示されている最大3つのIPPOのうちどれだけ達成できているかというものを表すという理解で合ってますでしょうか?

もし合っていれば、現在の実装では過去に登録した全てのIPPOがカウント対象となっていますので、

//未達成の数
let numberOfChallengingIPPO = fetch(IPPO.self).filter(NSPredicate(format: "_status = %@", argumentArray:[IPPOStatus.challenging.rawValue])).count
//達成中の数
let numberOfPerformedIPPO = fetch(IPPO.self).filter(NSPredicate(format: "_status = %@", argumentArray:[IPPOStatus.performed.rawValue])).count

ここに期間の指定を追加するなどで対象を絞る必要がありそうですね!

Sab-swiftlin commented 5 years ago

推測の話で恐縮なのですが、おそらくTaskButtonにIPPOを表示する実装を行う際に現在対象期間中のIPPOを検索して保持する部分を実装する事になるので、先にそちらを実装するとこのPRはもっと実装が楽になると思います!

riscait commented 5 years ago

過去に登録して今挑戦していないタスクはstockachievedになっているので challenging, performedの絞り込みだけで大丈夫かと思います👌 https://github.com/yasasii-team/Stepippo-iOS/blob/32f8b2dba308e349734dd6a706c9537a9bb72c61/Stepippo/Classes/Models/IPPO.swift#L12-L17

TaskButtonにIPPOを表示する実装を行う際に現在対象期間中のIPPOを検索して保持する部分を実装する事になるので、先にそちらを実装するとこのPRはもっと実装が楽になると思います!

こちらはその通りかと思います😊 プログレスでも良いですが、今挑戦中のタスクの内容を表示させる方を先にやったほうがどちらかというと効率良さそうですね👍

Sab-swiftlin commented 5 years ago

@Riscait なるほど、ありがとうございます! でしたらその絞り込みで大丈夫そうですね! ちなみに、challengingのまま期間を過ぎたものはstockとして保存される事になるのでしょうか?

eyener3 commented 5 years ago

Sabさん、龍之介さんありがとうございます^-^! 挑戦中のタスクの内容を表示させる実装を先にやってみますね💡

ちなみに、challengingのまま期間を過ぎたものはstockとして保存される事になるのでしょうか?

はい、そうしようかなと思っていました^^!

eyener3 commented 5 years ago

関数とプロパティにprivateつけられそうですね🙆‍♂️ RealmObjectAccessible でまかなっているのでこちらのimportは不要ですね👌

修正いたしました(o'ー'o)!

eyener3 commented 5 years ago

間違えました。。。

ドクロ💀りんごさん、Sabさんですか^0^? GIF?おもしろかったです!>ω<♪