Why Java cannot find my constructor?

Question

Well, maybe it is a stupid question, but I cannot resolve this problem.

In my ServiceBrowser class I have this line:

ServiceResolver serviceResolver = new ServiceResolver(ifIndex, serviceName, regType, domain);

And compiler complains about it. It says:

cannot find symbol
symbol : constructor ServiceResolver(int,java.lang.String,java.lang.String,java.lang.String)

This is strange, because I do have a constructor in the ServiceResolver:

public void ServiceResolver(int ifIndex, String serviceName, String regType, String domain) {
        this.ifIndex = ifIndex;
        this.serviceName = serviceName;
        this.regType = regType;
        this.domain = domain;
    }

ADDED: I removed void from the constructor and it works! Why?

@Roman did you just answer your own question with a different account? — Bozho
– Bozho, Commented Mar 16, 2010 at 18:54

Roman · Accepted Answer · 2010-03-16 14:33:47Z

9

delete void from signature

public ServiceResolver(int ifIndex, String serviceName, String regType, String domain) {
        this.ifIndex = ifIndex;
        this.serviceName = serviceName;
        this.regType = regType;
        this.domain = domain;
    }

answered Mar 16, 2010 at 14:33

Roman

66.5k93 gold badges247 silver badges342 bronze badges

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

Comments

Bozho · Accepted Answer · 2010-03-16 14:34:44Z

5

You have defined a method, not a constructor.

Remove the void

answered Mar 16, 2010 at 14:34

Bozho

599k147 gold badges1.1k silver badges1.2k bronze badges

1 Comment

Roman Over a year ago

Bonho, another Roman is another person. I do not answer my question from another account.

Timothy · Accepted Answer · 2010-03-16 14:35:10Z

2

That's no constructor... it's a simple method that returns nothing. Absolutely nothing!

Should be this:

public ServiceResolver(int ifIndex, String serviceName, String regType, String domain) {
        this.ifIndex = ifIndex;
        this.serviceName = serviceName;
        this.regType = regType;
        this.domain = domain;
    }

answered Mar 16, 2010 at 14:35

Timothy

2,47719 silver badges15 bronze badges

Comments

Kenned · Accepted Answer · 2010-03-16 14:35:45Z

0

Java constructors don't have return types on their signature - they implicitly return an instance of the class.

answered Mar 16, 2010 at 14:35

Kenned

5784 silver badges11 bronze badges

Comments

TofuBeer · Accepted Answer · 2010-03-16 14:41:16Z

Welcome to the mistake everyone makes once. As Roman points out you have to delete "void" from infront of the the constructor.

Constructors declare no return type - which may seem odd since you do things like x = new X(); but you can consider it like this:

// what you write...
public class X
{
    public X(int a)
    {
    }
}

x = new X(7);

// what the compiler does - well sort of... good enough for our purposes.
public class X
{
    // special name that the compiler creates for the constructor
    public void <init>(int a)
    {
    }
}

// this next line just allocates the memory
x = new X(); 

// this line is the constructor
x.<init>(7);

A good set of tools to run to find this sort of mistake (and many others) are:

That way when you make other common mistakes (you will, we all do :-), you won't spin your wheels so much looking for the solution.

Collectives™ on Stack Overflow

Why Java cannot find my constructor?

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related