Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > Центр помощи > [C++] Проблема при использовании указателей.

Автор: Haddingjar 14.10.2007, 21:17

Доброго времени суток!
Начал изучать С++. В данный момент столкнулся с небольшой неприятностью...
Вот код:

Код


#include <iostream>
using namespace std;
int slen(char* a)
{
        int i = 0;
        while(a[i] != '\0') i++;
        return i;
}

void scpy(char* a, char* b)
{
        int b_len = slen(b);

        if(a) delete [] a;
        a = new char[b_len + 1];
        for(int i = 0; i <= b_len; i++)
                a[i] = b[i];
}

int main()
{
        char *a = new char[slen("123") + 1]; a = "123";
        char *b = new char[slen("df33re") + 1]; b = "df33re";

        scpy(a, b); cout << a << endl;

        delete [] a; delete [] b;

        return 0;
}

А вот процесс компилляции:

Код


x@y:~/coding$ g++ foo.cpp -o q
x@y:~/coding$ ./q
*** glibc detected *** double free or corruption (!prev): 0x08048968 ***
Aborted
x@y:~/coding$

Не могли бы Вы "ткнуть носом" меня в ошибки? В каком месте я туплю?

Автор: sentry 14.10.2007, 21:41

Просто при выполнении a = "123" указатель перескакивает на эту строку, хранящуюся в сегменте данных, а не заполняет твой созданный массив. А потом ты пытаешься удалить, собственно, строку "123", но поскольку ты ее не выделял и она хранится в неизменяемом сегменте, возникает ошибка.

Добавлено через 6 минут и 39 секунд
И еще: после возвращения из процедуры scpy a не будет содержать указатель на новый созданный там массив, так как указатель в процедуру был передан по значению.

Автор: Haddingjar 14.10.2007, 21:52

Большое спасибо за ответ!
Действительно, если заполнять созданные массивы поэлементно, то все работает корректно...
Но мне все-же интересно, каким образом следует заполнять созданные массивы сразу одной строкой, указанной в двойных кавычках? Как бы Вы переписали вышеуказанный код?

Автор: Evjeny 14.10.2007, 22:05

Я бы сделал вот так:

Код


#include <iostream>

using namespace std;

int slen(char* a)
{
        int i = 0;
        while(a[i] != '\0') i++;
        return i + 1;
}

void scpy(char* a, char* b)
{
        while(*b)
            *a++ = *b++;
        *a = '\0';
}

int main()
{
        char* b = "123456789";

        char* a = new char[slen(b)];

        scpy(a, b);

        cout << a << endl;

        delete [] a;

        return 0;
}

Автор: sentry 14.10.2007, 22:47

Цитата(Haddingjar @ 14.10.2007, 23:22

)

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

Вроде можно только копированием.

Цитата(Evjeny @ 14.10.2007, 23:35

)

Я бы сделал вот так:

Тогда уж вот так:

Код


#include <iostream>

int slen(char *a)
{
    int result = 0;
    while (*a++) ++result;
    return result;
}

void scpy(char* a, char* b)
{
    while (*a++ = *b++);
}

void main()
{
    char* b = "123456789";
    char* a = new char[slen(b)+1];

    scpy(a, b);
    std::cout << a << std::endl;

    delete []a; 
    std::cin.get();
}

Автор: Dov 14.10.2007, 23:13

Или так:

Код


int slen(char* a)
{    
    for(char* p = a; *p; p++);

    return (p - a);
}

void scpy(char* a, char* b)
{
    if(slen(b) > slen(a))
        return;

    while(*a++ = *b++);
}

Автор: Haddingjar 15.10.2007, 20:52

Огромное всем спасибо за помощь!
Просто я многого захотел при работе с указателями =) Да и, скорее всего, недостаточно усвоил эту тему.
Вариант работы с оными:

Код


int main()
{
        char* b = "123456789";
        char* a = new char[slen(b)];
        scpy(a, b);
        cout << a << endl;
        delete [] a;
        return 0;
}

действительно работает, что и логично.