gustavotiecker / spacex-info-app

My first personal project, an app with information about SpaceX rockets and launches, built using the SpaceX-API.

SpaceX Info App

My first personal project, an app with information about SpaceX rockets and launches, built using the SpaceX-API.

Home Screen

Rockets Screen

Launches Screen

Getting Started

These instructions will get you a copy of the project up and running on your local machine for development and testing purposes.

Prerequisites

• To run any iOS application built with Swift you need to configure the environment on your machine.
• Xcode 11.0, *

Clone

• Clone this repo to your local machine using:

https://github.com/gustavotiecker/spacex-info-app

Built With

• Swift - A powerful open language that lets everyone build amazing apps.

License

This project is licensed under the MIT License - see the LICENSE.md file for details

spacex-info-app's Issues

Review

Quando trabalho com Swift, eu gosto de considerar três pilares importantes na construção de projeto que envolve a linguagem.

• Value Type
• Protocol Oriented Programming
• Functional Programming

organização de projeto,
Achei okay. Bem direto.

Extensions
Achei legal. Só mudaria para +Extension. Eu recomendo sempre evitar abreviação. Lembre-se que código é sempre para humano ler, 99% sabe o que significa Ext, mas aquele 1% : ).

Models

• Eu usaria mais de Protocolos. Suas models ficaram muito concretas. Trabalhe com POP mais. Gostei que sempre preferiu value type em vez de reference type.
• Algumas CodingKeys ficaram com valores associados ao case de forma confusa. Ex: Height só vai ler meters do JSON?. Se sim, ignora o que eu falei.

Network Manager

• Sua manager está muito amarrada e pouco escalável. O network manager foi zero protocolada e muito concreta. O Network poderia ser mais abstrato e ser em cima de Protocolos para criar todas as estruturas necessárias de uma NetworkLayer. Quando falo amarrada, meio que seu Network Manager é só testável se a baseURL for a que está dentro da classe.
• As funções são uns 95% iguais. Os métodos todos poderiam ser criados através de Protocolos e Generics.
• DownloadImage eu colocaria em uma extensão de UIImage e mudaria algumas coisas nela para que não fique tão vinculada a uma NetworkManager, pois nesse caso mesmo, ela não precisa saber de nada da classe onde ela se encontra. Princípio SOLID fala sobre isso.

Constants

• Achei bacana que você colocou os Assets Literal lá dentro. Depois dá uma olhada no Pod R.swift para gerar Static Files.
• SPXError para mim não deveria ficar ai.

View Controllers

• Todo "Configure"/"Setup" que seja relacionado a UI, pode ser feito uma BaseViewController e faz o seu ViewController final herdar desse BaseVIewController. Isso vai diminuir bastante o seu código em questão de Layout do ViewController.
• A sua tableview foi criada por uma variável. Poderia ser por variável computada, como o tableView sempre vai estar na tela, não precisaria ser lady.
• Toda parte de View-Code do projeto é repetido em diferentes ViewControllers, sendo assim, poderia ser criado algum Protocolo para lidar com View-Code
• Recomendo desafogar o seu ViewController através de DataSource e Delegate Pattern. Tirar aquele bloco dele onde ele tem que fazer todo o Data Source e Delegate na classe.
• Dentro do Data Source evita aquele Force Unwrap para fazer o Downcast. Aquilo ali é perigoso. Lida com um guard ali e no else do Guard, devolve um UITableViewCell()
• Em relação ao didSelect. O VC sabe de todos os outros Controllers que ele direciona a Model. Dá uma olhada em Coordinator Pattern depois, serve para desafogar esse tipo de problema.

Gostei de todo controle de memória. Houve Zero Leak de memória. Parabéns. Gostei também das classes de UIView Customizadas.

Artigos:
https://www.hackingwithswift.com/articles/159/how-to-refactor-massive-view-controllers
https://davedelong.com/blog/2020/06/27/http-in-swift-part-1/
https://www.toptal.com/swift/introduction-protocol-oriented-programming-swift
https://www.hackingwithswift.com/articles/71/how-to-use-the-coordinator-pattern-in-ios-apps

Pods:
https://github.com/mac-cain13/R.swift