Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Technical Support
 Issue with Refactoring Extract Method

You must be registered to post a reply.
Click here to register.

Screensize:
UserName:
Password:
Format: BoldItalicizeUnderlineStrikethrough Align leftCenterAlign right Insert horizontal ruleUpload and insert imageInsert hyperlinkInsert email addressInsert codeInsert quoted textInsert listInsert Emoji
   
Message:

Forum code is on.
Html is off.

 
Check to subscribe to this topic.
   

T O P I C    R E V I E W
nabulke Posted - Sep 21 2009 : 07:36:45 AM
Hello,

edit:
this problem is probably related to:
http://www.visualassist.com/forum/topic.asp?whichpage=1&TOPIC_ID=8686#34666
if yes, I apologize for the double post.


I have the following problem using the extract method refactoring with the 'extract to source' option checked:

if there is no newline after the last closing bracket in the C++ cpp file, the function declaration will be inserted INSIDE the last function of that cpp. To clarify things I attach an example showing the file before and after the refactoring. This happens not with all cpp files, but with most (those that use namespaces in .hpp, not sure yet). The resulting source code is not compilable.


Before refactoring:
[....]
BOOL CBild::IstFehlerortSuchergebnis(const CString& strFehlerortDBID) const
{
return m_pMapFESGefunden->Lookup(strFehlerortDBID, strValue);
}<End of file here>

After refactoring:
[....]
BOOL CBild::IstFehlerortSuchergebnis(const CString& strFehlerortDBID) const
{
return m_pMapFESGefunden->Lookup(strFehlerortDBID, strValue);

// INSERTS DECLARATION HERE, INSIDE LAST METHOD!
BOOL MyMethod( const CString& strFehlerortDBID );
}

// DEFINITION AT END OF FILE
BOOL CBild::MyMethod( const CString& strFehlerortDBID )
{
...
return TRUE;
}<End of file here>


Thanks in advance for any help

Horst


my setup:
- Visual Assist X 10.5.1727.0
- Visual Studio 2008 English
- Language C++
1   L A T E S T    R E P L I E S    (Newest First)
accord Posted - Sep 21 2009 : 4:41:42 PM
I believe this is

case=20072

I suggest to wait for the fix, and then check it again.

© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000