code refactoring tips
"Gregor J. Rothfuss" <[email protected]> Tue, 18 Feb 2003 20:24:21 +0100
| Newsgroups | gmane.comp.cms.wyona.devel |
|---|---|
| Message-ID | <001401c2d781$770737f0$2200a8c0@tarantula> |
here is a list that eclipse generated with suggestions to improve the code.. sorry about the html, but the list would be pretty unreadable otherwise. enjoy, -- Gregor J. Rothfuss [email protected] +41 1 272 9161 Software Architect Wyona AG, Hardstrasse 219, CH-8005 Zurich Open Source CMS http://www.wyona.org http://www.wyona.com Description Resource In Folder Location Assigning an Object to null is a code smell. Consider refactoring. Generator.java wyona/wyonacms/src/cocoon/WEB-INF/classes/org/apache/cocoon/samples/parentcm line 119 Avoid unused imports such as 'DateUtil' ArticleCreator.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 55 Avoid unused imports such as 'Calendar' ArticleCreator.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 60 Avoid unused imports such as 'GregorianCalendar' ArticleCreator.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 61 Avoid unused imports such as 'Configurable' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 46 Avoid unused imports such as 'Configuration' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 47 Avoid unused imports such as 'ConfigurationException' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 48 Avoid unused imports such as 'Constants' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 51 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 52 Avoid unused imports such as 'Redirector' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 53 Avoid unused imports such as 'Request' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 54 Avoid unused imports such as 'Session' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 55 Avoid unused imports such as 'SourceResolver' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 56 Avoid unused imports such as 'DefaultFilePublisher' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 70 Avoid unused imports such as 'DOM4JUtil' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 76 Avoid unused imports such as 'FileWriter' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 81 Avoid unused imports such as 'URL' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 83 Avoid unused imports such as 'HashMap' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 88 Avoid unused imports such as 'Map' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 90 Avoid unused local variables such as 'published' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 120 A method should have only one exit point, and that should be the last statement in the method ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 212 A method should have only one exit point, and that should be the last statement in the method ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 288 Avoid unused local variables such as 'year' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 341 Avoid unused local variables such as 'month' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 343 Avoid unused local variables such as 'day' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 345 Avoid unused local variables such as 'documentHelper' ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 350 Avoid empty catch blocks ArticleTask.java wyona/wyonacms/src/java/ch/computerworld/wyona/cms line 379 A method should have only one exit point, and that should be the last statement in the method GenericCreator.java wyona/wyonacms/src/java/ch/ethz/mat/wyona/cms/authoring line 83 Avoid unused imports such as 'Configurable' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 46 Avoid unused imports such as 'Configuration' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 47 Avoid unused imports such as 'ConfigurationException' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 48 Avoid unused imports such as 'Constants' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 51 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 52 Avoid unused imports such as 'Redirector' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 53 Avoid unused imports such as 'Request' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 54 Avoid unused imports such as 'Session' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 55 Avoid unused imports such as 'SourceResolver' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 56 Avoid unused imports such as 'DefaultFilePublisher' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 70 Avoid unused imports such as 'DOM4JUtil' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 76 Avoid unused imports such as 'FileWriter' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 81 Avoid unused imports such as 'URL' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 83 Avoid unused imports such as 'HashMap' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 88 Avoid unused imports such as 'Map' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 90 Avoid unused local variables such as 'published' ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 119 A method should have only one exit point, and that should be the last statement in the method ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 197 A method should have only one exit point, and that should be the last statement in the method ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 262 Avoid unnecessary temporaries when converting primitives to Strings ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 337 Avoid empty catch blocks ArticleTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 369 A method should have only one exit point, and that should be the last statement in the method ClearNewsletterTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 123 Avoid unused imports such as 'AbstractAction' DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 48 Avoid unused imports such as 'Node' DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 61 Avoid unused imports such as 'DOMReader' DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 64 Avoid unused imports such as 'FileWriter' DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 73 A method should have only one exit point, and that should be the last statement in the method DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 137 A method should have only one exit point, and that should be the last statement in the method DossierAction.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 327 Avoid unused imports such as 'DOM4JUtil' InsertDossierTask.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms line 63 Avoid unused imports such as 'FileWriter' NewArticleCreator.java wyona/wyonacms/src/java/ch/unizh/unipublic/wyona/cms/authoring line 51 Avoid reassigning parameters such as 'a' Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 84 Avoid reassigning parameters such as 'a' Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 91 Avoid reassigning parameters such as 'a' Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 98 Avoid reassigning parameters such as 'a' Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 105 Avoid reassigning parameters such as 'output' Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 112 A method should have only one exit point, and that should be the last statement in the method Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 266 Avoid empty catch blocks Md5.java wyona/wyonacms/src/java/org/w3c/tools/crypt line 287 Avoid using implementation types like 'Vector'; use the interface instead Identity.java wyona/wyonacms/src/java/org/wyona/cms/ac line 69 All methods are static. Consider using Singleton instead. Password.java wyona/wyonacms/src/java/org/wyona/cms/ac line 49 Avoid unused local variables such as 'b' Password.java wyona/wyonacms/src/java/org/wyona/cms/ac line 90 A method should have only one exit point, and that should be the last statement in the method Policy.java wyona/wyonacms/src/java/org/wyona/cms/ac line 148 A method should have only one exit point, and that should be the last statement in the method AbstractParentChildCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 91 A method should have only one exit point, and that should be the last statement in the method AbstractParentChildCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 143 No need to import a type that's in the same package DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 48 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 115 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 135 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 138 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 146 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 149 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 153 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 157 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 176 A method should have only one exit point, and that should be the last statement in the method DefaultCreator.java wyona/wyonacms/src/java/org/wyona/cms/authoring line 180 Avoid unused imports such as 'Constants' AbstractAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Tokenizer' AbstractAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 57 A method should have only one exit point, and that should be the last statement in the method AbstractAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 104 A method should have only one exit point, and that should be the last statement in the method AbstractAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 113 A method should have only one exit point, and that should be the last statement in the method AbstractAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 122 Avoid unused imports such as 'ThreadSafe' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'Constants' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'ValidatorActionHelper' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'Tokenizer' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'StringTokenizer' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 71 A method should have only one exit point, and that should be the last statement in the method AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 151 A method should have only one exit point, and that should be the last statement in the method AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 159 A method should have only one exit point, and that should be the last statement in the method AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 190 A method should have only one exit point, and that should be the last statement in the method AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 209 Avoid reassigning parameters such as 'pattern' AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 228 A method should have only one exit point, and that should be the last statement in the method AbstractAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 259 Avoid unused imports such as 'Configuration' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 46 Avoid unused imports such as 'ConfigurationException' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 47 Avoid unused imports such as 'Session' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'Document' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'Node' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'NodeList' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'Identity' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 56 Avoid unused imports such as 'Password' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 57 Avoid unused imports such as 'File' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 59 Avoid unused imports such as 'URL' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 61 Avoid unused imports such as 'DocumentBuilder' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 65 Avoid unused imports such as 'DocumentBuilderFactory' AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 66 A method should have only one exit point, and that should be the last statement in the method AbstractUsernamePasswordAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 84 Avoid unused imports such as 'Configurable' ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 46 Avoid unused imports such as 'Constants' ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 A method should have only one exit point, and that should be the last statement in the method ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 245 A method should have only one exit point, and that should be the last statement in the method ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 324 Avoid using implementation types like 'HashMap'; use the interface instead ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 338 Avoid using implementation types like 'HashMap'; use the interface instead ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 381 A method should have only one exit point, and that should be the last statement in the method ArticleImageUploadCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 421 Avoid unused imports such as 'IOUtil' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 47 Avoid unused imports such as 'Source' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'DOMReader' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 68 Avoid unused imports such as 'XMLWriter' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 69 Avoid unused imports such as 'FileWriter' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 82 Avoid unused imports such as 'InputStream' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 84 Avoid unused imports such as 'Reader' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 86 Avoid unused imports such as 'StringReader' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 87 Avoid importing anything from the package 'java.lang' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 89 A method should have only one exit point, and that should be the last statement in the method BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 181 Avoid unused local variables such as 'reqFile' BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 203 A method should have only one exit point, and that should be the last statement in the method BitfluxAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 276 Avoid unused imports such as 'Constants' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Context' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'Session' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 56 Avoid unused imports such as 'Tokenizer' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 58 Avoid unused imports such as 'Attribute' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'Document' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 61 Avoid unused imports such as 'DocumentHelper' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 62 Avoid unused imports such as 'Element' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 63 Avoid unused imports such as 'XPath' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 64 Avoid unused imports such as 'SAXReader' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 66 Avoid unused imports such as 'AbstractParentChildCreator' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 68 Avoid unused imports such as 'File' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 72 Avoid unused imports such as 'FileInputStream' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 73 Avoid unused imports such as 'HashMap' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 77 Avoid unused imports such as 'List' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 78 Avoid unused imports such as 'StringTokenizer' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 80 A method should have only one exit point, and that should be the last statement in the method DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 136 A method should have only one exit point, and that should be the last statement in the method DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 159 A method should have only one exit point, and that should be the last statement in the method DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 163 Avoid unused private methods such as 'intercept' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 178 Avoid unused formal parameters such as 'out' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 201 Avoid unused private methods such as 'intercept' DeleteDocumentAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 201 Avoid unused imports such as 'ThreadSafe' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'Constants' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'ValidatorActionHelper' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'Context' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 Avoid unused imports such as 'Tokenizer' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 61 Avoid unused imports such as 'Document' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 63 Avoid unused imports such as 'File' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 67 Avoid unused imports such as 'URL' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 71 Avoid unused imports such as 'Enumeration' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 73 Avoid unused imports such as 'StringTokenizer' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 76 Avoid unused imports such as 'DocumentBuilder' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 78 Avoid unused imports such as 'DocumentBuilderFactory' EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 79 A method should have only one exit point, and that should be the last statement in the method EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 122 A method should have only one exit point, and that should be the last statement in the method EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 131 A method should have only one exit point, and that should be the last statement in the method EditorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 164 Avoid unused imports such as 'Composable' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 46 Avoid unused imports such as 'ThreadSafe' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Constants' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'ValidatorActionHelper' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 Avoid unused imports such as 'PostInputStream' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 63 Avoid unused imports such as 'Tokenizer' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 64 Avoid unused imports such as 'Document' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 67 Avoid unused imports such as 'CharArrayWriter' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 75 Avoid unused imports such as 'IOException' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 78 Avoid unused imports such as 'Reader' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 80 Avoid unused imports such as 'Writer' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 82 Avoid unused imports such as 'URL' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 84 Avoid unused imports such as 'StringTokenizer' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 88 Avoid unused imports such as 'DocumentBuilder' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 90 Avoid unused imports such as 'DocumentBuilderFactory' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 91 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 115 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 124 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 203 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 206 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 210 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 226 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 232 Avoid unused local variables such as 'success' EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 249 A method should have only one exit point, and that should be the last statement in the method EditorMainAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 274 Avoid unused imports such as 'Configuration' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 46 Avoid unused imports such as 'ConfigurationException' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 47 Avoid unused imports such as 'Constants' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Context' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'Tokenizer' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 58 Avoid unused imports such as 'StringTokenizer' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 68 A method should have only one exit point, and that should be the last statement in the method EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 100 A method should have only one exit point, and that should be the last statement in the method EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 108 Avoid unused local variables such as 'editFile' EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 125 A method should have only one exit point, and that should be the last statement in the method EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 145 A method should have only one exit point, and that should be the last statement in the method EditorSaveAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 149 Avoid unused imports such as 'ThreadSafe' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'Constants' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'ValidatorActionHelper' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'Context' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 Avoid unused imports such as 'Tokenizer' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'Document' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 62 Avoid unused imports such as 'File' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 64 Avoid unused imports such as 'URL' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 66 Avoid unused imports such as 'StringTokenizer' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 70 Avoid unused imports such as 'DocumentBuilder' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 72 Avoid unused imports such as 'DocumentBuilderFactory' EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 73 A method should have only one exit point, and that should be the last statement in the method EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 117 A method should have only one exit point, and that should be the last statement in the method EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 126 A method should have only one exit point, and that should be the last statement in the method EditorTempfileAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 136 Avoid unused imports such as 'ThreadSafe' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'Constants' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'ValidatorActionHelper' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'Request' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 56 Avoid unused imports such as 'Session' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 57 Avoid unused imports such as 'Tokenizer' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 59 Avoid unused imports such as 'StringTokenizer' HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 63 A method should have only one exit point, and that should be the last statement in the method HTMLFormWriteAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 126 Avoid unused imports such as 'Node' IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 54 Avoid unused imports such as 'NodeList' IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 Avoid unused imports such as 'File' IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 A method should have only one exit point, and that should be the last statement in the method IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 157 Deeply nested if..then statements are hard to read IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 163 A method should have only one exit point, and that should be the last statement in the method IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 164 Deeply nested if..then statements are hard to read IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 169 A method should have only one exit point, and that should be the last statement in the method IMLAuthenticatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 176 Avoid unused imports such as 'Constants' NoPasswordYetAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 48 Avoid unused imports such as 'Request' NoPasswordYetAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'EntityResolver' NoPasswordYetAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 A method should have only one exit point, and that should be the last statement in the method NoPasswordYetAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 103 Avoid unused imports such as 'Constants' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Context' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'Tokenizer' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 58 Avoid unused imports such as 'FileInputStream' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 73 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 139 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 156 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 164 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 173 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 211 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 233 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 258 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 293 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 318 A method should have only one exit point, and that should be the last statement in the method ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 322 Avoid unused private methods such as 'intercept' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 337 Avoid unused formal parameters such as 'out' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 360 Avoid unused private methods such as 'intercept' ParentChildCreatorAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 360 Avoid unused imports such as 'Constants' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 50 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'ValidatorActionHelper' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'Redirector' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 Avoid unused imports such as 'SourceResolver' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 56 Avoid unused imports such as 'Tokenizer' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 57 Avoid unused imports such as 'HashMap' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 68 Avoid unused imports such as 'StringTokenizer' PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 70 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 157 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 169 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 184 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 188 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 196 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 207 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 214 A method should have only one exit point, and that should be the last statement in the method PMLAuthorizerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 221 Avoid unused imports such as 'Constants' RequestParamAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 53 A method should have only one exit point, and that should be the last statement in the method RequestParamAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 134 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' ReservedCheckinAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 A method should have only one exit point, and that should be the last statement in the method ReservedCheckinAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 109 A method should have only one exit point, and that should be the last statement in the method ReservedCheckinAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 115 Avoid unused imports such as 'Configurable' ReservedCheckoutAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 46 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' ReservedCheckoutAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 A method should have only one exit point, and that should be the last statement in the method ReservedCheckoutAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 120 A method should have only one exit point, and that should be the last statement in the method ReservedCheckoutAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 126 Avoid unused imports such as 'Constants' RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'Tokenizer' RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 58 Avoid unused imports such as 'FileReservedCheckOutException' RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 61 Avoid unused imports such as 'HashMap' RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 64 A method should have only one exit point, and that should be the last statement in the method RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 135 A method should have only one exit point, and that should be the last statement in the method RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 144 Assigning an Object to null is a code smell. Consider refactoring. RevisionControllerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 151 Avoid unused imports such as 'Constants' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 51 Avoid unused imports such as 'AbstractComplementaryConfigurableAction' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 52 Avoid unused imports such as 'Session' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 55 Avoid unused imports such as 'Document' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 58 Avoid unused imports such as 'Element' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 59 Avoid unused imports such as 'NodeList' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'FileReservedCheckInException' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 62 Avoid unused imports such as 'DOMParserFactory' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 64 Avoid unused imports such as 'DOMWriter' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 65 Avoid unused imports such as 'ByteArrayOutputStream' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 67 Avoid unused imports such as 'Date' RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 70 A method should have only one exit point, and that should be the last statement in the method RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 119 A method should have only one exit point, and that should be the last statement in the method RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 137 A method should have only one exit point, and that should be the last statement in the method RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 141 A method should have only one exit point, and that should be the last statement in the method RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 152 A method should have only one exit point, and that should be the last statement in the method RollbackAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 157 A method should have only one exit point, and that should be the last statement in the method TaskAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 158 A method should have only one exit point, and that should be the last statement in the method TaskAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 205 Avoid unused imports such as 'IOUtil' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 47 Avoid unused imports such as 'Source' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 60 Avoid unused imports such as 'DOMReader' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 68 Avoid unused imports such as 'XMLWriter' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 69 Avoid unused imports such as 'InputStream' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 83 Avoid unused imports such as 'Reader' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 85 Avoid unused imports such as 'StringReader' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 86 Avoid importing anything from the package 'java.lang' XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 88 A method should have only one exit point, and that should be the last statement in the method XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 173 A method should have only one exit point, and that should be the last statement in the method XopusHandlerAction.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/acting line 286 Avoid unused imports such as 'Constants' ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 51 Avoid unused imports such as 'HttpSession' ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 58 Avoid unused imports such as 'Iterator' ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 82 Avoid empty catch blocks ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 132 Avoid unused local variables such as 'is' ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 162 Avoid empty 'if' statements ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 168 Avoid empty 'if' statements ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 170 Avoid unused private methods such as 'attribute' ProxyGenerator.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/generation line 320 Avoid unused imports such as 'Constants' IncludeTransformer.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/transformation line 50 Avoid unused private fields such as 'port' IncludeTransformer.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/transformation line 67 A method should have only one exit point, and that should be the last statement in the method IncludeTransformer.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/transformation line 116 A method should have only one exit point, and that should be the last statement in the method IncludeTransformer.java wyona/wyonacms/src/java/org/wyona/cms/cocoon/transformation line 119 Avoid unused imports such as 'MimeMessage' MailTask.java wyona/wyonacms/src/java/org/wyona/cms/mail line 69 Deeply nested if..then statements are hard to read MailTask.java wyona/wyonacms/src/java/org/wyona/cms/mail line 136 Deeply nested if..then statements are hard to read MailTask.java wyona/wyonacms/src/java/org/wyona/cms/mail line 162 Avoid unused imports such as 'DateUtil' CMFProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 55 Avoid unused imports such as 'Calendar' CMFProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 60 Avoid unused imports such as 'GregorianCalendar' CMFProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 61 Avoid unused imports such as 'XPathAPI' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 47 Avoid unused imports such as 'Node' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 50 Avoid unused imports such as 'NodeList' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 51 Avoid unused imports such as 'DateUtil' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 54 Avoid unused imports such as 'FileWriter' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 59 Avoid unused imports such as 'Calendar' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 61 Avoid unused imports such as 'GregorianCalendar' CMSProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 62 Avoid unused imports such as 'DateUtil' EditorProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 55 Avoid unused imports such as 'Calendar' EditorProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 60 Avoid unused imports such as 'GregorianCalendar' EditorProjectCreator.java wyona/wyonacms/src/java/org/wyona/cms/pubs/oscom line 61 All methods are static. Consider using Singleton instead. Configuration.java wyona/wyonacms/src/java/org/wyona/cms/rc line 44 Avoid unused imports such as 'BufferedReader' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 58 Avoid unused imports such as 'FileNotFoundException' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 60 Avoid unused imports such as 'FileReader' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 61 Avoid unused imports such as 'FileWriter' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 62 Avoid unused imports such as 'OutputStream' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 64 Avoid unused imports such as 'AbstractList' RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 67 A method should have only one exit point, and that should be the last statement in the method RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 289 A method should have only one exit point, and that should be the last statement in the method RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 314 A method should have only one exit point, and that should be the last statement in the method RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 336 A method should have only one exit point, and that should be the last statement in the method RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 338 A method should have only one exit point, and that should be the last statement in the method RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 343 Avoid using implementation types like 'Vector'; use the interface instead RCML.java wyona/wyonacms/src/java/org/wyona/cms/rc line 357 Avoid unused imports such as 'HttpServlet' RCServlet.java wyona/wyonacms/src/java/org/wyona/cms/rc line 71 Avoid unused imports such as 'HttpSession' RCServlet.java wyona/wyonacms/src/java/org/wyona/cms/rc line 74 Avoid unused imports such as 'Document' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 48 Avoid unused imports such as 'Element' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 49 Avoid unused imports such as 'BufferedReader' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 54 Avoid unused imports such as 'FileOutputStream' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 58 Avoid unused imports such as 'FileReader' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 59 Avoid unused imports such as 'FileWriter' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 60 Avoid unused imports such as 'PrintWriter' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 64 Avoid unused imports such as 'Vector' RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 67 A method should have only one exit point, and that should be the last statement in the method RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 133 A method should have only one exit point, and that should be the last statement in the method RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 154 A method should have only one exit point, and that should be the last statement in the method RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 159 Avoid empty try blocks RevisionController.java wyona/wyonacms/src/java/org/wyona/cms/rc line 379 A method should have only one exit point, and that should be the last statement in the method JobDataMapWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 141 Avoid duplicate imports such as 'org.quartz.SchedulerFactory' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 49 Avoid unused imports such as 'SchedulerFactory' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 49 Avoid unused imports such as 'FileInputStream' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 63 Avoid unused imports such as 'Calendar' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 67 Avoid unused imports such as 'Collection' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 68 Avoid unused imports such as 'Iterator' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 72 Avoid unused imports such as 'Properties' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 73 Avoid unused local variables such as 'startTime' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 139 Avoid unused local variables such as 'doc_conf' LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 142 Assigning an Object to null is a code smell. Consider refactoring. LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 261 Avoid empty 'if' statements LoadQuartzServlet.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 298 No need to import a type that's in the same package SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 66 A method should have only one exit point, and that should be the last statement in the method SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 273 A method should have only one exit point, and that should be the last statement in the method SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 300 A method should have only one exit point, and that should be the last statement in the method SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 323 A method should have only one exit point, and that should be the last statement in the method SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 358 A method should have only one exit point, and that should be the last statement in the method SchedulerWrapper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 514 Avoid unused imports such as 'Parameters' ServletJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 46 Avoid unused imports such as 'ServletContext' ServletJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 50 All methods are static. Consider using Singleton instead. ServletJobFactory.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 49 A method should have only one exit point, and that should be the last statement in the method ServletJobFactory.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 73 A method should have only one exit point, and that should be the last statement in the method ServletJobFactory.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 92 Avoid unused imports such as 'Iterator' TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 68 Avoid unused imports such as 'List' TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 69 Avoid unused imports such as 'Map' TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 70 Avoid unused imports such as 'Properties' TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 71 Avoid unused local variables such as 'publicationPath' TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 113 A method should have only one exit point, and that should be the last statement in the method TaskJob.java wyona/wyonacms/src/java/org/wyona/cms/scheduler line 167 All methods are static. Consider using Singleton instead. TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 44 Avoid unused imports such as 'Iterator' TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 57 Avoid unused imports such as 'List' TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 58 A method should have only one exit point, and that should be the last statement in the method TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 134 A method should have only one exit point, and that should be the last statement in the method TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 186 A method should have only one exit point, and that should be the last statement in the method TriggerHelper.java wyona/wyonacms/src/java/org/wyona/cms/scheduler/xml line 207 Avoid unused imports such as 'StringTokenizer' AbstractTask.java wyona/wyonacms/src/java/org/wyona/cms/task line 48 Avoid unused imports such as 'ParameterException' CommandLineTask.java wyona/wyonacms/src/java/org/wyona/cms/task line 46 Avoid unnecessary constructors - the compiler will generate these for you CommandLineTask.java wyona/wyonacms/src/java/org/wyona/cms/task line 59 Avoid unnecessary constructors - the compiler will generate these for you HelloWorldTask.java wyona/wyonacms/src/java/org/wyona/cms/task line 59 Avoid unused imports such as 'Parameters' Task.java wyona/wyonacms/src/java/org/wyona/cms/task line 47 Avoid unused imports such as 'Configurable' TaskFactory.java wyona/wyonacms/src/java/org/wyona/cms/task line 46 A method should have only one exit point, and that should be the last statement in the method TaskFactory.java wyona/wyonacms/src/java/org/wyona/cms/task line 96 Avoid unused imports such as 'URL' TaskManager.java wyona/wyonacms/src/java/org/wyona/cms/task line 55 Avoid unused imports such as 'Configurable' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 51 Avoid unused imports such as 'DocumentFactory' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 58 Avoid unused imports such as 'Namespace' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 59 Avoid unused imports such as 'Arrays' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 61 Avoid unused imports such as 'Collections' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 62 Avoid unused imports such as 'HashMap' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 63 Avoid unused imports such as 'Map' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 64 Avoid unused imports such as 'SortedMap' TaskSequence.java wyona/wyonacms/src/java/org/wyona/cms/task line 65 All methods are static. Consider using Singleton instead. FileDocument.java wyona/wyonacms/src/java/org/wyona/lucene line 44 All methods are static. Consider using Singleton instead. HTMLDocument.java wyona/wyonacms/src/java/org/wyona/lucene line 44 Avoid unused local variables such as 'parser' HTMLDocument.java wyona/wyonacms/src/java/org/wyona/lucene line 146 A method should have only one exit point, and that should be the last statement in the method IndexEnvironment.java wyona/wyonacms/src/java/org/wyona/lucene line 170 All methods are static. Consider using Singleton instead. IndexFiles.java wyona/wyonacms/src/java/org/wyona/lucene line 44 A method should have only one exit point, and that should be the last statement in the method IndexFiles.java wyona/wyonacms/src/java/org/wyona/lucene line 66 All methods are static. Consider using Singleton instead. IndexHTML.java wyona/wyonacms/src/java/org/wyona/lucene line 44 A method should have only one exit point, and that should be the last statement in the method IndexHTML.java wyona/wyonacms/src/java/org/wyona/lucene line 79 Avoid duplicate imports such as 'java.io.File' ReTokenizeFile.java wyona/wyonacms/src/java/org/wyona/lucene line 53 Avoid reassigning parameters such as 'file' ReTokenizeFile.java wyona/wyonacms/src/java/org/wyona/lucene line 117 A method should have only one exit point, and that should be the last statement in the method ReTokenizeFile.java wyona/wyonacms/src/java/org/wyona/lucene line 153 All methods are static. Consider using Singleton instead. SearchFiles.java wyona/wyonacms/src/java/org/wyona/lucene line 44 Avoid unused imports such as 'IOException' SearchFiles.java wyona/wyonacms/src/java/org/wyona/lucene line 57 A method should have only one exit point, and that should be the last statement in the method SearchFiles.java wyona/wyonacms/src/java/org/wyona/lucene line 72 All methods are static. Consider using Singleton instead. Entities.java wyona/wyonacms/src/java/org/wyona/lucene/html line 44 Avoid reassigning parameters such as 'entity' Entities.java wyona/wyonacms/src/java/org/wyona/lucene/html line 314 A method should have only one exit point, and that should be the last statement in the method Entities.java wyona/wyonacms/src/java/org/wyona/lucene/html line 330 A method should have only one exit point, and that should be the last statement in the method Entities.java wyona/wyonacms/src/java/org/wyona/lucene/html line 335 A method should have only one exit point, and that should be the last statement in the method HtmlDocument.java wyona/wyonacms/src/java/org/wyona/lucene/html line 219 A method should have only one exit point, and that should be the last statement in the method HtmlDocument.java wyona/wyonacms/src/java/org/wyona/lucene/html line 245 Switch statements should have a default label HtmlDocument.java wyona/wyonacms/src/java/org/wyona/lucene/html line 306 Deeply nested if..then statements are hard to read HtmlDocument.java wyona/wyonacms/src/java/org/wyona/lucene/html line 315 Avoid unused private fields such as 'jj_semLA' HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 79 Avoid using implementation types like 'java.util.Vector'; use the interface instead HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 89 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 240 A high ratio of statements to labels in a switch statement. Consider refactoring. HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 495 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 500 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 514 A high ratio of statements to labels in a switch statement. Consider refactoring. HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 517 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 524 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 538 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 712 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 716 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 720 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 724 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 732 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 736 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 740 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 744 Assigning an Object to null is a code smell. Consider refactoring. HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 828 A method should have only one exit point, and that should be the last statement in the method HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 915 Switch statements should have a default label HTMLParser.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1040 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 127 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 130 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 135 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 138 Avoid unused private methods such as 'jjStartNfaWithStates_0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 156 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 163 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 172 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 185 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 196 Avoid reassigning parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 205 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 207 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 215 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 220 Avoid reassigning parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 229 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 231 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 239 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 246 Avoid unused private methods such as 'jjCheckNAddStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 282 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 288 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 295 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 714 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 730 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 737 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 782 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 833 Avoid unused formal parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 844 Avoid unused private methods such as 'jjStartNfa_6' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 851 Avoid unused private methods such as 'jjStartNfaWithStates_6' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 855 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 862 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 871 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 879 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 886 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 915 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 964 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 980 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 987 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1032 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1083 Avoid unused formal parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1094 Avoid unused private methods such as 'jjStartNfa_2' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1101 Avoid unused private methods such as 'jjStartNfaWithStates_2' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1105 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1112 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1121 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1124 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1132 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1139 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1200 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1255 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1273 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1276 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1286 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1289 Avoid unused private methods such as 'jjStartNfaWithStates_5' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1300 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1307 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1316 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1329 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1334 Avoid reassigning parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1343 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1345 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1353 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1360 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1373 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1380 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1425 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1476 Avoid unused formal parameters such as 'active0' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1487 Avoid unused private methods such as 'jjStartNfa_1' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1494 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1505 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1514 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1517 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1520 Avoid unused local variables such as 'nextStates' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1528 Avoid unused local variables such as 'j' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1535 Avoid unused local variables such as 'l' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1619 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1674 Avoid unused local variables such as 'kind' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1750 Avoid unused local variables such as 'specialToken' HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1751 A method should have only one exit point, and that should be the last statement in the method HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1763 Switch statements should have a default label HTMLParserTokenManager.java wyona/wyonacms/src/java/org/wyona/lucene/html line 1766 A method should have only one exit point, and that should be the last statement in the method ParseException.java wyona/wyonacms/src/java/org/wyona/lucene/html line 136 A method should have only one exit point, and that should be the last statement in the method SimpleCharStream.java wyona/wyonacms/src/java/org/wyona/lucene/html line 295 A method should have only one exit point, and that should be the last statement in the method SimpleCharStream.java wyona/wyonacms/src/java/org/wyona/lucene/html line 466 Assigning an Object to null is a code smell. Consider refactoring. SimpleCharStream.java wyona/wyonacms/src/java/org/wyona/lucene/html line 497 Assigning an Object to null is a code smell. Consider refactoring. SimpleCharStream.java wyona/wyonacms/src/java/org/wyona/lucene/html line 498 Assigning an Object to null is a code smell. Consider refactoring. SimpleCharStream.java wyona/wyonacms/src/java/org/wyona/lucene/html line 499 Avoid unused formal parameters such as 'lexState' TokenMgrError.java wyona/wyonacms/src/java/org/wyona/lucene/html line 196 A method should have only one exit point, and that should be the last statement in the method Configuration.java wyona/wyonacms/src/java/org/wyona/net line 160 Avoid using implementation types like 'Vector'; use the interface instead ProxyConf.java wyona/wyonacms/src/java/org/wyona/net line 62 Avoid unused imports such as 'Vector' ProxyItem.java wyona/wyonacms/src/java/org/wyona/net line 52 A method should have only one exit point, and that should be the last statement in the method ProxyItem.java wyona/wyonacms/src/java/org/wyona/net line 96 A method should have only one exit point, and that should be the last statement in the method ProxyItem.java wyona/wyonacms/src/java/org/wyona/net line 98 Avoid unused imports such as 'WildcardFilter' ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 50 Avoid unused imports such as 'IOException' ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 54 Avoid unused imports such as 'StringTokenizer' ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 57 Avoid using implementation types like 'Vector'; use the interface instead ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 70 A method should have only one exit point, and that should be the last statement in the method ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 137 Avoid using implementation types like 'Vector'; use the interface instead ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 156 A method should have only one exit point, and that should be the last statement in the method ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 169 A method should have only one exit point, and that should be the last statement in the method ProxyManager.java wyona/wyonacms/src/java/org/wyona/net line 179 Avoid unused formal parameters such as 'value' SMTP.java wyona/wyonacms/src/java/org/wyona/net line 253 Avoid empty catch blocks SMTP.java wyona/wyonacms/src/java/org/wyona/net line 257 Avoid unused local variables such as 'response' WGet.java wyona/wyonacms/src/java/org/wyona/net line 111 A method should have only one exit point, and that should be the last statement in the method CrawlerEnvironment.java wyona/wyonacms/src/java/org/wyona/search/crawler line 222 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 86 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 236 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 243 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 250 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 276 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 284 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 290 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 296 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 306 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 312 A method should have only one exit point, and that should be the last statement in the method HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 318 Switch statements should have a default label HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 373 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 414 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 416 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 418 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 420 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 422 Assigning an Object to null is a code smell. Consider refactoring. HTMLHandler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 424 Avoid using implementation types like 'java.util.Vector'; use the interface instead IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 71 Avoid using implementation types like 'java.util.TreeSet'; use the interface instead IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 72 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 215 Avoid unused local variables such as 'currentURLPath' IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 236 Avoid unused local variables such as 'lastModified' IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 243 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 249 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 290 Avoid reassigning parameters such as 'url' IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 314 Avoid using implementation types like 'java.util.TreeSet'; use the interface instead IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 314 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 328 Avoid reassigning parameters such as 'url' IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 346 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 348 Assigning an Object to null is a code smell. Consider refactoring. IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 373 Assigning an Object to null is a code smell. Consider refactoring. IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 377 Assigning an Object to null is a code smell. Consider refactoring. IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 381 A method should have only one exit point, and that should be the last statement in the method IterativeHTMLCrawler.java wyona/wyonacms/src/java/org/wyona/search/crawler line 410 A method should have only one exit point, and that should be the last statement in the method DateUtil.java wyona/wyonacms/src/java/org/wyona/util line 102 A method should have only one exit point, and that should be the last statement in the method DateUtil.java wyona/wyonacms/src/java/org/wyona/util line 117 All methods are static. Consider using Singleton instead. FileUtil.java wyona/wyonacms/src/java/org/wyona/util line 44 A method should have only one exit point, and that should be the last statement in the method FileUtil.java wyona/wyonacms/src/java/org/wyona/util line 77 A method should have only one exit point, and that should be the last statement in the method FileUtil.java wyona/wyonacms/src/java/org/wyona/util line 84 Avoid empty 'if' statements FileUtil.java wyona/wyonacms/src/java/org/wyona/util line 105 A method should have only one exit point, and that should be the last statement in the method FileUtil.java wyona/wyonacms/src/java/org/wyona/util line 204 Avoid unused imports such as 'Date' GetMillis.java wyona/wyonacms/src/java/org/wyona/util line 46 Avoid unused imports such as 'ArrayList' HTML.java wyona/wyonacms/src/java/org/wyona/util line 54 A method should have only one exit point, and that should be the last statement in the method HTML.java wyona/wyonacms/src/java/org/wyona/util line 135 A method should have only one exit point, and that should be the last statement in the method HTML.java wyona/wyonacms/src/java/org/wyona/util line 150 A method should have only one exit point, and that should be the last statement in the method HTML.java wyona/wyonacms/src/java/org/wyona/util line 165 A method should have only one exit point, and that should be the last statement in the method HTML.java wyona/wyonacms/src/java/org/wyona/util line 176 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 64 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 65 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 66 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 67 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 68 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 69 Deeply nested if..then statements are hard to read HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 97 Deeply nested if..then statements are hard to read HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 118 Deeply nested if..then statements are hard to read HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 130 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 142 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 151 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 160 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 169 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 178 Avoid using implementation types like 'ArrayList'; use the interface instead HTMLHandler.java wyona/wyonacms/src/java/org/wyona/util line 187 All methods are static. Consider using Singleton instead. StringUtil.java wyona/wyonacms/src/java/org/wyona/util line 44 All methods are static. Consider using Singleton instead. URLUtil.java wyona/wyonacms/src/java/org/wyona/util line 44 Assigning an Object to null is a code smell. Consider refactoring. URLUtil.java wyona/wyonacms/src/java/org/wyona/util line 102 Assigning an Object to null is a code smell. Consider refactoring. URLUtil.java wyona/wyonacms/src/java/org/wyona/util line 105 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 98 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 129 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 132 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 137 Avoid using implementation types like 'Vector'; use the interface instead WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 153 Avoid unnecessary if..then..else statements when returning a boolean WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 157 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 158 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 160 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 175 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 180 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 182 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 192 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 195 A method should have only one exit point, and that should be the last statement in the method WildcardFilter.java wyona/wyonacms/src/java/org/wyona/util line 203 All methods are static. Consider using Singleton instead. XmlUtil.java wyona/wyonacms/src/java/org/wyona/util line 44 Avoid unused imports such as 'Attributes' XmlUtil.java wyona/wyonacms/src/java/org/wyona/util line 48 Avoid unused imports such as 'File' XmlUtil.java wyona/wyonacms/src/java/org/wyona/util line 52 Avoid unused imports such as 'OutputStreamWriter' XmlUtil.java wyona/wyonacms/src/java/org/wyona/util line 54 Avoid unused imports such as 'Writer' XmlUtil.java wyona/wyonacms/src/java/org/wyona/util line 55 Avoid unused imports such as 'FileNotFoundException' XPSFileOutputStream.java wyona/wyonacms/src/java/org/wyona/util line 50 Avoid unused local variables such as 'cl' Configuration.java wyona/wyonacms/src/java/org/wyona/xml line 169 Avoid empty 'if' statements Configuration.java wyona/wyonacms/src/java/org/wyona/xml line 175 Assigning an Object to null is a code smell. Consider refactoring. Configuration.java wyona/wyonacms/src/java/org/wyona/xml line 202 Assigning an Object to null is a code smell. Consider refactoring. Configuration.java wyona/wyonacms/src/java/org/wyona/xml line 203 A method should have only one exit point, and that should be the last statement in the method Configuration.java wyona/wyonacms/src/java/org/wyona/xml line 249 Avoid unused imports such as 'Hierarchy' DocumentHelper.java wyona/wyonacms/src/java/org/wyona/xml line 67 Avoid unused imports such as 'Logger' DocumentHelper.java wyona/wyonacms/src/java/org/wyona/xml line 68 A method should have only one exit point, and that should be the last statement in the method DocumentHelper.java wyona/wyonacms/src/java/org/wyona/xml line 234 Avoid unnecessary constructors - the compiler will generate these for you DOM4JUtil.java wyona/wyonacms/src/java/org/wyona/xml line 65 Avoid duplicate imports such as 'java.util.Properties' DOMParserFactory.java wyona/wyonacms/src/java/org/wyona/xml line 57 A method should have only one exit point, and that should be the last statement in the method DOMParserFactory.java wyona/wyonacms/src/java/org/wyona/xml line 108 A method should have only one exit point, and that should be the last statement in the method DOMParserFactory.java wyona/wyonacms/src/java/org/wyona/xml line 118 A method should have only one exit point, and that should be the last statement in the method DOMParserFactory.java wyona/wyonacms/src/java/org/wyona/xml line 122 Avoid unused imports such as 'StringTokenizer' DOMUtil.java wyona/wyonacms/src/java/org/wyona/xml line 56 A method should have only one exit point, and that should be the last statement in the method DOMUtil.java wyona/wyonacms/src/java/org/wyona/xml line 211 A method should have only one exit point, and that should be the last statement in the method DOMUtil.java wyona/wyonacms/src/java/org/wyona/xml line 219 A method should have only one exit point, and that should be the last statement in the method DOMWriter.java wyona/wyonacms/src/java/org/wyona/xml line 126 A method should have only one exit point, and that should be the last statement in the method DOMWriter.java wyona/wyonacms/src/java/org/wyona/xml line 137 A method should have only one exit point, and that should be the last statement in the method DOMWriter.java wyona/wyonacms/src/java/org/wyona/xml line 141 Avoid unused imports such as 'DocumentBuilder' NamespaceHelper.java wyona/wyonacms/src/java/org/wyona/xml line 46 All methods are static. Consider using Singleton instead. Normalize.java wyona/wyonacms/src/java/org/wyona/xml line 46 Avoid empty 'if' statements XLink.java wyona/wyonacms/src/java/org/wyona/xml line 99 All methods are static. Consider using Singleton instead. XMLEncToJavaEnc.java wyona/wyonacms/src/java/org/wyona/xml line 44 Avoid unused local variables such as 'javaencoding' XMLEncToJavaEnc.java wyona/wyonacms/src/java/org/wyona/xml line 128 A method should have only one exit point, and that should be the last statement in the method XMLEncToJavaEnc.java wyona/wyonacms/src/java/org/wyona/xml line 131 A method should have only one exit point, and that should be the last statement in the method XPath.java wyona/wyonacms/src/java/org/wyona/xml line 47 A method should have only one exit point, and that should be the last statement in the method XPath.java wyona/wyonacms/src/java/org/wyona/xml line 62 A method should have only one exit point, and that should be the last statement in the method XPath.java wyona/wyonacms/src/java/org/wyona/xml line 73 A method should have only one exit point, and that should be the last statement in the method XPath.java wyona/wyonacms/src/java/org/wyona/xml line 89 Avoid unused imports such as 'StringUtil' XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 50 Avoid unused imports such as 'XercesParser' XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 53 Avoid using implementation types like 'Vector'; use the interface instead XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 166 Avoid using implementation types like 'Vector'; use the interface instead XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 183 Avoid using implementation types like 'Vector'; use the interface instead XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 207 Avoid using implementation types like 'Vector'; use the interface instead XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 236 Avoid using implementation types like 'Vector'; use the interface instead XPointerFactory.java wyona/wyonacms/src/java/org/wyona/xml line 298 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 270 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 290 Avoid using implementation types like 'Vector'; use the interface instead XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 306 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 315 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 332 Avoid unused local variables such as 'writtenToCache' XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 345 Avoid using implementation types like 'Vector'; use the interface instead XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 569 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 620 A method should have only one exit point, and that should be the last statement in the method XPSAssembler.java wyona/wyonacms/src/java/org/wyona/xml line 635 Avoid using implementation types like 'Vector'; use the interface instead XPSInclude.java wyona/wyonacms/src/java/org/wyona/xml line 64 Avoid using implementation types like 'Vector'; use the interface instead XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 68 Assigning an Object to null is a code smell. Consider refactoring. XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 81 Avoid empty 'if' statements XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 160 Avoid unnecessary if..then..else statements when returning a boolean XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 240 A method should have only one exit point, and that should be the last statement in the method XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 241 A method should have only one exit point, and that should be the last statement in the method XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 243 A method should have only one exit point, and that should be the last statement in the method XPSSourceInformation.java wyona/wyonacms/src/java/org/wyona/xml line 248 Avoid using implementation types like 'Vector'; use the interface instead XalanXPointer.java wyona/wyonacms/src/java/org/wyona/xml/xpointer line 123 Avoid unused local variables such as 'children' XalanXPointer.java wyona/wyonacms/src/java/org/wyona/xml/xpointer line 124 Avoid using implementation types like 'Vector'; use the interface instead XPointer.java wyona/wyonacms/src/java/org/wyona/xml/xpointer line 69