Sprint 05

[s1zzen]

Не придумал, как сделать так, чтобы вопросы не повторялись, если подскажете, в какую сторону думать, буду сильно признателен)

[helecta]

Стоит организовывать код следующим образом:

• Публичные переменные
• IBOutlet
• Приватные переменные
• Публичные методы(сначала initы, overrideы, и потом остальные)
• Приватные методы
• IBAction

Подробнее почитать можно тут: Организация кода

[helecta]

Для того, чтобы избежать повторений можно создать еще один массив с активными вопросами. и доставать вопросы из него, как только вопрос достали, удалять этот вопрос:

    /// Массив с вопросами, которых еще не было в игре. Если массив обнуляется, то его необходимо заново заполнить
    /// всеми вопросами
    private var activeQuestions: [QuizQuestion]
    
    init(delegate: QuestionFactoryDelegate?) {
        self.delegate = delegate
        self.activeQuestions = questions
    }
    
    func requestNextQuestion() {
        guard let question = getQuestion() else { return }
        
        delegate?.didReceiveNextQuestion(question: question)
    }
    
    private func getQuestion() -> QuizQuestion? {
        guard let index = (0..<activeQuestions.count).randomElement() else { return nil }
        
        let question = activeQuestions[safe: index]
        
        //удаление из массива вопроса, только что сформированного для показа
        activeQuestions.remove(at: index)
        
        // если были показаны все вопросы, то заполняем массив всеми вопросами заново
        if activeQuestions.count == 0 {
            activeQuestions = questions
        }
        
        return question
    }