Сделал дз. #7
Сделал дз. #7
Conversation
| private Paint dotPaint = new Paint(); | ||
| private Path dotPath = new Path(); | ||
| private static final int DEFAULT_POINT_COLOR = Color.BLACK; | ||
| private static final int DEFAULT_POINT_STROKE_WIDTH_DP = 5; |
There was a problem hiding this comment.
dimens.xml используют для хранения размеров
| int rightPadding = getPaddingRight(); | ||
| int topPadding = getPaddingTop(); | ||
| int bottomPadding = getPaddingBottom(); | ||
| int w = getWidth(); |
| @@ -1,13 +1,119 @@ | |||
| package ru.ok.technopolis.basketball; | |||
|
|
|||
| import android.animation.ValueAnimator; | |||
There was a problem hiding this comment.
Аниматоры стоит сохранять в поля и во время уничтожения активити останавливать анимации
| private float vx; | ||
| private float vy; | ||
|
|
||
| Point(float coordinate) { |
There was a problem hiding this comment.
Если этот класс в перспективе может быть использован во вне текущего пакета, то лучше сразу объявить public
| @@ -5,6 +5,47 @@ | |||
| android:layout_width="match_parent" | |||
| android:layout_height="match_parent" | |||
| tools:context=".MainActivity"> | |||
|
|
|||
| <FrameLayout | |||
There was a problem hiding this comment.
Не понял зачем тут констрейнт? Можно тогда сделать базовый лейаут фреймом
| @@ -0,0 +1,13 @@ | |||
| <?xml version="1.0" encoding="utf-8"?> | |||
There was a problem hiding this comment.
не могу понять, тут надо что-то исправлять?
There was a problem hiding this comment.
Нет, не нужно. Молодец что заюзал атрибуты
|
там же не просто центр кольца, а прямоугольник, в который нужно кидать мяч, чтобы он отскочил в кольцо. |
Это ок. Но почему я просто кликаю и выигрываю. Разве не нужно бросать мяч?) |
|
поправил, но что-то случилось и появилась эта ошибка, я не понимаю, что с этим делать C:\Android\basketball_homework\basketball_homework\app\build\intermediates\instant_run_split_apk_resources\debug\instantRunSplitApkResourcesDebug\out\slice_0\resources_ap |
| @@ -33,8 +30,8 @@ public FieldView(Context context, @Nullable AttributeSet attrs) { | |||
| super(context, attrs); | |||
|
|
|||
| DisplayMetrics displayMetrics = context.getResources().getDisplayMetrics(); | |||
| int pointStrokeWidthFromAttr = (int) (TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, DEFAULT_POINT_STROKE_WIDTH_DP, displayMetrics) + 0.5f); | |||
| int pointRadiusFromAttr = (int) (TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, DEFAULT_POINT_RADIUS_DP, displayMetrics) + 0.5f); | |||
| int pointStrokeWidthFromAttr = (int) (TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, R.dimen.default_point_stroke_width, displayMetrics) + 0.5f); | |||
There was a problem hiding this comment.
Что бы каждый раз не делать +0.5f и не забывать про это, лучше сделать метод для перевода из dp в px
| @@ -29,6 +29,7 @@ | |||
| private Context context; | |||
| private float G = 9800; | |||
| private float maxDistance; | |||
| ValueAnimator animator; | |||
| @@ -79,8 +80,12 @@ void calculateInitialCondition(float x0, float y0, float vx0, float vy0) { | |||
| } | |||
|
|
|||
| void initialiseAnimation() { | |||
| if (isScored(startPoint.getX(), fieldView.getHeight() - startPoint.getY())) { | |||
| Toast.makeText(context, R.string.cheater, Toast.LENGTH_SHORT).show(); | |||
| @Override | ||
| protected void onPause() { | ||
| super.onPause(); | ||
| animator.cancel(); |
There was a problem hiding this comment.
animator стоит проверить на нул. Иначе, если не делать бросков и свернуть приложение будет креш
Не сталкивался с таким. Попробуй почистить build папку |
|
я не понимаю, что произошло, но сегодня утром прокт начал собираться и запускаться. той ошибки больше не было |
|
нужно ли еще что-то исправлять? |
| 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
|
Работа принята |
Дз