C / C++ / MFC

Re: Use inline member function to refer different levels of classes.

24-Jan-08 20:12

Inline member function as follows.

public:
CMsgRecorder& GetMsgRecorder(void);

Class definition as follows.

class CMsgRecorder
{
public:
	void SetGroupState(char chState)
};

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 20:38

Re: Use inline member function to refer different levels of classes.

24-Jan-08 20:38

Yeah but were is the SetGroupState function defined ? There is just a declaration there. You need to provied a body for the function otherwise you'll get a linker error.
And BTW, you cannot double click on linker errors.

Also, modify your first post in order to remove the pre tags: if such longs strings we can only see a part of your message.

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 20:44

Re: Use inline member function to refer different levels of classes.

24-Jan-08 20:44

Ok here the body of SetGroupState()

void CMsgRecorder::SetGroupState(char chState)
{
	string strName = m_DiskFile.GetFileName();
}

I used this as

m_pCtlAgent->GetMsgRecorder().SetGroupState('R');

where "m_pCtlAgent" is the control agent.

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 20:50

Re: Use inline member function to refer different levels of classes.

24-Jan-08 20:50

I see. But why do you speak about inline functions ? Where is that inline coming form ? Did you forget something in your code ?

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 21:09

Re: Use inline member function to refer different levels of classes.

24-Jan-08 21:09

Why I use it, because that inline function(as I told earlier) is defined in another class. Inking to there is fine, just include header files. I guaranteed that it is work fine.

So basically this involve with three classes. I'll explain it much as follows,

CMsgRecorder& GetMsgRecorder(void);

defined in CFocusGroup.h

m_pCtlAgent->GetMsgRecorder().SetGroupState('R');

defined in CCmdLine.cpp

SetGroupState()

used in CMsgRecorder.cpp and CMsgRecorder.h

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 21:23

Re: Use inline member function to refer different levels of classes.

24-Jan-08 21:23

Sorry, but it is very confusing trying to understand you Unsure | :~

Why do you talk about inline functions if it has nothing to do with the problem and if you can guarantee that it works fine. You really need to explain your problem in a way that it is clear for people that don't see your screen.

Anyway here is your problem: you have defined a class CMsgRecorder in which you have a function (SetGroupState). You use this function somewhere else in the code and it compiles without error but fail for the linking. This is important because it tells you that the function declaration is found (so there is no problem with include files whatsoever, otherwise it would fail at compilation). The problem is that when the linker is doing its job, it cannot find a body (or a definition) for the function. You showed me the body of the function, so you've written one. The only possible 'solution' I have in mind is that you actually forgot to add the cpp file of CMsgRecorder to your project (meaning that this file is not compiled and then of course the linker cannot find the function). Are you sure it is contained in the files of your project ?

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 21:49

Re: Use inline member function to refer different levels of classes.

24-Jan-08 21:49

Yes, all files are in my project.

Also, all of these happened at compile time, not at the runtime. Frown | :(

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 22:07

Re: Use inline member function to refer different levels of classes.

24-Jan-08 22:07

Eranga Thennakoon wrote:
Also, all of these happened at compile time, not at the runtime

Well, I know that. That's wath I'm talking about in my previous post.

Eranga Thennakoon wrote:
Yes, all files are in my project.

Sorry, but then I can't help anymore. I explained the problem clearly in my previous message, so at least you have some clues to investigate.
Are you absolutely sure that the .cpp file is in your project ? I'm not talking about including the header file with an "include" statement.

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 22:22

Re: Use inline member function to refer different levels of classes.

24-Jan-08 22:22

Cedric Moonen wrote:
Are you absolutely sure that the .cpp file is in your project ?

Yes pal, this is sure. I've double checked after you ask that pal. Cry | :((

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 22:32

Re: Use inline member function to refer different levels of classes.

24-Jan-08 22:32

Is it one single project or is it a solution containing several sub-projects ?

Is it a big project ? If no, would it be possible that you send everything to me by mail (only the source and header files and the solution and project files: sln and vcproj if you are using VC2005)? You can send it to cedric_moonen at hotmail dot com.
Put all those files (and only those) in a zip file. I can have a look and see if I can fix the problem.

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 22:36

Re: Use inline member function to refer different levels of classes.

24-Jan-08 22:36

Thanks pal, it is collection of sub-projects. Actually there is 7 sub-projects. I've done all the things learn until now in a same project to use as a single one. Now the size is over 30MB, even delete all debug folders. I'll try to remove all un-wanted projects and send it to you.

Also I used Visual Studio .Net 2003.

I appreciate your help all the time...
Eranga Smile | :)

Cedric Moonen24-Jan-08 22:42

Re: Use inline member function to refer different levels of classes.

24-Jan-08 22:42

Eranga Thennakoon wrote:
it is collection of sub-projects.

That is probably your problem. This class (the CMsgRecorder) is part of one of your project but you try to use it in another one. The compiler can find the header file of this file but once it needs to link to it, it fails because it is not part of the same project. Am I right or not ?

You cannot do that, you cannot use classes from other projects. It is not because it is in the same solution that you can use everything from everywhere. It is exactly the same as if the projects were not in the same solution. Putting them together is just a matter of convenience, nothing else.

Cédric Moonen
Software developer

Charting control [v1.2]

CodingLover24-Jan-08 22:47

Re: Issues with std::vector

24-Jan-08 22:47

Cedric Moonen wrote:
That is probably your problem. This class (the CMsgRecorder) is part of one of your project but you try to use it in another one. The compiler can find the header file of this file but once it needs to link to it, it fails because it is not part of the same project. Am I right or not ?

Yes, I think so. Since I used large number of projects, it can be happened. Now first thing I'm going to do is, remove this project from the existing one and test. Let see what I can found.

I appreciate your help all the time...
Eranga Smile | :)

Issues with std::vector

Waldermort24-Jan-08 19:00

Waldermort

24-Jan-08 19:00

Is there some limitation on the vector template that I should know about?

I have a vector of 45,000 string pointers. It's a dictionary file Wink | ;)

. Anyway, I need to iterate through these strings checking if more than one definition is present on the same line. Multiple definition are seperated with the '|' character, when I find one I create a new string and push it to the end of the vector. Simple enough you would think.

So, here is my code.

vector< LPENTRY >::iterator vIter = vEntries.begin();
    
    while ( vIter != vEntries.end() )
    {
        LPSTR szEng = strchr( (*vIter)->English, '|' );
    
        if ( szEng != NULL )
        {
            // Create a new entry
            LPENTRY pEntry = new ENTRY;
    
            // Copy the contents ( the struct arranges new memory and copies the strings )
            *pEntry = *vIter;
    
            // Copy the new English
            strcpy_s( pEntry->English, MAX_LINE_LENGTH, &szEng[ 1 ] );
    
            // Shorten the old string
            *szEng = 0x00;
    
            vEntries.push_back( pEntry );
        }
    
        vIter++;
    }

The problem is, when the vector grows to a size of 61447 and when pushing a new value to the end, it causes the iterator to become invalid ( pointing to 0xfeeefeee ). Hence when trying to increase it it triggers a breakpoint. Any ideas what is going wrong?

Waldermort

Stephen Hewitt24-Jan-08 19:18

Stephen Hewitt

24-Jan-08 19:18

Firstly, see here[^] for the meaning of 0xfeeefeee.

See here[^] for a description of "iterator invalidation". Here's a quote:

[2] Memory will be reallocated automatically if more than
capacity() - size() elements are inserted into the vector. Reallocation does not change size(),
nor does it change the values of any elements of the vector. It does, however, increase capacity(),
and it invalidates [5] any iterators that point into the vector.

Steve

Re: Issues with std::vector

Waldermort24-Jan-08 19:22

Waldermort

24-Jan-08 19:22

That value of 0xfeeefeee should have given it away. I worked around the issue by throwing out the iterator and used the indexing method instead. Though thanks for the reply Smile | :)

Waldermort

ShellExecute

Paulraj G24-Jan-08 18:42

24-Jan-08 18:42

Hi...
CString str;
str = "www.google.com";
ShellExecute(NULL, "open" ,str, NULL, NULL, SW_SHOW);

this is my code....

but the error is
'ShellExecute': identifier not found.

what is the procedure for use shellexecute?

paulraj

Naveen24-Jan-08 18:44

Naveen

24-Jan-08 18:44

include "shellapi.h"

nave

[OpenedFileFinder]

Paulraj G24-Jan-08 18:52

24-Jan-08 18:52

already i included the header file...

paulraj

Don Box24-Jan-08 18:58

Don Box

24-Jan-08 18:58

include

shellapi.h enclosed with <>

(notice the '<>' instead of '""')

Come online at:-
jubinc@skype

Paulraj G24-Jan-08 19:30

24-Jan-08 19:30

i include like #include <shellapi.h>
now also same error

paulraj

Stephen Hewitt24-Jan-08 19:39

Stephen Hewitt

24-Jan-08 19:39

Show us more code. Are you including the file in the correct place?

Steve

Paulraj G24-Jan-08 19:43

24-Jan-08 19:43

#include "stdafx.h"
#include "Pdf_Example.h"

#include "Pdf_ExampleDoc.h"
#include "Pdf_ExampleView.h"
#include <shellapi.h>
...
...
...
void CPdf_ExampleView::OnBnClickedButton1()
{
CString str;
str = "www.google.com";
ShellExecute(NULL, "open" ,str, NULL, NULL, SW_SHOW);

}

thats all....
if i click a button that should be open....
shall i put instead of str, D:\\Paul.pdf?

paulraj

Naveen24-Jan-08 19:46

Naveen

24-Jan-08 19:46

Just for tracking the error. Please try like this and let me know the result...

#undef _INC_SHELLAPI<br />
#include <shellapi.h><br />
...<br />
...<br />
...<br />
void CPdf_ExampleView::OnBnClickedButton1()<br />
{<br />
CString str;<br />
str = "www.google.com";<br />
ShellExecute(NULL, "open" ,str, NULL, NULL, SW_SHOW);<br />
<br />
}

nave

[OpenedFileFinder]

Paulraj G24-Jan-08 23:53