Here at ABA we love Music and we know our users do it also!.
We want to give them the opportunity to learn English by singing their favourite songs.
Actually we have a current section that we would like to improve and restyle.
Usability is a most for us and we would like to have a fresh user experience.
Feel free to modify anything you think would improve the section.
This project is using cocoapods and the pods are included in the repo. Anyway if you have any problem please run the pod install command.
If you have any doubt about cocoapods you can check the reference here.
References:
Basically I have a protocol file for each scene in the app. This file defines the interaction between each layer as following:
- View - Presenter: protocols to notify changes and to inject information to the UI.
- Presenter - Interactor: protocols to request / receive information to / from the interactor.
- Presenter - Router: protocol to define the transitions between scenes.
Whith this protocols file is really easy to know how each layer notify / request / information to the other ones so we don't have any other way to communicate all the layers.
Another important point is because I'm using protocols it's really easy to define mocks views / presenters / interactors / routers for testing.
// View / Presenter
protocol SearchListViewInjection : class {
func showProgress(_ show: Bool, status: String)
func showProgress(_ show: Bool)
func showMessageWith(title: String, message: String, actionTitle: String)
func loadTracks(_ viewModels: [TrackViewModel], fromBeginning: Bool)
func loadSuggestions(_ suggestions: [SuggestionViewModel])
}
protocol SearchListPresenterDelegate : class {
func viewDidLoad()
func searchTrack(_ search: String?)
func trackSelectedAt(section: Int, index: Int)
func getSuggestions()
func suggestionSelectedAt(_ index: Int)
}
// Presenter / Interactor
typealias TracksGetTracksCompletionBlock = (_ viewModel: [TrackViewModel]?, _ success: Bool, _ error: ResultError?) -> Void
typealias TrackListGetSuggestionsCompletionBlock = ([SuggestionViewModel]) -> Void
protocol SearchListInteractorDelegate : class {
func getTracksList(search: String?, completion: @escaping TracksGetTracksCompletionBlock)
func clear()
func getTrackSelectedAt(section: Int, index: Int) -> TrackViewModel?
func getAllSuggestions(completion: @escaping TrackListGetSuggestionsCompletionBlock)
func getSuggestionAt(index: Int) -> SuggestionViewModel?
func getInitialSearch() -> String
}
// Presenter / Router
protocol SearchListRouterDelegate : class {
func showTrackDetail(_ track: TrackViewModel)
}I'm using the iTunes search api. You can check the API here.
public struct TracksResponse: Decodable {
let resultCount: UInt
let results: [TrackResponse]
}
public struct TrackResponse: Decodable {
let artistName: String
let trackName: String
let trackViewUrl: String
let previewUrl: String?
let artworkUrl100: String
let releaseDate: String
let primaryGenreName: String
}I'm using a Swift Standard Library decodable functionality in order to manage a type that can decode itself from an external representation (I really ❤ this from Swift).
Reference: Apple documentation
This model is used for the local suggestions:
class SearchSuggestion: Object {
@objc dynamic var suggestionId: String?
@objc dynamic var suggestion: String = ""
@objc dynamic var timestamp: TimeInterval = NSDate().timeIntervalSince1970
override class func primaryKey() -> String? {
return "suggestionId"
}
}As I'm using Realm for this it's important to define a class to manage each model in the database. In this case we only have one model (FavoriteRecipe)
Reference: Realm
I think using managers is a good idea but be careful!. Please don't create managers as if the world were going to end tomorrow.
I'm using only 3 here:
Used to manage the reachability. In this case I would like to notify a little issue related with the simulator. It seems Xcode has an issue with the simulator because if you try to turn off the wifi and turning on again, the observer for the state change is not triggering. It's working 100% fine in a real device.
Used to save the sucessfull searchs locally using a Realm database.
Used to process the images from iTunes API.
The API response has 3 different sizes for artists / tracks:
- 30x30 px
- 60X60 px
- 100x100 px
In this case if we're using the max size for images the result is not the best. I did some test related with this point and I noticed that the API is able to respond other sizes too. In this case I decided to use a 200x200 px. I know this is not the best option but I think this can create a better UX for the user.
- Realm migration process: It would be nice to add a process to migrate the realm database to a new model (just in case you need to add a new field into the database).
- Localizable strings files: I didn't add localizable strings files.
- Update the managers: I implemented the managers using singletons. The best option should be to remove the singletons in order to use dependency injections (to improve the testing).
- Add more UI tests: It would be great to add more UI tests to the project.
- Create a manager to manage the video player (AVPlayer).
- Swift 4.2
- Xcode 10.1
- Cocoapods 1.5.3
- Minimun iOS version: 11.0
- RealmSwift (3.13.1): A mobile database that runs directly inside phones, tablets or wearables.
- SVProgressHUD (2.2.5): A clean and lightweight progress HUD for your iOS and tvOS app.
- Haneke (1.0): A lightweight zero-config image cache for iOS, in Objective-C.
You can contact me using my email: [email protected]
Follow me @rcasanovan on twitter.