Converting Linked List toString into Generic

I am writing a Linked List class that takes in names or numbers, and then prints them out in a list. I managed to write the list normally. Here is what I did:

public String toString(){
    return list.toString; //where list is the LinkedList I am calling
}

That works correctly and returns my list after adding 4 elements like this:

[Joe, Jessica, Max, 5]

Now I am trying to convert that same method onto a generic method, so I did 2 things.

Here I created the collections object:

private Collection<E> collection;

public MyLinkedListG(Collection<E> _collection) {
    collection= _collection;
}

And here is how I wrote the new toString in collections:

 public String toString(){
    StringBuilder builder = new StringBuilder();
    for(E e : collection) {
        builder.append(e); //appends each string
    }
    return builder.toString();
}

The problem is that now my test class will not allow me to call the LinkedList object I had created before which was:

 MyLinkedListG x = new MyLinkedListG();

It states I need to input a collection inside the parameter. How can I call it now? Or am I doing it totally wrong?

If something is not clear please let me know so I can clarify as soon as possible. Thanks in advanced.

Solution

From what I can tell, your original class likely did not include a constructor. This means the no-arguments constructor new MyLinkedListG() is provided by default, which is likely what you used to construct an instance of your class.

After your modifications, you added a constructor MyLinkedListG(Collection<E> _collection). Now the no-arguments constructor is not provided by default anymore. If you want to continue to use it, it must be explicitly defined.

Your class will probably have two (or more) constructors in this case, perhaps something like this:

private Collection<E> collection;

public MyLinkedListG(Collection<E> _collection) {
    collection= _collection;
}

public MyLinkedListG() {
    collection=new LinkedList<E>();
}

Now you can use either constructor for your object.