Блог

Это простой класс и простая тестовая функция:

 #include <queue>
#include <string>

namespace {
    using namespace std;
}

class NameStream {
    queue<string> stream;

public:
    string operator * () { return stream.front(); }
    NameStream amp;operator   (int) { stream.pop(); return *this; }
    NameStream amp;operator   () { stream.pop(); return *this; }
    NameStream amp;operator <<(string name) { stream.push(name); }

    operator bool() { return !stream.empty(); }
};

inline void nameStreamTest() {
    NameStream amp;stream = *new NameStream;

    stream << "hi" << "hey" << "hoy";

    while (stream) {
        printf("%sn", (*stream  ).c_str());
    }
}
  

Он попадает в

 NameStream amp;operator <<(string name) { stream.push(name); }
  

внутри процедуры push queue, вот стек за пределами моего кода:

 #0  0x000b5079 in std::deque<std::string, std::allocator<std::string> >::push_back at stl_deque.h:1055
#1  0x000b50f2 in std::queue<std::string, std::deque<std::string, std::allocator<std::string> > >::push at stl_queue.h:204
#2  0x000b511c in NameStream::operator<< at NameStream.h:24
#3  0x000b520f in nameStreamTest at NameStream.h:32
  

Мой опыт в этом случае терпит неудачу. Что я делаю не так?

P.S.:

 NameStream amp;stream = *new NameStream;
  

Используется для очистки местоположения

 stream
  

объект по адресу (смещение?) 0x7d (!), который вызывает такое же исключение.

Поместите оператор return в конец вашей функции.

 NameStream amp;operator <<(string name) { stream.push(name); return *this; }
  

Добавить: неполучение допустимой ссылки / значения (при необходимости) вызовет UB, а это вызовет скрытые сбои или плохое поведение, которые часто трудно отлаживать (как в вашем случае). Поэтому никогда не игнорируйте предупреждение компилятора, если оно генерирует какое-либо.

Происходит сбой, поскольку постфиксный оператор имеет более высокий приоритет, чем operator * , поэтому, когда выполняется последняя итерация цикла, вы сначала pop удаляетесь из очереди, а затем пытаетесь выполнить, front к какому времени очередь пуста. Чтобы решить эту проблему, самый простой способ — разбить ее на два оператора типа:

 while (stream) {
        printf("%sn", (*stream).c_str());
        stream  ;
    }
  

Он попадает в

 NameStream amp;operator <<(string name) { stream.push(name); } 
  

Вы не возвращаете *this … связывая operator<<() вызовы в цепочку, второй и третий вызовы не имеют действительного адреса объекта для работы.

У вас есть ряд проблем в коде.

1. Отсутствует возврат в operator<< (как указано iammilind и другими), что вызывает ошибку времени выполнения, которую вы наблюдаете. Это можно исправить следующим образом:

    NameStream amp;operator <<(string name) { stream.push(name); return *this; }
     

2. Другая проблема связана с последовательностью вызовов в теле вашего цикла while, который:

   1. operator (int)
   2. оператор * ()

   Поэтому на последней итерации вы сначала pop() входите operator (int) , а затем пытаетесь вызвать front() in operator*() в пустой очереди, что вызовет ошибку времени выполнения.

   Самый простой способ исправить это — переписать цикл while следующим образом.

    while (stream) {
     printf("%sn", (*stream).c_str());
       stream;
   }
     

3. Утечка памяти. Вы вызываете выделение объекта NameStream с помощью new , но никогда не вызываете delete . Простым решением является размещение вашего объекта stream в стеке:

    NameStream stream;