Question

我在C ++上很安静，目前正在学习理解智能指针。因此，我目前正在开发一个用于插入，搜索和删除歌曲的小程序控制程序......出于学习目的习惯这些东西= ）

这是我的代码：

Song.hpp

#pragma once
#include <vector>
#include <memory>
#include <string>

class Song
{

public:
    typedef std::unique_ptr<Song> pSong;

public:
    Song();
    ~Song();
    void setTitle(std::string title);
    void setArtist(std::string artist);
    void checkSong(std::string item, int iterator);
    void get();

private:
    std::string _title;
    std::string _artist;
};

Song.cpp

#include "Song.hpp"
#include <iostream>



Song::Song()
{
}


Song::~Song()
{
}

void Song::setTitle(std::string title)
{
    _title = title;
}

void Song::setArtist(std::string artist)
{
    _artist = artist;
}

void Song::checkSong(std::string item, int iterator)
{
    if (_artist == item || _title == item)
    {
        std::cout << "Item found on Slot: " << iterator << std::endl;
    }
    else 
    {
        std::cout << "No item found!" << std::endl;
    }
}

void Song::get()
{
    std::cout << _artist << " - " << _title << std::endl;
}

Main.cpp的

#include <iostream>
#include <vector>
#include <algorithm>
#include <memory>
#include "Song.hpp"

//prototype
void IntVector();
void SongVector();
Song* setSong(std::string title, std::string artist);
void find(std::string item, std::vector<Song::pSong> v);


std::vector<Song::pSong> SongList;

int main()
{
    int k;
    SongVector();
    std::cin >> k;
    return 0;
}

void IntVector()
{
    // Create Vector
    std::vector<std::unique_ptr<int>> v;

    // Create a few unique_ptr<int> instances and fill them with ints
    v.push_back(std::unique_ptr<int>(new int(30)));
    v.push_back(std::unique_ptr<int>(new int(600)));
    v.push_back(std::unique_ptr<int>(new int(200)));
    v.push_back(std::unique_ptr<int>(new int(20)));
    v.push_back(std::unique_ptr<int>(new int(200)));
    v.push_back(std::unique_ptr<int>(new int(160)));
    v.push_back(std::unique_ptr<int>(new int(4)));
    v.push_back(std::unique_ptr<int>(new int(5)));
    v.push_back(std::unique_ptr<int>(new int(315)));


    // define vector<int> for storing values of the unique_ptr
    std::vector<int> intList;

    for (int i = 0; i < v.size(); i++)
    {
        // get memory-adress of each element
        auto result = v[i].get();
        // store value of result-pointer in Vector
        intList.push_back(*result);
        std::cout << *result << std::endl;
    }

    // Sort int of new Vector
    std::sort(intList.begin(), intList.end());

    // Loop through intList and cout
    for (int i = 0; i < intList.size(); i++)
    {
        std::cout << intList[i] << std::endl;
    }

}

void SongVector()
{

    Song* first = setSong("Afroki","Steve Aoki");
    Song* secound = setSong("Hype", "Steve Aoki");
    Song* third = setSong("Madness", "Steve Aoki");
    Song* fourth = setSong("Cake Face", "Steve Aoki");
    SongList.push_back(Song::pSong(first));
    SongList.push_back(Song::pSong(secound));
    SongList.push_back(Song::pSong(third));
    SongList.push_back(Song::pSong(fourth));

    for (const auto& song : SongList)
    {
        song->get();
    }

    find("Madness", SongList);
}

Song* setSong(std::string title, std::string artist)
{
    Song* song = nullptr;
    song = new Song;
    song->setArtist(artist);
    song->setTitle(title);
    return song;
}


void find(std::string item, std::vector<Song::pSong> v)
{

    int i = 0;
    for (const auto& song : v)
    {
        song->checkSong(item,i);
        i++;
    }
}

我收到了以下错误：

std::unique_ptr<Song,std::default_delete<_Ty>>::unique_ptr(const std::unique_ptr<_Ty,std::default_delete<_Ty>> &)': attempting to reference a deleted function
1>          with
1>          [
1>              _Ty=Song
1>          ]

我发现，这个错误只发生在调用我的find（...） - 方法时，所以我猜测那里的某个地方是我的错误，但我只能t find out, what I做错了。非常感谢您的帮助。

Answer 1

std::unique_ptr提供唯一所有权（名称），这意味着除了其他人之外，您无法复制std::unique_ptr的实例 - 这意味着共享所有权。当您按值传递std::vector<std::unique_ptr<whatever>>时，您将创建一个矢量实例的副本，该副本会尝试复制每个元素。最简单的解决方案是通过const引用传递std::vector实例（因为您无意修改向量）：

void find( const std::string &item, const std::vector<Song::pSong>& v);

除了修复你的问题传递（const）引用对于非平凡的对象更有效，所以你也可以将它用于std::string

在您的intVector()函数中：

for (int i = 0; i < v.size(); i++)
    {
        // get memory-adress of each element
        auto result = v[i].get();
        // store value of result-pointer in Vector
        intList.push_back(*result);           
        std::cout << *result << std::endl;
     }

你真的不需要获得原始指针，只需使用std::unique_ptr本身：

for (int i = 0; i < v.size(); i++)
{
    // get smart pointer for each element
    const auto &result = v[i];
    // store value of result-pointer in Vector
    intList.push_back(*result);
    std::cout << *result << std::endl;
}

Answer 2

void find(std::string item, std::vector<Song::pSong> v)

您需要通过引用传递向量。添加＆amp;。

void find(std::string item, std::vector<Song::pSong>& v)

不要忘记也改变功能的原型。

在vector <unique_ptr <x>＆gt;上获得错误v

2 个答案: