gaspersic/java-design-patterns
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #794 from iluwatar/Issue781

Browse Source 
#781 Resolved ClasscastException from Acyclic Visitor
This commit is contained in:
Ilkka Seppälä 2018-09-25 22:45:25 +03:00 committed by GitHub
commit b079aec1fd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
acyclic-visitor/src/main/java/com/iluwatar/acyclicvisitor/AllModemVisitor.java
View File

@ -23,8 +23,9 @@
package com.iluwatar.acyclicvisitor;
/**
* All ModemVisitor interface extends all visitor interfaces
* All ModemVisitor interface extends all visitor interfaces. This interface
* provides ease of use when a visitor needs to visit all modem types.
*/
public interface AllModemVisitor extends ModemVisitor, ZoomVisitor, HayesVisitor{
public interface AllModemVisitor extends ZoomVisitor, HayesVisitor{
}

5
acyclic-visitor/src/main/java/com/iluwatar/acyclicvisitor/ConfigureForUnixVisitor.java
View File

@ -27,9 +27,10 @@ import org.slf4j.LoggerFactory;
/**
* ConfigureForUnixVisitor class implements zoom's visit method for Unix
* manufacturer
* manufacturer, unlike traditional visitor pattern, this class may selectively implement
* visit for other modems.
*/
public class ConfigureForUnixVisitor implements ModemVisitor, ZoomVisitor {
public class ConfigureForUnixVisitor implements ZoomVisitor {
private static final Logger LOGGER = LoggerFactory.getLogger(ConfigureForUnixVisitor.class);

6
acyclic-visitor/src/main/java/com/iluwatar/acyclicvisitor/Hayes.java
View File

@ -37,10 +37,10 @@ public class Hayes extends Modem {
*/
@Override
public void accept(ModemVisitor modemVisitor) {
try {
if (modemVisitor instanceof HayesVisitor) {
((HayesVisitor) modemVisitor).visit(this);
} catch (ClassCastException e) {
LOGGER.error("Unable to cast to HayesVisitor");
} else {
LOGGER.info("Only HayesVisitor is allowed to visit Hayes modem");
}
}

6
acyclic-visitor/src/main/java/com/iluwatar/acyclicvisitor/Zoom.java
View File

@ -37,10 +37,10 @@ public class Zoom extends Modem {
*/
@Override
public void accept(ModemVisitor modemVisitor) {
try {
if (modemVisitor instanceof ZoomVisitor) {
((ZoomVisitor) modemVisitor).visit(this);
} catch (ClassCastException e) {
LOGGER.error("Unable to cast to ZoomVisitor");
} else {
LOGGER.info("Only ZoomVisitor is allowed to visit Zoom modem");
}
}