qq_32046433
arowana_
采纳率66.7%
2015-11-02 02:55 浏览 1.5k

代码出错,请帮助修改。

编译环境vs2010,c++

#include "stdafx.h"
#include <iostream>
#include <cstring>
using namespace std;

class Words{
public:
    Words(char *s)
    {
        len=strlen(s);
        str=new char[len+1];
        strcpy_s(str,len,s);
    }
    void disp();
    char operator[](int n);
private:
    int len;
    char *str;
};
char Words::operator[](int n)
{
    if(n<0||n>len-1)
    {
        cout<<"数组下标超界!\n";
        return ' ';
    }
    else
        return *(str+n);
}


void Words::disp()
{
    cout<<str<<endl;
}

int main()
{
    Words word("This is C++book.");
    word.disp();
    cout<<"第1个字符:";
    cout<<word[0]<<endl;
    cout<<"第16个字符:";
    cout<<word[15]<<endl;
    cout<<"第26个字符:";
    cout<<word[25]<<endl;
    return 0;
}
  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享
  • 邀请回答

2条回答 默认 最新

  • u014493323 xpbobcsdn 2015-11-02 03:31

    str=new char[strlen(s)+1];这句话怎么注释了,如果你不给指针分配内存,那就是个野指针了,根本不知道指针指向什么空间,你直接拷贝肯定有问题

    点赞 评论 复制链接分享
  • John_ToString John_ToString 2015-11-10 01:45

    通过调试发现 strcpy_s(str,len,s);这行代码出了问题...很明显你对strcpy_s这个函数的理解不是很深。第二个参数代表的是缓冲区的大小,即缓
    冲区的大小应该大于源字符串的长度至少为一,这也正是strcpy_s较strcpy函数的安全之处,你也看到了你的缓冲区大小不够的时候编译器是
    制止的...所以正确的修改方法是将len换为len+1,这样缓冲区大小就够了,因为你的len长度没有包含'\0'

    点赞 评论 复制链接分享

相关推荐