未解決の識別子 - inループロジック

Question

これは、下から上へのソートです。ループを繰り返している間は、配列内の最小の数で置換され、これが最後まで続きます。

ご覧のとおり、私はストライドを使用するようにリファクタリングしています。残念ながらvar lowest = firstIndexは私にいくつかの問題を与えています。

私はこの機能をストライドの右手で行うことができるはずですか？ stride: throughの代わりにstride: toを使用する必要があります。そのチップのTimに感謝します。私はそれがこのように動作するようになった

func selOrganize(myList: Array<Int>) -> Array<Int> { 1 

    var extract = myList 

    for firstIndex in 0..<extract.count { 

     var lowest = firstIndex 

     for var secondIndex = firstIndex + 1; secondIndex < extract.count; secondIndex++ { 

      if extract[lowest] > extract[secondIndex] { 
       lowest = secondIndex 
    } 
     } 

     if firstIndex != lowest { 
      swap(&extract[firstIndex], &extract[lowest]) 
     } 
    } 
    return extract 
} 

更新構文

func selOrganize(myList: Array<Int>) -> Array<Int> { 

var extract = myList 

    // var lowest = firstIndex 

    // Do I need 'key'? Should I declare 'lowest' as a variable here? 
    // If I do use it here I get a "'lowest' is unmutable because it's a let" error 
    for (firstIndex, key) in extract.enumerate() { 

     // < > stride uses 'to' and <= >= stride uses through 
     for secondIndex in (firstIndex).stride(to: 0, by: +1) { 

      if extract[lowest] > extract[secondIndex] { 
       lowest = secondIndex 
      } 
     } 

    if firstIndex != lowest { 
     swap(&extract[firstIndex], &extract[lowest]) 
    } 

} 

return extract 
} 

Answer 1

：

func selOrganize(myList: Array<Int>) -> Array<Int> { 

    var extract = myList 

    // Accessing indices is simpler than calling enumerate, and 
    // the function only needs access to the indices, not the 
    // values of the enumeration: 
    for firstIndex in extract.indices { 

     // lowest needs to be defined inside this loop if you 
     // are going to initialize it using firstIndex 
     // because firstIndex isn't defined outside the loop. 
     var lowest = firstIndex 

     // You need to stride from the firstIndex to the end of the 
     // array, so the call to stride should look like this: 
     for secondIndex in firstIndex.stride(to: extract.count, by: 1) { 

      if extract[lowest] > extract[secondIndex] { 
       lowest = secondIndex 
      } 
     } 

     if firstIndex != lowest { 
      swap(&extract[firstIndex], &extract[lowest]) 
     } 
    } 

    return extract 
} 

Answer 2

Strideが必要とされていない、あなたは標準for i in start..<end構文

func selOrganize(myList: Array<Int>) -> Array<Int> { 

    var extract = myList 

    for firstIndex in 0..<extract.count { 
    var lowest = firstIndex 
    for secondIndex in (firstIndex + 1)..<extract.count { 
     if extract[lowest] > extract[secondIndex] { 
     lowest = secondIndex 
     } 
    } 

    if firstIndex != lowest { 
     swap(&extract[firstIndex], &extract[lowest]) 
    } 
    } 
    return extract 
} 

であることを行うことができます

しかし、実際には1行

let sortedList = myList.sort{$0 < $1}