Ошибка с контейнером мап

@Kloody · Регистрация: 12.10.2019

Author24 — интернет-сервис помощи студентам

Столкнулся с проблемой. При компилировании программы - выводит ошибку, думаю это связано с контейнером map.
При удалении строчек из функции create, файла .h - всё запускается

Главный файл .cpp:

C++

#include <iostream> 
#include <SFML/Graphics.hpp>
#include <vector>
#include <map> 
 
#include "Anime.h"
 
using namespace sf;
using namespace std;
 
int main()
{
    RenderWindow window(VideoMode(1000, 800),"SHIT!");
 
    Texture Shrek;
    Shrek.loadFromFile("images/AnimeMan.png");
 
    Name_Animation NameAnim;
    NameAnim.create("walk", Shrek, 0, 0, 80, 110, 2, 0.005, 40);
 
    while (window.isOpen())
    {
        Event event;
        while (window.pollEvent(event))
        {
            if (event.type == Event::Closed)
                window.close();
        }
    }
 
}

Файл .h

C++

#include <iostream>
#include <string>
#include <vector>
#include <map> 
#include <SFML/Graphics.hpp>
 
using namespace sf;
using namespace std;
 
class Hero_Animation
{
public:
    vector<IntRect> FALSE_Mirror, TRUE_Mirror;
    float AnimationFrame, speed; bool MIRROR, PlayAnimation;
    Sprite HeroSprite;
 
    Hero_Animation(Texture &HeroTexture, int x, int y, int w, int h, int N_Anim, float SPEED, int Personnel_Size = 0)
    {
        speed = SPEED;
        HeroSprite.setTexture(HeroTexture);
 
        AnimationFrame = 0; PlayAnimation = true; MIRROR = false;
 
        for (int i = 0; i < N_Anim; i = i + 1)
        {
            FALSE_Mirror.push_back(IntRect(x + i * Personnel_Size, y, w, h));
            TRUE_Mirror.push_back(IntRect(x + i * Personnel_Size + w, y, -w, h));
        }
    }
 
    void Pocess_Frame(float time)
    {
        int i;
 
        if (!PlayAnimation) { return; }
        AnimationFrame = AnimationFrame + speed * time;
 
        if (AnimationFrame > FALSE_Mirror.size()) { AnimationFrame = AnimationFrame - FALSE_Mirror.size(); }
 
        i = AnimationFrame;
 
        HeroSprite.setTextureRect(FALSE_Mirror[i]);
 
        if (MIRROR) { HeroSprite.setTextureRect(TRUE_Mirror[i]); }
    }
};
 
class Name_Animation
{
public:
    
    String AnimationFrame_Name; //Имя действующей аинмации
    map<String, Hero_Animation> Sheet_With_Animation;
 
    Name_Animation()
    {}
 
    void create(String name, Texture &HeroTexture, int x, int y, int w, int h, int N_Anim, float SPEED, int Personnel_Size = 0)
    {
        Sheet_With_Animation[name] = Hero_Animation(HeroTexture, x, y, w, h, N_Anim, SPEED, Personnel_Size);
        AnimationFrame_Name = name;
    }
 
 
};

Алексей1153 · 12.10.2019, 10:04

а какую ошибку то пишут ?

@Kloody · 12.10.2019, 10:20 **[ТС]**

Алексей1153, Код ошибки = 0x80070002

Алексей1153 · 12.10.2019, 10:30

Сообщение от Kloody

При компилировании программы - выводит ошибку

- при компилировании где ошибка то ? Не в рантайме

0x80070002 - Не удается найти указанный файл.

и текст ошибки ?

@Kloody · 13.10.2019, 07:16 **[ТС]**

Алексей1153, 1>c:\program files (x86)\microsoft visual studio\2017\professional\vc\tools\msvc\14.16.27023\include\tuple(1305): error C2512: Hero_Animation::Hero_Animation: нет подходящего конструктора по умолчанию

Алексей1153 · 13.10.2019, 07:44

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

второе стреляет вот тут

C++

1	Sheet_With_Animation[name]

используй метод Sheet_With_Animation.insert()

@Kloody · 13.10.2019, 07:56 **[ТС]**

Алексей1153, спасибо, что откликаешься. Как сделать по умолчанию - не знаю.
Но почему Sheet_With_Animation.insert() , а не Sheet_With_Animation.emplace(), ведь данные я вношу с файла .cpp.
Да и вроде так правильно - Sheet_With_Animation[name], пересмотрел гайдов по этим случаям.

Алексей1153 · 13.10.2019, 08:32

вот тут требуется вызов конструктора по умолчанию (внутри оператора [] мапы это требуется)

C++

1	Sheet_With_Animation[name]

вставку можно произвести без конструктора по умолчанию так

C++

1	Sheet_With_Animation.insert(std::make_pair(name, Hero_Animation(HeroTexture, x, y, w, h, N_Anim, SPEED, Personnel_Size)));

конструктор по умолчанию, если нужен, будет примерно такой

C++

class Hero_Animation
{
public:
    std::vector<IntRect> FALSE_Mirror;
    std::vector<IntRect> TRUE_Mirror;
    float AnimationFrame;
    float speed;
    bool MIRROR;
    bool PlayAnimation;
    Sprite HeroSprite;
 
   //конструктор по умолчанию
   Hero_Animation()
    :AnimationFrame(0)
    ,speed(0)
    ,MIRROR(false)
    ,PlayAnimation(false)
   {
   }
 
    //конструктор с параметрами
    Hero_Animation(Texture &HeroTexture, int x, int y, int w, int h, int N_Anim, float SPEED, int Personnel_Size = 0)
    {
    }
...
...

@Kloody · 13.10.2019, 10:35 **[ТС]**

Алексей1153, то есть, мне нужно удалить конструктор с параметром и вставить по умолчанию? Или ничего не удалять и просто вставить конструктор по умолчанию?

Запись по моему пониманию

C++

class Hero_Animation
{
public:
    vector<IntRect> FALSE_Mirror, TRUE_Mirror;
    float AnimationFrame, speed; bool MIRROR, PlayAnimation;
    Sprite HeroSprite;
 
    Hero_Animation()
        :AnimationFrame(0)
        , speed(0)
        , MIRROR(false)
        , PlayAnimation(false)
    {
 
    }
 
    Hero_Animation(Texture &HeroTexture, int x, int y, int w, int h, int N_Anim, float SPEED, int Personnel_Size = 0)
    {
        speed = SPEED;
        HeroSprite.setTexture(HeroTexture);
 
        AnimationFrame = 0; PlayAnimation = true; MIRROR = false;
 
        for (int i = 0; i < N_Anim; i = i + 1)
        {
            FALSE_Mirror.push_back(IntRect(x + i * Personnel_Size, y, w, h));
            TRUE_Mirror.push_back(IntRect(x + i * Personnel_Size + w, y, -w, h));
        }
    }
 
    void Pocess_Frame(float time)
    {
        int i;
 
        if (!PlayAnimation) { return; }
        AnimationFrame = AnimationFrame + speed * time;
 
        if (AnimationFrame > FALSE_Mirror.size()) { AnimationFrame = AnimationFrame - FALSE_Mirror.size(); }
 
        i = AnimationFrame;
 
        HeroSprite.setTextureRect(FALSE_Mirror[i]);
 
        if (MIRROR) { HeroSprite.setTextureRect(TRUE_Mirror[i]); }
    }
};
 
class Name_Animation
{
public:
    
    String AnimationFrame_Name; //Имя действующей аинмации
    map<String, Hero_Animation> Sheet_With_Animation;
 
    Name_Animation()
    {}
 
    void create(String name, Texture &HeroTexture, int x, int y, int w, int h, int N_Anim, float SPEED, int Personnel_Size = 0)
    {   
        Sheet_With_Animation.insert(std::make_pair(name, Hero_Animation(HeroTexture, x, y, w, h, N_Anim, SPEED, Personnel_Size)));
 
        AnimationFrame_Name = name;
    }
 
 
 
};

Алексей1153 · 13.10.2019, 11:06

Kloody, если используешь insert , то конструктор по умолчанию в данном случае не нужен. Но можешь его и оставить, он не будет мешать. С ним твой прежний вариант получится, кстати, тоже. То есть с вызовом оператора []

@Kloody · 13.10.2019, 11:24 **[ТС]**

Алексей1153, если я использую инсерт, но удаляю конструктор по умолчанию, то программа не работает и выдает ошибку, если не удаляю - все работает. Не понимаю как это работает. Вы мне помогли, спасибо. Дальше, надеюсь, разберусь сам.

Алексей1153 · 13.10.2019, 11:36

Kloody, работает это так - если конструктор используется в программе, он должен быть объявлен )

Ошибка с контейнером мап

Решение

Решение

Алексей1153 фрилансер 5498 / 5094 / 1047 Регистрация: 11.10.2019 Сообщений: 13,341
	12.10.2019, 10:04	2
	а какую ошибку то пишут ? 0

@Kloody 1 / 1 / 0 Регистрация: 12.10.2019 Сообщений: 18
	12.10.2019, 10:20 [ТС]	3
	Алексей1153, Код ошибки = 0x80070002 0

Алексей1153 фрилансер 5498 / 5094 / 1047 Регистрация: 11.10.2019 Сообщений: 13,341
	12.10.2019, 10:30	4
	Сообщение от Kloody При компилировании программы - выводит ошибку - при компилировании где ошибка то ? Не в рантайме 0x80070002 - Не удается найти указанный файл. и текст ошибки ? 0

@Kloody 1 / 1 / 0 Регистрация: 12.10.2019 Сообщений: 18
	13.10.2019, 07:16 [ТС]	5
	Алексей1153, 1>c:\program files (x86)\microsoft visual studio\2017\professional\vc\tools\msvc\14.16.27023\include\tuple(1305): error C2512: Hero_Animation::Hero_Animation: нет подходящего конструктора по умолчанию 0

@Kloody 1 / 1 / 0 Регистрация: 12.10.2019 Сообщений: 18
	13.10.2019, 07:56 [ТС]	7
	Алексей1153, спасибо, что откликаешься. Как сделать по умолчанию - не знаю. Но почему Sheet_With_Animation.insert() , а не Sheet_With_Animation.emplace(), ведь данные я вношу с файла .cpp. Да и вроде так правильно - Sheet_With_Animation[name], пересмотрел гайдов по этим случаям. 1

Алексей1153 фрилансер 5498 / 5094 / 1047 Регистрация: 11.10.2019 Сообщений: 13,341
	13.10.2019, 11:06	10
	Kloody, если используешь insert , то конструктор по умолчанию в данном случае не нужен. Но можешь его и оставить, он не будет мешать. С ним твой прежний вариант получится, кстати, тоже. То есть с вызовом оператора [] 0

@Kloody 1 / 1 / 0 Регистрация: 12.10.2019 Сообщений: 18
	13.10.2019, 11:24 [ТС]	11
	Алексей1153, если я использую инсерт, но удаляю конструктор по умолчанию, то программа не работает и выдает ошибку, если не удаляю - все работает. Не понимаю как это работает. Вы мне помогли, спасибо. Дальше, надеюсь, разберусь сам. 0

Алексей1153 фрилансер 5498 / 5094 / 1047 Регистрация: 11.10.2019 Сообщений: 13,341
	13.10.2019, 11:36	12
	Сообщение было отмечено Kloody как решение Решение Kloody, работает это так - если конструктор используется в программе, он должен быть объявлен ) 0