Как уже указывалось, вам, скорее всего, нужно использовать ordNum
.
Кроме того, лучший ответ, который кто-либо может дать с кодом, который вы разместили, - "возможно".Ваша логика, безусловно, выглядит правильно и связана с тем, что вы описали, но то, будет ли она делать то, что должно, полностью зависит от вашей реализации в другом месте.
Как общий указатель (который может или не может быть применим в этом случае) вы должны убедиться, что ваш код имеет дело с крайними случаями и неправильными значениями.Поэтому, возможно, вы захотите пометить что-то не так, если, например, readyCollected.remove(b);
возвращает false, поскольку это означает, что b
не было в списке для удаления.
Как уже указывалось, взгляните на модульные тестыиспользуя JUnit для этого типа вещей.Его легко использовать, и писать тщательные юнит-тесты - очень хорошая привычка.