我想重构我的代码以应用干净的方法。

我有一个类 user

class User { 
    let name: String?
    let id: String
    var isOnline: Bool

    var mesaageHistory = [Message]()

    init(name: String, id: String, isOnlineStatus: Bool) {
        self.name = name
        self.id  = id
        self.isOnline = isOnlineStatus
    }

}

然后我使用结构模式来创建我的用户列表。

protocol CreateUserProtocol: class {
    func sortArray(inputArr: [User])
}


class CreateUserEntity: CreateUserProtocol {

    static let shared = CreateUserEntity()
    var users = [User]()

    func sortArray(inputArr: [User]){
        var applySortingByDate: Bool = false

        for user in inputArr {
            if !user.mesaageHistory.isEmpty {
                applySortingByDate = true
            }
        }

        if applySortingByDate {
            inputArr.sorted(by: { (first, second) -> Bool in
                (first.mesaageHistory.last?.messageTime)! < (second.mesaageHistory.last?.messageTime)!
            })
        } else {
            inputArr.sorted(by: { (first, second) -> Bool in
                first.name! < second.name!
            })
        }
    }

}

一个 Controller 负责添加新用户，而另一个 Controller 用于检索它们并将它们绑定(bind)到 tableView。一切正常，但我认为我的解决方案不足以扩展。

此外，在我的一个 VC 中，我用来将我的用户分类为在线和离线。我认为，我不应该在我的 VC 中这样做并将此逻辑放入我的 CreateUserEntity

var onlineUsersData = [User]()
var offlineUsersData = [User]()

private func classifyUsers() {
    for user in CreateUserEntity.shared.users {

        print("is user online: \(user.isOnline)")
        print(CreateUserEntity.shared.users.count)

        if user.isOnline == true && !onlineUsersData.contains(user) {
            onlineUsersData.append(user)
        }

        if user.isOnline == false && !offlineUsersData.contains(user) {
            offlineUsersData.append(user)
        }
    }
}

我想以适当的方式重写它，你有什么可以推荐给我的吗？

最佳答案

在我看来，首先尝试使用 struct 而不是 class。

例子:

struct User {
    let name: String
}

然后你应该弄清楚你将把这些用户存储在哪里？现在他们在内存中。因此，我们应该定义存储它们的位置和方式。

所以对于这种情况，我们可能会考虑将 NSUserDefaults 作为存储用户的类的核心。之后我们应该为此类创建外观来管理我们的用户。

protocol UserStoreFacade {
    func fetch(name withName:String) -> User
    func create(name withName:String) -> User
    func save(user:User)
    func update(name newName:String) -> User
    func delete(name withName:String)
}

和用于管理用户的UserStore。

class UserStore: UserStoreFacade {

    let defaults = UserDefaults(suiteName: "User")

    func fetch(name withName:String) -> User {
         let encodeData = defaults?.dictionary(forKey: withName)
         return User(dictionary: encodeData as! Dictionary<String, AnyObject>)
    }

    func create(name withName: String) -> User {
        return User(name: withName)
    }

    func save(user: User) {
        defaults?.set(user.encode(), forKey: user.name)
    }

    func update(name newName:String) -> User {
        return User(name: newName)
    }

    func delete(name withName:String) {
         defaults?.removeObject(forKey: withName)
    }
}

它非常原始，但仍然展示了如何实现的案例。

关于ios - 如何避免以正确的方式使用单例？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/43632295/