не был объявлен в этой области C++

Question

Зачем нужна эта ошибка в коде ниже?

class ST : public Instruction{ 
public: 
ST (string _name, int _value):Instruction(_name,_value){} 

    void execute(int[]& anArr, int aVal){ 
     //not implemented yet 
     cout << "im an st" <<endl; 
     anArr[value] = aVal; 
    } 
    virtual Instruction* Clone(){ 
     return new ST(*this); 
    } 
}; 



classes.h:81: error: ‘anArr’ was not declared in this scope 
classes.h:81: error: ‘aVal’ was not declared in this scope 

Answer 1

У вас возникла проблема с типом первого параметра вашей функции execute. Прочитайте this, чтобы узнать больше о том, как передавать массивы.

Answer 2

Попробуйте это:

недействительными выполнить (интермедиат anArr [], Int аваль)

, поскольку Вы не можете использовать массив ссылок.

Answer 3

Поскольку тип anArr недействителен.

Также вам может быть интересно использовать covariant return type по вашему методу клонирования. То есть он может вернуть указатель на ST вместо инструкции.

Answer 4

Если execute() предполагается принимать массив целых чисел, вероятно, вы должны объявить его как это:

void execute(int* anArr, int anArrLength, int aVal) 
{ 
    // ... 
} 

Обратите внимание, что есть несколько различий в ваш метод:

• anArr просеивает в качестве указатель в начало массива. Клиентский код может просто передать имя переменной массива, поскольку по определению это эквивалентно «указателю на начало массива».
• anArrLength передается для указания длины длины. Это необходимо для обеспечения того, чтобы метод execute() не получал доступ к памяти, которая находится вне границ массива (или того, что было выделено для массива). Это может привести к повреждению памяти .

Вы можете улучшить сигнатуру метода выше, добавив возвращаемое значение для указания успеха или сбоя. Это позволит коду клиента обнаруживать, были ли какие-либо проблемы. Например:

// Returns true on success, false on failure 
bool execute(int* anArr, int anArrLength, int aVal) 
{ 
    // Get "value" through whatever means necessary 
    // ... 

    if (value >= anArrLength) 
    { 
     // Out of bounds of array! 
     return false; 
    } 

    anArr[value] = aVal; 

    // Do whatever else you need to do 
    // ... 

    return true; 
}