Commit31bf130

and

authored

refactor: improvingMedian (#6404)

refactor: improving MedianCo-authored-by: Deniz Altunkapan <93663085+DenizAltunkapan@users.noreply.github.com>

1 parentb45fd2a commit31bf130Copy full SHA for 31bf130

File tree

+12

-0

lines changed

src
- main/java/com/thealgorithms/maths
  - Median.java
- test/java/com/thealgorithms/maths
  - MedianTest.java

+12

-0

lines changed

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,13 @@ private Median() {`
`13`	`13`	`* Calculate average median`
`14`	`14`	`* @param values sorted numbers to find median of`
`15`	`15`	`* @return median of given {@code values}`
	`16`	`+ * @throws IllegalArgumentException If the input array is empty or null.`
`16`	`17`	`*/`
`17`	`18`	`publicstaticdoublemedian(int[]values) {`
	`19`	`+if (values ==null \|\|values.length ==0) {`
	`20`	`+thrownewIllegalArgumentException("Values array cannot be empty or null");`
	`21`	`+ }`
	`22`	`+`
`18`	`23`	`Arrays.sort(values);`
`19`	`24`	`intlength =values.length;`
`20`	`25`	`returnlength %2 ==0 ? (values[length /2] +values[length /2 -1]) /2.0 :values[length /2];`

Lines changed: 7 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,7 @@`
`1`	`1`	`packagecom.thealgorithms.maths;`
`2`	`2`
`3`	`3`	`importstaticorg.junit.jupiter.api.Assertions.assertEquals;`
	`4`	`+importstaticorg.junit.jupiter.api.Assertions.assertThrows;`
`4`	`5`
`5`	`6`	`importorg.junit.jupiter.api.Test;`
`6`	`7`
`@@ -34,4 +35,10 @@ void medianNegativeValues() {`
`34`	`35`	`int[]arr = {-27, -16, -7, -4, -2, -1};`
`35`	`36`	`assertEquals(-5.5,Median.median(arr));`
`36`	`37`	`}`
	`38`	`+`
	`39`	`+@Test`
	`40`	`+voidmedianEmptyArrayThrows() {`
	`41`	`+int[]arr = {};`
	`42`	`+assertThrows(IllegalArgumentException.class, () ->Median.median(arr));`
	`43`	`+ }`
`37`	`44`	`}`

Comments

(0)