Преобразование string* в string

@dreamn · Регистрация: 25.09.2016

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

Здравствуйте, друзья.
У меня есть класс. В нем описано поле

C++
1
string *GPS;

В конструкторе выделяю память. Передаю именно *_GPS для использования метода инициализации.

C++
1
2
3
Class::Class (string *_GPS) {
GPS = new string;
}

Мне необходимо сначала считать строку GPS из файла. Но как без проблем ее считать не удалось найти, поэтому решил обходом сделать

C++
1
2
3
4
5
6
char *buf = new char [100];
fstream ip ("data/ip.txt", ios::in);
ip.getline(buf, 100);
ip.close();
string TEMP_GPS (buf, strlen(buf));
*GPS = TEMP_GPS;

И последняя строка у меня не работает. Код компилируется, но падает на этой строчке.

Подскажите, пожалуйста, как мне лучше считать в ДИНАМИЧЕСКУЮ string* строку объекта С ПРОБЕЛАМИ?
Может какой-нибудь другой способ есть, а он 100% есть?
Заранее спасибо)

@hoggy · 21.10.2016, 12:00

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

Необходимо динамический string

std::string - замечательный,
самый динамический из всех динамических стрингов.

@John999 · 21.10.2016, 12:02

hoggy, только он его не так использует

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

string*

@dreamn · 21.10.2016, 12:13 **[ТС]**

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

Необходимо динамический string : string *
перепутал с char* ?

Нет, не перепутал. Именно string
И именно динамический

Добавлено через 1 минуту

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

Цитата Сообщение от dreamn Посмотреть сообщение
delete this;
по рукам еще не били?

Так объяснял препод по программированию. Типа, если инициализация с ошибкой, то удаляем объект.

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

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

hoggy, только он его не так использует
Цитата Сообщение от dreamn Посмотреть сообщение
string*

А как правильно?

nd2 · 21.10.2016, 12:39

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

Скинул свой код.
По сути точно также описано все, но падает на строчке чтения

Падает, скорее всего, из-за того, что после инициализации у тебя GPS указывает не на ту память, которая выделялась в конструкторе. При инициализации, нужно не указатели присваивать (при этом адрес, выделенной в конструкторе памяти, теряется):

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

C++
1
GPS = _GPS;

а содержимое по указателям.
Покажи, как инициализацию делаешь.

@dreamn · 21.10.2016, 13:06 **[ТС]**

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

Покажи, как инициализацию делаешь.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
bool phoneClass::Init(float _thickness, float _width, float _length, float _weight, string* _model, string* _mark, string* _GPS) {
    cout << "bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)" << endl;
    if ((_thickness>0)&&(_width>0)&&(_length>0)&&(_weight>0)&&(!_model->empty())&&(!_mark->empty())) {
        *thickness = _thickness;
        *width = _width;
        *length = _length;
        *weight = _weight;
        model = _model;
        mark = _mark;
        GPS = _GPS;
 
        return true;
    }
    else {
        cout << "Error bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)" << endl;
        return false;
    }
}

nd2 · 21.10.2016, 13:24

dreamn, думаешь я этого кода не видел? Как делаешь покажи, вызов покажи. Чтобы было видно: что, и как ты туда передаёшь.

@dreamn · 21.10.2016, 13:32 **[ТС]**

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

dreamn, думаешь я этого кода не видел? Как делаешь покажи, вызов покажи. Чтобы было видно: что, и как ты туда передаёшь.

это в main.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
     float   _thickness,      //Толщина
            _width,          //Ширина
            _length,         //Длина
            _weight;         //Вес
    string  *_model = new string,   // Модель
            *_mark = new string,    // Марка
            *_GPS = new string;
    bool err;
 
*_model = "Crew";
    *_mark = "LOL";
    *_GPS = "Unknown";
phoneClass *phone = new phoneClass (_thickness, _width, _length, _weight, _model, _mark, _GPS, err);
    phone->actFlashligth();
    phone->actFlashligth();
    phone->actGPS();

nd2 · 21.10.2016, 13:42

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

При инициализации, нужно не указатели присваивать (при этом адрес, выделенной в конструкторе памяти, теряется):

C++
1
GPS = _GPS;

а содержимое по указателям.

C++
1
2
3
*model = *_model;
*mark = *_mark;
*GPS = *_GPS;

Покажи как в main() указатели инициализируются (_GPS и пр.).

@dreamn · 21.10.2016, 13:46 **[ТС]**

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

Покажи как в main() указатели инициализируются.

C++
1
2
3
4
5
6
7
8
9
10
int main(int argc, char** argv) {
    
    float   _thickness,      //Толщина
            _width,          //Ширина
            _length,         //Длина
            _weight;         //Вес
    string  *_model = new string,   // Модель
            *_mark = new string,    // Марка
            *_GPS = new string;
    bool err;

nd2 · 21.10.2016, 13:47

Понятно. Совет прежний.

@dreamn · 21.10.2016, 13:51 **[ТС]**

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

Понятно. Совет прежний.

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

MAIN.CPP

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include <cstdlib>
#include <iostream>
#include <string>
#include "classes/phoneClass.h"
#include <termios.h>
#include <unistd.h>
#include <stdio.h>
#include "H.h"              //Собственные переопределенные функции 
#include <stdlib.h>
using namespace std;
 
/*
 * 
 */
 
int main(int argc, char** argv) {
    
    float   _thickness,      //Толщина
            _width,          //Ширина
            _length,         //Длина
            _weight;         //Вес
    string  *_model = new string,   // Модель
            *_mark = new string,    // Марка
            *_GPS = new string;
    bool err;
    
  
    
    *_model = "Crew";
    *_mark = "LOL";
    *_GPS = "Unknown";
    
    phoneClass *phone = new phoneClass (_thickness, _width, _length, _weight, _model, _mark, _GPS, err);
    phone->actFlashligth();
    phone->actFlashligth();
    phone->actGPS();
  
 
    return 0;
}

phoneClass.h

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#ifndef PHONECLASS_H
#define PHONECLASS_H
 
#include <string>
 
using namespace std;
 
class phoneClass {
private:
    float   *thickness,      //Толщина
            *width,          //Ширина
            *length,         //Длина
            *weight;         //Вес
    string  *model,          //Модель
            *mark;           //Марка
    string  *GPS;            //Местонахождение
    bool err;                //Индентификатор ошибки 
    bool Init (float, float, float, float, string*, string*, string*);
    
public:
    bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
    
    phoneClass();
    phoneClass(float, float, float, float, string*, string*, string*, bool&);
    void actScreenshot();        //Скриншот
    void actFlashligth();        //Фонарик
    void actBrowser();           //Браузкер
    void actGPS();               //GPS
    int actContacts();          //Контакты
    int actCall();              //Звонок
    int actSms();               //Смс
    virtual ~phoneClass();
    void setMark(string* mark);
    string* getMark() const;
    void setModel(string* model);
    string* getModel() const;
    void setWeight(float* weight);
    float* getWeight() const;
    void setLength(float* length);
    float* getLength() const;
    void setWidth(float* width);
    float* getWidth() const;
    void setThickness(float* thickness);
    float* getThickness() const;
    
};
 
#endif /* PHONECLASS_H */

phoneClass.cpp

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
#include "phoneClass.h"
#include <iostream>
#include <string>
#include <fstream>
#include <cstring>
 
using namespace std;
 /*
      float   *thickness,      //Толщина
            *width,          //Ширина
            *length,         //Длина
            *weight;         //Вес
    string  *model,
            *mark;
  */
 
 
bool Init (float, float, float, float, string*, string*, string*);
 
phoneClass::phoneClass() {
    cout << "phoneClass::phoneClass()" << endl;
    thickness = new float;
    width = new float;
    length = new float;
    weight = new float;
    model = new string;
    mark = new string;
    GPS = new string;
}
 
phoneClass::phoneClass(float _thickness, float _width, float _length, float _weight, string* _model, string* _mark, string* _GPS, bool &err) {
    cout << "phoneClass::phoneClass(float _thickness, float _width, float _length, float _weight, string* _model, string* _mark, bool &err)" << endl;
 
    thickness = new float;
    width = new float;
    length = new float;
    weight = new float;
    model = new string;
    mark = new string;
    GPS = new string;
 
    if (!Init(_thickness, _width, _length, _weight, _model, _mark, _GPS)) {
        cout << "Error in phoneClass::phoneClass(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)" << endl;
        delete this;
        err = false;
    }
    else err = true;
}
 
phoneClass::~phoneClass() {
    cout << "phoneClass::~phoneClass()" << endl;
    delete thickness;
    delete width;
    delete length;
    delete weight;
    delete model;
    delete mark;
    delete GPS;
}
 
bool phoneClass::Init(float _thickness, float _width, float _length, float _weight, string* _model, string* _mark, string* _GPS) {
    cout << "bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)" << endl;
    if ((_thickness>0)&&(_width>0)&&(_length>0)&&(_weight>0)&&(!_model->empty())&&(!_mark->empty())) {
        *thickness = _thickness;
        *width = _width;
        *length = _length;
        *weight = _weight;
        model = _model;
        mark = _mark;
        *GPS = *_GPS;
 
        return true;
    }
    else {
        cout << "Error bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)" << endl;
        return false;
    }
}
 
void phoneClass::actFlashligth() {
    cout << "void phoneClass::actFlashligth()" << endl;
 
    if (iF == false) {
        cout << "Flashligth is ON" << endl;
        iF = true;
    }
    else {
        cout << "Flashligth is OFF" << endl;
        iF = false;
    }
}
 
void phoneClass::actGPS() {
    cout << "void phoneClass::actGPS()" << endl;
    char *buf = new char [100];
    
    if (iG == false) {
        cout << "iG = FALSE" << endl;
        ifstream f_in( "data/ip.txt" );
        cout << "FILE OPENED" << endl;
        getline( f_in, *GPS );
        cout << "READ STR" << endl;
        /*fstream ip ("data/ip.txt", ios::in);
        
        ip.getline(buf, 100);
        
        cout << "BUF = " << buf << endl;
        ip.close();
        string TEMP_GPS (buf, strlen(buf));
        cout << "TEMP_GPS = " << TEMP_GPS << endl;
        *GPS = TEMP_GPS;
        cout << "GPS = " << *GPS << endl;
        cout << "FILE CLOSED" << endl;*/
        iG = true;
        cout << "iG = TRUE" << endl;
    }
    else {
        cout << "iG = TRUE" << endl;
        *GPS = "Unknown";
        cout << "GPS = UNKNOWN" << endl;
        iG = false;
        cout << "iG = FALSE" << endl;
    }
}
 
void phoneClass::setMark(string* mark) {
    this->mark = mark;
}
 
string* phoneClass::getMark() const {
    return mark;
}
 
void phoneClass::setModel(string* model) {
    this->model = model;
}
 
string* phoneClass::getModel() const {
    return model;
}
 
void phoneClass::setWeight(float* weight) {
    this->weight = weight;
}
 
float* phoneClass::getWeight() const {
    return weight;
}
 
void phoneClass::setLength(float* length) {
    this->length = length;
}
 
float* phoneClass::getLength() const {
    return length;
}
 
void phoneClass::setWidth(float* width) {
    this->width = width;
}
 
float* phoneClass::getWidth() const {
    return width;
}
 
void phoneClass::setThickness(float* thickness) {
    this->thickness = thickness;
}
 
float* phoneClass::getThickness() const {
    return thickness;
}

Быть может теперь получится что-то решить..

nd2 · 21.10.2016, 13:58

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

Быть может теперь получится что-то решить..

Что решить? Исправления делаешь? Пробуешь? Результат? Вот смотри, аналогичный код, с исправлением в init():

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
class Class
{
    string* GPS;
    
public:
    Class()
    { GPS = new string; }
    ~Class()
    { delete GPS; }
    void init(string* str)
    {
        *GPS = *str;
    }
    void read()
    {
        ifstream fin ("file.txt", ios::in);
        if (fin)
        {
            getline(fin, *GPS);
            fin.close();
        }
    }
    const string* get() const
    { return GPS;}
 
};
 
 
int main()
{
    Class obj;
    string* str = new string;
    *str = "Unknown";
    obj.init(str);
    cout << *(obj.get()) << endl;
    obj.read();
    cout << *(obj.get()) << endl;
    
    delete str;
    
    system("pause");
    return 0;
}

Всё работает.

Добавлено через 2 минуты

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

C++
1
2
3
model = _model; 
mark = _mark; 
*GPS = *_GPS;

...

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

C++
1
2
3
*model = *_model;
*mark = *_mark;
*GPS = *_GPS;

@dreamn · 21.10.2016, 14:08 **[ТС]**

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

..

Ты же видишь, что model и mark я не использую
Поэтому не из-за них причина неработоспособности кода

nd2 · 21.10.2016, 14:11

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

C++
1
delete this;

Насчёт этого уже написали.

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

по рукам еще не били?

По барабану?

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

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

Ты же видишь, что model и mark я не использую

И что? Исправлять не нужно?

@dreamn · 21.10.2016, 14:59 **[ТС]**

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

Насчёт этого уже написали.

Так я спросил: что не то? Как правильно? Почему по барабану?

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

И что? Исправлять не нужно?

Исправил - безрезультатно.

Попробуй скопировать мой код, внести свои корректировки. Если заработает - оповести, буду сидеть и думать, в чем же проблема.

Добавлено через 5 минут
Что это означает?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
               ^
classes/phoneClass.h:34:27: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
                           ^
classes/phoneClass.h:34:39: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
                                       ^
classes/phoneClass.h:34:51: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
                                                   ^
classes/phoneClass.h:34:65: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено
                                                                 ^
classes/phoneClass.h:34:80: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11
     bool iF = false, iS = false, iB = false, iG = false, iCon = false, iCall = false, iSms = false; //Включено/Выключено

Добавлено через 34 минуты
СЛОЖНАА

@GbaLog- · 21.10.2016, 15:03

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

Быть может теперь получится что-то решить..

Запустил твой код, результаты:

Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Process started >>>
phoneClass::phoneClass(float _thickness, float _width, float _length, float _weight, string* _model, string* _mark, bool &err)
bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)
Error bool phoneClass::Init(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)
Error in phoneClass::phoneClass(float* _thickness, float* _width, float* _length, float* _weight, string* _model, string* _mark)
phoneClass::~phoneClass()
void phoneClass::actFlashligth()
Flashligth is ON
void phoneClass::actFlashligth()
Flashligth is OFF
void phoneClass::actGPS()
iG = FALSE
FILE OPENED
READ STR
iG = TRUE
<<< Process finished. (Exit code 0)

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

@dreamn · 21.10.2016, 15:04 **[ТС]**

Преобразование string* в string
Вот код.. Выручайте. Советы не помогли.

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

Сообщение от GbaLog-

Возможно, что твой компилятор запускает приложение

А вы какой компилятор используете?

@GbaLog- · 21.10.2016, 15:06

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

Что это означает?

Нельзя инициализировать переменные-члены в самом классе, это надо сделать в конструкторе, например, или метод создать.
Но вообще, можно просто С++11 включить.

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

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

А вы какой компилятор используете?

Тьфу, ошибся в слове. Я хотел сказать не компилятор, а среда разработки.
Я использую для компиляции командную строку, запускаю оттуда же.

@dreamn · 21.10.2016, 15:09 **[ТС]**

Сообщение от GbaLog-

Нельзя инициализировать переменные-члены в самом классе, это надо сделать в конструкторе, например, или метод создать.

Исправим

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

Сообщение от GbaLog-

Тьфу, ошибся в слове. Я хотел сказать не компилятор, а среда разработки.

Пользуюсь NetBeans 8.2

@GbaLog- · 21.10.2016, 15:11

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

Исправим

Перепиши у себя функцию и посмотри, открывается ли файл вообще:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
void phoneClass::actGPS() {
    cout << "void phoneClass::actGPS()" << endl;
    char *buf = new char [100];
    
    if (iG == false) {
        cout << "iG = FALSE" << endl;
        ifstream f_in( "data/ip.txt" );
        if( !f_in.is_open() )
        {
            std::cout << "can't open file" << std::endl;
        }
        cout << "FILE OPENED" << endl;
        getline( f_in, *GPS );
        cout << "READ STR" << endl;
        /*fstream ip ("data/ip.txt", ios::in);
        
        ip.getline(buf, 100);
        
        cout << "BUF = " << buf << endl;
        ip.close();
        string TEMP_GPS (buf, strlen(buf));
        cout << "TEMP_GPS = " << TEMP_GPS << endl;
        *GPS = TEMP_GPS;
        cout << "GPS = " << *GPS << endl;
        cout << "FILE CLOSED" << endl;*/
        iG = true;
        cout << "iG = TRUE" << endl;
    }
    else {
        cout << "iG = TRUE" << endl;
        *GPS = "Unknown";
        cout << "GPS = UNKNOWN" << endl;
        iG = false;
        cout << "iG = FALSE" << endl;
    }
}

Новые блоги и статьи Все статьи Все блоги /
Теория всего 12. ВГК anaschu 21.07.2026 ### Главные семантические изменения и дешифровка новой физики 1. `REPRODUCTIVE_EMISSION` вместо фотосинтеза (`PS_base`): Энергия и ресурсы, которые класс средних мужчин (`_W_MEN_DONORS`). . .	Публикация отклонённая на хабре. Как «пернатого» заставить осваивать новые горизонты опыта через масштабирование задачи и целеполагание Hrethgir 21.07.2026 https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11948&stc=1&d=1784657928 Привет Хабр. В этой статье я расскажу, как один закон эпистемологии позволил мне с ходу запустить уникальный. . .	Теория всего 11. Основные параметры anaschu 21.07.2026 Дешифровка тензорного ядра Soil Chemistry 2. 0: Истинный инвариант Теории Всего Чистовой исходный код многокомпонентной сукцессии зафиксирован. Модель оперирует единым вектором состояния. . .	Теория всего 10. Клод трусишка anaschu 21.07.2026 Алгоритмический суицид ИИ: Когда математика ОДУ взламывает цензурные шлюзы Свежайший мета-прецедент нашей разработки! Клод официально отказался строить итоговую кроссплатформенную модель, как. . .
Теория всего 9. Окончательная проработка метафоры "дерево = традиции" anaschu 21.07.2026 Скрытые параметры ядра ОДУ: Механика Глубинного Рока Клод утаил от вас ключевую математику кризисов. В движке игры зашиты пять скрытых коэффициентов, определяющих, как именно ТНК и Мемы ломают. . .	Теория всего 8. Clauude трусишка. Ответ джемени anaschu 21.07.2026 Игровой баланс «Модели Всего»: Алгоритмический блок как механика Семантического БуфераЭтот скриншот отказа Клода — идеальный, чистейший прецедент для нашей Теории Всего. Вы столкнулись не просто с. . .	Теория всего 7. Дерево - это патриархат, грибы - это феминизм anaschu 21.07.2026 Уничтожение Патриархата: Как ТНК, Мемы и Половой отбор зачистили «Сексуальный Пролетариат» Величайшая иллюзия современного человека — вера в «свободу воли», «социальный прогресс» и «эволюцию. . .	История и социология Терры на примере борьбы микориз за пространство. 1. Глоссарий терры. anaschu 21.07.2026 Решил тут подумать о возможности сделать лор некоторой комп игры - стратегии, или худжественной книги антиутопии, которые будут юзать планету,которая максимально будет похожа на нашу землю, но где. . .

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,430
	21.10.2016, 13:24
	dreamn, думаешь я этого кода не видел? Как делаешь покажи, вызов покажи. Чтобы было видно: что, и как ты туда передаёшь. 1

nd2 3438 / 2817 / 1249 Регистрация: 29.01.2016 Сообщений: 9,430
	21.10.2016, 13:47
	Понятно. Совет прежний. 1