Sprint 06 #712
Open
Sprint 06 #712
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Nikitaaeee
approved these changes
Feb 6, 2024
Comment on lines
10
to
16
| public extension Float { | ||
| static var randomMovieRating: Float { | ||
| let randomFloat = Float.random(in: 1.0...10.0) | ||
| let intVal:Int = Int(randomFloat*10) | ||
| return Float(intVal)/10.0 | ||
| } | ||
| } |
There was a problem hiding this comment.
extension не обязательно делать public, так как проект состоит из одного модуля и расширение Float используется только в рамках этого модуля.
Comment on lines
7
to
11
|
|
||
| import Foundation | ||
|
|
||
| import UIKit | ||
|
|
There was a problem hiding this comment.
UIKit включает в себя Foundation, поэтому можно импортировать только UIKit :)
Comment on lines
66
to
82
| } | ||
|
|
||
|
|
||
| private func convert(model: QuizQuestion) -> QuizStepViewModel { | ||
| return QuizStepViewModel( | ||
| image: UIImage(data: model.image) ?? UIImage(named: "The Godfather")!, | ||
| question: model.text, | ||
| questionNumber: "\(currentQuestionIndex + 1)/\(questionsAmount)") | ||
| } | ||
| func showLoadingIndicator() { | ||
| activityIndicator.isHidden = false | ||
| activityIndicator.startAnimating() | ||
| } | ||
|
|
||
| private func showNetworkError(message: String) { | ||
| hideLoadingIndicator() | ||
| let alertModel = AlertModel(title: "Ошибка", |
There was a problem hiding this comment.
- Между методами принято оставлять одну пустую строку, можно добавить там, где ее нет и удалить лишние там, где их больше одной.
- Метод showLoadingIndicator используется только внутри класса, поэтому ему можно добавить private.
Comment on lines
33
to
64
| // private let questions: [QuizQuestion] = [ | ||
| // QuizQuestion(image: "The Godfather", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "The Dark Knight", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "Kill Bill", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "The Avengers", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "Deadpool", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "The Green Knight", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: true), | ||
| // QuizQuestion(image: "Old", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: false), | ||
| // QuizQuestion(image: "The Ice Age Adventures of Buck Wild", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: false), | ||
| // QuizQuestion(image: "Tesla", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: false), | ||
| // QuizQuestion(image: "Vivarium", | ||
| // text: "Рейтинг этого фильма больше чем 6?", | ||
| // correctAnswer: false), | ||
| // ] |
There was a problem hiding this comment.
Мок-данные можно будет удалить, так как теперь фильмы приходят из сети :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.