The method onClick(View, String) from the type new View.OnClickListener(){} is never used locally

Question

These codes below are used to encrypt SMS using PRINCE algorithm and sends the SMS to the receiver. Currently I am facing a problem where public void onClick(View v,String args) gives an error message of "The method onClick(View, String) from the type new View.OnClickListener(){} is never used locally". I have also referred to The method onClick(View) from the type new Thread(){} is never used locally but the solutions given does not help me. Is there any other solution for this problem?

    btnSendSMS.setOnClickListener(new View.OnClickListener()
    {
    @Override
    public void onClick(View v) {
        String phoneNo = txtPhoneNo.getText().toString();
         String message = txtMessage.getText().toString();
         if (phoneNo.length()>0 && message.length()>0)
         {
              LongBuffer messageBuf = TooLong.messageToLongBuffer(message);
              messageBuf.flip();
              long[] messageData = new long[messageBuf.remaining()];
              LongBuffer i = messageBuf.get(messageData);
              String v1=prince.Encrypt(i, k0, kop, k1, t);
              sendSMS(phoneNo, v1);
          }
          else
               Toast.makeText(getBaseContext(),
               "Please enter both phone number and message.",
               Toast.LENGTH_SHORT).show();
    }
    });        
    }

//---sends a SMS message to another device---

   private void sendSMS(String phoneNumber, String v1)
   {      
         /*
         PendingIntent pi = PendingIntent.getActivity(this, 0,
                    new Intent(this, test.class), 0);                
         SmsManager sms = SmsManager.getDefault();
         sms.sendTextMessage(phoneNumber, null, message, pi, null);        
         */

         String SENT = "SMS_SENT";
         String DELIVERED = "SMS_DELIVERED";

         PendingIntent sentPI = PendingIntent.getBroadcast(this, 0,
                new Intent(SENT), 0);

         PendingIntent deliveredPI = PendingIntent.getBroadcast(this, 0,
                new Intent(DELIVERED), 0);


//---when the SMS has been sent---
      registerReceiver(new BroadcastReceiver(){
      @Override
         public void onReceive(Context arg0, Intent arg1) {
         switch (getResultCode())
         {
            case Activity.RESULT_OK:
             Toast.makeText(getBaseContext(), "SMS sent", 
               Toast.LENGTH_SHORT).show();
             break;
            case SmsManager.RESULT_ERROR_GENERIC_FAILURE:
             Toast.makeText(getBaseContext(), "Generic failure", 
               Toast.LENGTH_SHORT).show();
             break;
            case SmsManager.RESULT_ERROR_NO_SERVICE:
             Toast.makeText(getBaseContext(), "No service", 
               Toast.LENGTH_SHORT).show();
             break;
            case SmsManager.RESULT_ERROR_NULL_PDU:
             Toast.makeText(getBaseContext(), "Null PDU", 
               Toast.LENGTH_SHORT).show();
             break;
            case SmsManager.RESULT_ERROR_RADIO_OFF:
             Toast.makeText(getBaseContext(), "Radio off", 
               Toast.LENGTH_SHORT).show();
             break;
         }
      }
     }, new IntentFilter(SENT));

//---when the SMS has been delivered---
     registerReceiver(new BroadcastReceiver(){
     @Override
        public void onReceive(Context arg0, Intent arg1) {
        switch (getResultCode())
        {
            case Activity.RESULT_OK:
             Toast.makeText(getBaseContext(), "SMS delivered", 
               Toast.LENGTH_SHORT).show();
             break;
            case Activity.RESULT_CANCELED:
             Toast.makeText(getBaseContext(), "SMS not delivered", 
               Toast.LENGTH_SHORT).show();
             break;         
        }
       }
            }, new IntentFilter(DELIVERED));        

           SmsManager sms = SmsManager.getDefault();
           sms.sendTextMessage(phoneNumber, null, v1, sentPI, deliveredPI);               
        }    
       }

Answer 1

          btnSendSMS.setOnClickListener(new View.OnClickListener()
            {


    // why have you written this method it not called from anywhere remove this 
       method put  code in public void onClick(View v) method
                 public void onClick(View v,String args) 
                 {

                 }
            @Override
            public void onClick(View v) {
                String phoneNo = txtPhoneNo.getText().toString();
                 String message = txtMessage.getText().toString();
                 if (phoneNo.length()>0 && message.length()>0)
                 {
                      LongBuffer messageBuf = TooLong.messageToLongBuffer(message);
                      messageBuf.flip();
                      long[] messageData = new long[messageBuf.remaining()];
                      LongBuffer i = messageBuf.get(messageData);
                      String v1=prince.Encrypt(i, k0, kop, k1, t);
                      sendSMS(phoneNo, v1);
                  }
                  else
                       Toast.makeText(getBaseContext(),
                       "Please enter both phone number and message.",
                       Toast.LENGTH_SHORT).show();
            }
            });        
            }


if you require this method (onClick(View v,String args) )then comment.

Answer 2

The signature for an onClick() method is onClick(View), not onClick(View,String). The latter declares a new method that is never called and does not override the method in OnClickListener interface.

Remove the String argument from your method, and remove the other stub onClick(View).

Answer 3

public void `onClick(View v,String args)`

This method is never used locally because this method is an method written by you. OnButtonclick your method never get called. On click the onClick(View) method get invoked. Write your code inside onClick(View)