Make CraftingManager::sort() more cool with spaceship operators 😎

This commit is contained in:
Dylan K. Taylor 2018-03-27 19:48:10 +01:00
parent 145a4fad0f
commit bc836aaec1

View File

@ -145,21 +145,10 @@ class CraftingManager{
* @return int
*/
public static function sort(Item $i1, Item $i2){
if($i1->getId() > $i2->getId()){
return 1;
}elseif($i1->getId() < $i2->getId()){
return -1;
}elseif($i1->getDamage() > $i2->getDamage()){
return 1;
}elseif($i1->getDamage() < $i2->getDamage()){
return -1;
}elseif($i1->getCount() > $i2->getCount()){
return 1;
}elseif($i1->getCount() < $i2->getCount()){
return -1;
}else{
return 0;
}
//Use spaceship operator to compare each property, then try the next one if they are equivalent.
($retval = $i1->getId() <=> $i2->getId()) === 0 && ($retval = $i1->getDamage() <=> $i2->getDamage()) === 0 && ($retval = $i1->getCount() <=> $i2->getCount());
return $retval;
}
/**