Android onClick méthode ne fonctionne pas sur un affichage personnalisé

La création de contrôles personnalisés dans Android peut être difficile si vous n'êtes pas à l'aise avec le Framework d'INTERFACE fonctionne. Si vous ne l'avez pas déjà, je vous recommande la lecture de ces:

http://developer.android.com/guide/topics/ui/declaring-layout.html

http://developer.android.com/guide/topics/ui/custom-components.html

http://developer.android.com/guide/topics/ui/layout-objects.html

Avis que, lorsque les dispositions sont déclarées dans le fichier XML, les éléments sont imbriqués. Cela crée une mise en page de la hiérarchie que vous devez créer votre auto lors de la personnalisation d'un composant en utilisant uniquement du code Java.

Plus vous avez de chances d'être pris dans Android tactile de la hiérarchie. Contrairement à certaines autres plates-formes mobiles, Android offre les événements tactiles à partir du haut de la Vue de la hiérarchie et travaille son chemin vers le bas. Les classes qui occupent traditionnellement les niveaux supérieurs de la hiérarchie (Activité et modèles) logique dans leur vers l'avant, touche ils n'ont pas eux-mêmes consomment.

Donc, ce que je recommande de faire est de changer votre OpeningTimesView pour étendre un ViewGroup (la super-classe de toutes Android mises en page) ou une mise en page particulière (LinearLayout, RelativeLayout, etc.) et d'ajouter vos boutons comme des enfants. Maintenant, il ne me semble pas être une hiérarchie définie (les boutons ne sont pas vraiment des "contenus" dans le conteneur, ils sont tout les membres) qui peuvent être source de confusion sur la question de l'endroit où les événements sont vraiment.

1. La touche devrait plus s'écouler naturellement vers le bas pour les boutons, permettant à vos événements click pour déclencher
2. Vous pouvez profiter de Android mécanismes de mise en page pour dessiner votre point de vue au lieu de s'appuyer sur le code de dessin pour faire tout cela.

Choisissez une mise en page de la classe pour commencer, qui vous aidera à placer vos boutons dans leur FINAL endroits. Vous pouvez utiliser le cadre de l'animation dans Android ou code de dessin personnalisé (comme vous avez maintenant), afin de les animer, de toute façon vous le souhaitez jusqu'à ce point. L'emplacement d'un bouton et lorsque ce bouton est actuellement tiré sont autorisés à être très différente si nécessaire, et c'est ainsi que l'Animation en cours Framework fonctionne sous Android (antérieures à la version 3.0)...mais c'est une question distincte. Vous avez également AbsoluteLayout, qui vous permet de placer et à remplacer les objets n'importe où vous le souhaitez...mais attention à la façon dont votre application a l'air sur tous les appareils Android avec celui-ci (étant donné les différentes tailles d'écran).

Quant à votre deuxième point à propos de l'affichage info.
La méthode la plus simple est probablement d'utiliser Context.getResources().getDisplayMetrics() partout où vous en avez besoin. Activity hérite de Context, de sorte qu'ils peuvent appeler cette méthode directement. Vues de toujours avoir un Context vous pouvez accéder à la getContext(). Toutes les autres classes, vous pouvez simplement transmettre le Context en tant que paramètre dans la construction (c'est un modèle courant dans Android, vous pourrez voir beaucoup d'objets nécessitent un Context, principalement pour avoir accès à Resources).

Voici un squelette exemple, pour relancer les choses. Cette juste des lignes de trois jusqu'à l'horizontale une fois, comme d'un emplacement final:

Public class OpeningTimesView extends LinearLayout implements OnClickListener {
private MainMenuObjectView searchButton;
private MainMenuObjectView supportButton;
private MainMenuObjectView aboutButton;
private int screenWidth;
private int screenHeight;
public OpeningTimesView(Context context) {
this(context, null);
}
//Thus constructor gets used if you ever instantiate your component from XML
public OpeningTimesView(Context context, AttributeSet attrs) {
super(context, attrs);
/* This is a better way to obtain your screen info
DisplayMetrics display = context.getResources().getDisplayMetrics();
screenWidth = display.widthPixels;
screenHeight = display.heightPixels;
*/
//This way works also, without needing to customize the constructor
WindowManager wm = (WindowManager)context.getSystemService(Context.WINDOW_SERVICE);
Display dis = wm.getDefaultDisplay();
screenWidth = dis.getWidth();
screenHeight = dis.getHeight();
searchButton = new MainMenuObjectView(context, 200, MovingMode.RIGHT, R.drawable.search, dis);
supportButton = new MainMenuObjectView(context, 400, MovingMode.LEFT, R.drawable.support, dis);
aboutButton = new MainMenuObjectView(context, 600, MovingMode.RIGHT, R.drawable.about, dis);
//Even if they don't extend button, if MainMenuObjectView is always clickable
//this should probably be brought into that class's constructor
searchButton.setClickable(true);
supportButton.setClickable(true);
aboutButton.setClickable(true);
searchButton.setOnClickListener(this);
supportButton.setOnClickListener(this);
aboutButton.setOnClickListener(this);
//Add the buttons to the layout (the buttons are now children of the container)
setOrientation(LinearLayout.HORIZONTAL);
LinearLayout.LayoutParams params = new LinearLayout.LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT);
addView(searchButton, params);
addView(supportButton, params);
addView(aboutButton, params);       
}
@Override
public void onClick(View view){
Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
if(view == searchButton){
Toast.makeText(getContext(), "Search button pressed", Toast.LENGTH_SHORT).show();
}
else if(view == supportButton){
Toast.makeText(getContext(), "Support button pressed", Toast.LENGTH_SHORT).show();
}
else Toast.makeText(getContext(), "About button pressed", Toast.LENGTH_SHORT).show();
}
@Override
public void onDraw(Canvas canvas)
{
//Drawing the buttons
//This may only be necessary until they are in place, then just call super.onDraw(canvas)
this.searchButton.onDraw(canvas);
this.aboutButton.onDraw(canvas);
this.supportButton.onDraw(canvas);
}    
}

Vous pouvez la personnaliser à partir de là. Commençant peut-être les boutons de la visibilité de l'ensemble de la Vue.INVISIBLE jusqu'à ce que vous animez avec votre code de dessin ou une Animation personnalisée objet, puis de les rendre visible dans leur lieu de repos final.

La clé ici, c'est la mise en page est assez intelligent pour savoir que lorsqu'il reçoit un événement tactile il est censé transmettre à l'enfant correspondant. Vous pouvez créer une vue personnalisée sans cela, mais vous aurez à intercepter toutes les touches sur le récipient et faire le calcul pour déterminer la sous-vue manuellement avant l'événement. Si vous avez réellement ne peux pas faire de mise en page du gestionnaire de travail, c'est votre recours.

Espère que ça Aide!

Je le fais donc:

public class YourView extends LinearLayout implements OnClickListener {
OnClickListener listener;
//... constructors
public void setOnClickListener(OnClickListener listener) {
this.listener = listener;
}
@Override
public void onClick(View v) {
if (listener != null)
listener.onClick(v);
}
}

Mettre en œuvre les onClickListener dans le MainMenuObjectView de la classe, puisque ce sont les objets qui permettront de répondre aux clics.

Une autre alternative serait d'étendre Button au lieu de View, parce que vous utilisez seulement les boutons là

---

Mise à jour: exemple Complet

---

C'est l'idée de l'implanter directement dans le cliquables points de vue. Il y a un TestView classe qui étend la classe View et remplace onDraw, comme vous en avez besoin, et répond aussi aux clics. J'ai laissé de côté tout l'animation de la mise en œuvre que vous avez de la partie, et il n'est pas pertinent pour la ClickListener discussion.

Je l'ai testé dans un Eclair de l'émulateur et il fonctionne comme prévu (un Toast message après un clic).

fichier: Test.java

package com.aleadam.test;
import android.app.Activity;
import android.os.Bundle;
import android.widget.LinearLayout;
import android.widget.TextView;
public class Test extends Activity {
/** Called when the activity is first created. */
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
LinearLayout ll = new LinearLayout(this);
ll.setOrientation(LinearLayout.VERTICAL);
TextView label = new TextView(this);
LinearLayout.LayoutParams layoutParams = new LinearLayout.LayoutParams(
LinearLayout.LayoutParams.WRAP_CONTENT, LinearLayout.LayoutParams.WRAP_CONTENT);
label.setText("Click the circle!");
TestView testView = new TestView(this);
ll.addView(label, layoutParams);
ll.addView(testView, layoutParams);
setContentView(ll);
}
}

fichier: TestView.java

package com.aleadam.test;
import android.content.Context;
import android.graphics.Canvas;
import android.graphics.Color;
import android.graphics.Paint;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Toast;
public class TestView extends View implements OnClickListener {
Context context;
public TestView(Context context) {
super(context);
this.context = context;
setOnClickListener(this);
}
public void onClick(View arg0) {
Toast.makeText(context, "View clicked.", Toast.LENGTH_SHORT).show();
}
@Override
public void onDraw (Canvas canvas) {
super.onDraw(canvas);
this.setBackgroundColor(Color.LTGRAY);
Paint paint = new Paint (Paint.ANTI_ALIAS_FLAG);
paint.setColor(Color.RED);
canvas.drawCircle(20, 20, 20, paint);
}
}

Si vous avez besoin de quelques cliquable et certains pas cliquable, vous pouvez ajouter un constructeur avec un
argument booléen pour déterminer si le ClickListener est attaché ou non à la Vue:

public TestView(Context context, boolean clickable) {
super(context);
this.context = context;
if (clickable)
setOnClickListener(this);
}

J'ai eu le même problème. Dans mon cas, j'ai eu un LinearLayout comme un élément racine de ma vue personnalisée, avec clickable et focusable défini à true, et la vue personnalisée de la balise elle-même (utilisée dans un fragment de la mise en page) a aussi été mis à clickable et focusable. S'avère que la seule chose que j'avais à faire pour obtenir ce travail était de supprimer tous les clickable et focusable attributs dans le document XML 🙂 Contre-intuitif, mais cela a fonctionné.

Dans mon cas, j'ai eu un RelativeLayout en tant que parent dans mon affichage personnalisé et la seule façon de faire ce travail était de mettre en focusable et clickable à true dans le RelativeLayout et dans le constructeur de la vue personnalisée, après avoir gonflé la mise en page, ajoutez ceci:

View view = inflate(getContext(), R.layout.layout_my_custom_view, this);
view.findViewById(R.id.theparent).setOnClickListener(new OnClickListener() {
@Override
public void onClick(View view) {
performClick();
}
});