yasasii-team / Stepippo-iOS

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

IPPOのデータモデルを作成 #147

Closed naipaka closed 5 years ago

naipaka commented 5 years ago

fixes #140

Summary(要約)

IPPOのデータモデルを作成しました。 項目は以下の通りです。( #42 )

Tested(テストしたこと)

レビューポイント

Must Reviewer(必須レビュアー)

naipaka commented 5 years ago

レビューありがとうございます! 該当箇所修正しました!🙇

naipaka commented 5 years ago

image こんな感じで書き込めました!

riscait commented 5 years ago

修正ありがとうございます!

privateを外したことによって、statusとippoStatusが両方クラス外から見えるようになってしまって紛らわしいですね🤔わかりやすい名前を考えましょうか?

riscait commented 5 years ago

使ってほしい方がstatusだとして、使って欲しくない方(旧private)の候補出してみます🙋‍♂️

🤔

naipaka commented 5 years ago

privateでも書き込めるっぽい?です realm browserには表示されないんですが、print(ippo.status)で見ると設定できてますね🤔

image

image

riscait commented 5 years ago

privateでも書き込めるっぽい?です realm browserには表示されないんですが、print(ippo.status)で見ると設定できてますね🤔

realm.objects〜でstatusを取得できますか?🤔 取得できれば = 保存できているになりそうですが…

naipaka commented 5 years ago

realm.objects〜でstatusを取得できますか?🤔 取得できれば = 保存できているになりそうですが…

追加した内容を出力してるだけだから表示されるのは当たり前でしたごめんなさい!:sweat_drops: realm.objects〜ではいけませんでしたね…

naipaka commented 5 years ago

privateを使わない場合の変数の候補は

riscait commented 5 years ago

privateを使わない場合の変数の候補は

  • _status
  • statusString が個人的にいいですね

_statusにしちゃいましょうか☺️

riscait commented 5 years ago

ドキュメントを読んでいたら、 「一意の自動生成値が必要なほとんどの状況では、連続した整数IDを持つ必要はありません。通常、一意の文字列主キーで十分です」 って書かれてました笑

確かに、連番にこだわりなければランダムな文字列(UUID)で良いなと思いました… https://realm.io/docs/swift/latest#limitations-models

naipaka commented 5 years ago

ドキュメントを読んでいたら、 「一意の自動生成値が必要なほとんどの状況では、連続した整数IDを持つ必要はありません。通常、一意の文字列主キーで十分です」 って書かれてました笑

確かに、連番にこだわりなければランダムな文字列(UUID)で良いなと思いました… https://realm.io/docs/swift/latest#limitations-models

おっ!ドキュメント読むのやっぱり大事ですね〜 ではランダムな文字列にしようと思います! privateを使わない場合の変数も修正しますね☺️

naipaka commented 5 years ago

修正しました! 再レビューお願いします! (貼る画像間違えた。。。→修正しました!) image

naipaka commented 5 years ago

修正しました!何から何まで指摘していただいてありがとうございます:bow:

naipaka commented 5 years ago

承認ありがとうございます!マージします!