handle edge case

hackerSa3edy · hackerSa3edy · commit 7d52599f1b67 · 2023-12-17T17:54:50.000+02:00
diff --git a/2-selection_sort.c b/2-selection_sort.c
@@ -12,7 +12,7 @@ void selection_sort(int *array, size_t size)
 	size_t upperFor, innerFor, lowerInt_index;
 	int lowerInt;
 
-	if (size < 2 || array == NULL)
+	if (size < 2)
 		return;
 
 	for (upperFor = 0; upperFor < size - 2; upperFor++)
@@ -27,7 +27,7 @@ void selection_sort(int *array, size_t size)
 				lowerInt_index = innerFor;
 			}
 		}
-		if (array[upperFor] != lowerInt)
+		if (upperFor != lowerInt_index)
 		{
 			array[lowerInt_index] = array[upperFor];
 			array[upperFor] = lowerInt;

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ void selection_sort(int *array, size_t size)`
`12`	`12`	`size_t upperFor, innerFor, lowerInt_index;`
`13`	`13`	`int lowerInt;`
`14`	`14`
`15`		`- if (size < 2 \|\| array == NULL)`
	`15`	`+ if (size < 2)`
`16`	`16`	`return;`
`17`	`17`
`18`	`18`	`for (upperFor = 0; upperFor < size - 2; upperFor++)`
`@@ -27,7 +27,7 @@ void selection_sort(int *array, size_t size)`
`27`	`27`	`lowerInt_index = innerFor;`
`28`	`28`	`}`
`29`	`29`	`}`
`30`		`- if (array[upperFor] != lowerInt)`
	`30`	`+ if (upperFor != lowerInt_index)`
`31`	`31`	`{`
`32`	`32`	`array[lowerInt_index] = array[upperFor];`
`33`	`33`	`array[upperFor] = lowerInt;`