?
Solved

C2512 "no appropriate default constructor available for a class with constructor taking no parameters

Posted on 2010-11-22
11
Medium Priority
?
1,196 Views
Last Modified: 2012-05-10
error C2515 'CTextSignLine' : no appropriate default constructor available is occuring when m_SignLineArray = new CTextSignLine[m_pSignParams->m_nNumLines];

There is a default constructor that takes no parameters and everyone that has had this problem for gets to add a default constructor... So what ridiculous mistake am I making.
<(")
//TextSignLine.h
class CTextSignLine : public CObject
{
   public:
      CTextSignLine();
      ~CTextSignLine();
   ...
};
//TextSignLine.cpp
CTextSignLine::CTextSignLine()
{
   m_pSignParams = NULL;
   m_hWndParent = NULL;
   m_pBrSignBackground = NULL;
}
//TextSign.h
class CTextSign : public CObject
{
   private:
      CTextSignLine *m_pSignLineArray;
      ...
   public:
      CTextSign();
      ...
};
//TextSign.cpp
CTextSign::CTextSign(...)
{
   ...
   m_SignLineArray = new CTextSignLine[m_pSignParams->m_nNumLines];
   ...
}

Open in new window

0
Comment
Question by:graber
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
11 Comments
 
LVL 40

Accepted Solution

by:
evilrix earned 1500 total points
ID: 34192244
Can you show the CTextSignLine in its entirety please? don't assume that code you don't think isn't important isn't. For example, does that class have any constant or reference members? If so is the default constructor correctly initialising them? You may get this error if that isn't the case. What about CObject, does that have a default constructor too?

As an aside, the way you initialise your members in the constructor is inefficient. Use an initialiser list.
http://www.parashift.com/c++-faq-lite/ctors.html#faq-10.6
CTextSignLine::CTextSignLine():
   m_pSignParams(0),
   m_hWndParent(0),
   m_pBrSignBackground(0) 
{
}

Open in new window

0
 
LVL 2

Assisted Solution

by:graber
graber earned 0 total points
ID: 34192539
Thanks Evilrix.  I concur with your comment... at this point were trying to port some very old visual studio code to Visual Studio 2008.

I found the problem.  I had a forward declaration in the header but no corresponding header (TextSignLine.h) in the cpp (TextSign.cpp).  I was expecting to see an "unknown variable" but this wasn't going to happen because of the forward declaration.

I must be getting tired... Sorry about that.
//TextSignLine.h
...
class CTextSignLine : public CObject
{
   public:
      CTextSignLine();
};

//TextSignLine.cpp
CTextSignLine::CTextSignLine()
{
   m_pSignParams = NULL;
   m_hWndParent = NULL;
   m_pBrSignBackground = NULL;
}

//TextSign.h
class CSignFontMgr;
class CTextSignParams;
class CTextSignLine;

class CTextSign : public CObject
{
   friend class CTextSignPreview;
   private:
      ...
   public:
      CTextSign();
      ...
}
//TextSign.cpp
#include "TextSignLine.h" //<---- here was the culprit
CTextSign::CTextSign(...)
{
   ...
   m_SignLineArray = new CTextSignLine[m_pSignParams->m_nNumLines];
   for (int i = 0; i < m_pSignParams->m_nNumLines; ++i)
   {
      ...
   }
   ...
}

Open in new window

0
 
LVL 2

Expert Comment

by:calin131
ID: 34193163
You basically need a default constructor that gets as parameters the number of lines.
Something like:
CTextSignLine:: CTextSignLine( int numberOfLines )
{// set numberOfLines to your class attribute and then call your default
 // constructor without parameters or do what ever you need to initialize your object
}
0
VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

 
LVL 86

Expert Comment

by:jkr
ID: 34193669
>>You basically need a default constructor that gets as parameters the
>>number of lines.

You are aware that 'default constructor' and 'parameter' are mutually exclusive? ;o)
0
 
LVL 40

Expert Comment

by:evilrix
ID: 34194446
>> I must be getting tired... Sorry about that.

No worries. Pleased you resolved the issue.
0
 
LVL 2

Author Comment

by:graber
ID: 34196049
jkr
>> You are aware that 'default constructor' and 'parameter' are mutually exclusive? ;o)

As I understand it the default constructor has no parameters.  But this old dog could very be missing something important.  What do you have?

As the code sits there wasn't anything wrong with the constructor.  The forward declaration of CTextSignLine meant that the class instantiating it knew what it was but now how to 'Construct' it, ergo 'No appropriate constructor could be found.  Once I add the header for CTextSignLine to the cpp that was doing the constructing it was able to find an appropriate constructor.  Evil did I nail that.  

I've been doing this for some time and it still amazes me what I have yet to learn.  Thanks guys for all the support.  The turn around on this was great.
Gregg
0
 
LVL 40

Expert Comment

by:evilrix
ID: 34196151
>> As I understand it the default constructor has no parameters

Strictly speaking a default constructor is one that doesn't require arguments but it can have parameters as long as there are default values that mean the arguments are optional.

Both of these are default constructors and a class would only be able to have one of the other.

myclass(){}

myclass(bool b = false){}

>> Evil did I nail that.  

As I said before, there can be a number of reasons for this error and they are not all obvious without being able to see all the code. The lack of a constructor definition for a class that has been forward declared being one possible case.
0
 
LVL 2

Author Comment

by:graber
ID: 34196200
Yep that one evil that one escaped me....  
Thanks Evil ... I wish I could have shown you more of the source but I'm a contractor and even more sensitive to showing the customers stuff.
 
0
 
LVL 40

Expert Comment

by:evilrix
ID: 34213648
Since I've given you a good explanation as to what was generating this error I'm not sure I see why you would close it with only your own post as the solution. A more appropriate way to close this would be a split and possibly a B grade since my advice assisted you in resolving the issue rather than actually resolving it for you.

See the following link or the grading guidelines and how they should be applied.

http://www.experts-exchange.com/help.jsp#hs=29&hi=403
"If it requires you to do a little more research, or figure out one more piece of code, then it's worth a B."

If you disagree that's fine but I'd appreciate your thoughts on why.
0

Featured Post

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Unlike C#, C++ doesn't have native support for sealing classes (so they cannot be sub-classed). At the cost of a virtual base class pointer it is possible to implement a pseudo sealing mechanism The trick is to virtually inherit from a base class…
Introduction This article is the first in a series of articles about the C/C++ Visual Studio Express debugger.  It provides a quick start guide in using the debugger. Part 2 focuses on additional topics in breakpoints.  Lastly, Part 3 focuses on th…
The viewer will learn how to user default arguments when defining functions. This method of defining functions will be contrasted with the non-default-argument of defining functions.
The viewer will learn how to clear a vector as well as how to detect empty vectors in C++.
Suggested Courses
Course of the Month8 days, 22 hours left to enroll

764 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question