![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
MP9 DiscussionQ: Jeffrey Votteler - June 28th, 2007 - Can we use the test cases the do not Scamper as reference to see if our refactorings still work?Also for part 1, should we refactor the RetrieveContents class also (instead of just moved the method out of FtpUrl)? A: Nicholas Chen - June 28, 2007 1) Yes, you can use the SUnit test that directly grabs the page without using Scamper. 2) I am actually more interested in seeing what kinds of refactoring you could do IN ADDITION to the ones suggested. I would like to hear your explanations on it. This is an exercise to play around with the different kinds of refactoring you can perform. Elaine Savino - June 30, 2006: I am getting an error even when just testing before changing any code. The Scamper system window opens and in the main part of the window has an error message of: MessageNotUnderstood: Dictionary >> reset. UPDATE: I got a new image and now it works. I'm not sure what was different that caused the issue in the first place. Des Iorgova - June 28,2006 - For 2) roughtly how many methods should we expect to extract? Just a comment on the Refactoring Browser: I installed the Refactoring Browser and as a result Squeak is very slow and keeps crashing. Paul Adamczyk - June 28, 2006 - 3 or 4. As for RB, make sure that you install it only in a clean image, that way it will not affect any of your code. If you're still looking for a project idea, fixing the Refactoring Browser might be something to consider. All the background that is needed is programming languages concepts and basic knowledge of compilers. ELaine Savino - June 28, 2006: How can we get the refactoring browser? Is this necessary for MP9? I watched the lecture with the demo of the browser where Prof. Johnson said we could get it from Squeak Map, but I'm not sure where to find this. Could you please put a link, if we need one, along with any pertinent instructions on this page? Thanks. Paul Adamczyk - June 28, 2006 - Make sure to start from a clean image. Say open...->SqueakMap Package Loader. I'm getting intermittent errors in my image, but when it works, it opens a SqueakMap Package Loader. In top-left panel, select Refactoring Browser for 3.8 (3.8.41). Right-click and say install. When it finishes, say open...->Refactoring Browser. This will open the RB browser, which looks similar to the regular browser. Now you can use "extract method" and other refactorings directly from the right-click menu. Note that GUI of the Refactoring Browser is really bad. To do an "extract method" you need to type in the method name in the top panel of the dialog box, then accept it (Alt-S). You won't get any error messages if you do things wrong, but when everything is OK, the OK button (bottom left) will turn white. It's not required that you use the Refactoring Browser for MP9, because it's very badly implemented in Squeak. You should try it out to see how it works and you may find it useful for the MP as well. Link to this Page
|