SWIFTにてclass内で呼び出したstatic変数の値が保存されない

Question

困りごと

以下のコードの最初に宣言しているstatic変数６つ（A〜F）のうち、必ず３つ目の値が欠如する問題が起きており困っています。
３つ毎ではなく、３つ目のみです。

2019/12/24追記
現象はコード内最後に記述されている、PostNewRequest関数内のqueryParameters内で使う際に起きます。
■現象イメージ
以下の場合、Cに値渡しされない。

queryParameters = [
   "sample" :
   [
      [ "keyA": A, "keyB": B, "keyC": C],
      [ "keyA": A, "keyD": D, "keyE": E],
      [ "keyA": A, "keyB": F]
   ]
]

以下の場合、Aに値渡しされない。

queryParameters = [
   "sample" :
   [
      [ "keyA": A, "keyB": F],
      [ "keyA": A, "keyB": B, "keyC": C], <-ここ
      [ "keyA": A, "keyD": D, "keyE": E]
   ]
]

###やったこと
変数viewzDidLoadにて初期値を入れましたが、３つ目だけ初期値も入りません。
また、変数の順番も関係なく、変数の記述順を変えても必ず３つ目のみ値が入りません。（その他はちゃんと入る）
計算式でforced unwrapをしているのが悪さをしているのかと思いましたが、sliderを使わなかった場合も同様の現象が起きます。

現象確認方法

sendRequest関数にBrakePointを設定し、関数実行を途中で止めました。
その上でqueryParameters内の該当する変数の値を確認しています。

インターフェイス外観

![

import UIKit
import Foundation
import SwiftyJSON
import CryptoSwift

class ViewController: UIViewController, UIPickerViewDelegate, UIPickerViewDataSource {
    static var A: String = ""
    static var B: String = ""
    static var C: String = ""
    static var D: String = ""
    static var E: String = ""
    static var F: String = ""
    
    override func viewDidLoad() {
        super.viewDidLoad()
//現象確認用にここで初期値を宣言してもqueryParameters内の3つ目の値が入らない
        ViewController.A = "文字列"
        ViewController.B = "1000"
        ViewController.C = "2000"
        ViewController.D = "1900"
        ViewController.E = "900"
        ViewController.F = "950"
    }
//以下でスライダーで調整した値と「highMaximum」「lowMaximum」で入力した値をかけて元の値に足す
//以下の.floor(nearest:0.5)はFloatにextensionをつけて数字の丸め方を変えています
    @IBOutlet weak var inputA: UITextField!
    @IBOutlet weak var highMaximum: UITextField!
    @IBOutlet weak var lowMaximum: UITextField!
    @IBOutlet weak var highNum: UILabel!
    @IBAction func highRangeSelector(_ sender: UISlider) {//Sliderは0〜1
        let highNumCalc = Float(inputA.text!)! + Float(highMaximum.text!)! * sender.value
        highNum.text = String(highNumCalc.floor(nearest:0.5))
        ViewController.C = String(highNumCalc.floor(nearest:0.5))
        let highForD = highNumCalc - 100
        ViewController.D = String(highForD.floor(nearest: 0.5))
        ViewController.B = inputA.text!//念押しでsliderを調整する毎にBに値を入れておく
    }
    @IBOutlet weak var lowNum: UILabel!
    @IBAction func lowRangeSelector(_ sender: UISlider) {//Sliederは-1〜0
        let lowNumCalc = Float(inputA.text!)! + Float(lowMaximum.text!)! * sender.value
        lowNum.text = String(lowNumCalc.floor(nearest:0.5))
        ViewController.E = String(lowNumCalc.floor(nearest:0.5))
        let lowForF = lowNumCalc + 50
        ViewController.F = String(lowForF.floor(nearest: 0.5))
        ViewController.B = inputA.text!//念押しでsliderを調整する毎にBに値を入れておく
    }
//このボタンが押されると、PostNewRequestクラス内の変数でbuildURLRequestが実行される
    @IBOutlet weak var sendButton: UIButton!
    @IBAction func sendRequest(_ sender: Any) {
        let post = PostNewRequest()
//     （省略）
    }

//  To post
    final class PostNewRequest {
//      （省略）
        static var queryParameters: [String: Any] = [ここでstatic変数を使いたい]
//      （省略）
        func buildURLRequest() {}
        init() {}
    }
}

Comments

[退会済みユーザー]

"がおかしなことになってますが、転載する際のタイポとして扱っていいのでしょうか？

すでに実行されているでしょうが…
値が入っていないということでしたら、例えば、final class PostNewsRequest { の1行うえに
print("A: \(A)")
print("C: \(C)")とでも書いて実行すれば、問題の切り分けができると思います。

[masayoshi555]

ご指摘の転載上のタイポは修正いたしました。ありがとうございます。

また、検証方法のご提案ありがとうございます。
また後ほど検証結果を記載したいと思います。

Answer 1

takabosoftさんご指摘のstaticの件もポイントでしたが、もう一つ要因がありました。

これ以外にtextfieldが２つあり、計３つのtextfieldを持っています。
検証の際、textfieldへ入力し、最後のtextfieldへ入力後そのままボタン(sendButton)を押すと、textfieldの値が反映されず、sendRequest関数が実行されることでした。

なので、sendButton押下でsendRequest関数実行時、以下のコードを挟むことで解決出来ました。

@IBAction func sendRequest(_ sender: Any) {
        inputA.endEditing(true)
        highMaximum.endEditing(true)
        lowMaximum.endEditing(true)
        let post = PostNewRequest()
//     （省略）
    }

問題の一端及びそもそもの構成の指摘をして頂いたtakabosoftさんありがとうございました。
tyobigorouさんもありがとうございました。

Answer 2

検証をしていないので適当な事を書きますが、
静的変数の初期化値に静的変数を使うことは、
静的変数の初期化順の影響を受けるので、避けたほうがよいと思います。

おそらくstaticをすべて辞めれば解決する話だとは思います。
今回載せてもらった箇所で静的変数の必要性は感じません。

Comments

[masayoshi555]

@takabosoft さん回答ありがとうございます。
>静的変数の初期化値に静的変数を使うことは、静的変数の初期化順の影響を受けるので、避けたほうがよいと思います。
勉強不足でした。その点自分なりに調べてみます。

ご指摘の部分、実はエラーが出るからstaticにしていました。（上記のようなことは知らなかったです）
staticを外すと以下のエラーになります。
「Instance member '（AからF）' of type 'ViewController' cannot be used on instance of nested type 'ViewController.PostNewRequest'」

class内にclassを作っている上に、queryParametersもnestedになっております。
(queryParametersは関係ない？）

queryParameters = [
"sample" :
[
[ "keyA": A, "keyB": B, keyC": C],
[ "keyA": A, "keyD": D, keyE": E],
[ "keyA": A, "keyB": F]
]
]

staticだと解決出来たので安易にしていました。
ちなみに解決方法として、override func viewDidLoad() 内の初期の宣言をやめつつ、変数をViewControllerの外に出してグローバル変数にするというのはわかるのですが、メモリの消費を気にしていました。（実際はどうなのか理解していませんが）
もしグローバル変数以外の解決方法ご存知でしたら教えて頂けると幸いです。

[masayoshi555]

続報：上記に基づき、以下のコードにて実施したところ、「３つ目の変数が入らない。」現象は引き続き出てしまいました。

import ...

var A: String = ""
var B: String = ""
var C: String = ""
var D: String = ""
var E: String = ""
var F: String = ""

class ViewController: UIViewController, UIPickerViewDelegate, UIPickerViewDataSource {

　　　　（staticは削除）

override func viewDidLoad() {
super.viewDidLoad()

　　　　　　（staticは削除）

}

現象イメージを質問欄に追記しました。

[takabosoft]

AからFをグローバル変数にする必要もありません。というかグローバル変数もstatic変数も似たようなものなので、出来る限り利用を避けてください。

queryParametersもstaticである必要は全く無く、必要な時に動的に作成するべきです。
でないと二度目以降で変数が更新されないという不具合がついてくるはずです。

[masayoshi555]

takabosoftさん
たくさんのご指摘ありがとうございます。
ご指摘の通り、queryParametersをstaticにしていると２回目以降変数が反映されませんでした。
変数のスコープについては勉強しなおします。（static外し構成作り替えた結果しっかりと反映されました）