kridan-coder / ios-calculator Goto Github PK

1. Кейсы в enum следует именовать с маленькой буквы (например, https://docs.swift.org/swift-book/LanguageGuide/Enumerations.html).

@IBAction func plusMinusButton(_ sender: UIButton) - если ссылка на кнопку внутри метода не нужна, ее можно убрать при генерации IBAction (в настройке нижнее поле Sender меняется на None).

3. if (wasDividedByZero) - условие на Swift пишем без скобок!
4. В enum Operation прописано rawValue (соответствующие строки). Следовательно, можно получать конкретный объект по строке с помощью конструктора (https://docs.swift.org/swift-book/LanguageGuide/Enumerations.html#ID149) и убрать вот это вот все:

 switch sender.currentTitle! {
        case "÷":
            operation = Operation.Divide
        case "x":
            operation = Operation.Multiply
        case "+":
            operation = Operation.Summarize
        case "-":
            operation = Operation.Substract
        case "%":
            operation = Operation.Remaind
        default:
            operation = Operation.Nothing
        }

5. Подобный код:

 func basicDropShadow(button: UIButton){
        button.layer.shadowColor = #colorLiteral(red: 0.1764705926, green: 0.01176470611, blue: 0.5607843399, alpha: 1)
        button.layer.shadowOffset = CGSize(width: 1.0, height: 2.0)
        button.layer.shadowRadius = 2.0
        button.layer.cornerRadius = 4.0
    }

лучше убрать в extension для класса UIButton. Тогда он станет годным для повторного использования, то есть можно будет таскать из проекта в проект. Да и класс UIViewController к данному функционалу мало отношения имеет.

6. Не забывай про сочетание ctrl + I для причесывания кода.
7. На симуляторе (да и на устройстве) твои шрифты не отображаются, поскольку ты их не совсем корректно подключил. Нужно в структуре проекта создать не папку, а Group (желтая папка) и добавить файлы туда. Для каждого шрифта нужно проверить включение в Target (галочка справа).
8. Проверь тени, у меня они не отобразились.