Added logic to deal with roundabout instructions that involve a high …

CHANGELOG.md

@@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - Fixed the coordinate precision in the geojson export from 4 to 6 decimals (Issue #136)
 - Fixed the instructions='false' error when exporting as geojson (Issue #138)
 - Fixed missing summary in the geojson output (Issue #139)
+- Fixed error when a high exit number for a roundabout is used in instructions (Issue #145)
 
 ### Changed
 - Updated logging so that stack traces are only output when debug logging is enabled (Issue #148)

openrouteservice/src/main/java/heigit/ors/routing/instructions/InstructionTranslator.java

@@ -120,12 +120,28 @@ public String getRoundabout(int exitNumber, String wayName)
 		boolean isWayNull = Helper.isEmpty(wayName);
 		String str = isWayNull ? _actionRoundaboutDefault: _actionRoundaboutName;
 		boolean isExitNull = (exitNumber == 0);
+		boolean highNumber = false;
+
+		// We need to check if the exit number is greater than 10, as that is the most we have in the n-th representation
+		highNumber = (exitNumber > _numerals.length-1);
 
 		//If there was an error in finding the exit number, return "UNKNOWN". If there is no way name, don't return a way name
 		if(isExitNull)
 			str = str.replace("{exit_number}",  "UNKNOWN");
-		else
-			str = str.replace("{exit_number}",  _numerals[exitNumber]);
+		else {
+			String numeral = "";
+			if(highNumber) {
+				// if it is a high number which is very rare, then we dont use the numeral representation, just the
+				// number itself
+				// Converting to the th is too complicated due to exceptions and the position of the "th"
+
+				numeral = Integer.toString(exitNumber);
+			} else {
+				numeral = _numerals[exitNumber];
+			}
+
+			str = str.replace("{exit_number}", numeral);
+		}
 		if (isWayNull)
 			return str;
 		else