Conversation
| Random rand = new Random(); | ||
| View arc1; | ||
| Paint paintTemp; | ||
| Paint paintSmall; | ||
| Paint paintArc1; | ||
| Paint paintArc2; | ||
| Paint paintArc3; | ||
| Paint paintArc4; |
There was a problem hiding this comment.
нет модификаторов доступа. в данном случае это обычные поля класса. они должны быть приватными
|
|
||
| if (event.getAction() == MotionEvent.ACTION_DOWN) { | ||
| // Toast.makeText(this, "ergerg", Toast.LENGTH_SHORT).show(); | ||
| if ((event.getX() - centerX) * (event.getX() - centerX) + (event.getY() - centerY) * (event.getY() - centerY) <= 150 * 150) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
| paintArc2.setColor(paintArc1.getColor()); | ||
| paintArc1.setColor(paintTemp.getColor()); | ||
| invalidate(); | ||
| } else if ((event.getX() - centerX) * (event.getX() - centerX) + (event.getY() - centerY) * (event.getY() - centerY) <= 400 * 400) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
| paintArc1.setColor(paintTemp.getColor()); | ||
| invalidate(); | ||
| } else if ((event.getX() - centerX) * (event.getX() - centerX) + (event.getY() - centerY) * (event.getY() - centerY) <= 400 * 400) { | ||
| if (event.getX() > centerX && event.getX() < centerX + 400 && event.getY() > centerY && event.getY() < centerY + 400) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
| if (event.getX() > centerX && event.getX() < centerX + 400 && event.getY() > centerY && event.getY() < centerY + 400) { | ||
| paintArc1.setColor(rand.nextInt(2147483646)); | ||
| invalidate(); | ||
| } else if (event.getX() < centerX && event.getX() > centerX - 400 && event.getY() > centerY && event.getY() < centerY + 400) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
| } else if (event.getX() < centerX && event.getX() > centerX - 400 && event.getY() > centerY && event.getY() < centerY + 400) { | ||
| paintArc2.setColor(rand.nextInt(2147483646)); | ||
| invalidate(); | ||
| } else if (event.getX() < centerX && event.getX() > centerX - 400 && event.getY() < centerY && event.getY() > centerY - 400) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
| } else if (event.getX() < centerX && event.getX() > centerX - 400 && event.getY() < centerY && event.getY() > centerY - 400) { | ||
| paintArc3.setColor(rand.nextInt(2147483646)); | ||
| invalidate(); | ||
| } else if (event.getX() > centerX && event.getX() < centerX + 400 && event.getY() < centerY && event.getY() > centerY - 400) { |
There was a problem hiding this comment.
Я бы посоветовал вынести расчеты в отдельные методы и просто их здесь вызывать. Так код твой намного проще будет понимать
|
|
||
| import java.util.Random; | ||
|
|
||
| public class Custom extends View { |
There was a problem hiding this comment.
Не самое удачное имя для класса. Не сразу понятно что тут имеется ввиду.
| } | ||
|
|
||
| @Override | ||
| public boolean onTouchEvent(MotionEvent event) { |
There was a problem hiding this comment.
Смотри. У тебя есть твоя вьюха Custom. Именно она должна сообщать о том, что она нажата и передавать в активити координаты. В твоей же реализации onTouchEvent в этой активити отрабатывает на нажатие всего активити. В общем. Тебе надо создать свой лисенер и установить его в твою view. И если у тебя будет нажат сектор, то через этот лисенер надо передать в активити координаты.
There was a problem hiding this comment.
| import android.view.MotionEvent; | ||
| import android.widget.Toast; | ||
|
|
||
| public class MainActivity extends AppCompatActivity { |
There was a problem hiding this comment.
Если не ошибаюсь, то в задании надо было добавить Switch и показывать либо Toast, либо Snackbar
| private int centerY; | ||
| private float smallCircleArea; | ||
| private float bigCircleArea; | ||
| CoordinateListener coordinateListener; |
There was a problem hiding this comment.
Нет модификатора доступа =)
| invalidate(); | ||
| } else if (isSecondSector(event)) { | ||
| paintArc2.setColor(rand.nextInt(2147483646)); | ||
| coordinateListener.getCoordinates(event.getX(), event.getY(), paintArc2.getColor()); |
There was a problem hiding this comment.
coordinateListener тоже лучше проверять на null. Java же сама по себе подразумевает что любой объект может быть null, а тут ты его еще и устанавливаешь через сеттер. Так что ты точно не можешь сказать что он не null. В этом случае я бы еще порекомендовал вынести работу с coordinateListener в отдельный метод, а координаты и цвет передавать туда в качестве параметров метода
2 participants
No description provided.