reorder if and else to make more sense

src/check.rs

@@ -54,10 +54,10 @@ pub fn player_name<S: Into<String>>(name: S) -> PlayerCheck {
 		}
 	}
 
-	if !err_vec.is_empty() {
-		PlayerCheck::Err(err_vec)
-	} else {
+	if err_vec.is_empty() {
 		PlayerCheck::Ok(name.into())
+	} else {
+		PlayerCheck::Err(err_vec)
 	}
 }