Я деякий час не кодував на c ++, і застряг, коли намагався скомпілювати цей простий фрагмент:
class A
{
public:
void f() {}
};
int main()
{
{
A a;
a.f(); // works fine
}
{
A *a = new A();
a.f(); // this doesn't
}
}
Я деякий час не кодував на c ++, і застряг, коли намагався скомпілювати цей простий фрагмент:
class A
{
public:
void f() {}
};
int main()
{
{
A a;
a.f(); // works fine
}
{
A *a = new A();
a.f(); // this doesn't
}
}
Відповіді:
Це вказівник, тому замість цього спробуйте:
a->f();
В основному оператор .
(використовується для доступу до полів та методів об'єкта) використовується для об'єктів та посилань, тому:
A a;
a.f();
A& ref = a;
ref.f();
Якщо у вас тип вказівника, вам слід спершу його розмежувати, щоб отримати посилання:
A* ptr = new A();
(*ptr).f();
ptr->f();
a->b
Позначення, як правило , просто скорочення для (*a).b
.
Банку operator->
можна перевантажити, що особливо використовується розумними вказівниками. Коли ви використовуєте розумні вказівники , ви також використовуєте ->
для посилання на загострений об'єкт:
auto ptr = make_unique<A>();
ptr->f();
Дозволити аналіз.
#include <iostream> // not #include "iostream"
using namespace std; // in this case okay, but never do that in header files
class A
{
public:
void f() { cout<<"f()\n"; }
};
int main()
{
/*
// A a; //this works
A *a = new A(); //this doesn't
a.f(); // "f has not been declared"
*/ // below
// system("pause"); <-- Don't do this. It is non-portable code. I guess your
// teacher told you this?
// Better: In your IDE there is prolly an option somewhere
// to not close the terminal/console-window.
// If you compile on a CLI, it is not needed at all.
}
Як загальну пораду:
0) Prefer automatic variables
int a;
MyClass myInstance;
std::vector<int> myIntVector;
1) If you need data sharing on big objects down
the call hierarchy, prefer references:
void foo (std::vector<int> const &input) {...}
void bar () {
std::vector<int> something;
...
foo (something);
}
2) If you need data sharing up the call hierarchy, prefer smart-pointers
that automatically manage deletion and reference counting.
3) If you need an array, use std::vector<> instead in most cases.
std::vector<> is ought to be the one default container.
4) I've yet to find a good reason for blank pointers.
-> Hard to get right exception safe
class Foo {
Foo () : a(new int[512]), b(new int[512]) {}
~Foo() {
delete [] b;
delete [] a;
}
};
-> if the second new[] fails, Foo leaks memory, because the
destructor is never called. Avoid this easily by using
one of the standard containers, like std::vector, or
smart-pointers.
Як правило: якщо вам потрібно керувати пам’яттю самостійно, як правило, вже існує суперурочний менеджер або альтернатива, яка відповідає принципу RAII.
Короткий зміст : Замість a.f();
цього повинно бутиa->f();
В основному ви визначили a як вказівник на об'єкт A , щоб ви могли отримувати доступ до функцій за допомогою ->
оператора.
Альтернативний , але менш читається спосіб(*a).f()
a.f()
міг бути використаний для доступу до f (), якби a було оголошено як:
A a;
a
є покажчиком. Вам потрібно використовувати ->
, ні.