🚀 go-pugleaf

Harbinger of Doom wrote:
> De error doet zich voor in de copy-constructor van NodeHttpConnection.
> dit is'm:
> 
> NodeHttpConnection::NodeHttpConnection(class NodeHttpConnection& snhc)
> {
>      connection = snhc.connection;
>      port = snhc.port;
>      session = snhc.session; //<-- error
>      for(int i = 0; i < 23;i++)
>      {
>           url[i] = snhc.url[i];
>      }
> }
> 
> heb ik hier misschien ergens in gemist??

Je wijst hier een CInternetSession-object toe aan een ander
CInternetSession-object. Nog even afgezien van het feit of de compiler
dat al dan niet toelaat, denk ik dat dat een slecht idee is. Een sessie
is een sessie. Je zal ofwel dat object moeten delen tussen verschillende
NodeHttpConnection-instanties als je het wil hergebruiken, ofwel zal je
een nieuw moeten maken.

Als je die verschillende objecten eigenlijk niet nodig hebt, maar de
copy-constructor en operator= alleen om ze te kunnen doorgeven aan
functies, doe je er beter aan van pass-by-reference te gebruiken in
plaats van pass-by-value.

void ByValue(Node node)
{
    // ...
}

void ByReference(Node &node)
{
    // ...
}

// ...

Node node;

ByValue(node);      // de functie krijgt een kopie van node
ByReference(node);  // de functie werkt op node zelf

-- 
"Codito ergo sum"
Roel Schroeven