Просто не понимаю б чём проблема!!!!!!!(char* && index)

[artush1984]

Здраствуйте!!!!! помогите пожлуста разобраца!!!!! выдоёт ошибку "инструкция по какому то адрессу обратилас к памяти по какому то адрессу памят не может быт read"

/ Class automatically generated by Dev-C++ New Class wizard
#ifndef STRINGS_H
#define STRINGS_H

/*
 * No description
 */
class strings
{
	public:
		strings(char *p_string,int p_partSize = 128);
		~strings();
		char &operator[](int index); 
		void Add(char value);
		void Print(void);
		int GetSize(){return m_logicSize;};
	private:
       		 int m_logicSize;
      		  int m_partSize;
       		 int m_realSize;
       		 char *m_string;
       		 void Reallocate(void);
};

#endif // STRINGS_H

#include <iostream>
#include "strings.h"

using namespace std;

int main(int argc, char *argv[])
{
    strings *toh = new strings("artush");
    toh->Add('a');
    toh->Print();
    std::cout<<toh->GetSize()<<std::endl;
    system("PAUSE");
    return EXIT_SUCCESS;
}

#include "strings.h"
#include<iostream>
#define _INDEX_CONTROL
// class constructor
strings::strings(char *p_string,int p_partSize)
{
        if(p_partSize<=0)
        {
                m_partSize = 32;
        }
        else
        {
                m_partSize = p_partSize;
        }
        
        for(int i =0;p_string[i] != 0;i++)
        {m_logicSize = (i+1);}
        m_realSize = m_logicSize + p_partSize;
        if(m_logicSize>0)
        {
                m_string = p_string;
        }
        else
        {
                m_string = NULL;
                m_realSize = 0;
        }
}

// class destructor
strings::~strings()
{
	if (m_string !=NULL)
        	delete []m_string;
}

char &strings::operator[](int index) ///??????????????????????????????????????
{
#ifdef _INDEX_CONTROL
    assert(index > 0 && index <= m_logicSize);
#endif
    return m_string[index];    
}
void strings::Reallocate(void)
{
    char newstrings[m_realSize + m_partSize];
    m_realSize += m_partSize;
    for(int i = 0;i < m_logicSize;i++)
    {
        newstrings[i] = m_string[i];
    }
    delete []m_string;
    m_string = newstrings;
}
void strings::Add(char value)
{
    if(m_realSize == m_logicSize)
        Reallocate();
    m_logicSize++;
    m_string[m_logicSize - 1] = value;
    
}
void strings::Print(void)
{
    for(int i = 0; i < m_logicSize;i++)
    std::cout<<m_string[i]<<std::endl;
}

Модератор: использование аттачей для исходного кода запрещено!

[vasek123]

#include <iostream>
#include "strings.h"
using namespace std;
int main(int argc, char *argv[])
{
    strings *toh = new strings("artush");
    toh->Add('a');
    toh->Print();
    std::cout<<toh->GetSize()<<std::endl;
    system("PAUSE");
    return EXIT_SUCCESS;
    delete toh;
    toh=NULL;//По моему не мешает
    вставить
}

Упс... На файл s.txt не глянул... Перепишите свой код, заключив его в теги, а то без стакана не разобраться!

[artush1984]

Вообшем создаёт обект Print()иGetSize() работают а когда дело доходит до Add('a') то ERROR!!!!!!

[artush1984]

Извени но я нечего не понял!!!!!!!Но помоему тоже должно работать

[vvviperrr]

void strings::Reallocate(void)
{
    char newstrings[m_realSize + m_partSize];
    m_realSize += m_partSize;
    for(int i = 0;i < m_logicSize;i++)
    {
        newstrings[i] = m_string[i];
    }
    delete []m_string;
    m_string = newstrings;
}

ты объявил ЛОКАЛЬНЫЙ СТАТИЧЕСКИЙ (эти слова важны, поэтому капс лок) массив newstrings. в конце своей функции ты ка бы сохраняешь указатель на этот массив. но после выхода их функции твой массив уничтожается, и указатель становится недействительным. вот и все. код писать не буду - сам потренируйся.

[artush1984]

сапсибо!!!! За совет!!!!! А За то што код не написали большое спасибо!!!!

[vasek123]

Из листинга ясно что вы размещаете класс strings в динамической памяти но, потом, по завершению программы, не освобождаете память...

[vvviperrr]

2vasek123 в данном случае это делать не обязательно, т.к конец программы, система сама освободит всю занятую приложением память

[artush1984]

сапсибо!!!! учту обязательно!!!!! !!

[Sazary]

vvviperrr +1

Но тут до Reallocate дело вообще не дойдет.
Программа вылетает на строчке

m_string[m_logicSize - 1] = value;

в функции Add. И неудивительно.

В конструкторе m_string присваивается константа:

strings::strings(char *p_string,int p_partSize)
................
m_string = p_string;

В функции Add, для данного примера, вот сюда мы не зайдем:

if(m_realSize == m_logicSize)
        Reallocate();

А перескочим прямо сюда:

m_string[m_logicSize - 1] = value;

Но m_string - указатель на константу. Вот и выкидывает.