SWT FileDialog opening issue

Question

I need to open a FileDialog to open a file using SWT on Windows 7 Home using Java 7. I need this FileDialog open when BrowseBtn1 Button (SWT) is pressed. I am using addSelectionListener for this. Is this the correct approach? If yes, why my dialog won't open when it does not meet any run-time error. I am not using AWT at all, only using SWT.

import org.eclipse.swt.*;
import org.eclipse.swt.widgets.*;
import org.eclipse.swt.widgets.FileDialog;
import org.eclipse.swt.events.*;
import org.eclipse.swt.graphics.FontData;
import org.eclipse.swt.layout.*;

public class MySwtApp {
    public static Display display;
    public static Shell shell;

    public static void main(String[] args) {

        display = new Display ();
        shell = new Shell (display);
        shell.setText("MY TITLE");
        GridLayout gridLayout = new GridLayout (3, false);
        shell.setLayout (gridLayout);

        Label Label1 = new Label (shell, SWT.NONE);
        Label1.setText ("Select Message ");
        GridData data = new GridData (200, SWT.DEFAULT);
        Label1.setLayoutData (data);

        Combo Combo1 = new Combo (shell, SWT.NONE);
        Combo1.setItems (new String [] {"Option A", 
                "Option B"});
        // CaseStudyCombo.setText ("CaseStudyCombo");
        data = new GridData (200, SWT.DEFAULT);
        Combo1.setLayoutData (data);
        Combo1.addListener (SWT.DefaultSelection, new Listener () {
            public void handleEvent (Event e) {
                System.out.println (e.widget + " - Default Selection");
            }
        });

        Label emptyLabel = new Label (shell, SWT.NONE);
        emptyLabel.setText ("");
        data = new GridData (50, SWT.DEFAULT);
        emptyLabel.setLayoutData (data);

        Label Label2 = new Label (shell, SWT.NONE);
        Label2.setText ("Next Message: ");
        data = new GridData (200, SWT.DEFAULT);
        Label2.setLayoutData (data);

        final Text text4 = new Text (shell, SWT.BORDER);
        text4.setText ("");
        data = new GridData (215, SWT.DEFAULT);
        text4.setLayoutData (data);

        Button BrowseBtn1 = new Button (shell, SWT.PUSH);
        BrowseBtn1.setText ("Browse");
        data = new GridData (80, SWT.DEFAULT);
        BrowseBtn1.setLayoutData (data);
        BrowseBtn1.addSelectionListener(new SelectionAdapter() {
            String result = "";
            public void WidgetSelected(SelectionEvent e) {
                FileDialog dialog = new FileDialog (shell, SWT.OPEN);
                dialog.setFilterExtensions(new String [] {"*.html"});
                //dialog.setFilterPath("c:\\temp");
                result = dialog.open();
                text4.setText(result);
            }
        });

        Label Label3 = new Label (shell, SWT.NONE);
        Label3.setText ("Message Label 3: ");
        data = new GridData (200, SWT.DEFAULT);
        Label3.setLayoutData (data);

        Text text5 = new Text (shell, SWT.BORDER);
        text5.setText ("");
        data = new GridData (215, SWT.DEFAULT);
        text5.setLayoutData (data);

        Button BrowseBtn2 = new Button (shell, SWT.PUSH);
        BrowseBtn2.setText ("Browse");
        data = new GridData (80, SWT.DEFAULT);
        BrowseBtn2.setLayoutData (data);

        Label Label4 = new Label (shell, SWT.NONE);
        Label4.setText ("Message Label 4: ");
        data = new GridData (200, SWT.DEFAULT);
        Label4.setLayoutData (data);

        Combo Combo2 = new Combo (shell, SWT.NONE);
        Combo2.setText ("Options");
        data = new GridData (200, SWT.DEFAULT);
        Combo2.setLayoutData (data);

        Button searchBtn1 = new Button (shell, SWT.PUSH);
        searchBtn1.setText ("Search");
        data = new GridData (80, SWT.DEFAULT);
        searchBtn1.setLayoutData (data);

        Combo Combo3 = new Combo (shell, SWT.NONE);
        // CaseStudyCombo.setItems (new String [] {"Item 1", "Item 2", "Item 2"});
        Combo3.setText ("Options:");
        data = new GridData (200, SWT.DEFAULT);
        Combo3.setLayoutData (data);

        Combo Combo4 = new Combo (shell, SWT.NONE);
        Combo4.setText ("Options:");
        data = new GridData (200, SWT.DEFAULT);
        Combo4.setLayoutData (data);

        Button showDetailsBtn = new Button (shell, SWT.PUSH);
        showDetailsBtn.setText ("Show Details");
        data = new GridData (80, SWT.DEFAULT);
        showDetailsBtn.setLayoutData (data);

        shell.pack ();
        shell.open ();

        while (!shell.isDisposed ()) {
            if (!display.readAndDispatch ())
                display.sleep ();
        }
        display.dispose ();
    }
}

Please help solve this issue.

Answer 1

the problematic line is this:

public void WidgetSelected(SelectionEvent e)

note the uppercase 'W'. What you are currently doing is defining a new method (instead of overriding the existing widgetSelected(...) method) in that anonymous class. If you change this to lowercase 'w' like this, it will work:

public void widgetSelected(SelectionEvent e)

It is a good pratice (actually more than recommended) to use @Override annotation. If you had used that, compiler would have flagged you an error (refer eg. Joshua Bloch Effective Java 2nd edition Item 36: Consistently use the Override annotation)