Вираз повинен мати тип класу


82

Я деякий час не кодував на c ++, і застряг, коли намагався скомпілювати цей простий фрагмент:

class A
{
  public:
    void f() {}
};

int main()
{
  {
    A a;
    a.f(); // works fine
  }

  {
    A *a = new A();
    a.f(); // this doesn't
  }
}

2
рядок, в якому написано "цього немає", насправді нормальний, що робить ваше запитання незрозумілим.
juanchopanza

Відповіді:


166

Це вказівник, тому замість цього спробуйте:

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();

Тільки починаючи з C ++, все одно потрібно зробити це автоматизмом, щоб зрозуміти, використовувати вказівник чи посилання. У моєму конкретному випадку все, що мені потрібно було посилання, але я чомусь передав вказівник замість цього. У будь-якому разі, дякую за чітке пояснення!
Гійом М

13

Дозволити аналіз.

#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.


9

Короткий зміст : Замість a.f();цього повинно бутиa->f();

В основному ви визначили a як вказівник на об'єкт A , щоб ви могли отримувати доступ до функцій за допомогою ->оператора.

Альтернативний , але менш читається спосіб(*a).f()

a.f()міг бути використаний для доступу до f (), якби a було оголошено як: A a;


Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.