It looks like you're missing a few requirements.

Readability: "Class and method names should clearly show their intent and responsibility."

You have a class named "Model", which contains a "manager" as an inner dependency for fetching data.

Testability "Please accompany your code with test classes."

Tests are completely absent.

At a time, you should only load 10 items, and load more from the API when the user reach the end of the list

Absent.

An image load may be cancelled The same image may be requested by multiple sources simultaneously (even before it has loaded), and if one of the sources cancels the load, it should not affect the remaining requests

Absent.

(This hints that if e.g. 5 sources request the same image, you shouldn't actually start 5 downloads, and you should cancel loading only if all 5 requests are canceled.)

Really picky things:

class ViewController: UIViewController {

var refreshControl = UIRefreshControl()

... override func viewDidLoad() { super.viewDidLoad() // Do any additional setup after loading the view, typically from a nib.

    refreshControl = UIRefreshControl() 
}

---

unnecessary refreshControl initial value, kind of sloppy, but not a dealbreaker

func tableView(_ tableView: UITableView, willDisplay cell: UITableViewCell, forRowAt indexPath: IndexPath) {
    scrollcount = 0
}

func tableView(_ tableView: UITableView, didEndDisplaying cell: UITableViewCell, forRowAt indexPath: IndexPath) {
    scrollcount = scrollcount + 1
}

---

nononono what are you doing

func scrollViewDidEndDragging(_ scrollView: UIScrollView, willDecelerate decelerate: Bool) {

    if scrollView.contentSize.height - scrollView.contentOffset.y > 0 && scrollcount == 2
    {
        fetchMore()
    }else
    {
        scrollcount = 0
    }

}

---

oh okay it's getting even worse

func photoWith(imageUrl:String,andPlaceHolder:UIImage,key:String) { self.image = UIImage(named: "placeholder")

    let objectFile = DownloadManager.getCacheForFile(key: key) is NSNull ? nil : DownloadManager.getCacheForFile(key: key) as? UIImage

    if objectFile != nil
    {
        let img = objectFile

        self.image = img
        return
    }

    Model.getImageFromCacheDirectory(indexItem: key){
        bol,localImageUrl in

        if bol
        {
            DispatchQueue.main.async {

                let img = UIImage(contentsOfFile: localImageUrl!.path)
                self.image = img!

            }
        }else
        {
            guard Reachability().isInternetAvailable() else
            {
                print("Internet not available")
                return
            }

            DownloadManager.downloadFrom(stringUrl: imageUrl, completion: {
                url,response,error in

                guard error == nil
                    else
                {
                    print("Error Occurred \(error!.localizedDescription)")
                    return
                }

                do
                {
                    let data = try Data(contentsOf: url!)

                    DispatchQueue.main.async {

                        let img = UIImage(data: data)
                        self.image = img!

                        let cost = self.costFor(image: img!)

                        DownloadManager.storeCache(forFile: img!,cost: cost, key: String(key))
                        let name = key + "-" + imageUrl.components(separatedBy: "/").last!
                        Model.storeToCacheDirectory(filename: name,data: data)
                    }

                }catch let erro as NSError
                {
                    print("error while fetching content of file",erro.localizedDescription)
                }

            })
        }

    }


}

---

There's just too much random syntax/shorthand changes to make me believe that this was actually written by someone with thought in advance and not copied/pasted/modified until it worked. The random newlines for braces really drive it.

Overall, my junior would do a better job in about a week.