rild / android-joystick-test Goto Github PK

View Code? Open in Web Editor NEW

3.0 3.0 1.0 782 KB

clone joystick app

Java 100.00%

android-joystick-test's People

Contributors

Stargazers

Watchers

Forkers

stmen

android-joystick-test's Issues

OnJoyStickMoveListener 周りのリファクタリング

ネストが深くて闇が深まりそうなのでなんとかする

perform()
performInternal()

とかに切り分ける？

JoyStickSurfaceViewのサイズ

自動的にうまいことやってくれるようにしないといけない

画像リソース管理

コードの中に R.drawable. ... が埋もれていて、汎用性がないのでなんとかする

notification test

画像の読み込みに時間がかかる

unnecessary code, furthermore something wrong

private void drawLoadingAnimation() {
        while (isLoading) {
            Canvas canvas = surfaceHolder.lockCanvas();
            canvas.drawColor(Color.TRANSPARENT, PorterDuff.Mode.CLEAR);

            Matrix matrix = new Matrix();

            matrix.postRotate((float) 12.5);
            loadingImg = Bitmap.createBitmap(loadingImg, 0, 0, loadingImg.getWidth(), loadingImg.getHeight(), matrix, true);
            drawProcessingCircle(canvas);

            surfaceHolder.unlockCanvasAndPost(canvas);

            try {
                Thread.sleep(30);
            } catch (InterruptedException e) {

            }
        }
    }

    private void drawProcessingCircle(Canvas canvas) {
        int w = 0;
        int h = 0;
        if (params != null) {
            w = params.width;
            h = params.height;
        }
        canvas.drawBitmap(loadingImg, w / 2 - loadingImg.getWidth() / 2, h / 2 - loadingImg.getHeight() / 2, alphaSignal);
    }

make interval depend on distance

上記の通り

JoystickMoveListener cannot be renamed

when trying to change JoystickMoveListener to JoyStickMoveListener,
it causes compile error

押し込みイベント

押し込んだ時に今は interval を早くするだけだけど、

        MORE_UP,
        MORE_UPRIGHT,
        MORE_RIGHT,
        MORE_DOWNRIGHT,
        MORE_DOWN,
        MORE_DOWNLEFT,
        MORE_LEFT,
        MORE_UPLEFT,

これ追加する...?

イベント処理

それぞれの処理が連続で呼ばれるので、イベントハンドリングしなきゃ辛い

LongPushイベントを取る

ハンドラ使ってやる

postDelayed, removeCallbacks を使う

メモリ問題

リサイズ後のアドレスをリリースした (signal on)

private void resizeImages() {
        Bitmap tmp;

        if (shadow != null) {
            tmp = shadow;
            shadow = Bitmap.createScaledBitmap(tmp, shadowWidth, shadowHeight, false);
            tmp.recycle();
        }

        tmp = stick;
        stick = Bitmap.createScaledBitmap(tmp, stickWidth, stickHeight, false);
        tmp.recycle();

        tmp = background;
        background = Bitmap.createScaledBitmap(tmp, params.width, params.height, false);
        tmp.recycle();

        if (canUseSignal) {
            tmp = signalUp;
            signalUp = Bitmap.createScaledBitmap(tmp, params.width, params.height, false);
            tmp.recycle();
            tmp = signalRight;
            signalRight = Bitmap.createScaledBitmap(tmp, params.width, params.height, false);
            tmp.recycle();
            tmp = signalDown;
            signalDown = Bitmap.createScaledBitmap(tmp, params.width, params.height, false);
            tmp.recycle();
            tmp = signalLeft;
            signalLeft = Bitmap.createScaledBitmap(tmp, params.width, params.height, false);
            tmp.recycle();
        }
    }

カーソルを光らせると重くなる

それぞれのカーソルを光らせていると処理が重くなって、
操作しにくく感じる

LongPushイベントハンドリングが美しくない

この実装しかないのか....

Detecting a long press with Android