Fail the openDocument() validation

Question about the task

Resolved

I tested the code and the open document function works properly. Don't know what's missing, the validation showed even the view.has not been set to HTML tab!

23.1. Let's write the saveDocument() method for saving the current file. The method should work the same as saveDocumentAs(), but not ask the user for a file. Instead, it should use currentFile. If currentFile is null, then call the saveDocumentAs() method.
23.2. It's time to implement the openDocument() method. The method should work like the part of the saveDocumentAs() method that is responsible for file selection.

Hint: Pay attention to the name of the method used to display the dialog.

When a file is selected, you must:
23.2.1. Set the new value of currentFile.
23.2.2. Reset the document.
23.2.3. Set the file name as the title on the view.
23.2.4. Use currentFile to create a FileReader.
23.2.5. Use an HTMLEditorKit object to read data from the FileReader into the document field.
23.2.6. Reset the edits (call the view's resetUndo method).
23.2.7. If exceptions occur, log them but don't throw them upward.
Check that the Save and Open menu items work.

Requirements:

The saveDocument() method in the controller should switch the view to the HTML tab.
The saveDocument() method in the controller should create a FileWriter based on currentFile if currentFile != null.
The saveDocument() method in the controller should use HTMLEditorKit to copy the data from document to the FileWriter object if currentFile != null.
The saveDocument() method in the controller should call the saveDocumentAs() method if currentFile == null.
The saveDocument() method in the controller should not throw exceptions. It should only log them using ExceptionHandler.
The openDocument() method in the controller should switch the view to the HTML tab.
The openDocument() method in the controller should create a new JFileChooser object.
The openDocument() method in the controller should set an HTMLFileFilter object as the filter on the JFileChooser object.
The openDocument() method in the controller should use the JFileChooser.showOpenDialog() method to show the "Open File" file chooser dialog.
The openDocument() method in the controller should set the new value of currentFile if the user confirms the file selection.
The openDocument() method in the controller should reset the document if the user confirms the file selection.
The openDocument() method in the controller should set the file name as the window title on the view if the user confirms the file selection.
The openDocument() method in the controller should create a FileReader based on currentFile if the user confirms the file selection.
The openDocument() method in the controller should use HTMLEditorKit to read data from the FileReader into document if the user confirms the file selection.
The openDocument() method in the controller should reset the edits if the user confirms the file selection.
The openDocument() method in the controller should not throw exceptions. It should only log them using ExceptionHandler.

package com.codegym.task.task32.task3209; import com.codegym.task.task32.task3209.listeners.FrameListener; import com.codegym.task.task32.task3209.listeners.TabbedPaneChangeListener; import com.codegym.task.task32.task3209.listeners.UndoListener; import javax.swing.*; import javax.swing.undo.UndoManager; import java.awt.*; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; public class View extends JFrame implements ActionListener { private Controller controller; private JTabbedPane tabbedPane; private JTextPane htmlTextPane; private JEditorPane plainTextPane; private UndoManager undoManager = new UndoManager(); private UndoListener undoListener = new UndoListener(undoManager); public View() { try { UIManager.setLookAndFeel("javax.swing.plaf.metal.MetalLookAndFeel"); } catch (Exception e) { ExceptionHandler.log(e); } } public Controller getController() { return controller; } public void setController(Controller controller) { this.controller = controller; } public void update() { htmlTextPane.setDocument(controller.getDocument()); } public void showAbout() { JOptionPane.showMessageDialog(null,"This is an HTML Editor.", "Information", JOptionPane.INFORMATION_MESSAGE); } public UndoListener getUndoListener() { return undoListener; } @Override public void actionPerformed(ActionEvent actionEvent) { String action = actionEvent.getActionCommand(); switch(action) { case "New": controller.createNewDocument(); break; case "Open": controller.openDocument(); break; case "Save": controller.saveDocument(); break; case "Save as...": controller.saveDocumentAs(); break; case "Exit": controller.exit(); break; case "About": showAbout(); break; } } public void init() { tabbedPane = new JTabbedPane(); add(tabbedPane); htmlTextPane = new JTextPane(); plainTextPane = new JEditorPane(); initGui(); addWindowListener(new FrameListener(this)); setVisible(true); } public void initMenuBar() { JMenuBar menuBar = new JMenuBar(); MenuHelper.initFileMenu(this, menuBar); MenuHelper.initEditMenu(this, menuBar); MenuHelper.initStyleMenu(this, menuBar); MenuHelper.initAlignMenu(this, menuBar); MenuHelper.initColorMenu(this, menuBar); MenuHelper.initFontMenu(this, menuBar); MenuHelper.initHelpMenu(this, menuBar); getContentPane().add(menuBar, BorderLayout.NORTH); } public void initEditor() { htmlTextPane.setContentType("text/html"); JScrollPane scrollPane1 = new JScrollPane(htmlTextPane); tabbedPane.add("HTML", scrollPane1); JScrollPane scrollPane2 = new JScrollPane(plainTextPane); tabbedPane.add("Text", scrollPane2); Dimension paneSize = tabbedPane.getPreferredSize(); tabbedPane.setPreferredSize(paneSize); TabbedPaneChangeListener paneChangeListener = new TabbedPaneChangeListener(this); tabbedPane.addChangeListener(paneChangeListener); getContentPane().add(tabbedPane, BorderLayout.CENTER); } public void initGui() { initMenuBar(); initEditor(); pack(); } public void selectedTabChanged() { if (tabbedPane.getSelectedIndex() == 0) { controller.setPlainText(plainTextPane.getText()); } else { plainTextPane.setText(controller.getPlainText()); } resetUndo(); } public void selectHtmlTab() { tabbedPane.setSelectedIndex(0); resetUndo(); } public void undo() { try { undoManager.undo(); } catch (Exception e) { ExceptionHandler handler = new ExceptionHandler(); handler.log(e); } } public void redo() { try { undoManager.redo(); } catch (Exception e) { ExceptionHandler handler = new ExceptionHandler(); handler.log(e); } } public boolean canUndo() { return undoManager.canUndo(); } public boolean canRedo() { return undoManager.canRedo(); } public void resetUndo() { undoManager.discardAllEdits(); } public boolean isHtmlTabSelected() { return tabbedPane.getSelectedIndex() == 0; } public void exit() { controller.exit(); } }

Comments (5)

Popular
New
Old

You must be signed in to leave a comment

Elvis Lee

Level 41 , Aurora , United States

27 February 2020, 20:22

Line 73 is ok, it turned out that after changing line 76, it passed. Thank you Seb.

Elvis Lee

Level 41 , Aurora , United States

27 February 2020, 00:16

Strange, still not working after the change!

Seb

Level 41 , Crefeld , Germany

27 February 2020, 08:06

solution

If it's still not working, then also try to remove line 73. Adjust line 76 as follows: currentFile = chooser.getSelectedFile() Remove line 83.

Elvis Lee

Level 41 , Aurora , United States

27 February 2020, 00:10

Oh, got it! Thanks!!

Seb

Level 41 , Crefeld , Germany

26 February 2020, 23:39

solution

The main problem is on line 74. Instead of passing null to the showOpenDialog method, pass the view object into it. int returnVal = chooser.showOpenDialog(view);