Что не так в следующем фрагменте?

Question

Я знаю, что это довольно серьезное нарушение, но что это?

class xyz 
{ 
    void function1() 
    { 
     cout<<"in class"; 
    } 
}; 

int main() 
{ 
    xyz s1 = new xyz(100); 
    xyz s2 = s1; 
    s2.function1(); 
    delete s1; 
    return 0; 
} 

Что-то не так в распределении памяти, используя new. Я верю, но я не могу понять, что это за фундаменталь и резолюция.

Answer 1

new возвращает указатель (xyz *) на объект, а не объект, так что вы должны исправить тип s1:

xyz* s1=new xyz(100); 

и вызвать метод через указатель, вы должны использовать оператор ->:

s1->function1(); 

, который является эквивалентом разыменования указателя и вызова метода на объекте:

(*s1).function1(); 

Answer 2

Вы не можете назначить T*T (патологические случаи в стороне).

xyz * s1 = new xyz(); 
xyz * s2 = s1; 
s2->function1(); 
delete s1; 
return 0; 

еще лучше, не использовать голый new и delete и использовать смарт-указатель:

auto s1 = make_unique<xyz>(); 
xyz * s2 = s1.get(); // non-owning pointer 
s2->function1(); 
// no explicit delete necessary