C++のコンストラクタを使った自動ポインタでoperator関数の使い

C++のコンストラクタを使った自動ポインタでoperator関数の使い方で分からないところがあります。
環境下はVisual C++でC/C++のWin32コンソールアプリケーションを使って行っています。
下記のコードで実行させています。やっていることは文字列を反転させて表示させるだけのことです。

#include "stdafx.h"
#include <string>
#include <iostream>

class AutoPtr {
char *ptr;
public:
AutoPtr():ptr(0) {
}
~AutoPtr() {
delete [] ptr;
}
//
char *operator=(char *ptr) {
delete [] this->ptr;
this->ptr = ptr;
return this->ptr;
}
operator char *(){
return ptr;
}
char &operator[](int index) {
return ptr[index];
}
};

void reverse(char *str)
{
int i, n;
AutoPtr work;

n = strlen(str);
work = new char[n+1];
strcpy(work, str);
for(i=0; i<n; i++) {
str[i] = work[n-i-1];
}
printf("%s\n", str);
}


int _tmain(int argc, _TCHAR* argv[])
{
reverse("ABCDEFG");
return 0;
}

これを実行させると、reverse関数のfor()文の、str[i] = work[n-i-1];を実行させた所で実行エラーになってしまいます。その前の、strcpy(work, str);でworkにstrの内容が正常にコピーされているところまでは確認できています。operator関数の、

char &operator[](int index) {
return ptr[index];
}

で、operator[]はAutoPtrを配列のように扱っているはずなのですが、何故かstr[i] = work[n-i-1];
の所で実行エラーになってしまいます。
operaror関数の書き方が悪いのか、何が原因なのか分かりかねています。御経験のあるかたは、御教示いただけたらと思っています。
よろしくお願い致します。

No.1 ベストアンサー 回答者： magicalpass 回答日時： 2010/07/24 22:45

ポインタ変数strは何をさしていますか？

reverse関数のパラメータだから、"ABCDEFG"という文字列リテラルの先頭ポインタですよね。
文字列リテラルは定数値です。書き換えることは出来ません。

この回答への補足

reverse("ABCDEFG")は、

char *str = "ABCDEFG";
reverse(str);
と同じで駄目のようです。

char str[] = "ABCDEFG";
reverse(str);

とするとOKのようです。（文字型配列へのポインタとしてstrを定義する）

ありがとうございました。

補足日時：2010/07/25 01:21