Почему происходит ошибка при обращении к полю статического класса?

Question

[azaznioo]

Здравствуйте, есть классы с заголовками.

Класс LOGVOTING

#pragma once
#include "Fraction.h"
#include "vector"
using namespace std;
static class LogVoting
{
public:
 
    static  vector RegisteredDeputy;
    static      vector RegisteredFraction;
};

DEPUTY

#pragma once
#include "GovernmentEmployee.h"
#include "Human.h"

class Deputy : public GovernmentEmployee, public Human
{
public:
    Deputy(){ NumberOfPlaces = 0; };
    Deputy(unsigned number );
    Deputy(unsigned number, bool immunity, int age, string name, string surname, PeopleGender gender);
    virtual ~Deputy();

    unsigned get_Number();
    bool set_Number(int number);

protected:
    unsigned NumberOfPlaces;

};

FRACTION

#pragma once
#include "Deputy.h"
#include "vector"

class Fraction
{
public:
    Fraction();
    Fraction(string name, Deputy *leader,unsigned maxCountMembers);
    ~Fraction();

    bool set_Name(string name);
    string get_Name();
    unsigned get_MaxCountMembers();

    Deputy* get_Leader();
    bool set_Leader(Deputy *deputy);

    bool AddToDeputsList(Deputy* deputy);
    void DeleteDeputsFromList(Deputy * deputy);
protected:
    unsigned MaxCountMembers;
    string Name;
    vector DeputyList;
    Deputy  *Leader;
};

И при таком обращении LogVoting::RegisteredDeputy выдается куча ошибок вида:

Ошибка 12 error LNK2020: эхЁрчЁх°хээр ыхъёхьр (0A000438) "public: static class std::vector > LogVoting::RegisteredDeputy" (?RegisteredDeputy@LogVoting@@2V?$vector@PAVDeputy@@V?$allocator@PAVDeputy@@@std@@@std@@A)

Answer 1

[GavriKos]

Поставьте английскую студию - и она вполне внятно вам скажет в чем проблема. А еще укажите что вы за тип в векторе храните то.

Comments

[rich]

CP866 → CP1251. Неразрешенна лексема.

Answer 2

[Владимир Мартьянов]

Нужна инициализация static-member'ов.

Comments

[GavriKos]

А еще у вектора не задан тип хранимых данных. Или я не прав и он необязателен?

[azaznioo]

#pragma once

#include "Fraction.h"
#include
static class LogVoting
{
public:
static std::vector RegisteredDeputy;
static std::vector RegisteredFraction;
};


std::vector LogVoting::RegisteredDeputy;
std::vector LogVoting::RegisteredFraction;

Все равно ошибка

Ошибка 10 error LNK2005: "public: static class std::vector > LogVoting::RegisteredFraction" (?RegisteredFraction@LogVoting@@2V?$vector@PAVFraction@@V?$allocator@PAVFraction@@@std@@@std@@A) єцх юяЁхфхыхэ т DeputyReg.obj

[azaznioo]

GavriKos: Задан просто toster не хочет их отображать

[Владимир Мартьянов]

azaznioo: А инициализацию надо в CPP-файлы выносить.

[GavriKos]

azaznioo тогда в вопросе используйте тег для кода - и не будут куски пропадать

Answer 3

[Mercury13]

Дело в том, что static-член в теле класса — это только заголовок, говорящий, что такое есть, но не создающий ни кода, ни данных. Эквивалент extern. Где-то в CPP (т.е. в одной и только в одной единице компиляции — не в .h!) нужны…

std::vector LogVoting::RegisteredDeputy;

И так далее.

Если этим полям нужен нестандартный конструктор — пожалуйста…
std::vector LogVoting::RegisteredDeputy(42);

Comments

[Denis Seletkov]

А насчёт моего случая не можете подсказать? Такая же ошибка при компиляции "Ошибка LNK2001 неразрешенный внешний символ ""public: static int Student::cnt" (?cnt@Student@@2HA)" Проект2 C:\Users\User\Documents\Visual Studio 2015\Projects\FreeWork\Проект2\Source.obj Строка 1 ". Однако если убрать обращение к статической переменной, ошибки нет.

#include <iostream>
class Student {
public:
    static int cnt;
    Student()  {
        cnt++;
    };
};

int main(void) {
    Student::cnt = 0;
    Student s1;
    Student s2 = s1;
    return 0;
}

[Mercury13]

Denis Seletkov:
static int cnt; — это прототип. Extern.
Надо также в одной из единиц компиляции объявить int Student::cnt = 0;
А из main, соответственно, убрать.

[Denis Seletkov]

Mercury13: Объясните, пожалуйста, где это? В какой части кода? Никогда не слышал про единицы компиляции(

[Mercury13]

Единица компиляции — это CPP-файл с включёнными в него хедерами.
Имеется в виду где-то в CPP-файле, не в заголовочном.

[Denis Seletkov]

Mercury13: так это и есть .СРР файл

[Mercury13]

#include <iostream>

class Student {
public:
    static int cnt;
    Student()  {
        cnt++;
    };
};

int Student::cnt = 0;

int main(void) {
    Student s1;
    Student s2 = s1;
    std::cout << Student::cnt << std::endl;
    return 0;
}

Ну вот. Объяснишь сам, почему студент всего один?

[Denis Seletkov]

Mercury13: я просто всё лишнее убрал, чтоб проблему показать, а так всё дело в конструкторе копирования, который в данном примере не перегружен для увеличения счётчика

[Denis Seletkov]

Mercury13: спасибо, понял что надо было просто вне мэйна инициализировать

[Mercury13]

Не инициализировать, а завести!
Переменная может быть и не инициализированной.

[Denis Seletkov]

Mercury13: Понятно, спасибо. Просто в данном случае (как и в большинстве других) инициализация обязательна

[Mercury13]

Не инициализация, а объявление за пределами класса! Вот так тоже будет работать, если нет статических инициализаторов (большой бич «си с крестами», кто не знает).

int Student::cnt;

int main(void) {
    Student::cnt = 0;
…