憋屈的执行情况string类

Question

我的目标是创建一个返回的第i个元素的方法（从零开始，就像阵列），它被称为对String的。如果要求的位置超出字符串范围，此方法应打印出错误消息并返回空字符（\ 0）。

char String::element(int i) const 
{ 
    if (i < m_str1) 
    { 
     cout << s[i]; // also the s here is undefined 
    } 
    else 
    { 
     cout << "Error" << endl; 
    } 
    return (0); 
} 

所以我只是想知道这是正常的，或者我需要增加更多的，我怎么能解决这个不确定的变量？

Answer 1

为了使其正常工作，您需要将s更改为指向字符数组的成员变量。你没有提供的String类定义，所以很难说什么是成员变量的名字。

你也应该改变

char String::element(int i) const 

要么

char String::element(size_t i) const 

或

char String::element(unsigned int i) const 

这是因为你的字符数组不应该用负索引值来访问。如果你不改变i无符号价值，你需要确保它是等于或大于零，这不应该被允许反正。玩具也应该改变m_str1到size_t或unsigned int如果它已经不是因为字符串不应该具有负的长度。

应用这些建议将使String和element()看起来像下面这样...

class String 
{ 
    unsigned int m_str1; // length of character string 
    char* m_str; // pointer to the character string 
public: 
    char String::element(unsigned int i) const; 
}; 

char String::element(unsigned int i) const 
{ 
    if (i < m_str1) 
    { 
     return m_str[i]; // Changed s to m_str 
    } 

    cout << "Error" << endl; 
    return 0; 
}