TEXT_RESOURCE_HTML_R0

[Astea]

wiki page: TEXT_RESOURCE_HTML_R0 - effort: 1d

[meddle]

• About the code:
  • The assertEquals(buf.toString(),this.text); method has special arguments the first one is the EXPECTED one the second the ACTUAL, your case is not that one.
  • The indent your code, it's hardly readable that way.
  • Write your JavaDoc to be readable, put one new line between it's overview and parameters, indent it, write it carefully to be helpful, the good JavaDoc can be used as design documentation as well.
  • I think you should test your LogicR3 handler as well with SystemTest.
  • The current test of yours is not good, may be it will be no automatic, that method -> AppLogic.get().userCreateNewBook(mw.appView().get()); will make you use your BookPropertiesDialog, that's not good, in the SystemTest class there is a method for creating new book without it, you can use it in the SystemTest that you will write, and it's code when fixing this test that will test only the HtmlUtility logic and not use this many modules.
  • Are you diana or Diana, the author identificator is important thing.
  • Blocks like bodies of "if"s even if they are one row are surrounded with curly brackets.
  • Think of better names for your variables, one letter names are bad practice...
  • Flying JavaDoc is bad thing
  • I know that you are familiar with that page GoodCodeExamples, I don't know why don't you want to write your code using it, the other people do it and their reviews pass easily.
  • Why you write this twice : htmlResource.text().set(text);
• About the design:
  • So I explained why your test is a dummy test above.
  • Why you named this HtmlLogic.USER_ADD_HTML_FRAME, the user will import HTML to a text frame, may be HtmlLogic.USER_IMPORT_HTML is the better name.
  • Somebody have to fix the fact that the text related frames continue to be created by the FrameFactory, instead like the other resources, why don't you do it (optional).
  • Other thing like that is the repeating "for" cycle for finding the next contentLocation of the new frames, all the people just copy it. It can be extracted to a helper method.
  • Why the originUrl is a field and no property, if you want to continue that way you must explain why.
  • I think the methods of this HtmlUtility should be static, do you need special instantiation of it every time when you execute the html import?
  • What about the getCorrespondingVal method isn't it better for it to use enumeration of the html styles and switch operator. I mean this set of if-else statements is too large. I mean you can create enumeration for the possible (for now) html view attributes, it will be better.
  • You use the javax.swing.text library, you can explain more in details it's use and give a link to it's JavaDoc.
• And again you are the design owner, write your name in the ticket field, take more seriously our process it is important for the good product and documentation.

2p (1h)

[deyan]

Batch update from file query-obsoleted.csv