¿Porque no cambia el contenido de la variable?

[redtitle]

Buenas, no consigo comprender porque tras ejecutarse el método onCheckedChanged() no me cambia el valor de la variable... Gracias un saludo.

Código Java:

package juan.pruebas.smssender;
 
import android.app.Activity;
import android.app.PendingIntent;
import android.content.Intent;
import android.os.Bundle;
import android.telephony.SmsManager;
import android.view.View;
import android.widget.Button;
import android.widget.CheckBox;
import android.widget.CompoundButton;
import android.widget.EditText;
import android.widget.Toast;
 
public class GNOCAccesSmsSenderActivity extends Activity {
    /** Called when the activity is first created. */
    
    String telefonoGnoc ="605097654";
    public String entradaSalida = "";
    
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        
        final CheckBox checkBox = (CheckBox)findViewById(R.id.checkBoxEntradaSalida);
        
        EditText numCaso = (EditText)findViewById(R.id.numCaso);
        EditText emplazamiento = (EditText)findViewById(R.id.emplazamiento);
        EditText nomTec = (EditText)findViewById(R.id.nomTec);
        EditText empresa = (EditText)findViewById(R.id.empresa);
        EditText trabRealizar = (EditText)findViewById(R.id.trabRealizar);
        
        String var_numCaso = numCaso.getText().toString();
        String var_emplazamiento = emplazamiento.getText().toString();
        String var_nomTec = nomTec.getText().toString();
        String var_empresa = empresa.getText().toString();
        String var_trabRealizar = trabRealizar.getText().toString();
                
        Button botonEnvioSms = (Button)findViewById(R.id.botonEnvioSms);
        
        if (checkBox.isChecked()){
            checkBox.setChecked(false);
        }
        
        checkBox.setOnCheckedChangeListener(new android.widget.CompoundButton.OnCheckedChangeListener() {
            
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                // TODO Auto-generated method stub
                if (checkBox.isChecked()){
                    checkBox.setText("Entrada");
                    entradaSalida = "INI.";
                } else {
                    checkBox.setText("Salida");
                    entradaSalida = "FIN.";
                }
            }
        }); 
        
 
        final String sms = entradaSalida + "." + var_numCaso + "." + var_emplazamiento
                + "." + var_nomTec + "." + var_empresa + ".LOC:IN-" + var_trabRealizar;
        
        final SmsManager enviarSms = SmsManager.getDefault();
        String SENT = "SMS_SENT";
        String DELIVERED = "SMS_DELIVERED";
        final PendingIntent sentPI = PendingIntent.getBroadcast(this, 0, new Intent(SENT), 0);
        final PendingIntent deliveredPI = PendingIntent.getBroadcast(this, 0, new Intent(DELIVERED), 0);
        
        botonEnvioSms.setOnClickListener(new View.OnClickListener() {
            
            public void onClick(View v) {
                // TODO Auto-generated method stub
                
                
                //enviarSms.sendTextMessage(telefonoGnoc, null, sms, sentPI, deliveredPI);
                
                Toast.makeText(getApplicationContext(), sms, Toast.LENGTH_LONG).show();
            }
        });
    }
}
 

[Ronruby]

Te refieres a la variable entradaSalida o a la variable sms?

Si es la variable sms, primero que nada es final y aparte de todo siempre asignas el valor que trae por defecto. Deberia de reconstruirse la cadena al presionar el boton o lo que sea.

[redtitle]

Gracias por responder, sabía que iban por ahí los tiros, al introducir la construcción de la cadena dentro del botón si me funciona, pero me obliga a reconstruir toda la aplicación y a meterle todos los elementos gráficos en los botones...

Tendré que replantearlo de nuevo.

Un saludo.

[redtitle]

Ok, ya está el código listo.

Gracias, un saludo!!

Código Java:

package juan.pruebas.accesognoc;
 
import android.app.Activity;
import android.app.PendingIntent;
import android.content.Intent;
import android.os.Bundle;
import android.telephony.SmsManager;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;
import android.widget.Toast;
 
public class AccesoGnocActivity extends Activity {
    /** Called when the activity is first created. */
    
    final String telefonoGnoc ="605097654";
    
    EditText txnumCaso, txemplazamiento, txnomTec, txempresa, txtrabRealizar;
    String var_numCaso, var_emplazamiento, var_nomTec, var_empresa, 
        var_trabRealizar, entradaSalida, SENT, DELIVERED;
    String sms = "";
    SmsManager enviarSms;
    PendingIntent sentPI;
    PendingIntent deliveredPI;
    Button botonEntrada, botonSalida;
       
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        
        txnumCaso = (EditText)findViewById(R.id.numCaso);
        txemplazamiento = (EditText)findViewById(R.id.emplazamiento);
        txnomTec = (EditText)findViewById(R.id.nomTec);
        txempresa = (EditText)findViewById(R.id.empresa);
        txtrabRealizar = (EditText)findViewById(R.id.trabRealizar);
                       
        botonEntrada = (Button)findViewById(R.id.botonEntrada);
        botonSalida = (Button)findViewById(R.id.botonSalida);
                
        enviarSms = SmsManager.getDefault();
        SENT = "SMS_SENT";
        DELIVERED = "SMS_DELIVERED";
        sentPI = PendingIntent.getBroadcast(this, 0, new Intent(SENT), 0);
        deliveredPI = PendingIntent.getBroadcast(this, 0, new Intent(DELIVERED), 0);
        
        botonEntrada.setOnClickListener(new View.OnClickListener() {
            
            public void onClick(View v) {
                // TODO Auto-generated method stub
 
                var_numCaso = txnumCaso.getText().toString();
                var_emplazamiento = txemplazamiento.getText().toString();
                var_nomTec = txnomTec.getText().toString();
                var_empresa = txempresa.getText().toString();
                var_trabRealizar = txtrabRealizar.getText().toString();
                
                entradaSalida = "INI.";
                
                sms = entradaSalida + "." + var_numCaso + "." + var_emplazamiento
                        + "." + var_nomTec + "." + var_empresa + ".LOC:IN-" + var_trabRealizar;
                
                //enviarSms.sendTextMessage(telefonoGnoc, null, sms, sentPI, deliveredPI);
                
                Toast.makeText(getApplicationContext(), sms, Toast.LENGTH_LONG).show();
            }
        });
        
        botonSalida.setOnClickListener(new View.OnClickListener() {
            
            public void onClick(View v) {
                // TODO Auto-generated method stub
 
                var_numCaso = txnumCaso.getText().toString();
                var_emplazamiento = txemplazamiento.getText().toString();
                var_nomTec = txnomTec.getText().toString();
                var_empresa = txempresa.getText().toString();
                var_trabRealizar = txtrabRealizar.getText().toString();
                
                entradaSalida = "FIN.";
                
                sms = entradaSalida + "." + var_numCaso + "." + var_emplazamiento
                        + "." + var_nomTec + "." + var_empresa + ".LOC:IN-" + var_trabRealizar + ".";
                
                //enviarSms.sendTextMessage(telefonoGnoc, null, sms, sentPI, deliveredPI);
                
                Toast.makeText(getApplicationContext(), sms, Toast.LENGTH_LONG).show();
            }
        });
    }
}
 

Supongo que debe haber alguna forma de no tener que duplicar el código en cada botón, pero para empezar está bien :)