Some work on the UI.

Author: khatchad

edu.cuny.hunter.streamrefactoring.core/src/edu/cuny/hunter/streamrefactoring/core/utils/Util.java

@@ -59,6 +59,9 @@ public static ConvertToParallelStreamRefactoringProcessor createConvertToParalle
 
 	public static ConvertToParallelStreamRefactoringProcessor createConvertToParallelStreamRefactoringProcessor(
 			IJavaProject[] projects, Optional<IProgressMonitor> monitor) throws JavaModelException {
+		if (projects.length < 1) 
+			throw new IllegalArgumentException("No projects.");
+		
 		CodeGenerationSettings settings = JavaPreferencesSettings.getCodeGenerationSettings(projects[0]);
 		ConvertToParallelStreamRefactoringProcessor processor = new ConvertToParallelStreamRefactoringProcessor(
 				projects, settings, monitor);

edu.cuny.hunter.streamrefactoring.ui/src/edu/cuny/hunter/streamrefactoring/ui/handlers/OptimizeStreamHandler.java

@@ -23,11 +23,13 @@
 import org.eclipse.jdt.internal.ui.JavaPlugin;
 import org.eclipse.jdt.internal.ui.util.SelectionUtil;
 import org.eclipse.jdt.ui.JavaElementLabels;
+import org.eclipse.jface.dialogs.MessageDialog;
 import org.eclipse.jface.viewers.ISelection;
 import org.eclipse.swt.widgets.Shell;
 import org.eclipse.ui.handlers.HandlerUtil;
 import org.osgi.framework.FrameworkUtil;
 
+import edu.cuny.hunter.streamrefactoring.ui.messages.Messages;
 import edu.cuny.hunter.streamrefactoring.ui.wizards.OptimizeStreamRefactoringWizard;
 
 public class OptimizeStreamHandler extends AbstractHandler {
@@ -49,31 +51,23 @@ public Object execute(ExecutionEvent event) throws ExecutionException {
 					if (obj instanceof IJavaElement) {
 						IJavaElement jElem = (IJavaElement) obj;
 						switch (jElem.getElementType()) {
-						case IJavaElement.METHOD:
-							break;
-						case IJavaElement.TYPE:
-							break;
-						case IJavaElement.COMPILATION_UNIT:
-							break;
-						case IJavaElement.PACKAGE_FRAGMENT:
-							break;
-						case IJavaElement.PACKAGE_FRAGMENT_ROOT:
-							break;
 						case IJavaElement.JAVA_PROJECT:
 							javaProjectSet.add((IJavaProject) jElem);
 							break;
 						}
 					}
 
 				Shell shell = HandlerUtil.getActiveShellChecked(event);
-				OptimizeStreamRefactoringWizard.startRefactoring(
-						javaProjectSet.toArray(new IJavaProject[javaProjectSet.size()]), shell, Optional.empty());
+
+				if (javaProjectSet.isEmpty())
+					MessageDialog.openError(shell, Messages.Name, Messages.NoProjects);
+				else
+					OptimizeStreamRefactoringWizard.startRefactoring(
+							javaProjectSet.toArray(new IJavaProject[javaProjectSet.size()]), shell, Optional.empty());
 			} catch (JavaModelException e) {
 				JavaPlugin.log(e);
 				throw new ExecutionException("Failed to start refactoring", e);
 			}
-		// TODO: What do we do if there was no input? Do we display some
-		// message?
 		return null;
 	}
 

edu.cuny.hunter.streamrefactoring.ui/src/edu/cuny/hunter/streamrefactoring/ui/messages/Messages.java

@@ -11,6 +11,8 @@
  */
 public class Messages extends NLS {
 	private static final String BUNDLE_NAME = "edu.cuny.hunter.streamrefactoring.ui.messages.messages"; //$NON-NLS-1$
+	public static String Name;
+	public static String NoProjects;
 
 	static {
 		// initialize resource bundle

edu.cuny.hunter.streamrefactoring.ui/src/edu/cuny/hunter/streamrefactoring/ui/messages/messages.properties

@@ -0,0 +1,2 @@
+Name=Optimize Streams Refactoring
+NoProjects=No project(s) selected.