Some code style remarks
Code looks clean overall but I want to point out a few things:
- why not use an uninhabited type like an empty enum (enum with no cases) in RBConstants.swift to prevent instantiations of the
RBConstants
type ? - in RBManager.swift which seems to describe a singleton, why not clearly add a private initializer to make sure that only one instance will be created ?
- you seem to retrieve timestamps in a few places (in RBManager.swift or Message/MessageManager.swift) using a helper
timeIntervalSince1900
ofDate
which returns anInt
, so why those are wrapped again in Int values like this:Int(Date().timeIntervalSince1900)
?