What is wrong in C++?

Осваиваю язык С++ на MS VC++ 6.00 Win98.
Написал простую программу для работы с указателями, однако она не работает.
Что тут не так?
Дебагер пишет ошибку: Access Violation

Вот текст файла

// TestLearnDlg.cpp : implementation file
//

#include "stdafx.h"
#include "TestLearn.h"
#include "TestLearnDlg.h"

#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif

………

void CTestLearnDlg::OnButtontest1()
{
char* mystr1;
char* mystr2;

mystr1 = "mytest";
L= strlen(mystr1);
for (i =0; i<L; i++)
{
*mystr2 = *mystr1;
*mystr2 = mytest;
mystr2++;
mystr1++;
}

Вроде бы все учебнику….

Ну да, весь код тут ошибка

mystr2 не имеет своего места в памяти, а мы ему присваиваем значение
*mystr2 = *mystr1;

Скажите, что именно Вы собираетесь делать?

И пожалуйста, пользуйтесь тэгами [code=cpp] [/code]

Я пытаюсь скопировать строку как указано в учебниках, а также в файле strcopy.c.
Указатель mystr2 задан в памяти, поэтому нет проблем записать по этому адресу символ.
Например в strcopy.c. выполняется копирование так
*mystr2++=*mystr++;
Но у меня опять ошибка - Access Violation !
Ваше предложение относительно тэгов я не понял, я пока только учусь.
С большим удовольствием приму любую помощь.

Ок пройдёмся ещё раз по пунктам

Код

// TestLearnDlg.cpp : implementation file // #include "stdafx.h" #include "TestLearn.h" #include "TestLearnDlg.h" #ifdef _DEBUG #define new DEBUG_NEW #undef THIS_FILE static char THIS_FILE[] = __FILE__; #endif void CTestLearnDlg::OnButtontest1() { char* mystr1; char* mystr2;//Тут задекларирован указатель, но куда он показывает? пока не ясно.... mystr1 = "mytest";//Вот первый указатель показывает на константную строку. Это ОК L= strlen(mystr1);//Что такое L? Ладно, мы то тогадаемся , что это int for (i =0; i<L; i++)//Вообще то стринг всегда длиннее на один символ, //который и показывает, что стринг кончился { *mystr2 = *mystr1;//Итак здесь первому элементу mystr2 присваивается первый елемент mystr1. //Но ведь у  mystr2  не элементов.... Мы так и не определились, куда он показывает :) *mystr2 = mytest;// Странная переменная mytest. откуда, зачем? mystr2++; //mystr2 до сих пор не определён :) mystr1++;//Это ОК }

Насколько я понимаю, желается следующее:

Код

// TestLearnDlg.cpp : implementation file // #include "stdafx.h" #include "TestLearn.h" #include "TestLearnDlg.h" #ifdef _DEBUG #define new DEBUG_NEW #undef THIS_FILE static char THIS_FILE[] = __FILE__; #endif void CTestLearnDlg::OnButtontest1() { char* mystr1; char* mystr2; mystr1 = "mytest"; int L= strlen(mystr1); mystr2=new char[L+1];//Дадим всё таки память этому указателю :) for (i =0; i<=L; i++)//последний символ тоже скопируем :) { *mystr2 = *mystr1; //*mystr2 = mytest; смутно представляю, зачем эта строка вообще :) mystr2++; mystr1++; }

Вся конструкция однако заменяется (я бы так написал):

Код

// TestLearnDlg.cpp : implementation file // #include "stdafx.h" #include "TestLearn.h" #include "TestLearnDlg.h" #ifdef _DEBUG #define new DEBUG_NEW #undef THIS_FILE static char THIS_FILE[] = __FILE__; #endif void CTestLearnDlg::OnButtontest1() { char* mystr1; char* mystr2; mystr1 = "mytest"; mystr2=new char[strlen(mystr1)+1];//Дадим всё таки память этому указателю :) while(*mystr2++=*mystr1++); }

насчёт тэгов
сверху есть кнопка КОД
попробуйте вставить в свой пост:
[code=cpp]
#include "stdio.h"
int main(){/*итд.*/}
[/code]

Спасибо за хорошую вещь.
Однако у меня остается смутное впечатление, что все таки я далек от понимания.
Я хочу написать код не пользуясь функциями типа strlen. Опять же в файле strlen.c используется операция *p1=*p2. Почему у меня это не работает?

Относительно вставки [code=cpp] и т.д. то в какой файл это нужно добавить?
Я создал простой проект в MS VC, диалоговое окно с кнопкой OnButtontest1.
При нажатии на эту кнопку я пытаюсь выполнить код. Как в фале strcopy.c.
Еше раз прошу за дремучесть, но уж больно это все интересно.

Сам говорил и сам же ошибсяЖ)))

Код

mystr2=new char[strlen(mystr1)+1];//Дадим всё таки память этому указателю :)

Вставка [code] это в сообщение на форуме. Для того, чтобы код подсвечивался.

А насчёт без strlen так это надо просто mystr2 памяти сразу побольше дать и 3-ий вариант использовать

Цитата(alex22222 @ 21.11.2004, 21:15)

Еше раз прошу за дремучесть

Нет нет, абсолютно ничего страшного мы все здесь друг другу помогаем

PS:
manu, спасибо

А как все таки использовать тело функции strcopy or strrev напрямую, как это делается в файлах strcopy.с or strrev.с. Почему код типа *p2=*p1 не выполняется у меня.

char* mystr1;
char* mystr2;

mystr1 = "mytest";
mystr2 = "notest";

while(*mystr2++=*mystr1++);

тут опять ошибка - Access Violation !
Что не правильно?

Цитата

Почему код типа *p2=*p1 не выполняется у меня.

переключись в релиз версию

Код

char* mystr1; char* mystr2; mystr1 = "mytest"; mystr2 = "notest "; // тута пробельчик надо добавить :) while(*mystr2++=*mystr1++);

Не работает!

По вопросу.

• Во первых не рекомендуется присваивать константным строкам значения. От этого и виолэйшен.
  mystr2 должна указывать на свой участок памяти. Значит закладываться она должна например так:
  

  Код

  char mystr2[512]; //или char * mystr2=new char[512]; //а это означает, что память выделена один раз и менять её нельзя! Указатель всего лишь указывает на констаныную память. char* mystr2= "Вот тут константная строка!";

• Во вторых, дже сли бы строка не была константной, у Вас mystr1 на один символ длиннее чем mystr2 (как правильно заметил bel_nikita).

--------------------------------------------

Кстати
while(*mystr2++=*mystr1++);
Это bel_nikita написал первым (не хочу плагиатить )

И ещё кстати, bel_nikita, опять ты с константными строками балуешься Тем более новичку
Как же релиз потом дебагить?
Добавлено @ 00:20
Вот это вставьте в функцию и всё будет прекрасно работать

Код

char* mystr1; char* mystr2; mystr1 = "mytest"; mystr2=new char[strlen(mystr1)+1];//Дадим всё таки память этому указателю :) while(*mystr2++=*mystr1++); printf(mystr2);

Guest
Последуйте совету sergej.z, заверните свой код в функцию.
Вот, 100% рабочий пример:

Код

#include <stdio.h> void copy(char* out, const char* in) {  while (*out++ = *in++); } int main(int argc, char* argv[]) {  char string1[]="TRA-LA-LA";  char string2[]="LA-LA-TRA";  copy(string1,string2);  printf("\nstring1 = %s",string1);  printf("\nstring2 = %s",string2);  return 0; }

Цитата

Кстати while(*mystr2++=*mystr1++); Это bel_nikita написал первым (не хочу плагиатить )

До меня был Бьерн Страуструп

Цитата

И ещё кстати, bel_nikita, опять ты с константными строками балуешься

Кстати, тут в соседнем топике кое-чего про константы написал

Уважаемые специалисты!
Прошу прощения за дремучесть еще раз.
Проблема вот в чем.
Я создаю проект в MS VC++, диалоговое окно, с кнопкой Buttontest1.
затем я беру тело функции strcopy, please look at source strcopy.c.
затем я пытаюсь выполнить код
void CTestLearnDlg::OnButtontest1()
{
char* mystr1;
char* mystr2;

mystr1 = "mytest";
mystr2 = "notest";

while(*mystr2++=*mystr1++);

}
Все инициализировано!
Однако ошибка - Access Violation !

Я не хочу использовать стандартные функции типа strlen.
Я просто повторяю код из функции strcopy!
тут нет функции типа main() {}
ЭТО MFC!

Код

mystr1 = "mytest"; mystr2 = "notest";

Разве в стандартной strcpy такое есть?

Цитата(alex22222 @ 21.11.2004, 23:43)

тут нет функции типа main() {} ЭТО MFC!

Это не важно

Код

void CTestLearnDlg::OnButtontest1() { char* mystr1; char empty[512]; char* mystr2=empty; mystr1 = "mytest"; while(*mystr2++=*mystr1++); }

Вот рабочий вариант
-------------------------

bel_nikita да я имею ввиду, что в дебаге с таким извращением работать невозможно. То, что скомпиленая верся "пропускает" кучу ошибок я не спорю Просто зачем делать эти ошибки специально ?

alex22222

Код

void copy(char* out, const char* in) {   while (*out++ = *in++); } void CTestLearnDlg::OnButtontest1() {  char mystr1[] = "CONSOLE ";  char mystr2[] = "NEA, MFC";  copy(mystr1,mystr2);  // ну, а тут выводи строки куда хочешь :) }

Это сообщение отредактировал(а) bel_nikita - 22.11.2004, 01:04