for-in文内で最後の要素に到達した場合を簡潔に表現する方法を教えて下さい。

Question

いつもお世話になります。for-in文内で最後の要素に到達した場合を簡潔に表現する方法をご存知でしたら教えて下さい。

コードが適切でなかったため、訂正します
これでも適切かどうかわかりませんが…。

enum Type {
    case A
    case B
    case C
    case D
}

class Hoge {
    
    var name: String
    var type: Type
    var value: Int
    
    init(name: String, type: Type, value: Int = 1) {
        self.name = name
        self.type = type
        self.value = value
    }
  
}


var hoges = [Hoge]()

for i in hoges {
    
    if i.type == .C {
        // Cが見つかったときの処理
        break
    }

    if i.type == .D {
        // Dが見つかったときの処理
        break
    }
    
    // 最後の要素に到達したらという表現を簡潔に書きたいです
    if i === hoges.last {
        // 最後に到達した場合という表現が要素を比較する方法以外でありますか？
    }
    
}

訂正前コード

class Hoge {
    
    var name: String
    var value: Int
    
    init(_ name: String, value: Int = 1) {
        self.name = name
        self.value = value
    }
  
}


let hoges = [Hoge("A"), Hoge("B"), Hoge("A"), Hoge("A"), Hoge("A"), Hoge("A"), Hoge("C", value: 2), Hoge("A")]

let limitedHoges = hoges[hoges.firstIndex(where: { $0.name == "B" })!...hoges.firstIndex(where: { $0.name == "C" })!]


for i in limitedHoges {
    
    if i.name == "D" {
        print(i.value)
        break
    }
    
    // 最後の要素に到達したらという表現を簡潔に書きたいです
    if i === limitedHoges.last {
        print(i.value)
    }
    
}

// => 2

Comments

[MasakiHori]

ソースコードが省略されすぎていて質問者さんが何をしたいのかが分かりません。
実際に何がしたいのかが分かれば何らかの回答をすることが出来るかもしれません。

XY問題であるかもしれません。
https://ja.meta.stackoverflow.com/questions/2701/xy-%E5%95%8F%E9%A1%8C-%E3%81%A8%E3%81%AF%E4%BD%95%E3%81%A7%E3%81%99%E3%81%8B

[退会済みユーザー]

やりたいことはfor-in文内で最後の要素に到達したことを検知したいということだけです。

countでも、enumurated()でも、最後の要素を比較するでも実現したいことはできているのですが、
もうちょっと簡単に書く方法はないのかなと、

XY問題に近いと思います。
もっと簡単な記述方法があるはずだと自分で思いこんでいるフシがあります。

[gingertail]

XY問題と言っているのは「最後の要素に到達したことを検知」する必要がないのではないか？、ということではないでしょうか？
そもそもで言うと`if i === limitedHoges.last {`で十分簡潔だと思うのですがどうなのでしょうか？

[退会済みユーザー]

コードの例示がわかりづらくて良くなかったです
配列の途中にname == "D"という要素を挟むことによって最後の要素に到達する場合と到達しない場合があると明示すればよかったと思います。

[MasakiHori]

提示のコードであれば検出する必要はなくループを抜けた後で
limitedHoges.last.map { print($0.value) }
とすればいいだけですね。

Answer 1

let hoge = limitedHoges.first(where: {$0.name == "D"}) ?? limitedHoges.last!
print(hoge.name, hoge.value)

質問のコードであればこれで大丈夫？
Dがあればその要素を表示、無ければ最後の要素を表示しています。

Comments

[退会済みユーザー]

せっかかく回答頂いたのですが、
質問文のコードが適切ではありませんでした。
すいませんでした。

Answer 2

単純に最後の要素に処理を追加するだけなら、ループの外に出してしまうのが簡単だと思います。

for i in limitedHoges {
    // すべての要素に対して行う処理
}
if let i = limitedHoges.last {
    // 最後の要素に追加する処理
}

最後の要素とそれ以外で別の処理を行う場合も、

for i in limitedHoges.dropLast() {
    // 最後の要素以外に行う処理
}
if let i = limitedHoges.last {
    // 最後の要素だけに行うの処理
}

でできると思います。

Comments

[退会済みユーザー]

回答ありがとうございます。
breakなんで外に出すのはちょっと。

Answer 3

要素数をカウントする変数(count)を用意するのはどうでしょうか？

var count = 0
for i in limitedHoges {
    count += 1
    if count == limitedHoges.count {
        print("最後の要素に到達")
    }
}

enumeratedを使う方法もあります

for (i, value) in limitedHoges.enumerated() {
    if i == limitedHoges.count - 1 {
        print("最後の要素に到達")
    }
}

Comments

[退会済みユーザー]

回答ありがとうございます。i.offset/i.elementですね。

[退会済みユーザー]

要素で比較するか、要素で比較できないときはindexで判断するしかないようですね。