ошибка такова



вот код:
m_string.h

m_string.cpp

main


собственно вопрос вот в чём.Если в string.cpp поменять

на


то всё чудно работает - конструктор преобразования работает как надо ...

но всё так не понимаю почему так неправильно - объясните плз почему.

статические переменные инициализируются при первом входе в их область видимости. Другими словами, в твоем случае она инициализируется при первом входе в метод operator(). Причем, возможно аж до того, как s будет выделена память.

Кстати, твой код вообще какой-то странный. У тебя утечка памяти. Чтобы ее не было, перепиши так:

Этот код вызовет предупреждение, так как ты пользуешься некорректным преобразованием строкового литерала к неконстантному указателю. Чтобы предупреждения не было, конструктор m_string должен быть таким:
Чтобы не было утечек памяти тебе необходимо так же реализовать конструктор копирования и оператор присваивания. Так как оператор присваивания лучше всего реализуется через метод swap (m_string(param).swap(*this); return *this;) - обмен содержимым двух объектов, то рекомендую сначала сделать его.

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

а в чём утечка памяти то?

и всё таки очень интересно что значит этот BLOCK_TYPE......

у меня какая -то обще методическая ошибка как я понимаю..........

Добавлено через 7 минут и 52 секунды


а как это работает? ни встречал пока такой формы

Вот тут вы выделяете память но не освобождаете.

аа...понял но эта ошибка не в утечке....

неужели никто не знает?!

Это сообщение отредактировал(а) fuckingniger - 11.10.2010, 16:13

fuckingniger, я уже написал в чем ошибка. В том, что статическая переменная инициализируется чем-то непонятным, лежащим в s. Так как это происходит ДО выполнения тела функции, в которой она объявлена.

ну эта ошибка  меня убивает просто ...

переписал функцию operator+   как глобальную (как друг класса) чтоб можно было вызывать  в любом порядке

и опять та же ошибка хотя старую (operator() )я не вызываю

привожу код заново чтоб всё ясно было

m_string.h

m_string.cpp

main.cpp



ошибка та же самая...

создается новый объект типа m_string путем вызова подходящего по параметрам конструктора. Можно даже писать:

Добавлено через 1 минуту и 12 секунд

!  bsa

fuckingniger, ты читать умеешь? Я уже давно объяснил в чем у тебя проблема!

ну я дебагом просто проходил и всё нормально инициализируется

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

Компилятор сам делает конструктор копирования, просто присваивая поля объектов друг другу. Таким образом вы имеете несколько указателей на одну и ту же область данных и несколько раз ее освобождаете.

То, что без static работает, можно отнести на оптимизацию компилятора (или даже фичу языка - тут не возьмусь утверждать), когда он "проталкивает" возвращаемую переменную из функции наружу - со static, очевидно, так сделать нельзя - чтобы внешний код не повредил статичные данные функции.

Попробуйте сделать m_string& operator()(int).

Добавлено @ 16:42
bsa, вы не правы, по-моему. Бабах происходит из-за дабл фрии.

Добавлено через 7 минут и 1 секунду
fuckingniger,

>ну я дебагом просто проходил и всё нормально инициализируется 

Да там и без дебага понятно, что инициализируется нормально. static-данные инициализируеются не просто при входе в зону видимости, а когда до них черед выполнения доходит.

Это сообщение отредактировал(а) djamshud - 11.10.2010, 16:46

djamshud 

по ходу да ...ща попробую ...забыл что при возвращении по значению вызывается конструктор копии(хоть и узнал это часов 30 назад всего!)

Да, с оптимизациями и фичами я прогнал.

Тут просто указатель s преобразуется в один объект m_string, после чего благополучно уничтожится. Это все равно что сделать так:

Тут как раз наглядно видно, как "протолкнется" объект. А так

опять все сдохнет.

Это сообщение отредактировал(а) djamshud - 11.10.2010, 17:02

Конструктор копий:


Оператор+


Теперь вроде работает.

не совсем понял эту фразу....А так спасибо большое - после  написания конструктора копии всё запахало как надо

Добавлено через 2 минуты и 31 секунду

сдохнет из=за того что конструктор присваивания по умолчанию просто скопирует указатели(они будут указывать на 1 динамически выделенную память?)