String Swiftと同じ名前の変数の使用方法

Question

私は自分のコードを最適化しようとしています。私は次のことをしたい：dictの要素がfirstNamesの配列にある場合は、この要素をfirst変数に直接書き込むことができます。これはsecondなどと同じです。また、私は、ループを反復処理するとfirstに値を設定しようとしsecond等val

を使用しています> secondとなど - > first、 secondNames < - 私は、変数 firstNames <をリンク含ま names配列を作成しました

let dict = ["type": "1", "word": "abc"] // this is example 
    let firstNames = ["1", "2"] 
    let secondNames = ["3", "4"] 
    var first = String() 
    var second = String() 
    let names = [firstNames: first, secondNames: second] 
    for el in dict { 
     for var (key, val) in names { 
      if (key as! [String]).contains(el["type"]!) { 
       if ((val as! String) != "") { 
        val = (val as! String) + ", " + el["word"]! 
       } 
       else { 
        val = el["word"]! 
       } 
      } 
     } 
    } 

このコードには実行時エラーがありません。しかし、残念ながら私がループにvalを設定しようとしているときに何も起こっていません。私が間違ってやっているのはもっと洗練された解決策ですか？

ありがとうございました。

Answer 1

Swiftはコンパイルされた静的な言語であり、オプティマイザはこれらの一時変数を必要としない場合は完全に削除することができます。ローカル変数の名前は実行時には利用できません。このアプローチはうまくいきません。あなたが辞書を使って "first"と "second"をストリングキー（Paulw11があなたを指し示している場所）として保持していればうまくいくかもしれませんが、これはSwiftの間違ったアプローチです。

dictから始めましょう。これは辞書の適切な使用ではありません。これは、文字列と文字列との任意のマッピングではありません。これは、固有のフィールド識別子の値へのマッピングです。それが構造体の定義です。

「タイプ」は実際には「任意の整数」である可能性がありますが、実際には「値の制約付きリストからの値」であることが強く疑われます。それはenumです。

私たちは一緒にそれらを置く場合、辞書は本当に要素である：

enum ElementType { 
    case one 
    case two 
    case three 
    case four 
} 

struct Element { 
    let type: ElementType 
    let word: String 
} 

そして、あなたの名前のセレクタはElementTypesの任意のリストです：

let firstNameTypes = [ElementType.one, .two] 
let secondNameTypes = [ElementType.three, .four] 

その後、むしろただ一つのdictよりも、私たちは想像することができます要素のリスト（あなたが実際にこのループの外にいると仮定します）。

let elements = [ 
    Element(type: .one, word: "abc"), 
    Element(type: .two, word: "bcd"), 
    Element(type: .three, word: "efg"), 
    Element(type: .four, word: "fgh"), 
] 

私たちのループは些細なことであり、明らかにそれが何を意味していますか？

let firstNames = elements.filter { firstNameTypes.contains($0.type) } 
let secondNames = elements.filter { secondNameTypes.contains($0.type) } 

そして、我々の最後の文字列を取得することも簡単です：

let firstString = firstNames.map { $0.word }.joined(separator: ",") 
let secondString = secondNames.map { $0.word }.joined(separator: ",") 

このコードは非常に明確であるが、それは二回、各要素について一度「NameTypes」リスト上の要素を反復ありません。大多数の場合、それは問題ありません。それはコンピュータのことですが、何千もの要素がある場合（特に名前のフィルタリングリストが長い場合は）、コストが高くなる可能性があり、結果文字列の辞書を返す関数を書くことができます（手でfilterを書き換えます）。それがあなたの問題なら、それを書くことができます（根本的な問題をより明確に説明するならば）。しかし、ほとんどの場合、控えめなサイズのリストでは、これが最善の方法です。

Answer 2

私の推測では、 'val'は一時変数で、 'dict'辞書に影響するように割り当てることはできません。あなたはあなたのコードで達成しようとしているかを説明するかどう代わり

dict[key] = ... 

また使用

val = .. 

試してみるの

は、それは素晴らしいことです。