Swift Quiz

gistfile1.txt

import UIKit

protocol ReaderDelegate: class {
    func didReadData(data: Data)
}

class Reader{
    var file: String!
    var output: ReaderDelegate?
    var readCompleteBlock: (() -> Void)?
    
    func read() {
        let fileUrl = URL(fileURLWithPath: file!)
        let data = try? Data(contentsOf: fileUrl)
        self.output?.didReadData(data: data!)
        self.readCompleteBlock?();
    }
}

class orderReader: ReaderDelegate {
    public var reader: Reader
    init(_ file: URL) {
        self.reader = Reader()
        self.reader.file = file.absoluteString.replacingOccurrences(of: "file://", with: "")
        self.reader.output = self
        self.reader.readCompleteBlock = {
            self.didComplete()
        }
    }
    
    func Read() {
        self.reader.read()
    }
    
    func didComplete() {
        print("end of file")
    }
    
    func didReadData(data: Data) {
        print("\(data)")
    }
}

let filePath = Bundle.main.path(forResource: "myOrders.csv", ofType: nil)
let orderReader = orderReader(URL(fileURLWithPath: filePath!))
orderReader.Read()

1. Лучше при создании протокола, указывать AnyObject, а не class, есть разница
2. Class Reader надо пометить final, так как нет наследований, что оптимизирует его создание
3. Output надо указать weak, чтобы не было retain cycle’ов
4. file указывать через force unwrapped не есть хорошо. Лучше сделать его опционалом и в функции read использовать его через if let, а лучше guard в начале метода, так как без файла вся функция не имеет смысла
5. try лучше использовать через do catch
6. Опять же избавится от force uwrapped для data
7. Я бы покрыл протоколом класс Reader, чтобы Мы могли через DI отдать его классу orderReader, так же Мы будем следовать принципу D из солида. Этот класс так же должен быть final)
8. переменные и функции должны быть приватны
9. для filePath тоже надо убрать forceUwrapped
10. Для ресурсов лучше иметь какую-нибудь кодогенерацию, чтобы не писать через стринг доступ к файлам