Re: C4430 - C / C++ / MFC Discussion Boards

T.RATHA KRISHNAN31-Jul-08 3:30

31-Jul-08 3:30

I've to declare a function which is already defined in another class. I don't want to inherit from the existing class. The function is under a namespace in the first file. When I declare the function in the new class, I got the following errors:

d:\character\test.h(84) : error C4430: missing type specifier - int assumed. Note: C++ does not support default-int
d:\character\test.h(84) : error C2143: syntax error : missing ',' before '*'

I've declared the function in the public section only. How do I declare the function in the second class?

Re: C4430

Iain Clarke, Warrior Programmer31-Jul-08 3:33

Iain Clarke, Warrior Programmer

31-Jul-08 3:33

He didn't ask you to repeat your question - he asked you for some actual CODE.

If you can't post the actual code, try recreating the problem with similar code - this is probably a good idea anyway. I'm guessing your code is more than 10 lines long anyway. And by recreating the problem, you may even see your own soloution.

If you only post the error message, it is very unlikely anyone can or will help you.

Iain.

Re: C4430

T.RATHA KRISHNAN31-Jul-08 3:41

T.RATHA KRISHNAN

31-Jul-08 3:41

The function I want to declare is inside a namespace. It's coded like this:

namespace dtAI
{
 void AICharacter::GoToWaypoint(float dt, const Waypoint* pWaypoint)
   {  
      //simple... just rotate to the waypoint over time and set a
      //positive velocity to go there
      mCharacter->RotateToPoint(pWaypoint->GetPosition(), dt * 3.0f);            

      //osg::Vec3 pVector = pWaypoint->GetPosition() - GetPosition();

      //pVector [2] = 0.0f;

      //float dir = std::abs(osg::RadiansToDegrees(atan2(pVector[0], -pVector[1])) - mCharacter->GetRotation());

      //while(dir >= 360.0f) dir -= 360.0f;

      //std::cout << dir << std::endl;
      //if(dir < 5.0f)
      //{
      mCharacter->SetSpeed(-float(mSpeed));

      if(!mCharacter->IsAnimationPlaying("Run"))
      {
         mCharacter->ClearAllAnimations(0.5f);
         mCharacter->PlayAnimation("Run");
      }
      //}
      //else
      //{
      //   mCharacter->SetVelocity(0.0f);
      //}
   }
}

This function is declared in the header file under public section as below :

void GoToWaypoint(float dt, const Waypoint* pWaypoint);

I've to declare the function in some other header file. I can comment the function in the first file. When I declare the function in the second file, I got the above mentioned errors. What to do?

Re: C4430

Iain Clarke, Warrior Programmer31-Jul-08 4:34

Iain Clarke, Warrior Programmer

31-Jul-08 4:34

OK, I'm not much clearer, as you haven't show all the places.

But to expand on Alan Balkany's answer to you, I've done a similar solution to the one he proposes (though I didn't know it had a name).

//Iain.h

namespace Iain // let's use one, even though I normally don't
{
    class CIain
    {
        virtual BOOL DoSomethingExciting (char a, int b, void *p) = 0;
    }

    CIain *GetAnIainInterface ();  // declared so we can use it at some point later.
};

Now we have a pure virtual class and interface. Let's try and use it!

// UsingIain.c

#include "Iain.h"

...
    CIain *pIain = Iain::GetAnIainInterface ();
    ASSERT(pIain); // hey, I wanna exist!
    
    pIain->DoSomethingExciting ('a', 97, NULL);
...

So far so good - but we need to actually implement it somewhere!

// Iain.c

#include "Iain.h"

class CIainImpl : public Iain::CIain
{
....
      virtual BOOL DoSomethingExciting (char a, int b, void *p)
      {
          return TRUE; //ok, not all that exciting
      }
};


CIainImpl  m_Iain;   // singleton instance.

Iain::GetAnIainInterface ()
{
    return &m_Iain;
}

I hope that helps a little as an example!

Iain.

Re: C4430

CPallini31-Jul-08 3:38

CPallini

31-Jul-08 3:38

See here [^] (Iain expresses what I think even better than I'm able to Roll eyes | :rolleyes:

).

Probably the errors occurred have nothing to do with you requirement to include in a new class a method having the same name of some other class's method. This is the reason we're asking for the code. For sure we have no intention at all to copy it.
Smile | :)

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.

This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke

[My articles]

Re: C4430

T.RATHA KRISHNAN31-Jul-08 3:44

T.RATHA KRISHNAN

31-Jul-08 3:44

Sorry Gentlemen! I told my company's policies only. I never think anybody will copy my code. I feel really sorry if I hurt anybody.

Re: C4430

CPallini31-Jul-08 3:46

CPallini

31-Jul-08 3:46

Don't worry. Just kidding. Anyway if you can post a simple sample code emulating your troubled one...
Smile | :)

Re: C4430

T.RATHA KRISHNAN31-Jul-08 3:51

T.RATHA KRISHNAN

31-Jul-08 3:51

Here it is:

The function I want to declare is inside a namespace. It's coded like this:

namespace dtAI
{
void AICharacter::GoToWaypoint(float dt, const Waypoint* pWaypoint)
   {  
      //simple... just rotate to the waypoint over time and set a
      //positive velocity to go there
      mCharacter->RotateToPoint(pWaypoint->GetPosition(), dt * 3.0f);            

      //osg::Vec3 pVector = pWaypoint->GetPosition() - GetPosition();

      //pVector [2] = 0.0f;

      //float dir = std::abs(osg::RadiansToDegrees(atan2(pVector[0], -pVector[1])) - mCharacter->GetRotation());

      //while(dir >= 360.0f) dir -= 360.0f;

      //std::cout << dir << std::endl;
      //if(dir < 5.0f)
      //{
      mCharacter->SetSpeed(-float(mSpeed));

      if(!mCharacter->IsAnimationPlaying("Run"))
      {
         mCharacter->ClearAllAnimations(0.5f);
         mCharacter->PlayAnimation("Run");
      }
      //}
      //else
      //{
      //   mCharacter->SetVelocity(0.0f);
      //}
   }

   osg::Vec3 AICharacter::GetPosition() const
   {
     osg::Matrix mat = mCharacter->GetMatrixNode()->getMatrix();  
     osg::Vec3 pos(mat(3, 0), mat(3, 1), mat(3, 2));
     return pos;
   }
}

This function is declared in the header file under public section as below :

void GoToWaypoint(float dt, const Waypoint* pWaypoint);

I've to declare the function in some other header file. I can comment the function in the first file. When I declare the function in the second file, I got the above mentioned errors. What to do?

Re: C4430

CPallini31-Jul-08 4:02

CPallini

31-Jul-08 4:02

T.RATHA KRISHNAN wrote:
void GoToWaypoint(float dt, const Waypoint* pWaypoint);

The above declaration doesn' correnspond to the definition, unless it is enclosed inside the same namespace(dtAI) and class(AICharacter).
What do you need to do? Declare another function with the same signature outside the namespace?

Re: C4430

T.RATHA KRISHNAN31-Jul-08 4:08

T.RATHA KRISHNAN

31-Jul-08 4:08

I've to declare the function in another file not only the signature but I need the functionalites also in the function that is to be declared by me in the second file.(I can comment the function inside the first file i.e included in the namespace).

Re: C4430

Alan Balkany31-Jul-08 3:42

Alan Balkany

31-Jul-08 3:42

Ahh, it was so much clearer the second time!

You might consider the Facade design pattern (http://en.wikipedia.org/wiki/Facade_pattern[^]) to give one class access to a single function another class, without coupling the classes together.

The basic idea is to declare an interface that defines the one function, and make the existing class derive from that interface. The class that uses this function sees an instance of the existing class as the interface -- not the whole class.

Excel automation compatibility problem

followait31-Jul-08 1:26

followait

31-Jul-08 1:26

I import class from excel.exe in office 2003.
In the program, I use only basic operations to manipulate .xls files.
Will the program be compatible with .xls file create with other versions of Excel, for example Excel 2000 or 2007?

system

Re: Excel automation compatibility problem

vikas amin31-Jul-08 11:34

vikas amin

31-Jul-08 11:34

Excel 2007 has many changes i.e ribbon bar , but as far as just programming excel is concern , think a program with excel 2003 should work with excel 2007 .

Vikas Amin

My First Article on CP" Virtual Serail Port "[^]

modified on Thursday, July 24, 2008 5:33 PM

exe name

sunit531-Jul-08 0:30

sunit5

31-Jul-08 0:30

Is there any API through which processes can be enumerated on the basis of Port they are using.
Thanx

Sunit

never say die

Re: exe name

sunit531-Jul-08 2:27

sunit5

31-Jul-08 2:27

i feel this API <GetExtendedTcpTable> will work for me.If any body has another solution please let me know

Sunit Blush | :O

never say die

Re: exe name

Randor 31-Jul-08 10:17

Randor

31-Jul-08 10:17

You can use a combination of AllocateAndGetTcpExTableFromStack[^] or AllocateAndGetUdpExTableFromStack[^] from Iphlpapi.dll to find all open TCP/UDP ports and the PID of the process owning the handle. This technique should be used on OS prior to XP-SP2. Not available on 95/98/Me.

On XP SP2/Vista/2003/2008 the function GetExtendedTcpTable [^]is available as the prior mentioned functions have been deprecated.

Once you have the ports and the corresponding PID you can use CreateToolhelp32Snapshot toolhelp [^]functions to walk all available processes and find the name of the executable matching the PID.

Structures you will need:
MIB_TCPROW_OWNER_PID[^]
MIB_TCPTABLE_OWNER_PID[^]

Best Wishes,
-David Delaune

Re: exe name

sunit531-Jul-08 18:37

sunit5

31-Jul-08 18:37

Thanx a lot

never say die

Problem with friend keyword

T.RATHA KRISHNAN30-Jul-08 23:39

T.RATHA KRISHNAN

30-Jul-08 23:39

Hi!
I've two unrelated classes named "character" and "test". I've defined a variable "flag" inside test. I've to access this variable from the other class(i.e from "character"). For this I've declared the class "character" as friend in test.h in the private section.
i.e

friend class AICharacter;

and I've also forward declared the class test in character like this:

class test;

I've used the following code to access the variable:

test friend_test = new test();
  friend_test->char_flag1 = false;

But it shows the following errors:

(i)error C2079: 'friend_TestAI' uses undefined class 'TestAI'
(ii)error C2512: 'TestAI' : no appropriate default constructor available
(iii) left of '->char_flag1' must point to class/struct/union/generic type

Is there any other way to access a variable which is declared inside some other class?

Re: Problem with friend keyword

CPallini30-Jul-08 23:49

CPallini

30-Jul-08 23:49

Code missing in your post.
I don't see any TestAI class in posted code.
Smile | :)

Re: Problem with friend keyword

T.RATHA KRISHNAN30-Jul-08 23:56

T.RATHA KRISHNAN

30-Jul-08 23:56

Sorry. It's advised by my organisation not to post original code. Here is the modified post:

Hi!
I've two unrelated classes named "character" and "test". I've defined a variable "flag" inside test. I've to access this variable from the other class(i.e from "character"). For this I've declared the class "character" as friend in test.h in the private section.
i.e

<pre>friend class character;</pre>

and I've also forward declared the class test in character like this:

<pre>class test;</pre>

I've used the following code to access the variable:

<pre> test friend_test = new test();
friend_test->char_flag1 = false;</pre>

But it shows the following errors:

(i)error C2079: 'friend_test' uses undefined class 'test'
(ii)error C2512: 'test' : no appropriate default constructor available
(iii) left of '->char_flag1' must point to class/struct/union/generic type

Is there any other way to access a variable which is declared inside some other class?

Re: Problem with friend keyword

CPallini31-Jul-08 0:03

CPallini

31-Jul-08 0:03

It looks like you simply didn't include the 'test' header file. Forward declarations aren't enough when you need an instance of the object.
Smile | :)

Re: Problem with friend keyword

T.RATHA KRISHNAN31-Jul-08 0:11

T.RATHA KRISHNAN

31-Jul-08 0:11

I've included "test.h" also in character.h. Now it shows the following errors:

error C2440: 'initializing' : cannot convert from 'test *' to 'test'
No constructor could take the source type, or constructor overload resolution was ambiguous
.\character.cpp(124) : error C2819: type 'test' does not have an overloaded member 'operator ->'
d:\backupdelta3d\charactercontrol\test.h(40) : see declaration of 'test'
did you intend to use '.' instead?
.\character.cpp(124) : error C2232: '->test::char_flag1' : left operand has 'class' type, use '.'
Build log was saved at "file://d:\BackupDelta3D\AICharacterControl\Release\BuildLog.htm"
CharacterAnimation - 3 error(s), 0 warning(s)

Shall I use extern instead of friend? Which is good?

Re: Problem with friend keyword

CPallini31-Jul-08 0:17

CPallini

31-Jul-08 0:17

T.RATHA KRISHNAN wrote:
error C2440: 'initializing' : cannot convert from 'test *' to 'test'

Well, the compiler is (of course) right, change the wrong lines (I earlier overlooked the mistake)

test friend_test = new test();
friend_test->char_flag1 = false;

test friend_test;
friend_test.char_flag1 = false;

or to

test * pfriend_test = new test();
pfriend_test->char_flag1 = false;
delete pfriend_test;

Re: Problem with friend keyword

T.RATHA KRISHNAN31-Jul-08 0:21

T.RATHA KRISHNAN

31-Jul-08 0:21

OK.Thank u. What is ur opinion on using extern inplace of friend keyword?

Re: Problem with friend keyword

CPallini31-Jul-08 0:36

CPallini

31-Jul-08 0:36

T.RATHA KRISHNAN wrote:
OK.Thank u.

You're welcome.

T.RATHA KRISHNAN wrote:
What is ur opinion on using extern inplace of friend keyword?

They are unrelated.

BTW: I would use an accessor methods (get/set) for exposing the variable.

Smile | :)

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.