[SOLUCIONADO] (aprendiendo) en el videoTuto si funciona

[sonickseven]

Hola que tal comunidad, como están :D , resulta que me he decidido por aprender Android para poner a funcionar una pagina que he creado en los android (que rudo XD ) el problema es que estoy con un vídeo tutorial donde vi una manera muy buena de escuchar un click de todos los botones. Pero el que yo tengo es similar y no me funciona, no hice el del vídeo tuto, solo saco las partes que me interesan y esa fue. Como dije antes tengo ya un sitio web y tengo bastante experiencia con javascript(NODE) y php por lo que ya desarrollar no se me hace tan difícil como cuando comencé a programar. Bueno después de tanta charla les muestro el código para que me muestren el error:

Código Java:

@Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_calculadora);
 
 
        int idList[]={R.id.btnAdd, R.id.btnDiv, R.id.btnMul, R.id.btnSub};
        for (int id:idList){
            View v=(View) findViewById(id);
            v.setOnClickListener(btnClick);
        }
    }
 
private class  ButtonClickListeners implements View.OnClickListener{
        public void onClick(View v){
            result.setText("Hola amigos :D");//intente que me mostrara algo cuando le doy click a cualquier boton
            switch (v.getId()){
                case R.id.btnAdd:
                    Toast.makeText(getApplicationContext(), "Has oprimido algun boton", Toast.LENGTH_SHORT).show();
                    break;
            }
        }
    }
 

como ven ahi se guarda en un array(eso creo) todos los botones y en la función Onclick se captura el evento setOnClickListener como digo hasta ahora estoy comenzando. Puedo echarle la culpa a la versión de android que decidí escoger como mínima para funcionar pero no creo, además que el AndroidStudio no muestra ningún error por alguna parte.

Gracias chicos por la atención prestada

[ipraetoriux]

es que por lo que veo no tenes definido en ningun lugar "btnClick".

Lo que tendrias que hacer es quitar la clase ButtonClickListeners

y poner esto:

Código Java:

private OnClickListener btnClick = new OnClickListener() {
        
    @Override
    public void onClick(View v) {
        // y aca pones todo lo que necesites hacer
            
    }
};
 

[ipraetoriux]

Haber, disculpa que arriba te puse el código directamente sin explicar absolutamente nada...en Java como en la mayoria de los lenguajes de programación tenes varias formas de hacer diferentes cosas logrando mismo resultados.

En este ejemplo puntual, vos podes definir el "listener" para un Button de varias formas. Las mas usadas y frecuentes de encontrar son:

1) Definis el listener en el button y pasas como argumento la interface OnClickListener

Código Java:

public class MainActivity extends Activity{
 
    private Button miBtn;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
        this.miBtn = (Button) this.findViewById(R.id.mi_btn);
 
        this.miBtn.setOnClickListener(new OnClickListener() {
        
        @Override
        public void onClick(View v) {
            
        }
    });
    }
 
}
 

2) Implementas la interface OnClickListener directamente en la la clase. Ahora el argumento de setOnClickListener hace referencia a la clase que implementa la interface OnClickListener, y por otro lado debes implementar en la clase el metodo onClick

Código Java:

public class MainActivity extends Activity implements OnClickListener{
 
    private Button miBtn;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
        this.miBtn = (Button) this.findViewById(R.id.mi_btn);
 
        this.miBtn.setOnClickListener(this);
    }
 
    @Override
    public void onClick(View v) {
 
    }
 
}
 

3) Creas una variable de instancia del tipo OnClickListener que hará referencia a una nueva instancia de OnClickLister. Esta variable de instancia la pasas como argumento en el setOnClickListener.

Código Java:

public class MainActivity extends Activity{
 
    private Button miBtn;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
        this.miBtn = (Button) this.findViewById(R.id.mi_btn);
 
        this.miBtn.setOnClickListener(btnClick);
    }
 
    private OnClickLister btnClick = new OnClickListener() {
        
    @Override
    public void onClick(View v) {
            
    }
    };
 
}
 

[sonickseven]

la variable btnClick si esta definida, se me olvido ponerla, ya te pongo otra ves el codigo para que mires, También digo que el AndroidStudio no muestra ningún error. Ni el emulador tampoco, solo que no hace ninguna acción, Yo hice tal como tu me muestras ahí en tu segunda respuesta y si me funciona. Pero se me hace mas largo teniendo en cuenta que hay que hacer cuatro funciones, debido a que tengo cuatro botones, es una calculadora(suma, resta, divide, multiplica). Por eso esta forma, de guardar lo botones en un array se me hace mas funcional.

Código Java:

EditText num1, num2;
    TextView result;
    private ButtonClickListeners btnClick;//se declara
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_calculadora);
 
 
        int idList[]={R.id.btnAdd, R.id.btnDiv, R.id.btnMul, R.id.btnSub};
        for (int id:idList){
            View v=(View) findViewById(id);
            v.setOnClickListener(btnClick);
        }
    }
    public void operations(String opt){
        num1=(EditText) findViewById(R.id.num1);
        num2=(EditText) findViewById(R.id.num2);
        result=(TextView) findViewById(R.id.result);
        //Toast.makeText(getApplicationContext(), "Vamos a usar la operación "+opt, Toast.LENGTH_SHORT).show();
        result.setText("Hola amigos :D");
    }
 
    //new class ButtonClickListener
    private class  ButtonClickListeners implements View.OnClickListener{
        public void onClick(View v){
            result.setText("Hola amigos :D");
            switch (v.getId()){//captura todos id de  los botones
                case R.id.btnAdd:
                    operations("+");
                    break;
                case R.id.btnSub:
                    operations("-");
                    break;
                case R.id.btnDiv:
                    operations("/");
                    break;
                case R.id.btnMul:
                    operations("*");
                    break;
            }
        }
    }
 

bueno ese es el codigo completo. Como ves ahi cuatro botones y todos se guardan en un array.

olle amigo otra cosa que no es tan complicada, como puedo hacer algo como console.log de javascript aca en java, ya que ahí una forma pero no se como usarla, siempre me sale error, es Log.v() pongo los parámetros pero sigue con el error, y con System no se bn. Pues estoy buscando todabia en google :D

[ipraetoriux]

...haber...vamos por parte...

Tu error aca es el siguiente...

...linea 14: v.setOnClickListener(btnClick); ..mal, estas pasando una variable de instancia, pero que no hace referencia a ningun objeto...nunca has hecho una instancia de ButtonClickListeners..

...en la linea 14 deberia ir: v.setOnClickListener(new ButtonClickListeners()); y la variable de instancia btnClick la eliminas...

[ipraetoriux]

...despues...

esto de acá...es terrible, no es View sino Button..y después creas un 4 variables con el mismo nombre y diferente valor...mi hijo de 6 años que ya esta programando en C, sabe que sucede si a 4 variables les doy el mismo nombre con diferente valor...imagino que vos también ya que según vos tenes bastante experiencia con javascript...esto es fatal, de novato programador

Código Java:

for (int id:idList){
   View v=(View) findViewById(id);
   v.setOnClickListener(btnClick);
}
 

[ipraetoriux]

...despues linea 28...mal...mismo problema que el punto anterior...VARIABLES...básico...ninguna ciencia...como podes utilizar el método setText() en la variable result, sino todavia no la has inicializado...???...si lo haces en el metodo "operations"...pero ante el evento click y siguiendo el flujo del programa, estas empleando el metodo setText() en la variable result antes de invocar a "operations", y por ende result no esta inicializada...mal, por ellooo...se debe inicializar las variables en oncreate y onresume, o inicializas cada variable de instancia cuando le definis.

Código Java:

result.setText("Hola amigos :D");
 

Conclusion: Te recomendaria que leas un poco mas de variables de instancia, variables de clase, variables locales, clases, clases anonimas, metodos, listeners y el lifecycle de los activity...

Un saludo

[sonickseven]

ja ja aj ya con calma, olle tranquilo, te digo, lo veo en un vídeo tutorial en el cual si funciono, y si llevo bastante tiempo desarrollando con javascript. Tanto que vi en el vídeo tuto algo que yo hago comúnmente en javascript y es asignar un solo onCLick para varios botones, lo cual es lo que intento hacer.


aca yo lo hice con javascript

y aca esta el video How to create a Calculator App for Android - Peamon.com en minuto 12 mas o menos es donde el empieza a crear el array de los botones.

Pero ya viendo los comentarios, a algunos no les funciono, pero muestran la solucion solo que no la he logrado decifrar

"you need to instantiate your btnClick before you set the btnClick as a listener to v.
Ex. btnClick = new ButtonClickListener();

also I found that it works better if we make the ButtonClickListener public instead of private... it worked for me"

[sonickseven]

je ej ej ya despues de tanto cuadrar donde poner lo que el chico decia sobre el error lo encontre :D

era solo esto amigo, si ves que no soy tan malo!!!!

Código Java:

for (int id:idList){
       View v=(View) findViewById(id);
      
       btnClick=new ButtonClickListeners(); //esto era lo unico que faltaba :D
       v.setOnClickListener(btnClick);
    }
 

[ipraetoriux]

Iniciado por sonickseven

je ej ej ya despues de tanto cuadrar donde poner lo que el chico decia sobre el error lo encontre :D

era solo esto amigo, si ves que no soy tan malo!!!!

Código Java:

for (int id:idList){
       View v=(View) findViewById(id);
      
       btnClick=new ButtonClickListeners(); //esto era lo unico que faltaba :D
       v.setOnClickListener(btnClick);
    }
 

Lo encontré dice....jajajajajaja

Yo te indique este error acá

....haber...vamos por parte...

Tu error aca es el siguiente...

..linea 14: v.setOnClickListener(btnClick); ..mal, estas pasando una variable de instancia, pero que no hace referencia a ningun objeto...nunca has hecho una instancia de ButtonClickListeners..

...en la linea 14 deberia ir: v.setOnClickListener(new ButtonClickListeners()); y la variable de instancia btnClick la eliminas...