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

[Vladss]

Программирую недавно, над вопросом бъюсь второй день.
Вопрос следующий: при присвоении значения одного указателя другому, возникает ошибка "Access violation writing location". Что я делаю неправильно? Общий случай:

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

либо:

char *fnPlus(char *t, char *s)           //складываю две строки
{
 	char *temp = t;
	while(*s) *t++ = *s++;       //здесь и возникает ошибка
	*t = '\0';
	return temp;
}

Подскажите, пожалуйста, как избежать ошибки?

[Stilet]

wlen-i-1

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

[megachuhancer]

fnPlus работает вроде нормально, только она копирует одну строку в другую, а не складывает их. Ошибка может быть, например, из-за того, что нет нулевого символа в пределах s.

[Stilet]

while(*s) *t++ = *s++; //здесь и возникает ошибка

А с чего ты взял что этот код сложит тебе строки? И почему бы не воспользоваться стандартными функциями?

[pavelstraut]

скажите что делают эти 2 строки

int* *c;(особенно это интересует)
int* a;
c=&a;

[rangerx]

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

[lennon]

str[i] = str[wlen-i-1]; //здесь и возникает ошибка

размер массива не соответсвует wlen-i-1 в какомто такте из циклов, программа обратилася по несуществующему адрессу и поэтому вылетает.

while(*s) *t++ = *s++; //здесь и возникает ошибка
аналогично массив s может быть больше чем массив t тогда шоли условие написть надо так while (*s | *t) вообщем вердикт такой: у тебя memory leaks в этих вдух случаях из за неверного размера "массива- получателя" хехехе

[oleg kutkov]

Так же не могу не обратить внимание на переменную wlen, судя по всему это какая-то глобальная переменная. Это плохой стиль программирования.
Vladss, старайтесь не использовать глобальные переменные, хоть иногда это и наиболее простое решение, передавайте все что нужно функции через ее параметры.
Т.к. в данном случае не известно, как инициализируется wlen, то ошибка, судя по всему, возникает из-за некорректного значения этой переменной (обращение к несуществующему индексу массива, т.е. запрещенной, для данной программы, области памяти). Вообщем объявление wlen - в студию.
Вообще работа с указателям - опасная штука. Нужно быть очень внимательным, особенно при активном их использовании. В Вашем случае указатели используется очень рискованно и неразумно, поэтому вторую функцию предлагаю полностью переписать:

#include <stdio.h>
#include <malloc.h>
#include <string.h>

char *fnPlus(char *t, char *s)
{
	unsigned int len = strlen(t)+strlen(s)+1; //вычисляем длину итоговой строки
	char *temp = (char*)malloc(len); //выделяем необходимый объем памяти, и создаем указатель на нее - временный буфер
	memcpy(temp, t, strlen(t)); //копируем первую строку в наш буфер
	memcpy(temp+strlen(t), s, strlen(s)); //копируем в остаток буфера вторую
	temp[len-1] = 0; //финальный ноль
	return temp; //возвращаем результат
}

Остается одно большое НО! В функции мы динамически выделяем память, и после выхода из функции память остается выделенной. В результате получаем печальное последствие - утечку памяти, хоть маленькую. Эту память необходимо освободить. В самой функции этого делать нельзя, т.к. если мы сделаем освобождение до return temp; - возвращать уже будет нечего, а если после - этот код выполнятся уже не будет, т.к. функция завершится раньше. Для решения проблемы можно поступить так:

	char *res = fnPlus("one", "second");
	printf(res);
	free(res);

Результат вызова нашей функции мы присваиваем указателю res, т.е. теперь res = temp и указывают на ОДНУ И ТУ ЖЕ область памяти. Отображаем результат и теперь мы можем смело вызвать free(res); для освобождения занятой памяти.
А вообще, что бы не изобретать велосипед есть замечательные функции:
strcpy и strncpy.

[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]++).

[Vladss]

от автора))) Спасибо всем за отзыв..........
а)Прошу прощения - в строке
int len = strlen(str) имелось ввиду
int wlen = strlen(str); т.ч. это обычная переменная......
б)цит: "А с чего ты взял что этот код сложит тебе строки? И почему бы не воспользоваться стандартными функциями?" while(*s) *t++ = *s++;
Со стандартными всё ясно! Не могу понять, почему в аналогичных ситуациях я не могу присвоить одниму указателю значение другого! Может быть это очевидно, а может это очевидная чепуха, т.ч. простите за вопрос!