Thread: accept only assigned label dnd

What method? More needs to be done than change a method. Without a design coding is a waste of time.i

design is sorted

What is the design for how the program should work? Decide on what the program should do before writing code.

i would it to do as you suggested in post#4..

by design i am guessing you meant layout.. there in only one sentence and one draggable source in this code.

Does that code do what you want now?
What happens when you execute it?

it takes ANY draggable source

How are you testing it to see that it takes all draggable sources? What draggable sources are there in the program to test with?

Originally Posted by Norm

How are you testing it to see that it takes all draggable sources? What draggable sources are there in the program to test with?

These are the draggable sources:

Those draggable sources are NOT in the code in post#29 that I was using for testing.

What are all the Strings in the constructor for the DropTargetTextArea class for in post#33? They were NOT in the code in post#29.

Why did you change that?

this is t

Did you see my questions about the 4 Strings in the constructor? Why are they there? What does the class do with them?

What does the class do with the Strings that are passed to it?

if i add 5th drop label i will need to also add the string the the DropTargetTextArea for it to be dropped

What if there are 50 or 200?

What do you need to fix? What is the program supposed to do? What values does the DropTargetTextArea class need to have to do its job? What is the DropTargetTextArea class's job?

These are all questions that should have been asked and answered when the design for the program was made.

Originally Posted by Norm

What do you need to fix? What is the program supposed to do? What values does the DropTargetTextArea class need to have to do its job? What is the DropTargetTextArea class's job?

These are all questions that should have been asked and answered when the design for the program was made.

1) what the program should do
If the

the "cat jumped over the moon" label,

Where is that label? I don't see one with those words in the text.
The drop location in the code is EMPTY, there is no text in it.

have 1 only string value which it will either accept or reject

Yes, that sounds like what it should have and do.

Rewrite the code so it does that.

JLabel s1 =

if i was to add another 2 sentences how would i implement this?

Each new drop target would be a new instance of the DropTargetTextArea class.

Sorry, I don't know what you are asking for. What part are you having problems with?

Are you asking how to create an instance of a class? Use a new statement. The current code is full of them.
For example:

               JPanel gridPanel = new JPanel(new GridLayout(4, 4, 5, 5)); 
                JPanel flowPanel = new JPanel();
   ...
      Component droplabel = new DropTargetTextArea("moon", "dog","cat","mouse");

All those statements create new instances of some class.

but i

i thought i was doing them wrong and thats why i took them out.

You are correct, the code I posted was NOT how the code is supposed to be. I just copied some code from your last post to show you how you have used new statements many times in the code.

The constructor should have the String that the instance of the class will accept. Each instance can have its own String.