Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

最低限のリファクタリング を行った #1

Merged
merged 9 commits into from
Feb 3, 2021
Merged

Conversation

shinya-todaka
Copy link
Owner

Issue

ソースコードの可読性の向上

  • 命名規約(参考:Swift API Design Guidelines)
    • capitalCaseにした
    • ViewControllerの名前やUIの名前を分かりやすくした
  • ネスト
    • guard let を用いてネストを減らした
  • インデント
  • コメントの適切性
    • 不必要なコメントを消した
  • スペースや改行
    • 不必要なスペースや改行を減らした

ソースコードの安全性の向上

  • 強制アンラップ
    • guard let や if let で安全にoptionalをunwrapした
  • 強制ダウンキャスト
  • 不必要なIUO

バグを修正

  • レイアウトエラー
    • storyboardのレイアウトを直した
  • メモリリーク
  • どこがメモリリークしているのか分からなかった
  • パースエラー
    • Codableを使って安全にパースした

@shinya-todaka shinya-todaka merged commit 30fe841 into CI Feb 3, 2021
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.

1 participant