Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск  
 
 
Рейтинг 4.70/40: Рейтинг темы: голосов - 40, средняя оценка - 4.70
0 / 0 / 0
Регистрация: 12.05.2011
Сообщений: 17

strcpy - как быть?

19.09.2011, 22:11. Показов 9021. Ответов 36
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Всем Доброго времени суток.
Возник вопрос.
В процессе работы над заданием, столкнулся с проблемой, которую не хватает сил обойти.
Вот Код программы:
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
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
#include <iostream>
#include <string>
#include<iomanip>
 
using namespace std;
 
struct worker{
    char *name;
    char *spets;
    int   razread;
    
};
 
void init(worker *w, char* Name, char* Spets, int razread){
    w->razread = razread;
    w->name = new char[strlen(Name)+1];
    w->spets = new char[strlen(Spets)+1];
    strcpy(w->name, Name);
    strcpy(w->spets,Spets);
};
    void clearmem(worker *w) {
        delete[] w->name; w->name=NULL;
        delete[] w->razread; w->razread=NULL;
        delete[] w->spets; w->spets=NULL;
    };
 
    int compare(worker* w1, worker* w2) {
        if(w1->razread>w2->razread)
                return 1;
        else
                return 0;
    };
 
    int main(void){
        int n,r,option; //kolvo i razread
        worker W[10];
        worker tmp;
        char *nName, *nSpets;
 
        nName = new char [256];
        nSpets = new char [256];
 
        while(1) {
            cout << "Menu: " << endl;
            cout << "1. Data input " << endl;
            cout << "2. Data output " << endl;
            cout << "3. Data sorting " << endl;
            cout << "4. Data fields change" << endl;
            cout << "5.Exit" << endl;
            cout << "Choose an option: " << endl;
            cin >> option;
 
            switch(option){
            case 1:
                {
                    cout << "Enter the number of workers: "  << endl;
                    cin >> n;
                    for(int i=0;i<n;i++)
                    {
                        cout << "Name: " << endl;
                        cin >> nName;
                        cout << "Spetsialinosti:"  << endl;
                        cin >> nSpets;
                        cout <<"Razread : " << endl;
                        cin >> r;
                        init(&W[i], nName, nSpets, r);
                    }
                }
                break;
            case 2:
                {
                    cout<< "Data array : " << endl;
                    for(int i=0;i<n;i++)
                    {
                        cout << " Item no. " << i+1 << endl;
                        cout << " Name: " << W[i].name;
                        cout << " Spetsialinosti: " << W[i].spets;
                        cout << " Razread: " << W[i].razread << endl;
                    }
                }
                break;
            case 3:
                {
                    for(int i=1;i<n;i++)
                    {
                        if(compare(&W[i-1],&W[i]))
                        {
                            strcpy(nName, W[i].name);
                            strcpy(nSpets, W[i].spets);
                            r=W[i].razread;
                            clearmem(&W[i]);
                            init(&W[i], W[i-1].name, W[i].razread);
                            clearmem(&W[i-1]);
                            init(&W[i-1], nName, nSpets, r);
                        }
                        else break;
                    }
                }
                break;
            case 4:
                {
                    long i;
                    cout << "Enter the index of the item to be changed:"<<endl;
                    cin >> i;
                    cout << "Enter the new Name : " <<endl;
                    cin >> nName;
                    cout << "Enter the new spetsialinosti : "  <<endl;
                    cin >> nSpets ;
                    cout << " Enter the new razread: "  <<endl;
                    cin >> r;
 
                    init(&W[i-1],nName,nSpets,r);
                }
                break;
case5:
    {
        for(int i=0;i<n;i++)
        {
            clearmem(&W[i]);
        }
        return 1;
    }
    break;
            default:
                {
                    cout << "Wrong menu item chosen! Try again, please."<<endl;
                }
            }
        }
        delete [] nName; nName = NULL;
        delete [] nSpets; nSpets = NULL;
        cout << "Done !" << endl;
        return 1;
    };
Проблема в том, что компилятор кричит на "strcpy"... Как быть?
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
19.09.2011, 22:11
Ответы с готовыми решениями:

Функция strcpy () - строка должна быть пустой?
Здравствуйте! Возник вопрос: пусть имеются две непустые строки s1 и s2. Правильно ли копировать содержимое строки s2 в строку s1? Или...

Как должна быть организована сеть офиса? Что должно быть настроено обязательно, а чего быть не должно?
Добрый день. Сразу оговорюсь, что я инженер-строитель и настройкой сети занимаюсь только потому, что больше некому. Поэтому некоторые,...

Как работает strcpy(), где ошибка
Всем привет) Писал задачу, которая из 10 строк выводит самое короткое и первое в алфавитном порядке. Не понимаю почему не работает...

36
Эксперт С++
 Аватар для Nameless One
5828 / 3479 / 358
Регистрация: 08.02.2010
Сообщений: 7,448
21.09.2011, 15:34
Студворк — интернет-сервис помощи студентам
Цитата Сообщение от prik Посмотреть сообщение
Может покажите пример?
Начнем с того, что размер памяти, выделенный под массив динамически, ни С, ни в С++ узнать нельзя.
При передаче же статического массива в функцию strcpy происходит неявное преобразование к указателю на char* (т.е., по сути char dest[80] и char* dest - разные вещи). В качестве примера можешь посмотреть результат работы следующей программы:
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
#include <stdio.h>
#include <stdlib.h>
 
#define SIZE(ptr) sizeof(ptr) / sizeof(*ptr)
 
char* mystrcpy(char* dest, const char* src)
{
    printf("Allocated %u bytes for the destination string\n", SIZE(dest));
    
    char* result = dest;
    while(*dest++ = *src++);
    return result;
}
 
int main()
{
    char dest[80];
    const char* src = "Hello, strcpy!";
        
    printf("Allocated %u bytes for the destination string\n", SIZE(dest));
    mystrcpy(dest, src);
    printf("After copying: %s\n", dest);
    
    exit(0);
}
То есть, хотя мы можем внутри функции узнать число символов, которые содержит массив src, но мы не можем узнать допустимый объем памяти, который выделен (статически или динамически) для dest, значит, не можем и отследить переполнение dest.
В C++ есть хитрый трюк, который позволяет с помощью шаблонов узнать размер статического массива:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
#include <iostream>
 
template<size_t N>
char* mystrcpy(char (&dest)[N], const char* src)
{
    std::cout << "dest is " << N << " bytes long" << std::endl;
 
    char* ptr = dest;
    
    while(*ptr++ = *src++);
    
    return dest;
}
 
int main()
{
    char* src = "Hello, template hacks!";
    char dest[80];
    mystrcpy(dest, src);
    std::cout << dest << std::endl;
    
    return 0;
}
Кстати, если я правильно помню, одна из перегрузок strcpy_s как раз-таки написана подобным образом.
0
Эксперт С++
 Аватар для Nameless One
5828 / 3479 / 358
Регистрация: 08.02.2010
Сообщений: 7,448
21.09.2011, 15:35
Цитата Сообщение от Kastaneda Посмотреть сообщение
Ну да, без дополнительного параметра (как в strncpy()). В С++ можно реализовать, если передать указатель по ссылке и если это указатель на массив в стеке, а не на динамически выделенную память. Что говорит от том, что да - в общем случае невозможно

забыл, в С тоже можно, если ф-ция будет принимать указатель на указатель, а передавать в нее адрес указателя
разве? Можно примерчик?
0
725 / 224 / 73
Регистрация: 01.03.2011
Сообщений: 649
21.09.2011, 15:50
Цитата Сообщение от Nameless One Посмотреть сообщение
То есть, хотя мы можем внутри функции узнать число символов, которые содержит массив src, но мы не можем узнать допустимый объем памяти, который выделен (статически или динамически) для dest, значит, не можем и отследить переполнение dest.
Это вы показали продолжение все того же ублюдочного концепта бесконечной памяти, в веденного в язык с легкой руки K&R.
Я не вижу причин мешающих передавать в функцию копирования строк размер буфера...
0
 Аватар для kazak
3605 / 2745 / 356
Регистрация: 11.03.2009
Сообщений: 6,313
21.09.2011, 15:50
Цитата Сообщение от Nameless One Посмотреть сообщение
Начнем с того, что размер памяти, выделенный под массив динамически, ни С, ни в С++ узнать нельзя.
стандартными средствами нельзя.
0
Эксперт С++
 Аватар для Nameless One
5828 / 3479 / 358
Регистрация: 08.02.2010
Сообщений: 7,448
21.09.2011, 15:57
Цитата Сообщение от prik Посмотреть сообщение
Я не вижу причин мешающих передавать в функцию копирования строк размер буфера
Я тоже. Ну и в чем, собственно, вопрос? (Мой предыдущий пост относился к тому, что невозможно внутри функции char* strcpy(char*, const char*) узнать размер передаваемого буфера)
Цитата Сообщение от kazak Посмотреть сообщение
стандартными средствами нельзя.
А если нестандартными?
0
 Аватар для Kastaneda
5232 / 3206 / 362
Регистрация: 12.12.2009
Сообщений: 8,143
Записей в блоге: 2
21.09.2011, 16:00
Цитата Сообщение от Nameless One Посмотреть сообщение
разве? Можно примерчик?
Цитата Сообщение от Nameless One Посмотреть сообщение
забыл, в С тоже можно, если ф-ция будет принимать указатель на указатель, а передавать в нее адрес указателя
при этом

Цитата Сообщение от Nameless One Посмотреть сообщение
если это указатель на массив в стеке, а не на динамически выделенную память
вот это я тупанул!!!
0
725 / 224 / 73
Регистрация: 01.03.2011
Сообщений: 649
21.09.2011, 16:05
Цитата Сообщение от Nameless One Посмотреть сообщение
Я тоже. Ну и в чем, собственно, вопрос? (Мой предыдущий пост относился к тому, что невозможно внутри функции char* strcpy(char*, const char*) узнать размер передаваемого буфера)
С этим то никто и не спорит
Меня интересует, что может помешать писать strlcpy() ну или strncpy() на худой конец, вместо strcpy().
0
Эксперт С++
 Аватар для Nameless One
5828 / 3479 / 358
Регистрация: 08.02.2010
Сообщений: 7,448
21.09.2011, 16:14
Цитата Сообщение от prik Посмотреть сообщение
Меня интересует, что может помешать писать strlcpy() ну или strncpy() на худой конец, вместо strcpy().
Не знаю. Психические травмы или религиозная нетерпимость?
Ну, а в вырожденных случаях может сгодиться и strcpy:
C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
#define SIZE 80
 
int main()
{
    char dest[SIZE];
    char src[SIZE];
    fgets(src, SIZE, stdin);
    strcpy(dest, src);
    fputs(dest, stdout);
    
    exit(0);
}
0
 Аватар для kazak
3605 / 2745 / 356
Регистрация: 11.03.2009
Сообщений: 6,313
21.09.2011, 16:24
Цитата Сообщение от Nameless One Посмотреть сообщение
А если нестандартными?
Как передать массив в функцию не указывая размер
1
725 / 224 / 73
Регистрация: 01.03.2011
Сообщений: 649
21.09.2011, 16:27
Цитата Сообщение от Nameless One Посмотреть сообщение
Ну, а в вырожденных случаях может сгодиться и strcpy:
Это так кажется. fgets вернул NULL и приплыли.
0
 Аватар для kazak
3605 / 2745 / 356
Регистрация: 11.03.2009
Сообщений: 6,313
21.09.2011, 16:30
Цитата Сообщение от prik Посмотреть сообщение
Это так кажется. fgets вернул NULL и приплыли.
А проверить действительность параметров перед вызовом не судьба?
0
Эксперт С++
 Аватар для Nameless One
5828 / 3479 / 358
Регистрация: 08.02.2010
Сообщений: 7,448
21.09.2011, 16:32
Цитата Сообщение от prik Посмотреть сообщение
Это так кажется. fgets вернул NULL и приплыли
ну дык возвращаемое значение должно проверяться, а его ошибочное значение - обрабатываться. В примере выше обработка опущена для краткости.
0
725 / 224 / 73
Регистрация: 01.03.2011
Сообщений: 649
21.09.2011, 16:42
Ну я как бы откомментировал конкретный код, а не сферический)
0
 Аватар для resource2008
111 / 112 / 18
Регистрация: 11.03.2011
Сообщений: 421
21.09.2011, 17:55
ну раз уже развели тут такую дискуссию, добавлю от себя к вышесказанному:

strcpy() опирается на признак окончания строки (чем есть 0, в конце символьного массива), и если его в массиве не присутствует, то она отрабатывает до "посинения", чем является ошибка чтения памяти, с адресом, как правило, вне пределов адресного пространства программы.
0
Клюг
 Аватар для Charles Kludge
7677 / 3192 / 382
Регистрация: 03.05.2011
Сообщений: 8,380
21.09.2011, 18:14
Цитата Сообщение от Kastaneda Посмотреть сообщение
майкрософт активно агитирует прогеров использовать их ф-ции вместо стандартных, чтоб код, кроме как в MSVC больше ни где не собирался.
На самом деле отлично компилится на OpenWatcom, даже для Netware.
Цитата Сообщение от Open Watcom 1.8 C Library reference
#define __STDC_WANT_LIB_EXT1__ 1
#include <string.h>
errno_t strcpy_s( char * restrict s1, rsize_t s1max, const char * restrict s2 );
1
 Аватар для Kastaneda
5232 / 3206 / 362
Регистрация: 12.12.2009
Сообщений: 8,143
Записей в блоге: 2
21.09.2011, 19:20
Цитата Сообщение от Charles Kludge Посмотреть сообщение
На самом деле отлично компилится на OpenWatcom, даже для Netware.
Хм, не знал. Искрене думал, что это чисто майкрософтофская приблуда. Хотя в описании OpenWatcom что-то сказанно про "улучшенную совместимость с компиляторами Microsoft", может в этом дело?
0
Клюг
 Аватар для Charles Kludge
7677 / 3192 / 382
Регистрация: 03.05.2011
Сообщений: 8,380
21.09.2011, 19:51
Kastaneda, Waterloo всё-таки семейство компиляторов, да ещё и с внешними кодогенераторами, так что возможно всякое. Кстати есть у них и клоны МСовскких компайлера и линкера...
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
21.09.2011, 19:51

Как заменить функцию fgets на strlen и strcpy?
while (fgets(s1, sz_line, fp)) //будет прочитано 120 символов { // считаем длину текущей строки , будет в j j = 0; while...

Как написать функцию strcpy для UnicodeString?
Подскажите как написать функцию strcpy для UnicodeString, не может преобразовать E2034 Cannot convert 'UnicodeString' to 'char *' ...

Как правильно задействовать strcpy_s вместо strcpy?
Имеется простой класс, да, strcpy отрабатывает корректно, но, хотелось бы понять, как правильно задействовать strcpy_s вместо strcpy, что...

Быть любимым - это больше, чем быть богатым, потому что быть любимым означает быть счастливым?
Vourhey Melodie, &quot;Быть любимым - это больше, чем быть богатым, потому что быть любимым означает быть счастливым&quot; ээээ не...

Как работает strcpy с точки зрения распределения памяти?
Уважаемые знатоки С++, объясните, пожалуйста, как работает strcpy с точки зрения распределения памяти? Ламерский вопрос, но все-таки...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
37
Ответ Создать тему
Новые блоги и статьи
GitDOS: Когда я решил скрестить MS-DOS, GitHub и Claude
MaGz GoLd 08.07.2026
Привет, Киберфорум! Хочу поделиться своим проектом, который, надеюсь, вас заинтересует. GitDOS — это веб-приложение, которое позволяет запускать DOS в браузере с тремя крутыми фичами: диски живут в. . .
Учебный мини "цифровой двойник" = (Сервер = Java + Flask_питон )+(3D = Гугловскаая 4T nvideo + google colab) +( WMS = Odoo на Docker)
anaschu 07.07.2026
Увидел в требованиях к вакансии создателя имитационных моделей связку (REst + Api + WMS + Anylogic + питон + 3D Nvideo), и решил посмотреть, что это такое. Из этого всего я работал более менее. . .
Море в объективе
kumehtar 06.07.2026
Хотел написать много проникновенных слов, но передумал. Оно само - говорит. Кто слышит тот слышит.
Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii.
dcc0 05.07.2026
Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .
Установка статуса документа по условию
Maks 05.07.2026
Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .
Сезонность и суточность закисления почв
anaschu 04.07.2026
200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .
В чем ценность человеческого опыта в глобальном смысле?
kumehtar 03.07.2026
Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .
интеграция AnyLogic с самописным REST API и переход на Odoo
anaschu 03.07.2026
Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru