Browse Source

Changed the sort function to uses strcmp...

... instead of a self written compare. Thanks @Baachi.
Skamander 12 years ago
parent
commit
82fdd64d87

+ 2 - 14
php-symfony2/src/Skamander/BenchmarkBundle/Controller/BenchController.php

@@ -59,13 +59,7 @@ class BenchController extends Controller
         $fortunes[] = $runtimeFortune;
         $fortunes[] = $runtimeFortune;
 
 
         usort($fortunes, function($left, $right) {
         usort($fortunes, function($left, $right) {
-            if ($left->message === $right->message) {
-                return 0;
-            } else if ($left->message > $right->message) {
-                return 1;
-            } else {
-                return -1;
-            }
+            return strcmp($left->message, $right->message);
         });
         });
 
 
         return $this->render("SkamanderBenchmarkBundle:Bench:fortunes.html.twig", [
         return $this->render("SkamanderBenchmarkBundle:Bench:fortunes.html.twig", [
@@ -86,13 +80,7 @@ class BenchController extends Controller
         $fortunes[] = $runtimeFortune;
         $fortunes[] = $runtimeFortune;
 
 
         usort($fortunes, function($left, $right) {
         usort($fortunes, function($left, $right) {
-            if ($left->message === $right->message) {
-                return 0;
-            } else if ($left->message > $right->message) {
-                return 1;
-            } else {
-                return -1;
-            }
+            return strcmp($left->message, $right->message);
         });
         });
 
 
         return $this->render("SkamanderBenchmarkBundle:Bench:fortunes.html.php", [
         return $this->render("SkamanderBenchmarkBundle:Bench:fortunes.html.php", [