nillias / pokemondex Goto Github PK

View Code? Open in Web Editor NEW

4.0 1.0 1.0 2.35 MB

Swift 100.00%

pokemondex's Introduction

PokemonDex

pokemondex's People

Contributors

Stargazers

Watchers

Forkers

luizerz

pokemondex's Issues

Code Review

Desacoplamento

Primeiramente, parabéns pelo desempenho que tu teve Nillia, apesar de parecer simples eu sei que você teve muito trabalho!

Na linha de código seguinte, presente no arquivo API.swift, temos que o trecho de código que deveria está em um arquivo separado (model) se encontra dentro de outro.

PokemonDex/PokemonDex/Service/API.swift

Lines 10 to 22 in 5c78b84

    
           struct Data: Codable { 
        
               let data: [Card] 
        
           } 
        
           struct Card: Codable { 
        
               let id, name: String 
        
               let images: Images 
        
           } 
        
           struct Images: Codable { 
        
               let large: String 
        
           }

Para desacoplar essa sequencia de código devemos criar um novo arquivo .swift, basta apertar "cmd + n", escolher a opção swift file e nomear como uma model (ex: DataModel).

Outro arquivo que necessita de um desacoplamento é o ViewController.swift, esse é bem mais rápido já que você não precisará que criar um novo arquivo. Tendo o seguinte trecho de código:

class ViewController: UIViewController, UISearchResultsUpdating {
    
    //busca da search bar
    func updateSearchResults(for searchController: UISearchController) {
        guard let text = searchController.searchBar.text else {
            return
        }
        
        print(text)
    }

Podemos desacoplar o protocolo UISearchResultsUpdating utilizando um extension e trazendo a func dele para dentro desse extension

CodeExemple do review anterior:

extension ViewController: UISearchResultsUpdating {
 func updateSearchResults(for searchController: UISearchController) {
        guard let text = searchController.searchBar.text else {
            return
        }
}

Trabalhando nos apontamentos feitos, seu código se tornará mais legível, desacoplado, e padronizado. Voltando a inteirar parabéns por ter indo em frente com o projeto e não desistido 💯

Olá Nillia, seu projeto ficou bem legal, dá para notar que você se empenhou nele!

Percebi que você deixou todas as funções dos métodos da sua ViewController em apenas um lugar. Da seguinte forma:

PokemonDex/PokemonDex/View/CardSaveViewController.swift

Lines 56 to 73 in b9bc3b4

    
           func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int { 
        
               return dataSave.count 
        
           } 
        
           func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell { 
        
               let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! CardUICell 
        
               cell.configFromCoreData(with: dataSave[indexPath.row]) 
        
               cell.removeButton() 
        
               return cell 
        
           } 
        
           func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) { 
        
               tableView.deselectRow(at: indexPath, animated: true) 
        
               print(dataSave[indexPath.row].url ?? "") 
        
           }

Ficaria melhor organizado, bem como maia fácil de ler, se você fizesse uma extensão da sua classe para método.

Aqui um exemplo de como ficariam as extensões com seus respectivos métodos e func:

extension CardSaveViewController: UITableViewDataSource {
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
            return dataSave.count
        }
        
        func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
            let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! CardUICell
           
            cell.configFromCoreData(with: dataSave[indexPath.row])
            cell.removeButton()
            return cell
        }
}

extension CardSaveViewController: UITableViewDelegate {
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        tableView.deselectRow(at: indexPath, animated: true)
        print(dataSave[indexPath.row].url ?? "")
        
    }
}

nillias / pokemondex Goto Github PK

pokemondex's Introduction

PokemonDex

pokemondex's People

Contributors

Stargazers

Watchers

Forkers

pokemondex's Issues

Code Review

Desacoplamento

Primeiramente, parabéns pelo desempenho que tu teve Nillia, apesar de parecer simples eu sei que você teve muito trabalho!

CodeExemple do review anterior:

Olá Nillia, seu projeto ficou bem legal, dá para notar que você se empenhou nele!

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	struct Data: Codable {
	let data: [Card]
	}

	struct Card: Codable {
	let id, name: String
	let images: Images
	}

	struct Images: Codable {
	let large: String
	}

	func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
	return dataSave.count
	}

	func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

	let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! CardUICell

	cell.configFromCoreData(with: dataSave[indexPath.row])
	cell.removeButton()
	return cell
	}

	func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
	tableView.deselectRow(at: indexPath, animated: true)
	print(dataSave[indexPath.row].url ?? "")

	}