Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 21, средняя оценка - 4.95
manter
0 / 0 / 0
Регистрация: 12.05.2011
Сообщений: 17
#1

strcpy - как быть? - C++

19.09.2011, 22:11. Просмотров 2850. Ответов 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"... Как быть?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
19.09.2011, 22:11     strcpy - как быть?
Посмотрите здесь:

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

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

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

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

Как установить конец строки при своей реализации функции strcpy()? - C++
Доброго времени суток, попробовал сам реализовать функцию strcpy(). Программа работоспособная, но интересует один момент. Это рабочий...

strcpy - C++
прога пашет,но после сортировки по результату,он выдает какойто корявый список,тоесть strcpy неверно пашет...... че с ней теперь...

strcpy - C++
Недавно начал программировать на C++, в типах плохо ещё разбираюсь... не могу понять, что надо исправить, чтобы код: char szSendBuff; ...

После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,251
Записей в блоге: 1
Завершенные тесты: 1
21.09.2011, 14:59     strcpy - как быть? #16
Если целевая строка strcpy недостаточно велика (такое случается, если программист идиот/болван и не проверяет размер перед копированием), то может случится ужасное.
Ну на то он и Си, что нужно думать над тем, что пишешь Безопасность подобных ф-ций означает лишние проверки внутри нее, а значит затраченное время. А сишники, как известно, в этом отношении очень привередливы. Думаю именно поэтому в стандарт был внесен вариант strcpy() без дополнительных проверок, чтобы не увеличивать время работы и объем кода.
Nameless One
Эксперт С++
5769 / 3418 / 255
Регистрация: 08.02.2010
Сообщений: 7,446
21.09.2011, 15:01     strcpy - как быть? #17
Kastaneda, тем более проверка на переполнение буфера в общем случае невозможна.
prik
8 / 8 / 1
Регистрация: 01.03.2011
Сообщений: 62
21.09.2011, 15:04     strcpy - как быть? #18
Цитата Сообщение от Nameless One Посмотреть сообщение
Kastaneda, тем более проверка на переполнение буфера в общем случае невозможна.
Может покажите пример? язык си.
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,251
Записей в блоге: 1
Завершенные тесты: 1
21.09.2011, 15:12     strcpy - как быть? #19
Цитата Сообщение от Nameless One Посмотреть сообщение
Kastaneda, тем более проверка на переполнение буфера в общем случае невозможна.
Ну да, без дополнительного параметра (как в strncpy()). В С++ можно реализовать, если передать указатель по ссылке и если это указатель на массив в стеке, а не на динамически выделенную память. Что говорит от том, что да - в общем случае невозможно

Добавлено через 1 минуту
Цитата Сообщение от Kastaneda Посмотреть сообщение
В С++ можно реализовать, если передать указатель по ссылке
забыл, в С тоже можно, если ф-ция будет принимать указатель на указатель, а передавать в нее адрес указателя
yulicesar
4 / 4 / 0
Регистрация: 21.09.2011
Сообщений: 20
21.09.2011, 15:15     strcpy - как быть? #20
в начало файла
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_SECURE_NO_WARNINGS

или в свойствах проекта прописать
Nameless One
Эксперт С++
5769 / 3418 / 255
Регистрация: 08.02.2010
Сообщений: 7,446
21.09.2011, 15:34     strcpy - как быть? #21
Цитата Сообщение от 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 как раз-таки написана подобным образом.
Nameless One
Эксперт С++
5769 / 3418 / 255
Регистрация: 08.02.2010
Сообщений: 7,446
21.09.2011, 15:35     strcpy - как быть? #22
Цитата Сообщение от Kastaneda Посмотреть сообщение
Ну да, без дополнительного параметра (как в strncpy()). В С++ можно реализовать, если передать указатель по ссылке и если это указатель на массив в стеке, а не на динамически выделенную память. Что говорит от том, что да - в общем случае невозможно

забыл, в С тоже можно, если ф-ция будет принимать указатель на указатель, а передавать в нее адрес указателя
разве? Можно примерчик?
prik
8 / 8 / 1
Регистрация: 01.03.2011
Сообщений: 62
21.09.2011, 15:50     strcpy - как быть? #23
Цитата Сообщение от Nameless One Посмотреть сообщение
То есть, хотя мы можем внутри функции узнать число символов, которые содержит массив src, но мы не можем узнать допустимый объем памяти, который выделен (статически или динамически) для dest, значит, не можем и отследить переполнение dest.
Это вы показали продолжение все того же ублюдочного концепта бесконечной памяти, в веденного в язык с легкой руки K&R.
Я не вижу причин мешающих передавать в функцию копирования строк размер буфера...
kazak
3034 / 2355 / 155
Регистрация: 11.03.2009
Сообщений: 5,401
21.09.2011, 15:50     strcpy - как быть? #24
Цитата Сообщение от Nameless One Посмотреть сообщение
Начнем с того, что размер памяти, выделенный под массив динамически, ни С, ни в С++ узнать нельзя.
стандартными средствами нельзя.
Nameless One
Эксперт С++
5769 / 3418 / 255
Регистрация: 08.02.2010
Сообщений: 7,446
21.09.2011, 15:57     strcpy - как быть? #25
Цитата Сообщение от prik Посмотреть сообщение
Я не вижу причин мешающих передавать в функцию копирования строк размер буфера
Я тоже. Ну и в чем, собственно, вопрос? (Мой предыдущий пост относился к тому, что невозможно внутри функции char* strcpy(char*, const char*) узнать размер передаваемого буфера)
Цитата Сообщение от kazak Посмотреть сообщение
стандартными средствами нельзя.
А если нестандартными?
Kastaneda
Форумчанин
Эксперт С++
4514 / 2856 / 228
Регистрация: 12.12.2009
Сообщений: 7,251
Записей в блоге: 1
Завершенные тесты: 1
21.09.2011, 16:00     strcpy - как быть? #26
Цитата Сообщение от Nameless One Посмотреть сообщение
разве? Можно примерчик?
Цитата Сообщение от Nameless One Посмотреть сообщение
забыл, в С тоже можно, если ф-ция будет принимать указатель на указатель, а передавать в нее адрес указателя
при этом

Цитата Сообщение от Nameless One Посмотреть сообщение
если это указатель на массив в стеке, а не на динамически выделенную память
вот это я тупанул!!!
prik
8 / 8 / 1
Регистрация: 01.03.2011
Сообщений: 62
21.09.2011, 16:05     strcpy - как быть? #27
Цитата Сообщение от Nameless One Посмотреть сообщение
Я тоже. Ну и в чем, собственно, вопрос? (Мой предыдущий пост относился к тому, что невозможно внутри функции char* strcpy(char*, const char*) узнать размер передаваемого буфера)
С этим то никто и не спорит
Меня интересует, что может помешать писать strlcpy() ну или strncpy() на худой конец, вместо strcpy().
Nameless One
Эксперт С++
5769 / 3418 / 255
Регистрация: 08.02.2010
Сообщений: 7,446
21.09.2011, 16:14     strcpy - как быть? #28
Цитата Сообщение от 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);
}
kazak
3034 / 2355 / 155
Регистрация: 11.03.2009
Сообщений: 5,401
21.09.2011, 16:24     strcpy - как быть? #29
Цитата Сообщение от Nameless One Посмотреть сообщение
А если нестандартными?
Как передать массив в функцию не указывая размер
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.09.2011, 16:27     strcpy - как быть?
Еще ссылки по теме:

strcpy() - C++
Ребята! Помогите с задачкой! С помощью функции strcpy() заменить в строке три символа, начиная со второго. Для замены вводим короткую...

Strcpy не работает - C++
Привет :) Написал программу, не понимаю почему не работает. Объясните пожалуйста) КОД #include &lt;iostream&gt; #include...

Функция strcpy - C++
цель очевидна. почему не работает? :-| #include &lt;iostream&gt; void strcpy(char *from, char *to) { to=from; } int main() { ...

Реализация strcpy(); - C++
Какими способами функцию strcpy() библиотеки string можно выразить через указатели?

strdup(), strcpy() - C++
Помогите написать свою ф-ю реализующую те же действия, что и ф-и strdup() , strcpy()


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

Или воспользуйтесь поиском по форуму:
prik
8 / 8 / 1
Регистрация: 01.03.2011
Сообщений: 62
21.09.2011, 16:27     strcpy - как быть? #30
Цитата Сообщение от Nameless One Посмотреть сообщение
Ну, а в вырожденных случаях может сгодиться и strcpy:
Это так кажется. fgets вернул NULL и приплыли.
Yandex
Объявления
21.09.2011, 16:27     strcpy - как быть?
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru