Conversation
|
@MortyMerr добавил прямоугольник анимированный с тенью двигающийся бесконечно. Поменял немного структуру прошлых работ, добавил список из стран, из которых апи может выдать новости, добавил список избранных, поменял структуру Realm'a |
M0rtyMerr
left a comment
M0rtyMerr
left a comment
There was a problem hiding this comment.
Все ок, но я бы хотел немножко над стилем поработать. Ну там отступы чтобы везде были аккуратные, пробелы и так далее. А так молодец!
|
|
||
| import Foundation | ||
|
|
||
| final class Country { |
There was a problem hiding this comment.
А давай сделаем иммутабельным (let) и структурой? Это хорошая практика при хранении данных
|
|
||
| import UIKit | ||
|
|
||
| class CountryViewController: UIViewController, UITableViewDataSource, UITableViewDelegate { |
There was a problem hiding this comment.
Лучше разделять по отдельным extension, как в лекционном проекте
| tableView.rowHeight = 307 | ||
| } | ||
|
|
||
| @objc func refresh(_:AnyObject) { |
There was a problem hiding this comment.
С отступами надо поработать)
| } | ||
|
|
||
| func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int { | ||
| if (isFiltering) { |
| return newsCell | ||
| } | ||
|
|
||
| var isSearchBarEmpty: Bool { |
There was a problem hiding this comment.
Если бы был линтер, он бы попросил тебя объявлять все свойства наверху, а функции под ними. Я согласен в принципе для читаемости
|
|
||
| shadowView.addSubview(view) | ||
| self.view.addSubview(shadowView) | ||
| UIView.animate(withDuration: 1, |
There was a problem hiding this comment.
Молодец, что не поленился :)
|
@FrolovNV пофикси плз через нехочу. Для потомков сохранить хочется |
|
@MortyMerr , бать, все сделаю, дай только пожалуйста с курсовым закончу по джабе. А лекции все или будет последняя, сильная? |
|
@FrolovNV ну |
2 participants
@MortyMerr вот моя помойка, помогите пожалуйста