0

Sé que hay muchas preguntas sobre comprobar si CheckBox está seleccionado o si TextView está vacío, pero intento hacer una comprobación de ambos juntos y no la realiza, lo que he hecho es lo siguiente.

btn1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                // compruebo si opc1 u opc2 está seleccionado y si openCalendario no está vacío
                if (opc1.isChecked() || opc2.isChecked() || verificarCampoFecha()) {
                    // solamente ir si opc1 u opc2 está seleccionado y openCalendario no está vacío
                    Intent c4c5a = new Intent(Clase4.this, Clase5.class);
                    startActivity(c4c5a);
              } else {
                    // opc1 opc2, si alguno de los dos no está seleccionado, mostrar:
                    Toast.makeText(Clase4.this, "Por favor, seleccione una franja horaria", Toast.LENGTH_SHORT).show();
                    // verificarCampoFecha, si openCalendario está vacío mostrar:
                if (openCalendario.getText().toString().equals("")) {
                     Toast.makeText(Clase4.this, "Por favor, seleccione un día", Toast.LENGTH_SHORT).show();
               }
            }
       }
   });

// compruebo openCalendario

  private boolean verificarCampoFecha() {
        if (openCalendario.getText().toString().equals("")) {
            return false;
        }
        return true;
    }

El problema es que si TextView está vacío y CheckBox sin seleccionar, solamente muestra el Toast de CheckBox y si el CheckBox está seleccionado, pasa a la siguiente Activity sin tener en cuenta si TextView está vacío o no.

1 respuesta 1

2

Buenos días campañera @UserNameYo, según veo por el comentario que esta en el código requieres hacer:

 // compruebo si opc1 u opc2 está seleccionado y si openCalendario no está vacío

con este codigo estas preguntando esto:

// compruebo si opc1 u opc2 está seleccionado u openCalendario no está 
    if (opc1.isChecked() || opc2.isChecked() || verificarCampoFecha())

para realizar lo que esta puesto en el comentario combiene poner un "and" "&&" el codigo te quedaria algo asi:

if ((opc1.isChecked() || opc2.isChecked()) && (verificarCampoFecha()))

espero haberte ayudado.

Agregado a respuesta.

Referente al toast de la seccion de codigo.

// opc1 opc2, si alguno de los dos no está seleccionado, mostrar:
  Toast.makeText(Clase4.this, "Por favor, seleccione una franja horaria", Toast.LENGTH_SHORT).show();

aquí tendríamos que meter el mensaje si se cumple esa condicion algo asi como:

if (!opc1.isChecked() && !opc2.isChecked()){
// opc1 opc2, si alguno de los dos no está seleccionado, mostrar:
  Toast.makeText(Clase4.this, "Por favor, seleccione una franja horaria", Toast.LENGTH_SHORT).show();
}
4
  • me alegra que te haya servido en parte, eso ocurre si los 2 CheckBox están desmarcado y el TextView esta relleno ? Commented el 3 jun. 2017 a las 14:52
  • Ahh creo que entiendo, solo falta agregar la verificación antes de emitir el Toast, lo agrego a la respuesta para que se vea un poco mejor Commented el 3 jun. 2017 a las 15:02
  • ah ya si perdon puse or enves de and en la condiccion ahora lo edito Commented el 3 jun. 2017 a las 15:33
  • Perfecto! Ahora sí, gracias^^
    – UserNameYo
    Commented el 3 jun. 2017 a las 15:47

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.