Skip to main content
3 / 4
質問者からの指摘を受けて、修正を施しました。
Harawo
  • 4,599
  • 2
  • 13
  • 17

デクリメントのほうなんですけど、整数値が環状に並んでいるというイメージをしてください。そうすれば、increment戻るということは、range.length - increment進むとおなじであることに気づくはずです。

#import <Foundation/Foundation.h>

NSUInteger NSRangeIncrementLocationInRange(NSInteger location, NSInteger increment, NSRange range) {
    NSUInteger result;
    // locationがrangeの範囲外なら、NSNotFoundを返す。
    if (!NSLocationInRange(location, range)) return NSNotFound;
    if (increment > 0) { // incrementが正数の場合。
        result = (location - range.location + (NSUInteger)increment) % range.length + range.location;
    } else if (increment == 0) { // incrementが0の場合、locationをそのまま返す。
        result = location;
    } else { // incrementが負数の場合。range.lengthとincrementの差を、locationに加えるという計算をする。
        result = (location - range.location + (NSUInteger)((NSInteger)range.length + increment % (NSInteger)range.length)) % range.length + range.location;
// ((NSInteger)range.length + increment)) % range.length + range.location; 修正しました。
    }
    
    return result;
}

int main(int argc, const char * argv[]) {
    @autoreleasepool {
        NSUInteger inc = NSRangeIncrementLocationInRange(5, -8, NSMakeRange(4, 5));
        NSLog(@"Result = %ld", inc);
    }
    return 0;
}

編集しました。
引数incrementの型を、NSUIntegerからNSIntegerに変えました。きわめて初歩的な間違いですね。ご容赦。暗黙的に型キャストしてくれるであろう箇所も、明示的に型キャストしました。

二度目の編集をしました。負の向きに1巡以上するincrementで、正しい結果にならないという指摘を受けて、修正しました。
「increment戻るということは、range.length - increment進むとおなじ」を、
「increment戻るということは、range.length - increment % range.length進むとおなじ」に変更しました。

Harawo
  • 4,599
  • 2
  • 13
  • 17