Problème de visibilité avec la case à cocher

Question

J'utilise une case à cocher dans mon code qui, lorsqu'elle est cochée, rend un textview et un editText visibles, mais si je décoche une case à cocher, ils restent visibles au lieu de disparaître.

Voici le code:

final CheckBox save = (CheckBox) findViewById(R.id.checkbox); 
     save.setOnClickListener(new OnClickListener() { 
      public void onClick(View v) { 



       // Perform action on clicks, depending on whether it's now checked 
       if (((CheckBox) v).isChecked()) { 

        nameText.setVisibility(1); 
        editName.setVisibility(1); 

       } else { 

        nameText.setVisibility(0); 
        editName.setVisibility(0); 

       } 
      } 
     }); 

et une partie du xml qui se trouve dans une disposition relative:

<LinearLayout android:id="@+id/linearLayout3" 
      android:orientation="horizontal" 
      android:layout_width="fill_parent" 
      android:layout_height="wrap_content" 
      android:layout_below = "@+id/linearLayout2"> 

    <TextView android:id="@+id/text" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:text="@string/name" 
     android:visibility="invisible"/> 
    <EditText android:id="@+id/name" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_weight="1" 
     android:visibility="invisible"/>  


    <CheckBox android:id="@+id/checkbox" 
      android:layout_width="wrap_content" 
      android:layout_height="wrap_content" 
      android:text="@string/save" /> 
</LinearLayout> 

Que dois-je faire pour rendre le dissapear textView et EditText quand je décocher la case à cocher?

Merci!

Answer 1

utilisation View.VISIBLE, View.INVISIBLE, View.GONE pour contrôler la visibilité (au lieu de 0 & 1).

Answer 2

Deux choses:

1. Vous devez utiliser setOnCheckedChangeListener(), ce qui rendra la vie plus facile. Vous devez utiliser View.GONE et View.VISIBLE au lieu des entiers pour setVisibility().

Answer 3

le problème est dans cette partie:

// Perform action on clicks, depending on whether it's now checked 
       if (((CheckBox) v).isChecked()) { 

        nameText.setVisibility(1); 
        editName.setVisibility(1); 

       } else { 

        nameText.setVisibility(0); 
        editName.setVisibility(0); 

       } 

yout ne doivent pas utiliser des valeurs entières, mais au lieu d'utiliser les constantes fournies par la vue classe

// Exécuter l'action sur les clics, selon que l'on est maintenant cochée if (((CheckBox) v) .isChecked()) {

   nameText.setVisibility(View.VISIBLE); 
       editName.setVisibility(View.VISIBLE); 

      } else { 

       nameText.setVisibility(VIEW.INVISIBLE); 
       editName.setVisibility(VIEW.INVISIBLE); 

      } 

au lieu d'invisible, vous pouvez également utiliser GONE. alors votre TextEdit invisible n'a pas besoin d'espace dans la mise en page

si vous cochez la API vous verrez que 1 n'est pas un paramètre valide: