Skip to main content
図を添付して、理解しやすくしました。
ソース リンク
Harawo
  • 4,599
  • 2
  • 13
  • 17

デクリメントのほうなんですけど、整数値が環状に並んでいるというイメージをしてください。(下図)
画像の説明をここに入力
2戻るということは、3進むのと同じですそうすればこれを、一般化して表現すると、increment戻るということは、range.length - increment進むとおなじであることに気づくはずです。さらに厳密に表現すると、range.length - increment % range.length進むと同じです。これをそのままインクリメント(increment > 0)のときの式に代入してやります。具体的にコードにすると、こうなります。

#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進むとおなじ」に変更しました。

三度目の編集をしました。
「整数値を環状に並べる」を、図にして、わかりやすくしました。

デクリメントのほうなんですけど、整数値が環状に並んでいるというイメージをしてください。そうすれば、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進むとおなじ」に変更しました。

デクリメントのほうなんですけど、整数値が環状に並んでいるというイメージをしてください。(下図)
画像の説明をここに入力
2戻るということは、3進むのと同じですこれを、一般化して表現すると、increment戻るということは、range.length - increment進むとおなじであることに気づくはずです。さらに厳密に表現すると、range.length - increment % range.length進むと同じです。これをそのままインクリメント(increment > 0)のときの式に代入してやります。具体的にコードにすると、こうなります。

#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

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

#import <Foundation/Foundation.h>

NSUInteger NSRangeIncrementLocationInRange(NSInteger location, NSInteger increment, NSRange range) {
    NSUInteger result;
    // locationがrangeの範囲外なら、NSNotFoundを返す。
    if (!NSLocationInRange(nlocation, circleRangerange)) 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(45, -38, NSMakeRange(24, 65));
        NSLog(@"Result = %ld", inc);
    }
    return 0;
}

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

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

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

#import <Foundation/Foundation.h>

NSUInteger NSRangeIncrementLocationInRange(NSInteger location, NSInteger increment, NSRange range) {
    NSUInteger result;
    // locationがrangeの範囲外なら、NSNotFoundを返す。
    if (!NSLocationInRange(n, circleRange)) 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)) % range.length + range.location;
    }
    
    return result;
}

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

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

デクリメントのほうなんですけど、整数値が環状に並んでいるというイメージをしてください。そうすれば、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

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

#import <Foundation/Foundation.h>

NSUInteger NSRangeIncrementLocationInRange(NSUIntegerNSInteger location, NSUIntegerNSInteger increment, NSRange range) {
    NSUInteger result;
    // locationがrangeの範囲外なら、NSNotFoundを返す。
    if (!NSLocationInRange(n, circleRange)) 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)) % range.length + range.location;
    }
    
    return result;
}

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

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

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

#import <Foundation/Foundation.h>

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

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

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

#import <Foundation/Foundation.h>

NSUInteger NSRangeIncrementLocationInRange(NSInteger location, NSInteger increment, NSRange range) {
    NSUInteger result;
    // locationがrangeの範囲外なら、NSNotFoundを返す。
    if (!NSLocationInRange(n, circleRange)) 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)) % range.length + range.location;
    }
    
    return result;
}

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

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

ソース リンク
Harawo
  • 4,599
  • 2
  • 13
  • 17
読み込んでいます