So we have to use CoreData and we have to use a UITableView, now I'm asking how does that work with a NSFetchedResultsController? We have something already set up for other basic sectioned views where we dont need a fetchresultscontroller and that was pretty easy, but this seems a little trickier. There is a little support out there for our old pal Objective-C that has been around for years but we are looking for a Swift solution. Swift 5 at the time of this post.
Have already implemented the plain list (which is fairly well documented) and now we just want to add some sections...
So we have a UITableView and a NSFetchedResultsController and we get a nice list of Entities. Now we want to add a few sections into the tableview.
No Problem so we update the sortDescriptors to include a section:
fetchRequest.sortDescriptors =
[NSSortDescriptor(key: "sectionLabel", ascending: false),
NSSortDescriptor(key: "unixTimestamp", ascending: false)]
and the entity to include the sectionLabel, this part is a bit more custom depending on what kind of sections you want.
Then we add a few section functions :
func numberOfSections(in tableView: UITableView) -> Int {
return self.fetchController?.sections?.count ?? 0
}
func tableView(_ tableView: UITableView, viewForHeaderInSection section: Int) -> UIView? {
guard AllTheThings( fetchController -> sections -> entity -> cell )
if sections.count > 1 {
cell.configureCell(value: entity.sectionLabel ?? "myLabel")
return cell.contentView
} else { return UIView() }
}
func tableView(_ tableView: UITableView, heightForHeaderInSection section: Int) -> CGFloat {
return 20.0
}
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
guard let sections = fetchController?.sections else { return 0 }
return sections[section].numberOfObjects
}
So things are starting to shape up but theres still some work needed on the cellForRowAt indexPath and other standard cell functions. So in order to get an entity for the index to use in those we create a cool function:
func returnEntity(indexPath: IndexPath) -> MyEntity? {
guard let sections = fetchController.sections else {
return nil
}
guard let objects = sections[indexPath.section].objects else {
return nil
}
if objects.isEmpty {
return nil
}
guard let entity = objects[indexPath.row] as? MyEntity else {
return nil
}
return entity
}
Left the guards in this time ;)
Now all you have to do to get your entity is use returnEntity(indexPath: indexPath)
Related
Ok first thing first, what am I trying to do? Well I am trying to run something that is like a tag system where it filters out the data with the post by using a dictionary String, [String] type and displays it to the screen. I already figured this out on the console level, but I am stumped on how to do it with this which is kind of weird. I try and it returns nil inside the UI ,but works perfectly on the console level.
Simplified. I want the quick tags filtered array to show up in the tableview
I am repeating this again, console works perfect, but the UI gets wacky and returns nil or does nil. Ok here is the code.
Not 100% sure about this area causing problems ,but I displayed here just in case.
//this is the part where I add the stuff into the console
//and add the dictionary part.
#IBAction func ReplyAction(_ sender: UIButton) {
if !(TextFieldForComments.text?.isEmpty)! && TextFieldForComments.text != nil
{
CommentGlobals.shared.addToCommentSection(newElement: TextFieldForComments.text!)
let tagCheck = TextViewForComment.text
if !quickTags.FilteredComments.keys.contains(TextViewForComment.text){
quickTags.FilteredComments.updateValue(["\(String(describing: TextFieldForComments.text))"], forKey: tagCheck!)
print("Hey here is the dictinary you wanted \(quickTags.FilteredComments)")
}
else {
quickTags.FilteredComments[tagCheck!]?.append(CommentGlobals.shared.commentSection.last!)
print("Hey here is the dictinary you wanted wo wo \(quickTags.FilteredComments)")
}
TextFieldForComments.text = ""
//this line of code is important or it
//won't insert the table view right.
CommentFeed.insertRows(at: [IndexPath(row: CommentGlobals.shared.commentSection.count-1, section: 0)], with: .automatic)
}
Here is the problem area
//this is the problem area
func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
let ceal = CommentFeed.dequeueReusableCell(withIdentifier: "commentFeed", for: indexPath)
//guard let selectedDictionary = quickTags.FilteredComments["\(TextViewForComment)"] else {return ceal}
//this is the part that works, but noted it out for reference
//this doesn't work for what I am trying to do because
//I don't want to display the comments of every view
//ceal.textLabel?.text = "\(CommentGlobals.shared.commentSection[indexPath.row])"
//this is failure. I also tried another way ,but it just printed nil
//on to the UI
//ceal.textLabel?.text = "\(selectedDictionary[indexPath.row])"
return ceal
}
Ok if you need more information, please let me know.
Oh yea I can't say this enough it works on a console level perfectly, but not when I try to get it onto the UI.
I also stored the quick tags in a static array, and I stored the rest in a singleton
Here is the expected output (UI)
comment
comment
comment
the current output is like this(UI)
(it does nothing, runs nil, or crashes)
some other sources lead the thing to be like this
comment comment comment
all of them on the same line which is not what I want.
func numberOfSections(in tableView: UITableView) -> Int {
// #warning Incomplete implementation, return the number of sections
return 1
}
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
// #warning Incomplete implementation, return the number of rows
// return CommentGlobals.shared.commentSection.count
print(quickTags.FilteredComments.count)
return CommentGlobals.shared.commentSection.count
}
UITableViews do not contain their own data set, they take it from the dataSource. insertRows(at:, with:) is only for cell animation, and should be wrapped with begin/endUpdates().
class ViewController: UITableViewDataSource {
#IBAction func ReplyAction(_ sender: UIButton) {
// ...
CommentFeed.beginUpdates()
CommentFeed.insertRows(at: [IndexPath(row: CommentGlobals.shared.commentSection.count - 1, section: 0)], with: .automatic)
CommentFeed.endUpdates()
}
func numberOfSections(in tableView: UITableView) -> Int {
return 1
}
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return CommentGlobals.shared.commentSection.count
}
func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCell(withIdentifier: "commentFeed", for: indexPath)
cell.textLabel?.text = CommentGlobals.shared.commentSection[indexPath.row]
return cell
}
}
Edit: Note that your data set's number of rows and sections MUST match the expected results at the end of the animation, or you will get an exception when calling endUpdates().
I'm building an app, where I got several sections in an UITableView. My current solution is collecting my data in a dictionary, and then pick one key for every section. Is there a better solution?
One of the good ways to it - direct model mapping, especially good with swift enums.
For example you have 2 different sections with 3 different type of rows. Your enum and ViewController code will look like:
enum TableViewSectionTypes {
case SectionOne
case SectionTwo
}
enum TableViewRowTypes {
case RawTypeOne
case RawTypeTwo
case RawTypeThreeWithAssociatedModel(ModelForRowTypeNumberThree)
}
struct ModelForRowTypeNumberThree {
let paramOne: String
let paramTwo: UIImage
let paramThree: String
let paramFour: NSData
}
struct TableViewSection {
let type: TableViewSectionTypes
let raws: [TableViewRowTypes]
}
class ViewController: UIViewController, UITableViewDataSource {
var sections = [TableViewSection]()
func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return sections[section].raws.count
}
func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let raw = sections[indexPath.section].raws[indexPath.row]
switch raw {
case .RawTypeOne:
// Here return cell of first type
case .RawTypeTwo:
// There return cell of second type
case .RawTypeThreeWithAssociatedModel(let modelForRawTypeThree):
// And finally here you can use your model and bind it to your cell and return it
}
}
}
What benefits? Strong typization, explicit modelling, and explicit handling of your various cell types. The only simple thing that you have to do in that scenario it is parse your data into this enums and structs, as well as you do it for your dictionaries.
Here is a quick example that I wrote. Please note, it error-prone since it is not checking wether the keys exists not does it create a proper cell.
You could do this with a dictionary as well, since you can iterate over its content.
Hope it helps:
class AwesomeTable: UITableViewController {
private var tableContent: [[String]] = [["Section 1, row 1", "Section 1, row 2"], ["Section 2, row 1", "Section 2, row 2"]]
override func numberOfSectionsInTableView(tableView: UITableView) -> Int {
return tableContent.count
}
override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return tableContent[section].count
}
override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCellWithIdentifier("reuseIdentifier", forIndexPath: indexPath)
let item = tableContent[indexPath.section][indexPath.row]
cell.textLabel?.text = item
return cell
}
}
Implement the table view datasource as follows:-
1) Set number of sections = no of keys in dictionary
2) No of rows in section = no of values in dictionary at index(section)
I use enum for building my UITableView cells:
enum VAXSections: Int
{
case Segmented = 0
case Scrollable = 1
case ScheduledMode = 2
case ScheduledFooter = 3
case SilentHours = 4
case SilentFooter = 5
}
here how I use it:
func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell
{
guard let unwrappedSection = VAXSections(rawValue: indexPath.section) else {
showFatalError()
return nil
}
Few problem here I want to guard my section value if it's out of max case in my enum. For example if indexPath.section will be bigger then 5 then app should fall back. But if you see we can't return nil here, as cellForRowAtIndexPath has to return cell in any case.
I can solve problem by providing more readability with replacing showFatalError() fiction with:
guard let unwrappedSection = VAXSections(rawValue: indexPath.section) else {
fatalError("Check \(String(VAXUnitDashboardViewController.self)) UITableView datasource or check \(String(VAXSections.self)) enum.")
}
then I don't need to return any value. But then I turned in another problem. As I need to specify at least 3 datasource functions of my UITableView I need to duplicate fatal error which I wish to replace with one function that do the same all the time:
fatalError("Check \(String(VAXUnitDashboardViewController.self)) UITableView datasource or check \(String(VAXSections.self)) enum.")
enum VAXItems: String {
case Item1
case Item2
case Item3
}
enum VAXSections: String {
case Segmented
case Scrollable
case ScheduledMode
case ScheduledFooter
case SilentHours
case SilentFooter
}
struct VAXModel {
var type: VAXSections
var items: [VAXItems]
}
Then on your UIViewController you can have:
class ViewController: UIViewController {
private var model: [VAXModel] = []
override func viewDidLoad() {
super.viewDidLoad()
model = [
VAXModel(type: .ScheduledMode, items: [.Item1, .Item2, .Item3]),
VAXModel(type: .SilentFooter, items: [.Item1])
]
}
func numberOfSectionsInTableView(tableView: UITableView) -> Int {
return model.count
}
func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return model[section].items.count
}
func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCellWithIdentifier(String(UITableViewCell), forIndexPath: indexPath)
let item = model[indexPath.section].items[indexPath.row]
switch item {
case .Item1: cell.textLabel?.text = item.rawValue
case .Item2: // Config
case .Item3: // Config
}
return cell
}
}
I don't think you need to have it in 3 places actually. Assuming that the 3 data source methods you are talking about are :
func numberOfSectionsInTableView(_ tableView: UITableView) -> Int
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int
func tableView(_ tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell
You actually need to have your guard in only one of them.
numberOfSectionsInTableView is the first method that will be called, so if you fail here, the other two methods won't be called. If the number of sections is based on some calculations, you can also cut off the value, something like this : if calculatedNumberOfSections > 6 { return 6 } else { return calculatedNumberOfSections } (remember that section numbering is 0 based)
numberOfRowsInSection - if you guard here you have two options - either fail with fatalError or (better in my opinion) - return 0 if a section number higher than 5 gets passed. Returning 0 will result in cellForRowAtIndexPath not being called with that section.
cellForRowAtIndexPath - you already got this :)
I want to show 25 of the songs I have in my library. This is my code:
var allSongsArray: [MPMediaItem] = []
let songsQuery = MPMediaQuery.songsQuery()
func numberOfSectionsInTableView(tableView: UITableView) -> Int {
return 1
}
func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return 25 //allSongsArray.count
}
func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let cell = tableView.dequeueReusableCellWithIdentifier("cell")
let items = allSongsArray[indexPath.row]
cell?.textLabel?.text = items.title
cell?.detailTextLabel?.text = items.artist
cell?.imageView?.image = items.artwork?.imageWithSize(imageSize)
return cell!
}
When I run this, it crashes, because:
fatal error: Index out of range
I tried to change the numberOfRowsInSection to allSongsArray.count, but it ends up with the same error.
You should return allSongsArray.count and to avoid being returned empty cells use yourTableView.reloadData after you fill your allSongsArray.
When you first create the array it is empty. Hence, it will give you out of bound error.
Try to return the count of the songs array instead.
1st you need to get the data into the array and then update the table view.
Here is a sample code:
#IBAction private func refresh(sender: UIRefreshControl?) {
if myArray.count > 0 {
self.tableView.reloadData()
sender?.endRefreshing()
} else {
sender?.endRefreshing()
}
}
In case your app crashes in
let items = allSongsArray[indexPath.row]
as you check if the allSongsArray.count, u can safe guard it by making sure that the [indexPath.row] doesn't exceed your array count. so u can write a simple if condition as;
if allSongsArray.count > 0 && indexPath.row < allSongsArray.count {
//Do the needful
}
Please return the actual array count instead of static
return allsongsarray.count
For folks looking for a solution to swiping on a filtered list, you need to provide the section as well as the row or xcode throws this error.
func swipe(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
let myRow = filteredList[indexPath.section].items[indexPath.row]
}
not
func swipe(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
let myRow = filteredList[indexPath.row]
}
This is a bit of a continuation of this question, but basically I am trying to figure out how I can return no cell if the result of a function is nil.
This is my code:
override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
var cell = tableView.dequeueReusableCellWithIdentifier("rideCell", forIndexPath: indexPath) as! RideCell
var ride = DataManager.sharedInstance.getRideByName(favouritesArray[indexPath.row] as! String)
println(ride)
if ride != nil {
cell.rideNameLabel.text = ride!.name
var dateFormat = NSDateFormatter()
dateFormat.dateFormat = "h:mm a"
cell.updatedLabel.text = dateFormat.stringFromDate(ride!.updated!)
if ride!.waitTime! == "Closed" {
cell.waitTimeLabel.text = ride!.waitTime!
} else {
cell.waitTimeLabel.text = "\(ride!.waitTime!)m"
}
}
return cell
}
So at the moment everything works, however wherever ride does equal nil, I just get the prototype cell returned, whereas I would like it to return nothing.
I have tried hiding or changing the height of these cells to nil, but it just seems like messy solution. Anyone know a better one?
Thanks.
determine the number of rows by implementing tableView(_ tableView: UITableView, numberOfRowsInSection section: Int)
override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) {
if section == 0 {
return arrayName.count
}
return 0
}
I figured out a solution. As a few people mentioned, I had to figure out what to display before dealing with cellForRowAtIndexPath.
Basically I added some code to figure out which favourites could be found in the array and put it in viewWillAppear.
for index in 0...favouritesArray.count - 1 {
var ride = DataManager.sharedInstance.getRideByName(favouritesArray[index] as! String)
if ride != nil {
favouritesFound.append(ride!.name!)
println(favouritesFound)
}
}
Works perfectly now! Thanks everyone.
It is assumed that you always know how many valid rows do you have and you publish this number by implementing UITableViewDataSource's tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int method. So if at some moment number of rows changes you should call UITableView.reloadData not to try to hide excess rows.
To get an array of valid rides try something like:
var newArray:[String] = []
for str in favouritesArray where DataManager.sharedInstance.getRideByName(str!) != nil
{
newArray += [str]
}