nerthase / tablemanager Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 21 KB

Swift 100.00%

tablemanager's People

Contributors

Watchers

tablemanager's Issues

Come fare un update condizionale?

TableManager/TableManager/ViewController.swift

Lines 79 to 84 in 391a6d7

 let cellLabel = CellLabelManager(text: self.user.description) 

 tmpCellManager.append([cellLabel]) 

 tmpIndexPaths.append(cellLabel.indexPath(inArray: tmpCellManager)) 

 callback(tmpCellManager, tmpIndexPaths)

Questa soluzione funziona molto bene in questo caso specifico. Non mi sembra però opportuno che sia il metodo populateTable a decidere quale cella aggiornare, solitamente questa logica nasce da altre parti del codice. Immagina che dovrà esserci qualche altro parametro passato al metodo, per decidere chi/quando aggiornare.

TableManager/TableManager/Protocols/Protocols.swift

Lines 17 to 39 in 391a6d7

 extension CellManager { 

 func indexPath(inArray array: [[CellManager]]) -> IndexPath { 

 let sectionElement = array.first(where: { (section) -> Bool in 

 section.contains(where: { (row) -> Bool in 

 row === self 

 }) 

 }) 

 let sectionIndex = array.index(where: { (section) -> Bool in 

 section.contains(where: { (row) -> Bool in 

 row === self 

 }) 

 }) 

 let rowIndex = sectionElement?.index(where: { (row) -> Bool in 

 row === self 

 }) 

 guard let section = sectionIndex, let row = rowIndex else { return IndexPath() } 

 return IndexPath(row: row, section: section) 

 } 

 }

Visto che c'ero ho racchiuso un po' di logica per generare un IndexPath a partire da un elemento all'interno del dataSource

Codice duplicato per passare informazioni da Controller a Cell

TableManager/TableManager/Cells/Field/CellField.swift

Lines 42 to 50 in 391a6d7

 class CellFieldManager: CellManager { 

 var placeholder: String? 

 var text: String? 

 var keyboardType: UIKeyboardType = .default // Necessario per poi passarlo a CellField 

  var isSecureTextEntry: Bool = false // Necessario per poi passarlo a CellField 

 var blockText: Blocks.stringBlock?

Introducendo il CellManager è necessario duplicare le proprietà che il ViewController passa alla Cell. C'è sempre una sorta di codice duplicato tra Manager e Cell, mentre prima il ViewController passava direttamente queste informazioni alla Cell

Il Manager deve "auto aggiornarsi"

TableManager/TableManager/Cells/Field/CellField.swift

Lines 61 to 71 in 391a6d7

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

 tableView.register(CellField.self) 

 let cell: CellField = tableView.deque(indexPath) 

 cell.build(placeholder: self.placeholder, text: self.text, keyboardType: self.keyboardType, isSecureTextEntry: self.isSecureTextEntry, block: { 

 self.text = $0 // Necessario per mantenere aggiornato CellFieldManager, che altrimenti continuerebbe ad avere un valore vecchio 

  self.blockText?($0) 

 }) 

 return cell 

 } 

 }

Il Manager non può limitarsi a passare i valori dalla Cell al Controller, ma deve anche preoccuparsi di mantenersi aggiornato. Questo succede perché il valore originale, disponibile nell'oggetto User è disaccoppiato dal valore memorizzato nel Manager.

Ho provato a passare tutto l'oggetto User ma andrebbe implementata una logica ulteriore per individuare la proprietà da considerare. Inoltre questa Cell generica non sarebbe poi adatta ad altri tipi di oggetti, in quanto accoppiata fortemente con User.

Ho provato anche con protocolli/subclass, ma passare il nome del parametro tramite String proprio non mi piace.

nerthase / tablemanager Goto Github PK

tablemanager's People

Contributors

Watchers

tablemanager's Issues

Come fare un update condizionale?

Codice duplicato per passare informazioni da Controller a Cell

Il Manager deve "auto aggiornarsi"

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

	let cellLabel = CellLabelManager(text: self.user.description)
	tmpCellManager.append([cellLabel])

	tmpIndexPaths.append(cellLabel.indexPath(inArray: tmpCellManager))

	callback(tmpCellManager, tmpIndexPaths)

	extension CellManager {
	func indexPath(inArray array: [[CellManager]]) -> IndexPath {
	let sectionElement = array.first(where: { (section) -> Bool in
	section.contains(where: { (row) -> Bool in
	row === self
	})
	})

	let sectionIndex = array.index(where: { (section) -> Bool in
	section.contains(where: { (row) -> Bool in
	row === self
	})
	})

	let rowIndex = sectionElement?.index(where: { (row) -> Bool in
	row === self
	})

	guard let section = sectionIndex, let row = rowIndex else { return IndexPath() }

	return IndexPath(row: row, section: section)
	}
	}

	class CellFieldManager: CellManager {

	var placeholder: String?
	var text: String?
	var keyboardType: UIKeyboardType = .default // Necessario per poi passarlo a CellField
	var isSecureTextEntry: Bool = false // Necessario per poi passarlo a CellField

	var blockText: Blocks.stringBlock?

	func getCell(tableView: UITableView, indexPath: IndexPath) -> UITableViewCell {
	tableView.register(CellField.self)

	let cell: CellField = tableView.deque(indexPath)
	cell.build(placeholder: self.placeholder, text: self.text, keyboardType: self.keyboardType, isSecureTextEntry: self.isSecureTextEntry, block: {
	self.text = $0 // Necessario per mantenere aggiornato CellFieldManager, che altrimenti continuerebbe ad avere un valore vecchio
	self.blockText?($0)
	})
	return cell
	}
	}