while(true)を撲滅したい

[Lucky3028]

SimpleSlider/src/main/kotlin/click/seichi/simpleslider/listener/PlayerClickListener.kt

Lines 68 to 111 in a56ebce

	when(defaultRegions.size) {
	0 -> for (i in 1..maxDistanceOfSearching()) {
	val nextLocation = defaultLocation.clone().apply {
	when (direction) {
	NORTH -> z -= i
	EAST -> x += i
	WEST -> x -= i
	SOUTH -> z += i
	}
	}
	// ERR: その座標にあるブロックを取得できない
	val nextLocBlock = nextLocation.block ?: return Err(BlockNotFoundException)
	// SUC: スライダーであるかつSliderTypeが同じならLocationを返し、ERR: でないならループを戻る
	if (isSlider(nextLocBlock) && sliderType == getSliderType(nextLocBlock)!!)
	return Ok(nextLocation)
	}
	1 -> {
	val defaultRegion = defaultRegions.iterator().next()
	var i = 1
	while (true) {
	val nextLocation = defaultLocation.clone().apply {
	when (direction) {
	NORTH -> z -= i
	EAST -> x += i
	WEST -> x -= i
	SOUTH -> z += i
	}
	}
	// ERR: 保護が見つからない or 同じ保護で保護されていない
	if (getRegions(nextLocation).none { it.id == defaultRegion.id })
	return Err(SliderNotFoundInSameRegion)
	// ERR: その座標にあるブロックを取得できない
	val nextLocBlock = nextLocation.block ?: return Err(BlockNotFoundException)
	// SUC: スライダーであるかつSliderTypeが同じなら、Locationを返す
	// ERR: そうでないなら、カウンタをインクリメントして再びループ
	// `getSliderType(nextLocBlock)`は、`isSlider(nextLocBlock)`の後に指定してあるのでnullにはならない
	if (isSlider(nextLocBlock) && sliderType == getSliderType(nextLocBlock)!!)
	return Ok(nextLocation)
	i++
	}
	}
	else -> return Err(RegionsDuplicatedException)
	}

このwhen節の中のwhile(true)を撲滅できれば関数化できるのに...

[Lucky3028]

保護があろうがなかろうが最大値は決めたほうがよさそう