Run-Time Check Failure #2 - Stack around the variable 'fish2' was corrupted

@kriks_ · Регистрация: 24.10.2018

Author24 — интернет-сервис помощи студентам

C++

while (str1[size1] != '\0') {
        str1[size1] = ' ';
        size1++;
    }

Вот часть кода. Этот цикл должен найти длину строки и заполнить его пробелами. Но проблема в том что из-за строки "str1[size1] = ' ';" видает исключение в конце программы "Run-Time Check Failure #2 - Stack around the variable 'fish2' was corrupted.". Также если смотреть в переменную size1, в ней постоянно число около 300 (что намного больше чем должно быть в программе). В чём проблема? Кажется этот цикл сначала убирает '\0' а потом идёт заново. Если это так, то почему?
При этом программа работает правильно.

@zayats80888 · 12.05.2019, 22:21

C++

size_t pos = 0;
while (str1[pos] != '\0') {
        str1[pos] = ' ';
        pos++;
    }

@kriks_ · 12.05.2019, 22:27 **[ТС]**

zayats80888, ошибка не пропала. Это не помогает.

@oleg-m1973 · 12.05.2019, 22:35

Сообщение от kriks_

zayats80888, ошибка не пропала. Это не помогает.

Очевидно, что проблема не в этом цикле. Покажи весь код.

@kriks_ · 12.05.2019, 22:41 **[ТС]**

C++

void strcpy_k(char str1[], const char str2[]) {
 
    int size1 = 0;
    int size2 = 0;
 
    while (str1[size1] != '\0') {
        str1[size1] = ' ';
        size1++;
    }
 
    while (str2[size2] != '\0') {
        size2++;
    }
 
    for (int i = 0; i < size2; i++) {
        str1[i] = str2[i];
    }
 
    str1[size2] = '\0';
    std::cout << size1 << "\t" << size2 << "\n";
}

Вот. Но я проверял. Когда убирал строку "str1[size1] = ' ';" всё работало без ошибок. Как я знаю, это исключение может быть из-за выхода за границу массива. Скорее всего это так. Ведь у меня в переменной size1 всегда выходит около 300 (а должно 4). А эта переменная меняется только в этом цикле.

@zayats80888 · 12.05.2019, 22:50

во первых

C++

    int min = (size1 < size2) ? size1 : size2;
    for (int i = 0; i < min; i++) {
        str1[i] = str2[i];
    }

во вторых, зачем пробелами заполнять?
в третьих, вы уверены что в функцию передаются корректные строки? (c '\0' в конце)

@oleg-m1973 · 12.05.2019, 22:57

Сообщение от kriks_

Вот. Но я проверял. Когда убирал строку "str1[size1] = ' ';" всё работало без ошибок.

Значит ошибка ещё выше.

А вообще, строка копируется обычно в неинициализированный буфер (скорее всего у тебя так и есть). Т.е. поиск нуля в str1 - абсолютно бессмысленная вещь - он будет пробегать всю доступную память пока случайно не встретится ноль.

Покажи остальной код.

@kriks_ · 12.05.2019, 23:01 **[ТС]**

zayats80888
1. Это не работает (скрин ниже).
2. Чтоб когда первая строка была больше чем вторая, не оставалось лишних символов.
3. Да. Ведь всё работает нормально. Просто у меня постоянно это исключение.

@kriks_ · 12.05.2019, 23:05 **[ТС]**

oleg-m1973, там есть ноль. Второй цикл (без заполнения пробелами) работает так как надо. Этот, если без заполнения пробелами, тоже. И вот всё что как-то связано с этими строками:

C++

void strcpy_k(char str1[], const char str2[]) {
 
    int size1 = 0;
    int size2 = 0;
 
    while (str1[size1] != '\0') {
        str1[size1] = ' ';
        size1++;
    }
 
    while (str2[size2] != '\0') {
        size2++;
    }
 
    for (int i = 0; i < size2; i++) {
        str1[i] = str2[i];
    }
 
    str1[size2] = '\0';
    std::cout << size1 << "\t" << size2 << "\n"; //для отладки
}
 
class fish {
private:
 
    char clr1[16];
    char clr2[16];
    int size;
 
public:
 
    fish() {}
 
    fish(const char clr1[], const char clr2[], const int size = rand() % 3) {
        strcpy_k(this->clr1, clr1);
        strcpy_k(this->clr2, clr2);
        this->size = size;
    }
 
    void print_data() {
        std::cout << "clr1 = " << clr1 << "\t" << "clr2 = " << clr2 << "\t" << "size = " << size << "\n";
    }
 
};
 
int main() {
    srand(time(NULL));
    
    fish fish1("red", "blue");
    fish fish2("white", "yelow", 2);
 
    fish1.print_data();
    fish2.print_data();
 
    return 0;
}

@zayats80888 · 12.05.2019, 23:10

C++

#include <iostream>
 
void strcpy_k(char str1[], const char str2[]) {
 
    int size1 = 0;
    int size2 = 0;
 
    while (str1[size1] != '\0') {
        //str1[size1] = ' ';
        size1++;
    }
 
    while (str2[size2] != '\0') {
        size2++;
    }
 
    int min = (size1 < size2) ? size1 : size2;
    for (int i = 0; i < min; i++) {
        str1[i] = str2[i];
    }
 
    str1[min] = '\0';
    std::cout << size1 << "\t" << size2 << "\n";
}
 
int main()
{
    char str1[]{"red"};
    const char* str2{ "blue" };
    std::cout << str1 << '\n' << str2 << '\n';
 
    strcpy_k(str1, str2);
    std::cout << str1 << '\n' << str2 << '\n';
}

@oleg-m1973 · 12.05.2019, 23:17

Сообщение от kriks_

oleg-m1973, там есть ноль

Нет, нету. strcpy_k(this->clr1, clr1); в this->clr1 - мусор

Убери этот while (str1[size1] != '\0') { - Он не нужен вообще ни под каким видом

@zayats80888 · 12.05.2019, 23:23

так короче

C++

void strcpy_k(char str1[], const char str2[]) {
 
    while (*str1 && *str2) *str1++ = *str2++;
}

@oleg-m1973 · 12.05.2019, 23:24

Сообщение от zayats80888

так короче

И не менее бессмысленно. Единственное, что нужно контролировать при копировании строк - это размер выходного буфера, str1. Здесь этого нет.

@kriks_ · 12.05.2019, 23:24 **[ТС]**

zayats80888 и oleg-m1973, спасибо огромное. zayats80888 уже дал рабочий код. oleg-m1973 ожидается, что эту функцию не только для этого класса буду использовать.
Извините если был в каких то моментах слишком глупый. Теорию вроде знаю, а практикуюсь слишком мало, поэтому много туплю.

@zayats80888 · 12.05.2019, 23:30

Сообщение от oleg-m1973

Единственное, что нужно контролировать при копировании строк - это размер выходного буфера, str1.

согласен

@oleg-m1973 · 12.05.2019, 23:30

Сообщение от kriks_

zayats80888 уже дал рабочий код. oleg-m1973 ожидается, что эту функцию не только для этого класса буду использовать.

Это не рабочий код (как минимум ноль в конце надо добавить). Ещё раз повторяю - в str1 мусор, нельзя полагаться на его содержимое.

Вот классический цикл для копирования строк

C++

while (*str2) 
    *(str1++) = *(str2++);
 
*str1 = 0;

@kriks_ · 12.05.2019, 23:33 **[ТС]**

oleg-m1973, спасибо.

@zayats80888 · 12.05.2019, 23:35

так безопаснее:

C++

void strcpy_k(char* str1, const char* str2, size_t count) {
 
    while (--count && *str2) *str1++ = *str2++;
    *str1 = 0;
}

@Yetty · 12.05.2019, 23:38

Сообщение от oleg-m1973

как минимум ноль в конце надо добавить

Сообщение от zayats80888

str1[min] = '\0';

zayats80888, а без этой строки будет работать так ?

C++

1
2
3

int min = (size1 < size2) ? size1 : size2+1;
for (int i = 0; i < min; i++) 
str1[i] = str2[i];

@kriks_ · 12.05.2019, 23:41 **[ТС]**

zayats80888, oleg-m1973, если есть время, можете объяснить зачем в цикле (*str2)?

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	12.05.2019, 23:17	11
	Сообщение от kriks_ oleg-m1973, там есть ноль Нет, нету. strcpy_k(this->clr1, clr1); в this->clr1 - мусор Убери этот while (str1[size1] != '\0') { - Он не нужен вообще ни под каким видом 0

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	12.05.2019, 23:24	13
	Сообщение от zayats80888 так короче И не менее бессмысленно. Единственное, что нужно контролировать при копировании строк - это размер выходного буфера, str1. Здесь этого нет. 1

@kriks_ -4 / 1 / 0 Регистрация: 24.10.2018 Сообщений: 23
	12.05.2019, 23:24 [ТС]	14
	zayats80888 и oleg-m1973, спасибо огромное. zayats80888 уже дал рабочий код. oleg-m1973 ожидается, что эту функцию не только для этого класса буду использовать. Извините если был в каких то моментах слишком глупый. Теорию вроде знаю, а практикуюсь слишком мало, поэтому много туплю. 0

@zayats80888 6105 / 3460 / 1405 Регистрация: 07.02.2019 Сообщений: 8,791
	12.05.2019, 23:30	15
	Сообщение от oleg-m1973 Единственное, что нужно контролировать при копировании строк - это размер выходного буфера, str1. согласен 1

Run-Time Check Failure #2 - Stack around the variable 'fish2' was corrupted

Решение

Решение

@kriks_ -4 / 1 / 0 Регистрация: 24.10.2018 Сообщений: 23
	12.05.2019, 22:27 [ТС]	3
	zayats80888, ошибка не пропала. Это не помогает. 0

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	12.05.2019, 22:35	4
	Сообщение от kriks_ zayats80888, ошибка не пропала. Это не помогает. Очевидно, что проблема не в этом цикле. Покажи весь код. 1

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	12.05.2019, 22:57	7
	Сообщение от kriks_ Вот. Но я проверял. Когда убирал строку "str1[size1] = ' ';" всё работало без ошибок. Значит ошибка ещё выше. А вообще, строка копируется обычно в неинициализированный буфер (скорее всего у тебя так и есть). Т.е. поиск нуля в str1 - абсолютно бессмысленная вещь - он будет пробегать всю доступную память пока случайно не встретится ноль. Покажи остальной код. 1

@kriks_ -4 / 1 / 0 Регистрация: 24.10.2018 Сообщений: 23
	12.05.2019, 23:01 [ТС]	8
	zayats80888 1. Это не работает (скрин ниже). 2. Чтоб когда первая строка была больше чем вторая, не оставалось лишних символов. 3. Да. Ведь всё работает нормально. Просто у меня постоянно это исключение. Миниатюры 0

@kriks_ -4 / 1 / 0 Регистрация: 24.10.2018 Сообщений: 23
	12.05.2019, 23:33 [ТС]	17
	oleg-m1973, спасибо. 0

@kriks_ -4 / 1 / 0 Регистрация: 24.10.2018 Сообщений: 23
	12.05.2019, 23:41 [ТС]	20
	zayats80888, oleg-m1973, если есть время, можете объяснить зачем в цикле (*str2)? 0