VPF::[C++] Проблема при использовании указателей.

Haddingjar

Дата 14.10.2007, 21:17 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 10
Регистрация: 11.10.2007

Репутация: нет
Всего: нет

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

Код


#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 (ссылка)

(нет голосов)

Загрузка ...

Code Monkey

Профиль
Группа: Участник
Сообщений: 133
Регистрация: 29.1.2007
Где: Москва

Репутация: 4
Всего: 10

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

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

Haddingjar

Дата 14.10.2007, 21:52 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 10
Регистрация: 11.10.2007

Репутация: нет
Всего: нет

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

Evjeny

Дата 14.10.2007, 22:05 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 88
Регистрация: 14.8.2006

Репутация: 3
Всего: 5

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

Код


#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 (ссылка)

(нет голосов)

Загрузка ...

Code Monkey

Профиль
Группа: Участник
Сообщений: 133
Регистрация: 29.1.2007
Где: Москва

Репутация: 4
Всего: 10

Цитата(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 (ссылка)

(нет голосов)

Загрузка ...

аСинизатор

Профиль
Группа: Завсегдатай
Сообщений: 1721
Регистрация: 10.5.2003
Где: Эрец-Исраэль

Репутация: 50
Всего: 88

Или так:

Код


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 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 10
Регистрация: 11.10.2007

Репутация: нет
Всего: нет

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

Код


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

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

Это сообщение отредактировал(а) Haddingjar - 16.10.2007, 19:51

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Центр помощи \| Следующая тема »