Конструктор игнорируется

Question

Мой конструктор почему-то игнорируется. Вот мой код:

Моего класса:

class field 
{ 
private: 
    char PlayField[5][5]; 

public: 
    char o = 'o'; 
    field() 
    { 
     char PlayField[5][5] = { { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o } }; 
    } 


    void setTile(int x_val, int y_val) 
    { 
     PlayField[x_val][y_val] = 'x'; 
    } 
    char getTile(int x_val, int y_val) 
    { 
     return PlayField[x_val][y_val]; 
    } 



    /*field::~field();*/ 
}; 

Поле конструктора() должны инициализировать поле моих 4 побед с «о, и если я хочу, чтобы добавить плитки она будет й знаком, где плитка. Но если я

int main() 
{ 
    char x; 

    field FourWins; 
    //FourWins.setTile(3, 2); 
    x = FourWins.getTile(3, 2); 
    std::cout << x << std::endl; 

    return 0; 
} 

Конструктор будет игнорируются, и я получаю weired знак, который, скорее всего, только в настоящее время, где я смотрю. Позиционирование работает, потому что, если я сначала установил и x (3,2), он напечатает мне x.

Любые идеи?

Answer 1

Ideone example here

Синтаксис инициализации для char[][] вы использовали разрешено, но только при строительстве - не присвоение (и ваш пример сконструированной новую переменную, а что назначен к переменной-члена). По крайней мере, с C++ 14 вы можете сделать это следующим образом:

class field 
{ 
private: 
    char o = 'o'; 
    char PlayField[5][5]; 
public: 
    field() : PlayField{{ o, o, o, o, o }, { o, o, o, o, o }, 
         { o, o, o, o, o }, { o, o, o, o, o }, 
         { o, o, o, o, o }} 
    {} 
}; 

Рассмотрите возможность использования std::vector из std::vector или std::array из std::array вместо этого. Только вниз сторона использования std::array по сравнению с char[5][5] является то, что размеры (5x5 в вашем случае) должны быть известны во время компиляции (так же, как в вашем примере)

#include <iostream> 
#include <array> 
#include <vector> 

using namespace std; 

int main() { 
    char o='A'; 

    // C++11 or later: 
    std::array<std::array<char,5>, 5> PlayField2{{{o,o,o,o,o},{o,o,o,o,o}, 
            {o,o,o,o,o},{o,o,o,o,o},{o,o,o,o,o}}}; 

    // or: (C++11 or later) 
    std::array<std::array<char,5>, 5> PlayField; 
    for(auto& row : PlayField){ 
     for(auto& place : row){ 
      place=o; 
     } 
    } 
    cout << PlayField[2][3] << std::endl; 

    // or: (C++98 or later) 

    std::vector<std::vector<char>> PlayFieldVec(5,std::vector<char>(5,o)); 
    cout << PlayFieldVec[2][3] << std::endl; 

} 

синтаксис, который используется для инициализации массива отлично, просто убедитесь, что вы не назначаете переменную new, если вы хотите инициировать существующую.

Answer 2

Вы не можете присвоить значение массиву, подобному этому. Вы определяете локальную переменную в своем конструкторе. Вы можете сделать это в своем конструкторе.

for(int i = 0; i < 5; i++){ 
    for (int j = 0; j < 5; ++j) { 
     PlayField[i][j] = o; 
    } 
} 

Answer 3

Ошибка вы сделали это, что вы определяете Playfield дважды:

class field 
{ 
private: 
    char PlayField[5][5]; 
     ^^^^^^^^ class attribute 

public: 
    char o = 'o'; 
    field() 
    { 
     char PlayField[5][5] = { { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o } }; 
     ^^^^ this 'char' made a second PlayField local! a common mistake 
    } 

так, то инициализировать только локальный массив, а не атрибут класса один.

К сожалению -по Насколько я ноу-нет никакого способа явно инициализации массива рядом с помощью двух петель для:

for(int x=0;x<5;x++) 
    for(int y=0;y<5;y++) 
    PlayField[x][y]=o; 

В конце концов, ваш конструктор не был проигнорирован, он не имеет эффекта вы хотели :-)

Answer 4

Конструктор вызывается, но не инициализирует член CharField. Вместо этого он создает локальный массив с именем CharField (который, хотя и имеет то же имя, отличается от элемента структуры с именем CharField), инициализируя его, и этот массив больше не существует, когда конструктор завершается.

Реализация конструктора

field() 
    { 
     char PlayField[5][5] = { { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o } }; 
    } 

имеет точно такой же эффект (без изменения элемента структуры с именем CharField) как

field() 
    { 
     char unused_data[5][5] = { { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o } }; 
    } 

Единственное, что изменилось, так это имя временного массива созданный в теле конструктора.

В самом деле инициализация PlayField элемента, изменить этот застройщик, чтобы использовать список инициализатора

field() : PlayField { { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o }, { o, o, o, o, o } } 
    { // body of constructor is here 
    }; 

или (до C++, 11), где эта форма не поддерживается.

field() 
    { 
     for (int i = 0; i < 5; ++i) 
      for (int j = 0; j < 5; ++j) 
       CharField[5][5] = o; 
    };