Пытаюсь изучить ООП на базе С++(с помощью книги Б. Джонса и Д. Либерти), работаю с M Visual Studio, 2013, но есть и 2008. Раньше все было идеально, но вот недавно столкнулся с такой проблемой: при использовании функций strcpy() и strncpy() выдает вот такую ошибку:

Error    1    error C4996: 'strcpy': This function or variable may be unsafe. Consider using strcpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.    c:\users\глеб\documents\visual studio 2013\projects\consoleapplication6\consoleapplication6\source.cpp    20

и при чем оба компилятора. Что делать? Вроде все делаю как нужно, по книге код воспроизвожу, все одно и то же.




И еще, если кому не лень, пробегитесь глазами, скажите что не так: 

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


class String
{
public:
    //конструкторы/деструкторы
    String();
    String(const char *const);
    String(const String &);
    ~String();

    //операторы
    char & operator[] (unsigned short offset);
    char operator[] (unsigned short offset) const;
    String operator+(const String&);
    void operator+=(const String&);
    String & operator=(const String&);

    //методы доступа
    unsigned short GetLen() const { return itsLen; }
    const char* GetString() const { return itsString; }

private:
    unsigned short itsLen;
    char * itsString;
    String(unsigned short);

};

String::String(unsigned short len)
{
    itsLen = len;
    itsString = new char[itsLen];
    for (int i = 0; i <= itsLen; i++)
        itsString[i] = '\0';
}

String::String()
{
    itsLen = 0;
    itsString = new char[1];
    itsString[1] = '\0';
}

String::String(const char * const pStr)
{
    itsLen = strlen(pStr);
    itsString = new char[itsLen+1];
    for (int i = 0; i < itsLen; i++)
    {
        itsString[i] = pStr[i];
    }
    itsString[itsLen] = '\0';
}

String::String(const String & rStr)
{
    itsLen = rStr.GetLen();
    itsString = new char[itsLen + 1];
    for (int i = 0; i < itsLen; i++)
    {
        itsString[i] = rStr[i];
    }
}

String::~String()
{
    delete[] itsString;
}

char & String::operator[] (unsigned short offset)
{
    if (itsLen < offset)
        return itsString[itsLen - 1];
    else
        return itsString[offset];
}

char String::operator[] (unsigned short offset) const
{
    if (itsLen < offset)
        return itsString[itsLen - 1];
    else
        return itsString[offset];
}

String String::operator+(const String & rStr)
{
    unsigned short totalLen = itsLen + rStr.GetLen();
    String temp(totalLen);
    int i=0;
    for (i; i < itsLen; i++)
        temp[i] = itsString[i];
    for (int j = 0; j < rStr.GetLen(); i++, j++)
        temp[i] = rStr[j];
    temp[totalLen] = '\0';
    return temp;
}

void String::operator+=(const String & rStr)
{
    unsigned short totalLen = itsLen + rStr.GetLen();
    String temp(totalLen);
    int i = 0;
    for (i; i < itsLen; i++)
        temp[i] = itsString[i];
    for (int j = 0; j < rStr.GetLen(); i++, j++)
        temp[i] = rStr[j];
    delete[] itsString;
    *this = temp;
}

String & String::operator=(const String & rStr)
{
    if (this == &rStr)
        return *this;
    delete[] itsString;
    itsLen = rStr.GetLen();
    itsString = new char[itsLen];
    for (int i = 0; i < itsLen; i++)
        itsString[i] = rStr[i];
    itsString[itsLen] = '\0';
    return *this;
}


int main()
{
    String s1("initial test");
    String s2("test initial");
    s1 += s2;
    std::cout << s1.GetString();
    
    return 0;
}


Автор книги объясняет работу со строками и как бы создает калеку вот подобную) Идею я думаю вы поняли.. А я вник, понял смысл и спустя какое то время просто хотел воспроизвести что запомнилось.. Вот пробую этот класс использовать, а компилятор матерится уже после компиляции(всплывает окошко). Но саму ошибку в коде не показывает.. ( Что не так? 
Только основной вопрос вверху, это так - дополнительный, если кто поможет буду благодарен, но главное помогите с функцией strcpy() разобраться.
Всем огромное спасибо, кто уделил мне свое драгоценное время!

Это не ошибка, это предупреждение, которое впринципе достаточно понятно говорит, что strcpy небезопасна (так как может привести к переполению буфера - она не проверяет границы), и советует использовать strcpy_s вместо нее. Вы можете это предупреждение проигнорировать вот так, если мне не изменяет память


А лучше, воспользуйтесь советом и используйте strcpy_s, указывая границу:


Посмотрел ваш код, во первых в следующий раз вставляйте его в тег код, а во вторых, в коде вы не используете strcpy, откуда же берется ошибка? - Понял, код к первому вопросу не относится. Может кто другой посмотрит тогда, сейчас нету компилятора под рукой

Это сообщение отредактировал(а) sQu1rr - 20.11.2014, 18:08

Вы меня уж извините, я тут новичок, еще не знаю что и как тут..
А код это отдельный вопрос. Спасибо вам большое.

http://forum.vingrad.ru/forum/topic-48015.html пункт А)

Ну мы же не упражняемся в дословном переводе с английского, господин baldina
Я к тому что это не "ошибка", а компилятор мелкомягких просто хочет ограничить переполнение буфера.

http://stackoverflow.com/questions/2187304...fe-when-complin

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


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


для всяких размеров лучше использовать size_t


зачем строку нулями забивать?


вместо циклов копирования можно использовать функцию std::copy (объявлена в <algorithm>)
отдельно записывать 0 нет необходимости, его можно скопировать из исходной строки


operator= и конструктор копирования делают почти одно и то же. подумайте, как избежать дублирования кода

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

да это вообще не ошибка (скорее, это не соответствие VC++ стандарту). я, кстати, удивлен, т.к. в 10 студии (а другой нет под рукой) это таки warning
но человек сказал, что не может откомпилировать, и приводит текст где видно что это ошибка. предупреждение ему не помешало бы получить результат.

Добавлено через 8 минут и 13 секунд
а, понятно. там новая опция

Пробовал через #define убрать вот это предупреждение, все та же ошибка и не компилирует.. Ну не важно, так как помогло второе решение, вот с функцией strcpy_s(). Но не могу понять, ведь strncpy() по сути выполняет те же действия, только там указывается конкретное количество символов для копирования. Этим она как бы и безопаснее, чем strcpy(), но и с ней та же ошибка..

Почитайте тут: http://msdn.microsoft.com/en-us/library/5dae5d43(VS.80).aspx
Безопасная версия принимает 4 аргумента а не 3.

Нужно в cpp добавить 


или



Это сообщение отредактировал(а) Dem_max - 24.11.2014, 11:03

Возражаю. По возможности используйте стандартные, более переносимые функции.

Так уже 3 года прошло с С11
http://en.wikipedia.org/wiki/C11_(C_standard_revision)
И вот
https://www.securecoding.cert.org/confluenc...+or+target+size

Это сообщение отредактировал(а) sQu1rr - 26.11.2014, 02:06

sQu1rr, в С но не С++

Ну да, этот вариант я пропустил, так как функции вроде strcpy у меня ассоциируются именно с С. В С++ все таки std::string надо использовать. А из безопасных варинатов strcpy можно memcpy использовать