Skip to content

Sprint_16#8

Open
rfhfv wants to merge 8 commits into
Sprint_15from
Sprint_16
Open

Sprint_16#8
rfhfv wants to merge 8 commits into
Sprint_15from
Sprint_16

Conversation

@rfhfv

@rfhfv rfhfv commented Jan 29, 2024

Copy link
Copy Markdown
Owner

No description provided.

return pageControl
}()

lazy var pages: [UIViewController] = [

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Наверное можно приватным это поле сделать


// MARK: - Actions

@objc func pageControlTapped(_ sender: UIPageControl) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тоже приватно

// MARK: - UIPageViewControllerDataSource

extension OnboardViewController: UIPageViewControllerDataSource {
func pageViewController(_ pageViewController: UIPageViewController, viewControllerBefore viewController: UIViewController) -> UIViewController? {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Рекомендую придерживаться следующего стиля написания длинных вызовов и длинных объявлений функций.

func pageViewController(
        _ pageViewController: UIPageViewController,
        viewControllerBefore viewController: UIViewController
    ) -> UIViewController? {

Такой формат принят за основу во многих компаниях и следование ему даст несколько очков к карме при работе над боевыми проектами


let window = UIWindow(windowScene: windowScene)
window.rootViewController = TabBarController()
window.rootViewController = dataStorage.firstLaunchApplication ? (TabBarController()) : (OnboardViewController())

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вместо dataStorage.firstLaunchApplication можно писать сразу DataStorege.shared.firstLaunchApplication

Также TabBarController() и OnboardViewController() можно не заключать в скобки

extension CategoryViewController: UITableViewDataSource {
func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
return category.count
guard let count = viewModel?.categoriesCount() else { return 0 }

@amardanov amardanov Jan 29, 2024

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно записать в одну строку с помощью дефектного значения если не получается развернуть опциона. Делается это так:

return viewModel?.categoriesCount() ?? 0

В таком случае сначала будет попытка развернуть и вернуть опциональное значение из categoriesCount() и если оно nil, то вернется значение после ??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants