C / C++ / MFC

Mario M.10-Jan-02 9:28

10-Jan-02 9:28

I have other source codes which uses vector and compiles fine, I think there is something which was changed by MS in this new VC++ release...

James R. Twine10-Jan-02 10:15

James R. Twine

10-Jan-02 10:15

Mario M. wrote:
I have other source codes which uses vector and compiles fine, I think there is something which was changed by MS in this new VC++ release...

I have seen the same thing happen when people moved off of VC++ 6.0's STL code to Dinkumware's STL or STLPort. The reason I suggested is still a possibility. So again, can we get a code snippet (not a code archive) that shows the code around, and including the problem (include things like variable declarations, STL typedefs, etc.)

Peace!

-=- James.

James R. Twine10-Jan-02 8:51

James R. Twine

10-Jan-02 8:51

Mario M. wrote:
cannot convert from 'NETRESOURCE *' to 'std::vector<_Ty,_Ax>::_Tptr

Misuse of STL?

Peace!

-=- James.

Christian Graus10-Jan-02 9:19

10-Jan-02 9:19

My first guess is someone tried to return an iterator from a function that wants to return a pointer, which VC6 probably would be more likely to let you get away with.

As has been said, it is kind of pointless asking these questions if you're not going to post some code.

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Mario M.10-Jan-02 9:35

10-Jan-02 9:35

Hi,
You can find the complete source code project here http://www.codeproject.com/dialog/selectcomputerdialog.asp

Mario

Christian Graus10-Jan-02 9:47

10-Jan-02 9:47

I'm not going to reboot into W2000, run .Net ( which I still don't know my way around ), reboot again, redial the internet, when you can just post the offending line of source yourself.

To be honest, if it wasn't that much hassle, I would do it gladly, although I still think you should have posted the source that is crashing, but to test under .Net is simply too much hassle for me.

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Mario M.10-Jan-02 10:22

10-Jan-02 10:22

The error comes from the first line here...
----
vector <CNETRESOURCE> vecnrDomains;
if (!EnumDomains(NULL, vecnrDomains))
return false;

for (vector<CNETRESOURCE>::const_iterator ci = vecnrDomains.begin();
ci != vecnrDomains.end(); ++ci)
vecstrDomains.push_back(ci->GetRemoteName());
------

I am not the author of these sources, someone else posted it to codeguru.com, and it works fine with VC++ 6.

-
Mario

Christian Graus10-Jan-02 11:01

10-Jan-02 11:01

Mario M. wrote:
The error comes from the first line here...
----
vector vecnrDomains;
if (!EnumDomains(NULL, vecnrDomains))
return false;

for (vector::const_iterator ci = vecnrDomains.begin();
ci != vecnrDomains.end(); ++ci)
vecstrDomains.push_back(ci->GetRemoteName());
------
You mean declaring the vector ? Your <> pairs were lost because you didn't post 'as-is' ( see the check box below as you're typing your post ), but whatever is in the vector: does it have a copy constructor ?

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Mario M.10-Jan-02 11:34

10-Jan-02 11:34

I have modified the post to 'as-is'. And here is the vector class...
also you can read again the previous message with 'as-is'...

class CSelectComputerNetwork
{
public:
CSelectComputerNetwork();
virtual ~CSelectComputerNetwork();

public:
static bool GetDomains(vector<basic_string<TCHAR> >& vecstrDomains);
static bool GetServers(const basic_string<TCHAR>& strDomain, vector<basic_string<TCHAR> >& vecstrServers);

private:
//NETRESOURCE structure wrapper used to save text data that freed by WNetEnumClose(_);
class CNETRESOURCE
{
public:
CNETRESOURCE(const NETRESOURCE& nr):
m_nr(nr)
{
m_strLocalName = nr.lpLocalName ? nr.lpLocalName : _T("");
m_strRemoteName = nr.lpRemoteName ? nr.lpRemoteName : _T("");
m_strComment = nr.lpComment ? nr.lpComment : _T("");
m_strProvider = nr.lpProvider ? nr.lpProvider : _T("");

m_nr.lpLocalName = new TCHAR [m_strLocalName.size() + 1];
_tcscpy(m_nr.lpLocalName, m_strLocalName.c_str());

m_nr.lpRemoteName = new TCHAR [m_strRemoteName.size() + 1];
_tcscpy(m_nr.lpRemoteName, m_strRemoteName.c_str());

m_nr.lpComment = new TCHAR [m_strComment.size() + 1];
_tcscpy(m_nr.lpComment, m_strComment.c_str());

m_nr.lpProvider = new TCHAR [m_strProvider.size() + 1];
_tcscpy(m_nr.lpProvider, m_strProvider.c_str());

}

CNETRESOURCE(const CNETRESOURCE& rhs):
m_nr(rhs.m_nr),
m_strLocalName(rhs.m_strLocalName),
m_strRemoteName(rhs.m_strRemoteName),
m_strComment(rhs.m_strComment),
m_strProvider(rhs.m_strProvider)
{
m_nr.lpLocalName = new TCHAR [m_strLocalName.size() + 1];
_tcscpy(m_nr.lpLocalName, m_strLocalName.c_str());

m_nr.lpRemoteName = new TCHAR [m_strRemoteName.size() + 1];
_tcscpy(m_nr.lpRemoteName, m_strRemoteName.c_str());

m_nr.lpComment = new TCHAR [m_strComment.size() + 1];
_tcscpy(m_nr.lpComment, m_strComment.c_str());

m_nr.lpProvider = new TCHAR [m_strProvider.size() + 1];
_tcscpy(m_nr.lpProvider, m_strProvider.c_str());
}

~CNETRESOURCE()
{
delete [] m_nr.lpLocalName;
delete [] m_nr.lpRemoteName;
delete [] m_nr.lpComment;
delete [] m_nr.lpProvider;
}

public:
NETRESOURCE* operator &()
{
return &m_nr;
}

operator NETRESOURCE() const
{
return m_nr;
}

void CorrectRemoteName()
{
m_strRemoteName = m_strRemoteName.substr(2);//skip 2 leading slashes

delete [] m_nr.lpRemoteName;

m_nr.lpRemoteName = new TCHAR [m_strRemoteName.size() + 1];
_tcscpy(m_nr.lpRemoteName, m_strRemoteName.c_str());
}

basic_string<TCHAR> GetRemoteName() const
{
return m_strRemoteName;
}

private:
NETRESOURCE m_nr;

private:
basic_string<TCHAR> m_strLocalName;
basic_string<TCHAR> m_strRemoteName;
basic_string<TCHAR> m_strComment;
basic_string<TCHAR> m_strProvider;

};
//NETRESOURCE structure wrapper

private:
static bool EnumDomains(NETRESOURCE* pnr, vector<CNETRESOURCE>& vecnrDomains);
static bool EnumServers(NETRESOURCE* pnr, vector<CNETRESOURCE>& vecnrServers);
};

Christian Graus10-Jan-02 11:42

10-Jan-02 11:42

If it's a vector of CNETRESOURCE, then maybe the problem is absence of operator NETRESOURCE in CNETRESOURCE, but to be honest, I'm guessing because I can't test it.

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Mario M.10-Jan-02 11:51

10-Jan-02 11:51

CNETRESOURCE does have an operator NETRESOURCE... Cry | :((

else it would not compile in VC++ 6.0

Thanks for your time.

-
Mario

Tim Smith10-Jan-02 12:15

Tim Smith

10-Jan-02 12:15

The two NETRESOURCE operators need to be removed. They will cause all sorts of problems with standard STLs

NETRESOURCE* operator &()
  {
      return &m_nr;
  }

  operator NETRESOURCE() const
  {
      return m_nr;
  }

Specificall,y the operator & is causing the problems. The & operator is used heavily in standard STL. When STL applies the & operator, it is expecting a reference to the class, not a NETRESOURCE *, thus the problem.

Tim Smith
Descartes Systems Sciences, Inc.

Mario M.10-Jan-02 12:33

10-Jan-02 12:33

Thanks!

, this removes the error, but now I have to find something to replace the operator, since I get this error...

SelectComputerNetwork.cpp(58) : error C2664: 'CSelectComputerNetwork::EnumServers' : cannot convert parameter 1 from 'CSelectComputerNetwork::CNETRESOURCE *' to 'NETRESOURCE *'
Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or function-style cast

-
Mario

Tim Smith10-Jan-02 12:49

Tim Smith

10-Jan-02 12:49

Just make a new routine to get that resource. IMHO, I don't understand why NETRESOURCE was made private when the operator & provided unrestricted access to the member. I would just make NETRESOURCE public and be done with it.

Tim Smith
Descartes Systems Sciences, Inc.

Mario M.10-Jan-02 13:25

10-Jan-02 13:25

I will try to make a new routine to get NETRESOURCE, I also made NETRESOURCE & CNETRESOURCE public, but I get the same error...

Thanks for your time.