Added: nomes, funções e classe e OO - LoginViewController#17
Added: nomes, funções e classe e OO - LoginViewController#17lfoliveir4 wants to merge 2 commits intoppedroam:mainfrom
Conversation
lfoliveir4
changed the title
lfoliveir4
changed the title
| return .lightContent | ||
| } | ||
|
|
||
| func verifyLogin() { |
There was a problem hiding this comment.
sobre essa logica de verificar se o usuario ta logado tenho 2 pontos:
1 - é interessante criar um useCase para isso, assim essa logica pode ser reaproveitada pelo projeto
2 - nao faz sentido essa logica estar dentro dessa controller, é uma logica que viria antes de login
ou seja, se o usuario estive logado, manda para hoje, senão, para login
logo sugiro criar um InitialRouter ou um InitalCoordinator para lidar com esse roteamento inicial, e chamar essa classe dentro desse case no SceneDelegate:
There was a problem hiding this comment.
salve salve @ppedroam !
essa função eu movi para a validateSession ! Faz sentido?
| Globals.alertMessage(title: "Ops..", message: "Houve um problema, tente novamente mais tarde.", targetVC: self) | ||
| } | ||
|
|
||
| AF.shared.request( |
There was a problem hiding this comment.
criar viewModel ou service para chamada de api
|
|
||
|
|
||
| @IBAction func createAccountButton(_ sender: Any) { | ||
| @IBAction func createAccountDidTap(_ sender: Any) { |
There was a problem hiding this comment.
didTapCreateAccountButton*
o nomenclatura deve se assemelhar a forma inglesa de ler
| } | ||
|
|
||
| @IBAction func resetPasswordButton(_ sender: Any) { | ||
| @IBAction func resetPasswordDidTap(_ sender: Any) { |
| } | ||
|
|
||
| // MARK: - IBActions | ||
| @IBAction func loginDidTap(_ sender: Any) { |
| let vc = storyboard.instantiateViewController(withIdentifier: "LuzResetPasswordViewController") as! LuzResetPasswordViewController | ||
| vc.modalPresentationStyle = .fullScreen | ||
| present(vc, animated: true) | ||
| let viewController = storyboard.instantiateViewController( |
There was a problem hiding this comment.
criar factory para ResetViewController
| button.layer.borderColor = borderColor.cgColor | ||
| button.setTitleColor(borderColor, for: .normal) | ||
| button.backgroundColor = .white | ||
| } |
There was a problem hiding this comment.
normalmente os projeto adotam padrões de layout (DesignSystem)
logo podemos presumir que essa configuração de botao vai acontecer em varias telas
pode entao criar se uma extension de UIButton para configurar os botões
o certo mesmo seria criar uma classe do tipo UIButton que tivesse a configuração que queriamos
mas precisaríamos de viewCode
There was a problem hiding this comment.
salve salve @ppedroam !
Então, este PR ainda não contempla a refatoração para viewCode. Este ajuste pode ficar para o proximo PR que conterá view code? Como sugere?
| do { | ||
| let session = try JSONDecoder().decode(Session.self, from: data) | ||
| Task { @MainActor in self.setupHomeViewController() } | ||
| UserDefaultsManager.UserInfos.shared.save(session: session, user: nil) |
There was a problem hiding this comment.
no useCase de login teria uma funcao para salvar infos do usuario
ai vc chamaria esse useCase
| } | ||
|
|
||
| // MARK: - MakeParams | ||
| extension LuzLoginViewController { |
There was a problem hiding this comment.
passar para service ou viewmodel
| // MARK: - Setup HomeViewController | ||
| extension LuzLoginViewController { | ||
| func setupHomeViewController() { | ||
| let navigationController = UINavigationController(rootViewController: LuzHomeViewController()) |
There was a problem hiding this comment.
utilizar Factory
e criar coordinator para essa viewcontroller
| import UIKit | ||
|
|
||
| final class LuzLoginViewController: UIViewController { | ||
| private let viewModel: LuzLoginViewModel |
There was a problem hiding this comment.
fazer viewModel ser orientada a protocolo
| @IBOutlet weak var emailTextField: UITextField! | ||
| @IBOutlet weak var passwordTextField: UITextField! | ||
|
|
| emailTextField.text = "clean.code@devpass.com" | ||
| passwordTextField.text = "111111" | ||
| #endif | ||
| viewModel.setupDebugConfiguration( |
There was a problem hiding this comment.
logica de controle em elementos UI devem ser feitas na viewController
| func validateSession() { | ||
| guard UserDefaultsManager.UserInfos.shared.readSesion() != nil else { return } | ||
| setupHomeViewController() | ||
| viewModel.validateSession { [weak self] in |
There was a problem hiding this comment.
essa logica deve ser feita antes de abrir a LoginViewController
igual eu te falei, cria um InitialCoordinatol que retorna a primeira controller que vai ser aberta
dentro do coordinator vai ter essa logica de validar sessao
se estiver logado, retorna HomeViewController
se nao, retorna sua Login
isso não é uma logica que é responsabilidade da login
assim o codigo fica acoplado
| self.handleError() | ||
| Task { | ||
| do { | ||
| let data = try await viewModel.login( |
There was a problem hiding this comment.
nao precisa chamar a funcao login na viewModel pare receber o dat aqui e novamente chamar uma funcao na viewModel
vc pode chamar só um método na viewModel (login) e la dentro, a própria viewModel, chama a funcao privada handleLoginSuccess
| ) as! LuzResetPasswordViewController | ||
|
|
||
| @IBAction func didTapResetPassword(_ sender: Any) { | ||
| let viewController = LuzResetPasswordFactory.make() |
There was a problem hiding this comment.
essa linha d baixo pode ficar na factory
| completion: @escaping (UIAlertController) -> Void | ||
| ) { | ||
| if !ConnectivityManager.shared.isConnected { | ||
| let alertController = UIAlertController( |
There was a problem hiding this comment.
a logica de alerta nao deve ficar na viewModel
viewModel é apenas regra de negocio
vc pode colocar no coordinator ou na própria viewController
| } | ||
| } | ||
|
|
||
| func makeParams( |
| UserDefaultsManager.UserInfos.shared.save(session: session, user: nil) | ||
| } | ||
|
|
||
| func validateButton( |
There was a problem hiding this comment.
confesso que achei essa codigo complexo para uma logica simples
pq nao criar uma funcao de uma linha na viewModel assim:
func isEmailValid(email: String?) -> Bool { return LuzLoginEmailValidator.validate(email)
vc esta trabalhando com duas closures escaping apenas para validar uma string
isso é um caso de overEngineering
ppedroam
left a comment
ppedroam
left a comment
There was a problem hiding this comment.
code review 2
ignora o comentario de fazer protocolo para viewModel pq essa tarefa nao abrange POP e DIP
2 participants
Este PR contempla a refatoração de mais uma view controller:
tratar classe
tratar nome de funções
tratar nomes
um pouco do OO (encapsulamento)