Ошибка "Неправильный указатель"

@HAIRY · Регистрация: 24.01.2011

Author24 — интернет-сервис помощи студентам

C++

#include <conio.h>
#include "stdafx.h"
#include <cstdlib>
#include <iostream>
#include <windows.h>
 
using namespace std;
 
class Account 
{     public:
      void   setName(char*);
      char*  getName();
 
      private:
      char*  Name;
   
};
 
void Account::setName(char* n)
{
     Name = n;
}
 
char* Account::getName()
{
      return Name;
}
 
 
int main(int argc, char *argv[])
{
    Account a;
    char* n;
    cin >> n;
    a.setName(n);
    cout << a.getName();
    system("PAUSE");
    return EXIT_SUCCESS;
}

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

@nmcf · 17.04.2014, 20:36

Так ты память сначала выдели под строку n.
У тебя в целом идея неверная. Класс должен иметь указатель на свою копию строки. Ну мне так кажется. Иначе зачем он вообще нужен?

@~~newbie666~~ · 17.04.2014, 20:38

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

почему у меня этот код не хочет работать?

лично я даже читать твой код не хочу, зачем изучать твой код за тебя? Если б это было две строки...
Грамотный человек скажет, на какой строчке выскакивает ошибка, это раз, во вторых, что значит компилятор говорит" По русский он говорит? FATAL_ERROR программировать на не ENG версиях VS.

@HAIRY · 17.04.2014, 20:44 **[ТС]**

вот так в видеоуроке и у него все работает

@HAIRY · 17.04.2014, 20:47 **[ТС]**

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

лично я даже читать твой код не хочу, зачем изучать твой код за тебя? Если б это было две строки...

так это итак две строки

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

Грамотный человек скажет, на какой строчке выскакивает ошибка, это раз,

на строчке

C++

cin >> n;

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

во вторых, что значит компилятор говорит" По русский он говорит? FATAL_ERROR программировать на не ENG версиях VS.

поставил ENG

@nmcf · 17.04.2014, 20:47

Это заголовок класса. Где остальное?
Читай, что я написал cin >> n не будет работать с пустым указателем. надо либо память выделить для n, либо объявить char n[50], к примеру.

@HAIRY · 17.04.2014, 20:53 **[ТС]**

вот остальное

@nmcf · 17.04.2014, 21:10

Здесь инициализируют константами, поэтому ошибки нет. Сделай для начала как в моём предыдущем сообщении.

alsav22 · 17.04.2014, 21:18

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

хотя в видеоуроке так же все написано и все работает.

И где там так же (в main())?

@HAIRY · 17.04.2014, 22:10 **[ТС]**

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

И где там так же (в main())?

ну раньше у меня и так работало. я вводил с клавиатуры имя.

Добавлено через 48 секунд

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

Здесь инициализируют константами, поэтому ошибки нет. Сделай для начала как в моём предыдущем сообщении.

так все работает, ну я не хочу, чтобы выделялась лишняя память

alsav22 · 17.04.2014, 22:13

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

ну раньше у меня и так работало. я вводил с клавиатуры имя.

Тогда как понять это?

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

почему у меня этот код не хочет работать?

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

я не хочу, чтобы выделялась лишняя память

Кто вам сказал, что она лишняя?

@HAIRY · 17.04.2014, 22:34 **[ТС]**

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

Кто вам сказал, что она лишняя?

ну например я выделю массив [50], а займу только 10 символов. [40] лишнии

@nmcf · 17.04.2014, 22:41

Ты как-то слишком рано задумался об оптимизации.
cin сам память не выделяет. Других вариантов нет. Определи, сколько может быть максимально введно символов и столько задай.

@HAIRY 32 / 25 / 12 Регистрация: 24.01.2011 Сообщений: 564
	17.04.2014, 22:34 [ТС]	12
	Сообщение от alsav22 Кто вам сказал, что она лишняя? ну например я выделю массив [50], а займу только 10 символов. [40] лишнии 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	17.04.2014, 20:36	2
	Так ты память сначала выдели под строку n. У тебя в целом идея неверная. Класс должен иметь указатель на свою копию строки. Ну мне так кажется. Иначе зачем он вообще нужен? 0

@~~newbie666~~ Заблокирован
	17.04.2014, 20:38	3
	Сообщение от HAIRY почему у меня этот код не хочет работать? лично я даже читать твой код не хочу, зачем изучать твой код за тебя? Если б это было две строки... Грамотный человек скажет, на какой строчке выскакивает ошибка, это раз, во вторых, что значит компилятор говорит" По русский он говорит? FATAL_ERROR программировать на не ENG версиях VS. 0

@HAIRY 32 / 25 / 12 Регистрация: 24.01.2011 Сообщений: 564
	17.04.2014, 20:44 [ТС]	4
	вот так в видеоуроке и у него все работает Миниатюры 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	17.04.2014, 20:47	6
	Это заголовок класса. Где остальное? Читай, что я написал cin >> n не будет работать с пустым указателем. надо либо память выделить для n, либо объявить char n[50], к примеру. 1

@HAIRY 32 / 25 / 12 Регистрация: 24.01.2011 Сообщений: 564
	17.04.2014, 20:53 [ТС]	7
	вот остальное Миниатюры 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	17.04.2014, 21:10	8
	Здесь инициализируют константами, поэтому ошибки нет. Сделай для начала как в моём предыдущем сообщении. 0

alsav22 5499 / 4894 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	17.04.2014, 21:18	9
	Сообщение от HAIRY хотя в видеоуроке так же все написано и все работает. И где там так же (в main())? 0

@HAIRY 32 / 25 / 12 Регистрация: 24.01.2011 Сообщений: 564
	17.04.2014, 22:10 [ТС]	10
	Сообщение от alsav22 И где там так же (в main())? ну раньше у меня и так работало. я вводил с клавиатуры имя. Добавлено через 48 секунд Сообщение от nmcf Здесь инициализируют константами, поэтому ошибки нет. Сделай для начала как в моём предыдущем сообщении. так все работает, ну я не хочу, чтобы выделялась лишняя память 0

alsav22 5499 / 4894 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	17.04.2014, 22:13	11
	Сообщение от HAIRY ну раньше у меня и так работало. я вводил с клавиатуры имя. Тогда как понять это? Сообщение от HAIRY почему у меня этот код не хочет работать? Сообщение от HAIRY я не хочу, чтобы выделялась лишняя память Кто вам сказал, что она лишняя? 0

@nmcf 7803 / 6567 / 2988 Регистрация: 14.04.2014 Сообщений: 28,705
	17.04.2014, 22:41	13
	Ты как-то слишком рано задумался об оптимизации. cin сам память не выделяет. Других вариантов нет. Определи, сколько может быть максимально введно символов и столько задай. 0