Search and Tableview - ios

Question

I need little help.
Peace everyone, hope that you are doing great.
I made an app with tableview, I wanted to add a searchbar to my tableview, so I found a tutorial how to add searchbar to tableview, but the in the tutorial it was just an list with item but not items that when you type those items they move you to Viewcontroller with textview. Different from tutorial, my app is so, when you type one of those items they move you to Viewcontroller with textview. But better you can see how my tablview looks like:
[![enter image description here][1]][1]
It looks nice, it also looks nice when I use search-bar :
[![enter image description here][2]][2]
But problem is, when you use search, it gives me different data from different variable, as in image:
[![enter image description here][3]][3]
Hope that I was clear, and hope someone will explain me how to fix this issue.
Peace.
Here is my tablview code:
import UIKit
class TableViewController: UITableViewController, UISearchResultsUpdating {
var FirstTableArray = [String]()
var passThisArray = [String]()
var filteredVerbs = [String]()
var resultSearchController = UISearchController()
override func viewDidLoad() {
super.viewDidLoad()
self.resultSearchController = UISearchController(searchResultsController: nil)
self.resultSearchController.searchResultsUpdater = self
self.resultSearchController.dimsBackgroundDuringPresentation = false
self.resultSearchController.searchBar.sizeToFit()
self.tableView.tableHeaderView = self.resultSearchController.searchBar
self.tableView.reloadData()
// This array will display on your tableviewcell.
FirstTableArray = [åadlyde, åanbefale, åansette, åarbeide, åavslå, åavstå, åbade, åbanke, åbarbere, åbe, åbearbeide, åbedra, åbedøve, åbegynne, åbehandle, åbeite, åbeskrive, åbestemme…]
//You can pass element of this array
passThisArray = ["å adlyde", "å anbefale", "å ansette", "å arbeide", "å avslå", "å avstå", "å bade", "å banke", "å barbere", "å be", "å bearbeide", "å bedra", "å bedøve", "å begynne", "å behandle"…]
}
override func didReceiveMemoryWarning()
{
super.didReceiveMemoryWarning()
// Dispose of any resources that can be recreated.
}
override func numberOfSectionsInTableView(tableView: UITableView) -> Int
{
return 1
}
override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
if(passThisArray.count > 3){
if (self.resultSearchController.active)
{
return self.filteredVerbs.count;
}
else
{
return self.passThisArray.count
}
}
return FirstTableArray.count
}
override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let Cell = self.tableView.dequeueReusableCellWithIdentifier("Cell", forIndexPath: indexPath) as! UITableViewCell
Cell.textLabel?.text = passThisArray[indexPath.row]
if (self.resultSearchController.active)
{
Cell.textLabel?.text = self.filteredVerbs[indexPath.row]
return Cell
}
else
{
Cell.textLabel?.text = self.passThisArray[indexPath.row]
return Cell
}
}
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
if(segue.identifier == "detailView") {
let vc = segue.destinationViewController as! ViewwController
//Get the Index of selected Cell
var indexPath : NSIndexPath = self.tableView.indexPathForSelectedRow!
//assign string to next view controller instance from selected cell.
vc.FirstString = FirstTableArray[indexPath.row]
if (self.resultSearchController.active) {
vc.FirstString = FirstTableArray[filteredVerbs.count]
}
}
}
func updateSearchResultsForSearchController(searchController: UISearchController)
{
self.filteredVerbs.removeAll(keepCapacity: false)
let searchPredicate = NSPredicate(format: "SELF CONTAINS[c] %#", searchController.searchBar.text!)
let array = (self.passThisArray as NSArray).filteredArrayUsingPredicate(searchPredicate)
self.filteredVerbs = array as! [String]
self.tableView.reloadData()
}
}

Answer 1

So it seems that your problem lies in this method, in the selected line :
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
if(segue.identifier == "detailView") {
let vc = segue.destinationViewController as! ViewwController
//Get the Index of selected Cell
var indexPath : NSIndexPath = self.tableView.indexPathForSelectedRow!
//assign string to next view controller instance from selected cell.
vc.FirstString = FirstTableArray[indexPath.row]
if (self.resultSearchController.active) {
vc.FirstString = FirstTableArray[filteredVerbs.count] // <<---
}
}
}
If the search is active, you should rather take your FirstString this way self.filteredVerbs[indexPath.row].
With knowledge from comments :
It seems there is no easy way to map between values stored in these two arrays.
Also, this is not an ideal way to store this data. It is not flexible and mixes data with presentation, which should generally be avoided. I'd suggest creating a struct to store the data, something like this :
class Flexion : NSObject {
var infinitiv:String = ""
var presens:String = ""
var preteritum:String = ""
var perfektum:String = ""
var english:String = ""
init(infinitiv:String, presens:String, preteritum:String, perfektum:String, english:String) {
self.infinitiv = infinitiv
self.presens = presens
self.preteritum = preteritum
self.perfektum = perfektum
self.english = english
}
}
The main benefit in your case would be to have only one array with objects from which your table view is sourced. Thw whole thing would also get more readable and become easier for further upkeep.
So your code from the original question would become something like this :
class TableViewController: UITableViewController, UISearchResultsUpdating {
var FirstTableArray = [Flexion]()
var filteredVerbs = [Flexion]()
var resultSearchController = UISearchController()
override func viewDidLoad() {
super.viewDidLoad()
self.resultSearchController = UISearchController(searchResultsController: nil)
self.resultSearchController.searchResultsUpdater = self
self.resultSearchController.dimsBackgroundDuringPresentation = false
self.resultSearchController.searchBar.sizeToFit()
self.tableView.tableHeaderView = self.resultSearchController.searchBar
//create your objects like this :
var åadlyde = Flexion(infinitiv: "å adlyde", presens: "adlyder", preteritum: "adlød", perfektum: "har adlydt", english: "obey")
//create the rest of them
FirstTableArray = [åadlyde, åanbefale, åansette, åarbeide, åavslå, åavstå, åbade, åbanke, åbarbere, åbe, åbearbeide, åbedra, åbedøve, åbegynne, åbehandle, åbeite, åbeskrive, åbestemme…]
}
override func tableView(tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
if (self.resultSearchController.active) {
return self.filteredVerbs.count
}
return FirstTableArray.count
}
override func tableView(tableView: UITableView, cellForRowAtIndexPath indexPath: NSIndexPath) -> UITableViewCell {
let Cell = self.tableView.dequeueReusableCellWithIdentifier("Cell", forIndexPath: indexPath) as! UITableViewCell
if (self.resultSearchController.active) {
let verb = self.filteredVerbs[indexPath.row] as Flexion
Cell.textLabel?.text = verb.infinitiv
} else {
let verb = self.FirstTableArray[indexPath.row] as Flexion
Cell.textLabel?.text = verb.infinitiv
}
return Cell
}
override func prepareForSegue(segue: UIStoryboardSegue, sender: AnyObject?) {
if(segue.identifier == "detailView") {
let vc = segue.destinationViewController as! UIViewController
var indexPath : NSIndexPath = self.tableView.indexPathForSelectedRow!
if (self.resultSearchController.active) {
vc.verb = filteredVerbs[indexPath.row]
} else {
vc.verb = FirstTableArray[indexPath.row]
}
}
}
func updateSearchResultsForSearchController(searchController: UISearchController) {
self.filteredVerbs.removeAll(keepCapacity: false)
let searchPredicate = NSPredicate(format: "SELF.infinitiv CONTAINS[c] %#", searchController.searchBar.text!)
let array = (self.FirstTableArray as NSArray).filteredArrayUsingPredicate(searchPredicate)
self.filteredVerbs = array as! [Flexion]
self.tableView.reloadData()
}
}
You'd also need to make neccessary changes in "detailView" view controller - adding a verb variable of Flexion type and changing the way you present the data to user.
This way, there is no need for this "mapping" between arrays.
NOTE: note the change in the NSPredicate - it will now check only the infinitiv version of the word.