[W1174503]

Eclipse allows me to fix the undefined constructor (line 4 of TestToString.java) by removing the arguments but that is not what I am trying to do. I am attempting to get the program to execute and say (“Id is 1 The name is Fluffy). Can anyone please help?

public class TestToString {

	public static void main(String[] args) {
		ToString obj1 = new ToString(1, "Fluffy");
		System.out.println(obj1.toString(obj1));

	}

}

public class ToString {
	private int iD;
	private String name;
	
	public void toString(int iDArg, String nameArg){
		iD = iDArg;
		name = nameArg;
	}

	public String toString(ToString Obj1) {
		String ToString = "Id is " + iD + "The name is " + name; 
		return ToString;
	}
	
}

[Ryano121]

Java is case sensitive - toString != ToString

[Kilorn]

The error you're getting is because your constructor is named "toString" while your class name is "ToString". There is a difference in those two spellings, and therefore it is not recognized as a constructor for the class.

EDIT: Quick fingers there, Ryano121. Well done.

This post has been edited by Kilorn: 15 January 2013 - 02:27 PM

[darek9576]

It is a terrible idea to name your class ToString. First of all, there is a toString() method inside the Object class that everyone extends. Then, if ToString was a good idea, it sounds like a method, whereas classes tend to be nouns.

[jon.kiparsky]

Your toString method shouldn't take an argument.

And for Pete's sake can't you think of a name that isn't "toString"?

This post has been edited by jon.kiparsky: 15 January 2013 - 02:31 PM

[NathanMullenax]

The constructor should have the same name as the class, and the class name is case-sensitive. Also, constructors don't specify a return type. I think you might want something like this:

public class ToString {
	private int iD;
	private String name;
	
	public ToString(int iDArg, String nameArg){
		iD = iDArg;
		name = nameArg;
	}

        // ... not really sure about the second method. 
        // obj1 is never used. Is there a reason for it?
}

[pbl]

and this is wrong

return ToString; // constructor don't return anything

Nobody catch it

*Edit: but NathanMullenax

This post has been edited by pbl: 15 January 2013 - 02:37 PM
Reason for edit:: Yes somebody catched it

[darek9576]

*caught it

[jon.kiparsky]

pbl, on 15 January 2013 - 04:36 PM, said:

and this is wrong
return ToString; // constructor don't return anything
Nobody catch it

To be honest, there's so much wrong in those few lines, it would have been difficult to get it all in.

[farrell2k]

jon.kiparsky, on 15 January 2013 - 09:29 PM, said:

And for Pete's sake can't you think of a name that isn't "toString"?

I vote for TooString

[W1174503]

I understand what I have done wrong and fixed it. Thanks everyone for your help.