Android-Invalidate()

[TeraTroll]

Guten Abend,

ich habe mich, nach der vergangenen Zeit in welche ich viel für die Schule machen musste, mal wieder an Java gesetzt und wollte ein bisschen mit der Android SDK rumspielen.
Nun habe ich allerdings ein Problem, bei dem "refreshen" meines Canvas.

Code:

 Spoiler

package com.high_guy.tut;

import android.app.Activity;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.os.Bundle;
import android.view.View;
import android.view.WindowManager;
import java.lang.Math;

public class MainActivity extends Activity {
    private int wy,t;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(new MyView(this));
    }

    public class MyView extends View {
        public MyView(Context context) {
            super(context);
            // TODO Auto-generated constructor stub
        }

        @Override
        protected void onDraw(Canvas canvas) {
            getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN,WindowManager.LayoutParams.FLAG_FULLSCREEN);
            // TODO Auto-generated method stub
            super.onDraw(canvas);
            int x = getWidth();
            int y = getHeight();
            int radius;
            radius = 10;
            Paint paint = new Paint();
            paint.setStyle(Paint.Style.FILL);
            paint.setColor(Color.WHITE);
            canvas.drawPaint(paint);
            // Use Color.parseColor to define HTML colors
            //paint.setColor(Color.parseColor("#CD5C5C"));
            paint.setColor(Color.BLUE);
            for(int i=0;i<y-20;i++) {
                t++;
                wy = (int)Math.sin(t)*20;
                canvas.drawCircle(x/2+wy, 10+i*20, radius, paint);
            }
            invalidate();
        }
    }
}

Vielleicht kann mir ja einer von euch helfen,
vielen Dank im Voraus.

MfG Joshi

[Dantox]

Wo liegt denn dein Problem? Siehst du nichts? Auf den ersten Blick, kann ich dir auch nicht sagen, wo dein Fehler liegt, da ich mit SurfaceView's arbeite, die schneller als normale Views sind. Ich schau' heute nachmittag nochmal drüber.

Vorab:
Deine Fullscreen Methode gehört in onCreate() und Berechnungen, sowie Paints erstellen/stylen, packt man wegen der Performance in eine eigene Methode, die man vorher aufruft.

[TeraTroll]

Ich habe vor die Blöcke testweise bewegen zu lassen, jedoch tun sie dies nicht, sie bleiben in der Mitte stehen, weswegen ich denke, dass es sich nicht redrawed.
Habe noch nie mit surfaceview gearbeitet. ^^

[Dantox]

Dein Problem ist, dass du eine for-schleife benutzt. Die Idee ist nicht schlecht, für solch ein View aber nicht. Der for-loop wird ganz(!) ausgeführt, sobald er einmal(!) aufgerufen. Deine Idee war wahrscheinlich, dass es nach jedem refresh den for-loop weiterführt.
Hier ist eine Lösung, wie es funktioniert:

 Spoiler

package com.high_guy.tut;

import android.app.Activity;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.os.Bundle;
import android.view.View;
import android.view.Window;
import android.view.WindowManager;
import java.lang.Math;

public class MainActivity extends Activity {
    private int wy,t,x,y,radius,i;
    private Paint paintC;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        // requesting to turn the title OFF
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        // full screen
        getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN, WindowManager.LayoutParams.FLAG_FULLSCREEN);
        // set layout
        setContentView(new MyView(this));
    }

    public class MyView extends View {
        public MyView(Context context) {
            super(context);
            init();
        }

        private void init() {
            radius = 10;
            
            paintC = new Paint();
            paintC.setStyle(Paint.Style.FILL);
            paintC.setColor(Color.BLUE);
	}

	@Override
        protected void onDraw(Canvas canvas) {
            super.onDraw(canvas);
            // draw background
            canvas.drawColor(Color.WHITE);
            
            // set width and height
            x = getWidth();
            y = getHeight();

            // change circle
            if(i<y-20){
                t++;
                wy = (int)Math.sin(t)*20;
                canvas.drawCircle(x/2+wy, 10+i*20, radius, paintC);
                i++;
            }
            
            // refresh view
            invalidate();
        }
    }
}

[TeraTroll]

Danke sehr dies hat nun gut funktioniert(es bewegt sich sozusagen ein Circle von oben nach unten und verschwindet.)
Nun habe ich versucht, einen Circle in einem Kreis bewegen zu lassen(sin/cos),
das will nun aber nicht funktioniere.
Wodran liegt es nun?

 Spoiler

package com.guy.high.first;

import android.app.Activity;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.os.Bundle;
import android.view.View;
import android.view.Window;
import android.view.WindowManager;
import java.lang.Math;

public class MainActivity extends Activity {
    private int t,x,y,radius,sin,cos;
    private Paint paintC;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        // requesting to turn the title OFF
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        // full screen
        getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN, WindowManager.LayoutParams.FLAG_FULLSCREEN);
        // set layout
        setContentView(new MyView(this));
    }

    public class MyView extends View {
        public MyView(Context context) {
            super(context);
            init();
        }

        private void init() {
            radius = 10;

            paintC = new Paint();
            paintC.setStyle(Paint.Style.FILL);
            paintC.setColor(Color.BLUE);
        }

        @Override
        protected void onDraw(Canvas canvas) {
            super.onDraw(canvas);
            // draw background
            canvas.drawColor(Color.WHITE);

            // set width and height
            x = getWidth();
            y = getHeight();

            // change circle
            canvas.drawCircle(x/2+sin, y/2+cos, radius, paintC);

            t++;
            sin = (int)Math.sin(t)*50;
            cos = (int)Math.cos(t)*50;

            // refresh view
            invalidate();
        }
    }
}

[Dantox]

Schau dir nochmal die Unterschiede der verschiedenen Datentypen in Java an. Mit einem Integer sin/cos kommst du wegen der Nachkommastellen nicht weit. Benutz stattdessen bspw. float.

 Spoiler

package com.guy.high.first;

import android.app.Activity;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.view.Window;
import android.view.WindowManager;

public class MainActivity extends Activity {
    private int x,y,radius;
    private double t;
    private float sin, cos;
    private Paint paintC;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        // requesting to turn the title OFF
        requestWindowFeature(Window.FEATURE_NO_TITLE);
        // full screen
        getWindow().setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN, WindowManager.LayoutParams.FLAG_FULLSCREEN);
        // set layout
        setContentView(new MyView(this));
    }

    public class MyView extends View {
        public MyView(Context context) {
            super(context);
            init();
        }

        private void init() {
            radius = 10;

            paintC = new Paint();
            paintC.setStyle(Paint.Style.FILL);
            paintC.setColor(Color.BLUE);
        }

        @Override
        protected void onDraw(Canvas canvas) {
            super.onDraw(canvas);
            // draw background
            canvas.drawColor(Color.WHITE);

            // set width and height
            x = getWidth();
            y = getHeight();

            // draw circle
            canvas.drawCircle(x/2+sin, y/2+cos, radius, paintC);

            // math
            t+=0.1;
            sin = (float)Math.sin(t)*50;
            cos = (float)Math.cos(t)*50;

            // check values for sin/cos in logcat
            Log.d("MainActivity", sin + " " + cos);
            
            // refresh view
            invalidate();
        }
    }
}

Ich habe den Datentyp für t zu double geändert, damit man in kleineren Schritten den Circle drehen kann. t kannst du natürlich beliebig verändern. Damit du selbst schneller deine Fehler erkennst, habe ich dir mal ein Beispiel gezeigt, wie du mit LogCat die Werte der Variablen checken kannst, während die App läuft.

[TeraTroll]

Vielen Dank es funktioniert wunderbar