File path: /apache-ant-1.7.0/src/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessage.java | File path: /apache-ant-1.7.0/src/org/apache/tools/ant/taskdefs/optional/ssh/ScpToMessageBySftp.java | |||
Method name: void sendDirectory(Directory, InputStream, OutputStream)
|
Method name: void sendDirectory(ChannelSftp, Directory)
|
|||
Number of AST nodes: 5 | Number of AST nodes: 5 | |||
1 | for (Iterator fileIt = current.filesIterator(); fileIt.hasNext();) {↵ | 1 | for (Iterator fileIt = current.filesIterator(); fileIt.hasNext();) {↵ | |
2 | sendFileToRemote((File) fileIt.next(), in, out);↵ | 2 | sendFileToRemote(channel, (File) fileIt.next(), null);↵ | |
3 | }↵ | 3 | }↵ | |
4 | for (Iterator dirIt = current.directoryIterator(); dirIt.hasNext();) {↵ | 4 | for (Iterator dirIt = current.directoryIterator(); dirIt.hasNext();) {↵ | |
5 | Directory dir = (Directory) dirIt.next();↵ | 5 | Directory dir = (Directory) dirIt.next();↵ | |
6 | sendDirectoryToRemote(dir, in, out);↵ | 6 | sendDirectoryToRemote(channel, dir);↵ | |
7 | } | 7 |
| |
See real code fragment | See real code fragment |
Number of common nesting structure subtrees | 1 |
Number of refactorable cases | 0 |
Number of non-refactorable cases | 1 |
Time elapsed for finding largest common nesting structure subtrees (ms) | 0.6 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 17 |
Number of mapped statements | 3 |
Number of unmapped statements in the first code fragment | 2 |
Number of unmapped statements in the second code fragment | 2 |
Time elapsed for statement mapping (ms) | 12.7 |
Clone type | Type 3 |
ID | Statement | ID | Statement | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | for (Iterator fileIt = current.filesIterator(); fileIt.hasNext(); ) |
| 3 | for (Iterator dirIt = current.directoryIterator(); dirIt.hasNext(); ) | |||||||||||||||||||
|
| 4 | Directory dir = (Directory)dirIt.next(); | ||||||||||||||||||||
2 | sendFileToRemote((File)fileIt.next(), in, out); |
| 5 | sendDirectoryToRemote(channel, dir); | |||||||||||||||||||
3 | for (Iterator dirIt = current.directoryIterator(); dirIt.hasNext(); ) |
| 1 | for (Iterator fileIt = current.filesIterator(); fileIt.hasNext(); ) | |||||||||||||||||||
|
| 2 | sendFileToRemote(channel, (File)fileIt.next(), null); | ||||||||||||||||||||
4 | Directory dir = (Directory)dirIt.next(); |
| | ||||||||||||||||||||
5 | sendDirectoryToRemote(dir, in, out); |
| |
Row | Violation |
---|---|
1 | Unmatched statement Directory dir=(Directory)dirIt.next(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Expression sendFileToRemote((File)fileIt.next(),in,out) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Expression sendFileToRemote((File)fileIt.next(),in,out) is a void method call, and thus it cannot be parameterized |
4 | Expression sendDirectoryToRemote(channel,dir) is a void method call, and thus it cannot be parameterized |
5 | Expression sendFileToRemote((File)fileIt.next(),in,out) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
6 | Expression sendFileToRemote((File)fileIt.next(),in,out) is a void method call, and thus it cannot be parameterized |
7 | Expression sendDirectoryToRemote(channel,dir) is a void method call, and thus it cannot be parameterized |
8 | Unmatched statement sendFileToRemote(channel,(File)fileIt.next(),null); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
9 | Unmatched statement Directory dir=(Directory)dirIt.next(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
10 | Unmatched statement sendDirectoryToRemote(dir,in,out); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
11 | Clone fragment #1 returns variables dirIt , while Clone fragment #2 returns variables fileIt, dirIt |