julia0028 / quaternion Goto Github PK

1. Названия некоторыхм етодов не совсем удачны: quaternionSum, quaternionMinus, scalarMultiplication, quaternionMultiplication, а ещё и не консистентны. Предлагаю два варианта переименования:

• add, subtract, multiply, divide - более общий и традиционный вариант, называть методы, вычисляющие что-то глаголами.
• plus, minus, times, div - общепринятые названия арифметических операций.

Обратите внимание, что можно создавать методы с одинаковыми навзаниями, но разными типами аргументов.

Вместо quaternionVector и quaternionScalar лучше сделать getVector/getScalar, либо vector/scalar

2. А нужен ли класс Coordinates вообще? У вас сейчас есть класс, представляющий вектор. Утилиты можно оформить следующим образом:

• методы getAxis/getRotationAngle у кватерниона, вычисляющие ось и угол поворота. и статический метод у кватерниона createQuaternion(double angle, Vector axis). Сейчас Coordinates по сути альтернативное представление кватерниона, с которым нельзя делать никаких операций.

3. В тестах хотелось бы побольше кейсов: следует добавить граничные и ошибочные ситуации.

4. Определите в тестах вспомогательный assertEquals для кватернионов, принимающий точность сравнения (лучше всего измеряемую в Math.ulp для конкретного числа).