swiftで電卓 改良したい

Question

長くなってしまいますが、電卓作りました。
ただ、現状だと、２＊２のように一回の計算しかできず、いちいち＝してそれから＊とかしないといけない仕様になっています。

２＊２＊３＋３のように複数回計算ができる仕様に変更したいと思い
改良したのが下記ですがどうもうまく動作しません。

とりあえず４回までは計算できるようにしたいです。
そこがクリアできればあとはなんとかなるかと、、、

変更点としては、number3とnumber4の変数を用意しました。
どうロジックを組んだら良いでしょうか？
動作も近い数字は出ているので後一押しなんです、、、
よろしくお願いします。

//
//  ViewController.swift
//  Sample003_WinePiano
//
//  Created by MasayaGoto on 2016/09/13.
//  Copyright © 2016年 Jwebgate. All rights reserved.

import UIKit

class ViewController: UIViewController {
    var resultLabel = UILabel()    //計算結果表示エリアのラベル
    let xButtonCount = 4	//1行に配置するボタンの数
    let yButtonCount = 4	//1列に配置するボタンの数
    
    var number1:NSDecimalNumber = 0.0 // 計算前の変数
    var number2:NSDecimalNumber = 0.0 // １回目の数値追加用の変数
    var number3:NSDecimalNumber = 0.0 // ２回目の数値追加用の変数
    var number4:NSDecimalNumber = 0.0 // ３回目の数値追加用の変数
    var result:NSDecimalNumber = 0.0 // 計算結果の変数
    var operatorId:String = "" // 演算子の変数
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        //画面の横幅
        let screenWidth:Double = Double(UIScreen.mainScreen().bounds.size.width)
        //画面の縦幅
        let screenHeight:Double = Double(UIScreen.mainScreen().bounds.size.height)
        //ボタン間の余白(縦) & (横)
        let buttonMargin = 10.0
        // 計算結果表示エリアの縦幅
        var resultArea = 0.0
        
        // 画面全体の縦幅に応じて計算結果表示エリアの縦幅を決定
        switch screenHeight {
        case 480:   // 画面全体の縦幅が480なら
            resultArea = 200.0  // 計算結果表示エリアの縦幅は200
            
        case 568:   // 画面全体の縦幅が480なら
            resultArea = 250.0  // 計算結果表示エリアの縦幅は200
            
        case 667:   // 画面全体の縦幅が480なら
            resultArea = 300.0  // 計算結果表示エリアの縦幅は200
            
        case 736:   // 画面全体の縦幅が480なら
            resultArea = 350.0  // 計算結果表示エリアの縦幅は200
            
        default:
            resultArea = 0.0
        }
        
        
        //計算結果ラベルのフレームを設定。
        //左から10px,上から30pxに配置 全体の横幅から20px引いた数値が幅,計算結果フレームから30px引いた数値が縦幅
        resultLabel.frame = CGRect(x: 10, y: 30, width: screenWidth - 20, height: resultArea - 30)
        
        
        //計算結果ラベルの背景色を灰色にする
        resultLabel.backgroundColor = UIColor.grayColor()
        //計算結果ラベルのフォントと文字サイズを設定
        resultLabel.font = UIFont(name: "Arial", size: 50)
        //計算結果ラベルのアラインメントを右揃えに設定
        resultLabel.textAlignment = NSTextAlignment.Right
        //計算結果ラベルの表示行数を4行に設定
        resultLabel.numberOfLines = 4
        //計算結果ラベルの初期値を"0"に設定
        resultLabel.text = "0"
        //計算結果ラベルをViewControllerクラスのviewに設置
        self.view.addSubview(resultLabel)
        
        // ボタンのラベルタイトルを配列で用意
        let buttonLabels = [
            "7","8","9","×",
            "4","5","6","-",
            "1","2","3","+",
            "0","C","÷","="
        ]
        
        //二次元配列で１６個のボタンを並べる
        //[0][0] = 7
        //[0][1] = 8
        //[0][2] = 9
        //[0][3] = ×
        //[1][0] = 4
        //[1][1] = 5
        //[1][2] = 6
        //[1][3] = -
        //[2][0] = 1
        //[2][1] = 2
        //[2][2] = 3
        //[2][3] = +
        //[3][0] = 0
        //[3][1] = C
        //[3][2] = ÷
        //[3][3] = =
        for var y = 0; y < yButtonCount; y++ {
            for var x = 0; x < xButtonCount; x++ {
                
                //計算機のボタンを作成
                let button = UIButton()
                
                // ボタンの横幅サイズ作成
                let buttonWidth =  (screenWidth - (buttonMargin * (Double(xButtonCount)+1))) / Double(xButtonCount)
                //ボタンの縦幅サイズ作成
                let buttonHeight = (screenHeight - resultArea - ((buttonMargin*Double(yButtonCount)+1))) / Double(yButtonCount)
                
                //ボタンのX座標
                let buttonPositionX = (screenWidth - buttonMargin) / Double(xButtonCount) *  Double(x) + buttonMargin
                //ボタンのY座標
                let buttonPositionY =
                    ( screenHeight - resultArea - buttonMargin ) / Double(yButtonCount) * Double(y) + buttonMargin + resultArea
                
                // ボタンの縦幅サイズ作成
                button.frame = CGRect(x:buttonPositionX,y: buttonPositionY, width:buttonWidth,height:buttonHeight)
                // ボタン背景色設定
                button.backgroundColor = UIColor.greenColor()
                
                //ボタンのラベルタイトルを取り出すインデックス番号
                //１回目の内ループで０掛け後のプラスがミソ
                let buttonNumber = y * xButtonCount + x
                
                //ボタンのラベルタイトルを設定
                button.setTitle(buttonLabels[buttonNumber], forState: UIControlState.Normal)
                // ボタンタップ時のアクション設定
                button.addTarget(self, action: "buttonTapped:",  forControlEvents: UIControlEvents.TouchUpInside)
                
                // ボタン配置
                self.view.addSubview(button)
            }
        }
        //デバッグ
        print("縦画面サイズ\(screenHeight) 横画面サイズ\(screenWidth)")
    }
    
    
    
    
    //ここから裏処理
    // ボタンがタップされた時のメソッド
    func buttonTapped(sender:UIButton){
        let tappedButtonTitle:String = sender.currentTitle!
        print("\(tappedButtonTitle)ボタンが押されました!")
        // ボタンのタイトルで条件分岐
        switch tappedButtonTitle {
        
        //数字ボタンタップ時
        case "0","1","2","3","4","5","6","7","8","9":
            numberButtonTapped(tappedButtonTitle)
        
        //演算子ボタンタップ時
        case "×","-","+","÷":
            operatorButtonTapped(tappedButtonTitle)
        
        //結果ボタンタップ時
        case "=":
            equalButtonTapped(tappedButtonTitle)
        
        //クリアボタンタップ時
        default:
            clearButtonTapped(tappedButtonTitle)
        }
    }
    
    //数字ボタンタップ時
    func numberButtonTapped(tappedButtonTitle:String) {
        print("数字ボタンタップ：\(tappedButtonTitle)")
        
        let tappedButtonNum:NSDecimalNumber =   NSDecimalNumber(string: tappedButtonTitle)
        // 入力されていた値を10倍にして1桁大きくして、その変換した数値を加算
        number1 = number1.decimalNumberByMultiplyingBy (NSDecimalNumber(string: "10")).decimalNumberByAdding(tappedButtonNum)
        
        //タップしたボタン（数字）を表示
        resultLabel.text = number1.stringValue
    }
    
    //演算子ボタンタップ時
    func operatorButtonTapped(tappedButtonTitle:String) {
        print("演算子ボタンタップ：\(tappedButtonTitle)")
        
        //ここでは押された演算子を変数に格納
        operatorId = tappedButtonTitle
        
        //これも同じように１回目にタップした数字を変数に格納
        
        
        
       
        
        number2 = number1
        
        if number2 != 0 {
            number3 = number2
        }
        
        if number3 != 0 {
            number4 = number3
        }
        
        //次の数字をタップする用に０に戻す
        number1 = 0
    }
    
    //結果ボタンタップ時
    func equalButtonTapped(tappedButtonTitle:String) {
        print("等号ボタンタップ：\(tappedButtonTitle)")
        
        //押された演算子が何だったのかによってSwitchする
        switch operatorId {
            
        //押された演算子が+なら足し算
        case "+":

            result = number2.decimalNumberByAdding(number1)
            
            if number3 != 0 {
                result = result.decimalNumberByAdding(number3)
            }
            
            if number4 != 0 {
                result = result.decimalNumberByAdding(number4)
            }
            
        
        //押された演算子が-なら引き算
        case "-":
            result = number2.decimalNumberBySubtracting(number1)
            
        //押された演算子が×なら掛け算
        case "×":
            result = number2.decimalNumberByMultiplyingBy(number1)
            
        //押された演算子が÷なら割り算
        case "÷":
            if(number1.isEqualToNumber(0)){
                number1 = 0
                resultLabel.text = "無限大"
                return
            } else {
                result = number2.decimalNumberByDividingBy(number1)
            }
        default:
            print("その他")
        }
        
        //まだ計算を続ける場合を想定してnumber1に代入
        //次に押すボタンが演算子だとして、number1はnumber2にすぐ代入される
        //この３つのメソッドの連動性を考えるのが重要
        number1 = result
        resultLabel.text = String("\(result)")
    }
    
    //クリアボタンタップ時
    func clearButtonTapped(tappedButtonTitle:String) {
        print("クリアボタンタップ：\(tappedButtonTitle)")
        
        //全部初期化
        number1 = 0
        number2 = 0
        number3 = 0
        number4 = 0
        result = 0
        operatorId = ""
        resultLabel.text = "0"
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    
}

Answer 1

コードは一切読んでいませんが、とりあえず仕様をまとめるところから始めてはいかがでしょうか。
実際の電卓の操作と内部構造を順番に記載して、それをコードにするだけだと思います。

Comments

[earnest_gay]

ロジカルな思考ができない部分ですので質問しているのです。