Вызов strcpy для char* дважды
Доброго всем времени суток!:)
Хочу спросить у вас об одной проблеме, внятного объяснения которой я не смог найти.
Проблема вот в чем:
создаются 3 указателя на char:
Затем происходит копирование из st1 в st2:
Здесь все проходит хорошо, но
или
выводит ошибку отказа доступа к памяти.
Если же сделать указатели массивами или выделить динамически под них память все проходит как по маслу.
Я сделал альтернативу strcpy:
но ничего не изменилось.
Я понимаю различия между char* и char [], но тем не менее не могу понять, где собака зарыта. :confused:
Почему в первый раз вызывается нормально, а во второй вылетает?
Почему именно с ошибкой запрета доступа к памяти?
Объясните пожалуйста в чем здесь проблема. :confused:
P.S. Использовался компилятор Builder C++ 6.0, если это имеет значение
Хочу спросить у вас об одной проблеме, внятного объяснения которой я не смог найти.
Проблема вот в чем:
создаются 3 указателя на char:
Код:
char * st1="text", *st2, *st3;
Затем происходит копирование из st1 в st2:
Код:
strcpy(st2,st1)
Здесь все проходит хорошо, но
Код:
strcpy(st3,st1)
Код:
strcpy(st3,st2)
Если же сделать указатели массивами или выделить динамически под них память все проходит как по маслу.
Я сделал альтернативу strcpy:
Код:
void copyString(char *in, char *out)
{
for(int i=0; (out=in)!='\0';i++);
}
{
for(int i=0; (out=in)!='\0';i++);
}
но ничего не изменилось.
Я понимаю различия между char* и char [], но тем не менее не могу понять, где собака зарыта. :confused:
Почему в первый раз вызывается нормально, а во второй вылетает?
Почему именно с ошибкой запрета доступа к памяти?
Объясните пожалуйста в чем здесь проблема. :confused:
P.S. Использовался компилятор Builder C++ 6.0, если это имеет значение
У тебя st2 и st3 не инициализированы и память не выделена. Т.е. st2 И st3 содержат какие-то "мусорные" значения которые интерпретируются как адреса destination буферов.
И что удивительного в данной ситуации?? То, что вылетает ошибка, когда происходит обращение к невыделенной памяти?
Цитата:
Почему в первый раз вызывается нормально
Это частный случай, в общем случае - это грубейшая ошибка.
Цитата: Salamansar
Я сделал альтернативу strcpy:
Код:
void copyString(char *in, char *out)
{
for(int i=0; (out=in)!='\0';i++);
}
{
for(int i=0; (out=in)!='\0';i++);
}
но ничего не изменилось.
Не верю. Наверняка на эту альтернативу уже компилятор ругался, а не программа вылетала.
Цитата: Kogrom
Не верю. Наверняка на эту альтернативу уже компилятор ругался, а не программа вылетала.
А чего ему ругаться?
Кстати, можно было сделать функцию на основе классического примера:
Код:
void copyString(const char* in, char* out)
{
while(*out++ = *in++);
}
{
while(*out++ = *in++);
}
Цитата: Green
А чего ему ругаться?
Происходит обращение к элементам массивов (in, out), которые не были объявлены.
(Изменил ответ, так как ранее он был грубоват и не ясен)
В чем ошибка, уже было сказано. А вот правильный пример (для Си):
З.Ы. При работе со строками в Билдере обычно используют класс AnsiString. В этом случае память выделяется и освобождается автоматически.
Код:
#include <stdio.h>
#include <string.h>
#include <malloc.h>
int main()
{
char * st1="text", *st2, *st3;
// выделяем память
st2=malloc(strlen(st1)+1);
st3=malloc(strlen(st1)+1);
// копируем строки
strcpy(st2,st1);
strcpy(st3,st1);
// выводим строки
printf("%s\n%s\n%s\n", st1, st2, st3);
// освобождаем память
free(st2);
free(st3);
// конец
return 0;
}
#include <string.h>
#include <malloc.h>
int main()
{
char * st1="text", *st2, *st3;
// выделяем память
st2=malloc(strlen(st1)+1);
st3=malloc(strlen(st1)+1);
// копируем строки
strcpy(st2,st1);
strcpy(st3,st1);
// выводим строки
printf("%s\n%s\n%s\n", st1, st2, st3);
// освобождаем память
free(st2);
free(st3);
// конец
return 0;
}
З.Ы. При работе со строками в Билдере обычно используют класс AnsiString. В этом случае память выделяется и освобождается автоматически.
Цитата: Ден Зурин
З.Ы. При работе со строками в Билдере обычно используют класс AnsiString. В этом случае память выделяется и освобождается автоматически.
При работе со строками на C++ обычно используют класс std::string.
А ещё есть функция
char *strdup(const char *).
malloc() и strcpy() в одном флаконе.
#include <stdio.h>
#include <string.h>
#include <malloc.h>
int main()
{
char * st1="text", *st2, *st3;
st2=strdup(st1);
st3=strdup(st1);
// выводим строки
printf("%s\n%s\n%s\n", st1, st2, st3);
// освобождаем память
free(st2);
free(st3);
// конец
return 0;
}
char *strdup(const char *).
malloc() и strcpy() в одном флаконе.
#include <stdio.h>
#include <string.h>
#include <malloc.h>
int main()
{
char * st1="text", *st2, *st3;
st2=strdup(st1);
st3=strdup(st1);
// выводим строки
printf("%s\n%s\n%s\n", st1, st2, st3);
// освобождаем память
free(st2);
free(st3);
// конец
return 0;
}
Цитата: Kogrom
Происходит обращение к элементам массивов (in, out), которые не были объявлены.
(Изменил ответ, так как ранее он был грубоват и не ясен)
(Изменил ответ, так как ранее он был грубоват и не ясен)
Смотрим стандарт:
Цитата:
5.2.1 Subscripting [expr.sub]
1 A postfix expression followed by an expression in square brackets is a postfix expression. One of the expressions shall have the type “pointer to T” and the other shall have enumeration or integral type. The result is an lvalue of type “T.” The type “T” shall be a completely-defined object type.56) The expression E1[E2] is identical (by definition) to *((E1)+(E2)). [Note: see 5.3 and 5.7 for details of * and + and 8.3.4 for details of arrays. ]
Цитата: Green
Смотрим стандарт:
Извиняюсь. Ошибался. Привык к использованию адресной арифметики при работе с указателями.
То, что из-за памяти вылетает это понятно, но почему всеже все нормально проходит в первый раз и только потом память для доступа "закрывется"?:confused:
[QUOTE=Ден Зурин]При работе со строками в Билдере обычно используют класс AnsiString. В этом случае память выделяется и освобождается автоматически.[/QUOTE]
[QUOTE=Green]При работе со строками на C++ обычно используют класс std::string.[/QUOTE]
Спасибо за совет, но мне нужно обрабатывать строки именно таким примитивным образом - через char* :)
[QUOTE=Trilobit]
char *strdup(const char *)
[/QUOTE]
Спаисбо, приму к сведению :)
[QUOTE=Kogrom]Не верю. Наверняка на эту альтернативу уже компилятор ругался, а не программа вылетала.[/QUOTE]
Уж поверь - мне врать ни к чему :)
[QUOTE=Ден Зурин]При работе со строками в Билдере обычно используют класс AnsiString. В этом случае память выделяется и освобождается автоматически.[/QUOTE]
[QUOTE=Green]При работе со строками на C++ обычно используют класс std::string.[/QUOTE]
Спасибо за совет, но мне нужно обрабатывать строки именно таким примитивным образом - через char* :)
[QUOTE=Trilobit]
char *strdup(const char *)
[/QUOTE]
Спаисбо, приму к сведению :)
[QUOTE=Kogrom]Не верю. Наверняка на эту альтернативу уже компилятор ругался, а не программа вылетала.[/QUOTE]
Уж поверь - мне врать ни к чему :)
Цитата: Salamansar
То, что из-за памяти вылетает это понятно, но почему всеже все нормально проходит в первый раз и только потом память для доступа "закрывется"?:confused:
Да ничего потом не закрывается - сказали же, это частный случай.
Просто первый раз не сделало ничего критического, чтобы сразу порушить программу, или сделала но это не обнаружилось сразу. А второй раз уже обнаружилось.
Просто запомни, что так делать нельзя - что то писать по неизвестно куда указываемому указателю.