create constructor string string1,string string2,string string3,Object object1

Question

i want to create a class in my MainActivity:

MyCard customCard1 = new MyCard(cardTitle.getText().toString(),
                cardbuttonon.getText().toString(),
        cardbuttonon.getText().toString(),
        cardbuttonon.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                String[] url={"url"};
                new onoff(MainActivity.this).execute(url);
            }});
        mCardView.addCard(customCard1);
        mCardView.refresh();

but he say me that the constructor MyCard(String,String,String,Object undefined is.

How can i define him on my MyCard:

package de.heron.cloudbox;

import android.content.Context;
import android.view.LayoutInflater;
import android.view.View;
import android.widget.TextView;

import com.fima.cardsui.objects.Card;

public class MyCard extends Card {

    public MyCard(String title, String desc, String string, Object setOnClickListener){
        super(title, desc); 
    }
    @Override
    public View getCardContent(Context context) {
        View view = LayoutInflater.from(context).inflate(R.layout.card_ex, null);

        ((TextView) view.findViewById(R.id.title)).setText(title);
        ((TextView) view.findViewById(R.id.description)).setText(desc);

        return view;
    }
}

can someone help me by my problem?

why can i make it with 2 onclicklisteners:

MyCardXml customCard1 = new MyCardXml(cardTitle.getText().toString(),cardbuttonon = (Button) findViewById(R.id.cardbuttonon),cardbuttonoff = (Button) findViewById(R.id.cardbuttonoff),
                cardbuttonon.getText().toString(),cardbuttonoff.getText().toString(),
        cardbuttonon.getText().toString(),cardbuttonoff.getText().toString(),
        cardbuttonon.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {

                String[] url={"http://everhome.de/api/applive/", "8ef43502ad3dc04f87b4a48b993878c0","/" + KEY_ID,"/" + KEY_COST};
                new onoff(MainActivity.this).execute(url);
                }}

                ),cardbuttonoff.setOnClickListener(new View.OnClickListener() {
                public void onClick(View v) {
                    String[] url={"http://everhome.de/api/applive/", "8ef43502ad3dc04f87b4a48b993878c0","/" + KEY_ID,"/" + KEY_COST};
                    new onoff(MainActivity.this).execute(url);
                }}

                ));
        mCardView.addCard(customCard1);
        mCardView.refresh();

I'm not sure what you're doing, but MyCard's constructor isn't even using half its arguments, your variable names are bizarre, and your indentation is screwed up. Why is MyCard's constructor taking a parameter called setOnClickListener? That name doesn't sound like it describes an object; that sounds like it describes an action. — user2357112
– user2357112, Commented Jul 5, 2013 at 9:14
Can you please share exact exception. Beacuse MyCard customCard1 = new MyCard(..) is not completely closed. — Ashish Aggarwal
– Ashish Aggarwal, Commented Jul 5, 2013 at 9:15
To elaborate on Ashish's comment: you're missing a closing parenthesis on the MyCard constructor call. — user2357112
– user2357112, Commented Jul 5, 2013 at 9:16

oks16 · Accepted Answer · 2013-07-05 09:18:02Z

1

Your constructor needs an Object as the fourth parameter. You are not passing an object in the fourth parameter when you are constructing the object.This will solve the error

MyCard customCard1 = new MyCard(cardTitle.getText().toString(),
            cardbuttonon.getText().toString(),
    cardbuttonon.getText().toString(),
    new View.OnClickListener() {
        public void onClick(View v) {
            String[] url={"url"};
            new onoff(MainActivity.this).execute(url);
        }});

answered Jul 5, 2013 at 9:18

oks16

1,27611 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Fabian Over a year ago

but whats when i have another button with a onclicklistener

Blackbelt · Accepted Answer · 2013-07-05 09:12:17Z

1

you should pass View.OnClickListener to your MyCard constructor instead of Object

Change

   public MyCard(String title, String desc, String string, Object setOnClickListener){
        super(title, desc); 
    }

with

   public MyCard(String title, String desc, String string,  View.OnClickListener setOnClickListener){

answered Jul 5, 2013 at 9:12

Blackbelt

158k31 gold badges308 silver badges310 bronze badges

Comments

R9J · Accepted Answer · 2013-07-05 09:14:55Z

0

You have to change your constructor like this

public MyCard(String title, String desc, String string, OnClickListener clickListener){

}

answered Jul 5, 2013 at 9:14

R9J

6,7254 gold badges21 silver badges25 bronze badges

Collectives™ on Stack Overflow

create constructor string string1,string string2,string string3,Object object1

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related