Ошибка, связанная с переполнением буфера

@tyreu · 17.09.2014, 23:58. **Показов** 1350. **Ответов** 7

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

В общем, данный код рабочий, собираю на VS2013, при компиляции выдает переполнение буфера. Другие люди пробовали собирать у себя, у них всё нормально. Кто может, исправьте, пожалуйста.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
#include "stdafx.h"
#include<iostream>       //Содержит cin,coute
#include<fstream>        //Содержит тип fstream
#include<string.h>         //Содержит строковые функции
#include<conio.h>          //Содержит getch();
#include<stdio.H>          //Содержит getchar();
#include <locale>
 
int iProverka(const int *);
char cTransTsifr(const int *);
 
int main(void)
{
    setlocale(LC_ALL, "Rus");
    const int N = 256;
    char cStroka[N];
    char cBufStroka[N];
    int i = 0, i2 = 0, j = 0, k = 0, n = 0, m = 0, iPr = 0;
    for (m = 0; m <= N; m++)
    {
        cStroka[m] = 0;
        cBufStroka[m] = 0;
    };
    m = 0;
    printf("Введите строку не длинней ", N, " символов:\n");
    while (((cStroka[m] = getchar()) != '\n') && (N != m)) m++;
    cStroka[m] = 0;
    while (cStroka[i])
    {
        if (cStroka[i] == cStroka[i + 1])
        {
        k = 2;
        j = i;
        while (cStroka[i + 1] == cStroka[i + 2])
            {
            //считаем совпадающие символы
            k++;
            i++;
            }
        i = j + k - 1;
        iPr = iProverka(&k);
        if (k>3)//Нет смысла заменять меньше 3 цифр
            {
            cBufStroka[i2] = cStroka[j];
            cBufStroka[i2 + 1] = '(';
            cBufStroka[i2 + 2 + iPr] = ')';
            for (n = iPr; n >= 1; n--)
                {
                m = k;
                m = m % 10;
                k = k / 10;
                cBufStroka[i2 + 1 + n] = cTransTsifr(&m);
                }
            i2 = i2 + iPr + 2;
            }
        else
            {
        for (n = i2; n <= i2 + k - 1; n++) cBufStroka[n] = cStroka[j];
        i2 = i2 + k - 1;
            }
        }
        else
        {
        cBufStroka[i2] = cStroka[i];
        }
        i++;
        i2++;
        }
        printf("Старая строка: ", cStroka, "\n", "Новая строка: ", cBufStroka,"\n");
        _getch();
        return 1;
        }
        //---------------------FUNKTSII---------------------------------------------
        //Подсчет количества цифр в числе
        int iProverka(const int *cC){
        int k = 0, m = *cC;
        do{
        k++;
        m = m / 10;
        } while (m != 0);
        return k;
        }
 
        //Преобразует числа от 0 до 9 в текст
        char cTransTsifr(const int *iTsifra)
        {
        char cTsifra;
        switch (*iTsifra)
            {
        case 0: cTsifra = '0'; break;
        case 1: cTsifra = '1'; break;
        case 2: cTsifra = '2'; break;
        case 3: cTsifra = '3'; break;
        case 4: cTsifra = '4'; break;
        case 5: cTsifra = '5'; break;
        case 6: cTsifra = '6'; break;
        case 7: cTsifra = '7'; break;
        case 8: cTsifra = '8'; break;
        case 9: cTsifra = '9'; break;
        default: break;
            }
        return cTsifra;
        }

alsav22 · 18.09.2014, 00:38

tyreu, смотрели, как код работает? printf() неправильно сделан.

Добавлено через 4 минуты
Вот это - выход за границы массивов:

C++
1
2
3
4
5
6
7
8
char cStroka[N];
    char cBufStroka[N];
    int i = 0, i2 = 0, j = 0, k = 0, n = 0, m = 0, iPr = 0;
    for (m = 0; m <= N; m++)
    {
        cStroka[m] = 0;
        cBufStroka[m] = 0;
    };

Нужно просто: m < N.

@tyreu · 18.09.2014, 08:19 **[ТС]**

alsav22, переделал. В результате получаются каракули и только 1 символ, а не какое-то слово

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
#include "stdafx.h"
#include<fstream>        //Содержит тип fstream
#include<string.h>         //Содержит строковые функции
#include<conio.h>          //Содержит getch();
#include<stdio.H>          //Содержит getchar();
#include <locale>
 
int iProverka(const int *);
char cTransTsifr(const int *);
 
int main(void)
{
    setlocale(LC_ALL, "Rus");
    const int N = 256;
    char cStroka[N];
    char cBufStroka[N];
    int i = 0, i2 = 0, j = 0, k = 0, n = 0, m = 0, iPr = 0;
    for (m = 0; m < N; m++)
    {
        cStroka[m] = 0;
        cBufStroka[m] = 0;
    };
    m = 0;
    printf("Введите строку не длинней 256 символов:\n");
    while (((cStroka[m] = getchar()) != '\n') && (N != m)) m++;
    cStroka[m] = 0;
    while (cStroka[i])
    {
        if (cStroka[i] == cStroka[i + 1])
        {
        k = 2;
        j = i;
        while (cStroka[i + 1] == cStroka[i + 2])
            {
            //считаем совпадающие символы
            k++;
            i++;
            }
        i = j + k - 1;
        iPr = iProverka(&k);
        if (k>3)//Нет смысла заменять меньше 3 цифр
            {
            cBufStroka[i2] = cStroka[j];
            cBufStroka[i2 + 1] = '(';
            cBufStroka[i2 + 2 + iPr] = ')';
            for (n = iPr; n >= 1; n--)
                {
                m = k;
                m = m % 10;
                k = k / 10;
                cBufStroka[i2 + 1 + n] = cTransTsifr(&m);
                }
            i2 = i2 + iPr + 2;
            }
        else
            {
        for (n = i2; n <= i2 + k - 1; n++) cBufStroka[n] = cStroka[j];
        i2 = i2 + k - 1;
            }
        }
        else
        {
        cBufStroka[i2] = cStroka[i];
        }
        i++;
        i2++;
        }
    printf("Старая строка: ");
    printf("%c", cStroka);
    printf("\nНовая строка: "); 
    printf("%c", cBufStroka);
    printf("\n");
        _getch();
        return 1;
        }
        //---------------------FUNKTSII---------------------------------------------
        //Подсчет количества цифр в числе
        int iProverka(const int *cC){
        int k = 0, m = *cC;
        do{
        k++;
        m = m / 10;
        } while (m != 0);
        return k;
        }
 
        //Преобразует числа от 0 до 9 в текст
        char cTransTsifr(const int *iTsifra)
        {
        char cTsifra;
        switch (*iTsifra)
            {
        case 0: cTsifra = '0'; break;
        case 1: cTsifra = '1'; break;
        case 2: cTsifra = '2'; break;
        case 3: cTsifra = '3'; break;
        case 4: cTsifra = '4'; break;
        case 5: cTsifra = '5'; break;
        case 6: cTsifra = '6'; break;
        case 7: cTsifra = '7'; break;
        case 8: cTsifra = '8'; break;
        case 9: cTsifra = '9'; break;
        default: break;
            }
        return cTsifra;
        }

@zss · 18.09.2014, 08:38

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

while (((cStroka[m] = getchar()) != '\n') && (N != m)) m++;

надо m<N
А еще лучше

C++
1
fgets(cStrora,N-1,stdin);

@tyreu · 18.09.2014, 11:46 **[ТС]**

alsav22,

C++
1
2
3
4
5
6
7
    printf("Старая строка: ");
    printf("%c", cStroka);
    printf("\nНовая строка: "); 
    printf("%c", cBufStroka);
    printf("\n");
        _getch();
        return 1;

а принтф нормально? всё равно переделал и выдаёт по одному символу неясному. попробуйте у себя запустить.

@castaway · 18.09.2014, 12:13

Замени %c на %s

@tyreu · 18.09.2014, 12:22 **[ТС]**

castaway, спасибо.

@zss · 18.09.2014, 15:21

Кстати, я не вижу, что вы используете функции из других файлов, кроме
<stdio.h> <conio.h> и <locale.h>
зачем их подключать?

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

Ошибка, связанная с переполнением буфера

Решение

Решение

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,170 Записей в блоге: 10
	18.09.2014, 12:13
	Сообщение было отмечено tyreu как решение Решение Замени %c на %s 1

@tyreu Заблокирован
	18.09.2014, 12:22 [ТС]
	castaway, спасибо. 0

@zss Модератор 13773 / 10966 / 6491 Регистрация: 18.12.2011 Сообщений: 29,244
	18.09.2014, 15:21
	Кстати, я не вижу, что вы используете функции из других файлов, кроме <stdio.h> <conio.h> и <locale.h> зачем их подключать? 0