rhiskey / ciphereverything

Collaborative open-source project

Greetings 👋

---

SpotyVK: music export app

---

📌 Pinned Repositories

Data Science Hero

Commertial Projects

More Projects

💼 Skills

More Skills

(https://github.com/anuraghazra/github-readme-stats)

• 🔭 I’m currently working as ent-r

• 🌱 I’m currently learning Swift, UIKit, SwiftUI, Android SDK, Kotlin, PyTorch, Keras, Tensorflow, Deep Learning frameworks

• 👯 I’m looking to collaborate on pet projects, startaps

• • 📫 How to reach me: Discord: rhiskey#2088

ciphereverything's Issues

Features

• Copy/paste password from field
• iCloud backup and sync
• Subfolders/subcategories
• Encryption
• Paid version

Code Review by Evgenya Bruyko

Приложение выглядит отлично, все продумано, даже проверка логина на сложность:)

LoginViewController

Это свойство должно быть приватным. Но если вообще думать глобально, то это должно быть свойством человека, который залогинился! Создайте еще одну модель, которая будет содержать имя и пароль основного пользователя:

потому как ты же будешь в дальнейшем сохранять этого человечка в памяти, что бы при повторном запуске приложения ему не пришлось заново логинится!
Или нужно было вообще не создавать экран логина, раз это не создает какого то человека!

не нашла к чему относится этот метод! откуда он возвращает, видимо задумка была, но не реализовали:) Лишний код плохой код!

TabBarViewController

Этот класс у вас определенно лишний! Название для заголовка вы определяете в сториборде, а кнопки у вас и без того не будет!

AccountsTableViewController

ключевое слово return здесь лишнее, и открывающую скобку нужно определять на той же строке что и возвращаемый тип данных:

В остальном в этом классе все хорошо!

###TeamTableViewController

ключевое слово self лишнее
В остальных вьюконтроллерах не нашла замечаний! Структуру класса соблюдаете, делегирование усвоили, в целом логика не плохая! Одним словом молодцы, главное в этом задании, было понять как работать в команде, разобраться в этом и не подраться:)) И с этой задачей вы справились;) На собеседовании очень часто спрашивают опыт командной работы, и это приложение можно использовать как портфолио;)

Improvements - exit from acc, search bar

Не хватает только выхода из учетки, я бы добавила еще одну кнопочку с лева в виде человечка, что бы можно было сменить аккаунт либо просто выйти из него. У меня просто есть потребность хранить пароли мамы и папы, и было бы очень удобно хранить их в отдельной учетке;) И если еще поисковую строку запилите будет вообще 🔥