Onclick Avtivity funktioniert nicht

[Lilithian]

Wenn du den Code so wie er da oben steht benutzt, fehlt das:

startActivity(intent);

Sonst legst du den Intent nur an.

 

[rufux]

hast natürlich recht, danke

jetzt habe ich das ganze zusammengebastelt.

Aber zurückgehen tut er immer noch nicht. naja mal schauen

 

[the_alien]

Ruf einfach finish() auf.

 

[rufux]

das habe ich gerade mal getestet.

funktioniert...

Nur jetzt sendet er wieder nicht eine sms sondern hunderte...

habe in meinem Manifest bei der sms_sendingactivity
android:launchMode="singleInstance" hinzugefügt, womit ich vorher nur eine geschickt hatte.

Was kann ich machen, damit er nur einmal eine SMS sendet und nicht 100mal?

 

[the_alien]

Es wird kaum an der Activity liegen, sonst würden ja auch davon 100 erstellt. Dein Code zum SMS versenden wird falsch sein.

 

[rufux]

ich irritiert nur, dass es ja funktioniert hat

aber hier meine sendSMS Funktion:

PendingIntent pi =PendingIntent.getActivity(this,0,new Intent(this,sms_sending.class),0);
SmsManager sms = SmsManager.getDeffault();
sms.sendTextMessage(number,null,message,pi,null);

vg

 

[the_alien]

Ist das ein geheimes Projekt? Hau mal mehr Quelltext raus, darum hilft auch kaum jemand, es gibt zu wenig Infos.

 

[rufux]

ne nix ist geheim...

will nur das froum nicht unendlich vollspamen

aber dann kommt jetzt code:

meine main ohne imports

package com.test.main;
public class main extends Activity implements View.OnClickListener{
 
 public static final String LOG_TAG = "debug";
 Button sendButton;
 Button recieveButton;
 Button chk1;
 Button chk2;
 Button chk3;
 Button chk4;
 Button chk5;
 Button chk6;
 Button chk7;
 Button chk8;
 Button chk9;
 Button chk10;
 public int[] checked = new int[10];
 int i = 0;
 
 private Intent intentService;
 
 
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        TextView tv = new TextView(this);
        tv.setText("main");
        setContentView(R.layout.main);
        sendButton = (Button) findViewById(R.id.sms_sending_button);
        recieveButton = (Button) findViewById(R.id.sms_recieving_button);
        chk1 = (Button) findViewById(R.id.chk1);
        chk2 = (Button) findViewById(R.id.chk2);
        sendButton.setOnClickListener(this);
        recieveButton.setOnClickListener(this);
        chk1.setOnClickListener(this);
        chk2.setOnClickListener(this);
}
 public void onClick(View v) {
  if(v == sendButton){
  Intent intent = new Intent(this,sms_sending.class);
  intent.putExtra("com.test.intArray",checked);
 
  startActivity(intent);   
}
  if(v == recieveButton){
  Intent intent = new Intent(this,sms_recieving.class);
    }
 
  /*Map<String, String>map=new HashMap<String,String>();
  for(int x=1;x<=16;x++){
  map.put("chk" +i, new String());
   if (v==map.get("chk"+i)){
   checked[x-1]=x;
 
  }*/
  }
 }
}

meine sms_sending ohne imports

package com.test.smssending;
public class sms_sending extends Activity  {
     private Intent intentService;
 
    String number = "5556" ;
    StringBuilder msg = new StringBuilder(160);
    int i = 0;
    int[] temp = new int[2];
    public static final String LOG_TAG = "debug";
 
    public void onCreate(Bundle savedInstanceState)  {
         super.onCreate(savedInstanceState);
         msg.append("query:");
         TextView tv = new TextView(this);
         tv.setText("send");
         setContentView(R.layout.sending);
         Intent i = this.getIntent();
         temp = i.getIntArrayExtra("com.test.intArray");
         if(temp[0]==1){
         msg.append("1");
         }
         if(temp[1]==1){
         msg.append("2");
         }
 
         if(msg.toString()!="query:"){
 
         sendSMS(number,msg.toString());}
 
 
 /*     intent.setClass(this, main.class);
      finish();*/
      }
 
 
   private void sendSMS(String number, String message){
 
 PendingIntent pi = PendingIntent.getActivity(this, 0, new Intent(this,sms_sending.class), 0);   
    SmsManager sms = SmsManager.getDefault();
    sms.sendTextMessage(number, null, message, pi, null);
   }
public IBinder onBind(Intent intent) {
 // TODO Auto-generated method stub
 
 return null;
}
}

Baue gerade an den variablen Variablen und an dem zurückgehen auf die main (wobei aber nur eine SMS gesendet wird)


Also und meine Idee dahinter...

Sms wird an eine GSM Box versendent und schickt dieser PAramter, welche man vorher per Buttons/CHeckboxen übergibt.

So ... nix geheim

 

[Lilithian]

Was mir nur so grad auffällt:

Bei Strings musst du zum vergleichen .equals anstatt == oder != benutzen, deine Abfrage müsste sonst glaub ich immer true sein.

if (!msg.toString().equals("query:")) {
                sendSMS(number,msg.toString()); }