ошибка при работе с указателем

[lennon]

Сообщение от rangerx

вторую можно переписать так:

void fnPlus(char *dest, char *src, int dest_buf_size) //dest_buf_size это реальный размер dest массива
{
    /*здесь должны быть все необходимые проверки на то хватит ли места для копирования и т.д.
        ...
    */

    char *p = dest + strlen(dest);
    while(*src) *(p++) = *(src++);
}

P.S. Переход к след элементу осуществляется так p++, а не так, как у вас. Так *p++ вы увеличиваете само значение(если вам так будет понятней, то вы по сути делаете p[1]++).

а попробуйте теперь освободить указатель p и увидете что будет, не советую указатели массивов приравнивать, по сути вы присваиваете адресс в памяти одного указтеля другоу и получаеться тогда что это обсолютно одинаковые указетели. и по сути он не делает p[1]++, во первых нумерация в С++ начинаеться с нуля, во вторых p[1]++ изменяет значение одного байта в массиве на еденицу, а именно этой вашей еденицы. p[1]++ и *p++ не одно и тоже. звездочка указывает на самый первый элемент в массиве а два плюса это начало двигают ) ггггг

[Vladss]

Сообщение от rangerx

В первом случае никакой ошибки, на первый взгляд, быть не должно.
Вопрос скорее в том, что вы передаёте ей в качестве аргумента.

Ответ: массив символов типа char, либо указатель на строку char *....

[Vladss]

Сообщение от Stilet

А где wlen вычисляется?

Прости, ошибся в написании переменной - не wlen, а len......

[rangerx]

а попробуйте теперь освободить указатель p и увидете что будет

Зачем?

не советую указатели массивов приравнивать, по сути вы присваиваете адресс в памяти одного указтеля другоу и получаеться тогда что это обсолютно одинаковые указетели.

В смысле приравнивать? Где? Вы явно что-то напутали.

по сути он не делает p[1]++, во первых нумерация в С++ начинаеться с нуля

1 здесь была как пример, причём здесь начинается с нуля или не начинается?

вторых p[1]++ изменяет значение одного байта в массиве на еденицу, а именно этой вашей еденицы. p[1]++ и *p++ не одно и тоже.

да, здесь я ошибся. Глаз замылился )
++*p или (*p)++; - изменяем само значение.
*p++(но это не тоже самое, что p++) или p++ или p+=1 - изменяем адрес.

звездочка указывает на самый первый элемент в массиве а два плюса это начало двигают ) ггггг

звездочка может указывать хоть на десятый элемент массива, на то это и указатель(см. мой предыдущий пост). Насчёт "двигающих плюсов" ответил выше.

[rangerx]

Сообщение от Vladss

Ответ: массив символов типа char, либо указатель на строку char *....

приведите пример неработающего кода.
P.S. во второй функции вы просто перезаписываете массив, а не присоединяете один к другому. Смотрите пример, который я вам привёл.

[Vladss]

Сообщение от oleg kutkov

Так же не могу не обратить внимание на переменную wlen, судя по всему это какая-то глобальная переменная. //***
Вообще работа с указателям - опасная штука. Нужно быть очень внимательным, особенно при активном их использовании.
Вторую функцию предлагаю полностью переписать/****
А вообще, что бы не изобретать велосипед есть замечательные функции:
strcpy и strncpy.

Спасибо за столь развернутый и познавательный ответ, но!
а) переменная wlen - это я по ошибке....Реально len....
б) насчет strcpy и strncpy я полностью согласен. Просто не могу понять - почему в тех примерах просто не могу присвоить значение *(p++) = *(src++);

[ISergeyN]

void fnReverse(char *str) //переворачиваю строку
{
        char temp;
        int len = strlen(str);
        for(int i = 0; i < len/2; i++) 
        {                   
                temp    =       str[i];    
                str[i]    =       str[len-i-1]; //здесь и возникает ошибка
                str[len-i-1] = temp;   
        }
}

теперь найди отличие в двух листингах.
1, Рабочий

#include <iostream>

void fnReverse(char *str) //переворачиваю строку
{
	char temp;
	int len = strlen(str);
	for(int i = 0; i < len/2; i++) 
	{                   
		temp    =       str[i];    
		str[i]  =       str[len-i-1]; //здесь и возникает ошибка
		str[len-i-1] = temp;   
	}
}

int main() {
	char c[] = "forum";
	fnReverse(c);
	std::cout << c << '\n';
	return 0;
}

2, Не рабочий

#include <iostream>

void fnReverse(char *str) //переворачиваю строку
{
	char temp;
	int len = strlen(str);
	for(int i = 0; i < len/2; i++) 
	{                   
		temp    =       str[i];    
		str[i]  =       str[len-i-1]; //здесь и возникает ошибка
		str[len-i-1] = temp;   
	}
}

int main() {
	char *c = "forum";
	fnReverse(c);
	std::cout << c << '\n';
	return 0;
}

[lennon]

Сообщение от rangerx

Зачем?

В смысле приравнивать? Где? Вы явно что-то напутали.

1 здесь была как пример, причём здесь начинается с нуля или не начинается?

да, здесь я ошибся. Глаз замылился )
++*p или (*p)++; - изменяем само значение.
*p++(но это не тоже самое, что p++) или p++ или p+=1 - изменяем адрес.

звездочка может указывать хоть на десятый элемент массива, на то это и указатель(см. мой предыдущий пост). Насчёт "двигающих плюсов" ответил выше.

зачем нафлудил то?
значит первое: ненужно умничать.
второе: объявлен указатель значит он раположить в динамиской памяти, его нужно освобождать, только переменная в стеке разрушаеться сама когда выходит из области видимости, указатели надо освобождать самому иначе программа будет требовать много ОЗУ а потом когда оно закончиться не будет работать.
третье: ненужно неправильных примеров.
четвертое: двигающиеся плюсы это двигающиеся плюсы
пятое: зачем расписано мне

да, здесь я ошибся. Глаз замылился ) 
++*p или (*p)++; - изменяем само значение.
*p++(но это не тоже самое, что p++) или p++ или p+=1  - изменяем адрес.

я сказал что p[0]++ не одноо и тоже что и *p++ во втором случае вы поставили указатель которые делает смещает адресс в памяти, в первом всего лишь добавилася еденица. ЛОЛ ))) ненадо пожалуйста больше спорить : )

[Vladss]

Сообщение от ISergeyN

теперь найди отличие в двух листингах.
1, Рабочий

char c[] = "forum";
fnReverse(c);

2, Не рабочий
char *c = "forum";
fnReverse(c);

}[/CODE]

Ага!!! Спасибо, теперь сам характер ошибки понятен и теперь всё встает на свои места ....код начинает работать. НО! Объясните, пожалуйста, какая между ними разница при передаче оных в качестве агрументов функции?...........
Имхо *str в void fnReverse(char *str) в любом случае указывает на начало массива!???
Заранее спасибо за ответ!

[ISergeyN]

char *c = "forum";

это костанта и вот по этому у тебя прога и падала при попытке изненить значение.

char c[] = "forum";

а это просто обычный массив с которым можно делать что угодно.