Automatic Pattern creation from Input Image

Question

Hi all,
I've created a program to automatically create a patterns from an input image
1. The user selects image
2. with a rectangle he selects a zone of that image
3. An automatic pattern image is created as result of repeating that rectangle
It's working most of the times but sometimes:
1. The red rectangle which I use to select the area I want to repeat is not displayed
2. The area is not correctly handled and strange blank areas appear
Do you know how to solve these issues?
I put you here the whole source code so you can execute it and have fun creating patterns!
http://tlloreti.googlepages.com/src.rar
Regards
T

Answer 1

Part 2*
Now, about your red rectangle
class Selector extends MouseInputAdapter
    ImageSelectionPanel selectionPanel;
    Point start;
    Point end;
    boolean dragging, isClipSet;
    public Selector(ImageSelectionPanel isp)
        selectionPanel = isp;
        dragging = false;
        isClipSet = false;
    public void mousePressed(MouseEvent e)
        if(isClipSet)             // clear existing clip
            selectionPanel.setClipFrame(start, start);
            isClipSet = false;
        else                      // or start new clip
            start = e.getPoint();
            dragging = true;
            isClipSet = true;
    public void mouseReleased(MouseEvent e)
        dragging = false;
        end = e.getPoint();
        System.out.println("START="+start);
        System.out.println("END="+end);
        int width=(int) (end.getX()-start.getX()) ;
        int height=(int) (end.getY() - start.getY());
        if ((width>0)  && (height>0))
        MergeImages.GenerateFourImagesDragAndDrop(crop1.fileName, start.getX(), start.getY(),width ,height );
    public void mouseDragged(MouseEvent e)
        if(dragging)
            selectionPanel.setClipFrame(start, e.getPoint());
}The problem stems from the isClipSet and dragging variables. I have no idea what you were trying to achieve with these. I'm guessing you're making certain assumptions about the way mouse events are received that are just plain wrong. Also, the logic in the MouseReleased method is wrong. The point you define as "start" is not necessarily the top left of the clip rectangle as you have it set up. But in this call
MergeImages.GenerateFourImagesDragAndDrop(crop1.fileName, start.getX(), start.getY(),width ,height );you're assuming it is the top left point. I imagine this might contribute to your "blank areas" issue, but I'm not sure.
Change your Selector class to the code below, and the rectangle will always work (at least it does on my machine --> java 1.6.0+)
class Selector extends MouseInputAdapter
    ImageSelectionPanel selectionPanel;
    Point start;
    public Selector(ImageSelectionPanel isp)
        selectionPanel = isp;
    public void mousePressed(MouseEvent e)
        start = e.getPoint();
        selectionPanel.setClipFrame(start, start);
    public void mouseReleased(MouseEvent e)
        Point end = e.getPoint();
        end.x = Math.min(Math.max(end.x,0),selectionPanel.image.getWidth());
        end.y = Math.min(Math.max(end.y,0),selectionPanel.image.getHeight());
        selectionPanel.setClipFrame(start, end);
        System.out.println("START="+start);
        System.out.println("END="+end);
        Rectangle clip = selectionPanel.clip;
        if(clip.width > 0 && clip.height > 0)
        MergeImages.GenerateFourImagesDragAndDrop(crop1.fileName,
                                                  clip.x,clip.y,
                                                  clip.width,clip.height );
    public void mouseDragged(MouseEvent e)
        selectionPanel.setClipFrame(start, e.getPoint());
}You will see that a new problem surfaces. That problem is that generated patterns affect previous ones. I imagine this has to do with your extensive use of static variables. Any swing person will tell you that a static GUI component constitutes as a bad idea in the majority of situations. But I'll let you deal with that.