Basketball HW #13
Basketball HW #13
Conversation
|
Сила броска зависит от вертикального слайда по мячу. Область попадания сделал соразмерной щиту, чтобы почаще попадать) |
|
|
||
| final View ball = getBall(); | ||
| ViewTreeObserver ballObserver = ball.getViewTreeObserver(); | ||
| ballObserver.addOnGlobalLayoutListener(new ViewTreeObserver.OnGlobalLayoutListener() { |
There was a problem hiding this comment.
Можно конечно и GlobalLayoutListener использовать, но тут достаточно post
| public boolean onTouch(View v, MotionEvent event) { | ||
| switch (event.getAction()) { | ||
| case MotionEvent.ACTION_DOWN: { | ||
| velocityTracker = VelocityTracker.obtain(); |
There was a problem hiding this comment.
Лучше перед тем как брать новый VelocityTracker отпустить старый методом recycle
| TypeEvaluator<Float> xEvaluator = new TypeEvaluator<Float>() { | ||
| @Override | ||
| public Float evaluate(float fraction, Float startValue, Float endValue) { | ||
| // float result = startValue + fraction * xVelocity; |
There was a problem hiding this comment.
Это была попытка учесть горизонтальную скорость, но там надо подзапариться с формулами из физики, чтобы получить startValue и endValue. По закоменченой формуле анимация заканчивается раньше, чем необходимо, поэтому не стал ее использовать.
| }); | ||
| } | ||
|
|
||
| private void animateBall(final View ball, final float xVelocity, final float yVelocity) { |
| @Override | ||
| protected void onStop() { | ||
| super.onStop(); | ||
| velocityTracker.recycle(); |
There was a problem hiding this comment.
Если ни разу не кинуть и свернуть приложение будет NPE. Т.к. velocityTracker и animatorSet могут быть еще не созданы
| float rating = ((float) successfulThrows / allThrows) * starCount; | ||
| ratingView.setActiveStarCount(Math.round(rating)); | ||
| TextView counter = findViewById(R.id.point_counter); | ||
| counter.setText(String.format("%d/%d", successfulThrows, allThrows)); |
There was a problem hiding this comment.
Когда работаешь с числами в строках, лучше явно указывать локаль. Т.к. не на всех языках цифры выглядят так, как мы привыкли
String.format(Locale.ENGLISH, "%d/%d", successfulThrows, allThrows)
| } | ||
|
|
||
| private void updateViews() { | ||
| RatingView ratingView = findViewById(R.id.star_rating); |
There was a problem hiding this comment.
Не обязательно каждый раз искать view, можно сохранить его в поле класса
|
Работа принята |
No description provided.