"Array index is out of range:" when get a range from an array

Question

I have an array anArray and I want to create a sub array by specifying a range range, like:

 var range: Range<Int>
 guard let slice = anArray[range] else {
      return nil
 }

I run into a problem where the code throws an "Array index is out of range:" exception.

 var range: Range<Int>
 guard range.upperBound > anArray.count - 1 else {
        return nil
    }
 guard let slice = anArray[range] else {
      return nil
 }

But I see get the Array out of range exception. Can you please tell me what did I miss?

Can you update the question with the complete error message? — Phillip Mills
– Phillip Mills, Commented Sep 30, 2020 at 19:26

Corbell · Accepted Answer · 2020-09-30 19:28:31Z

4

Your operator is backwards, the guard should be ensuring

guard range.upperBound <= anArray.count - 1 else { ...

not >

(The guard will skip the else block if its condition is true - it isn't an if, it's an else).

answered Sep 30, 2020 at 19:28

Corbell

1,4231 gold badge9 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Leo Dabus · Accepted Answer · 2020-10-01 17:20:30Z

1

Besides the fact that you were using the wrong operator beware that not all collections starts at zero and not all collections have all elements up to its index before the endIndex. Thats the case of the result of your subscription which returns an ArraySlice. You should always check the collection indices, not its count. Resuming you should check if your range fits within the collection indices. Check this post. Note that RandomAccessCollection's subscript does NOT return an optional, it would crash if you pass an invalid range.

guard range.clamped(to: anArray.indices) == range else {
    return nil
}

let slice = anArray[range]

edited Oct 1, 2020 at 17:20

answered Sep 30, 2020 at 19:54

Leo Dabus

237k61 gold badges512 silver badges624 bronze badges

Collectives™ on Stack Overflow

"Array index is out of range:" when get a range from an array

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related