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

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

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

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

C++

1	string *GPS;

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

C++

1
2
3

Class::Class (string *_GPS) {
GPS = new string;
}

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

C++

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++

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++

     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++

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++

#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++

#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++

#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++

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++

     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

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

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

Код

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++

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;
    }
}

@John999 230 / 113 / 79 Регистрация: 17.10.2016 Сообщений: 312
	21.10.2016, 12:02	22
	hoggy, только он его не так использует Сообщение от dreamn string* 1

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

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

@dreamn 0 / 0 / 0 Регистрация: 25.09.2016 Сообщений: 27
	21.10.2016, 14:08 [ТС]	33
	Сообщение от nd2 .. Ты же видишь, что model и mark я не использую Поэтому не из-за них причина неработоспособности кода 0

@dreamn 0 / 0 / 0 Регистрация: 25.09.2016 Сообщений: 27
	21.10.2016, 15:04 [ТС]	37
	Преобразование string* в string Вот код.. Выручайте. Советы не помогли. Добавлено через 47 секунд Сообщение от GbaLog- Возможно, что твой компилятор запускает приложение А вы какой компилятор используете? 0

@hoggy 8739 / 4317 / 960 Регистрация: 15.11.2014 Сообщений: 9,760
	21.10.2016, 12:00	21
	Сообщение от dreamn Необходимо динамический string std::string - замечательный, самый динамический из всех динамических стрингов. 1

@dreamn 0 / 0 / 0 Регистрация: 25.09.2016 Сообщений: 27
	21.10.2016, 12:13 [ТС]	23
	Сообщение от John999 Необходимо динамический string : string * перепутал с char* ? Нет, не перепутал. Именно string И именно динамический Добавлено через 1 минуту Сообщение от Croessmah Цитата Сообщение от dreamn Посмотреть сообщение delete this; по рукам еще не били? Так объяснял препод по программированию. Типа, если инициализация с ошибкой, то удаляем объект. Добавлено через 23 секунды Сообщение от John999 hoggy, только он его не так использует Цитата Сообщение от dreamn Посмотреть сообщение string* А как правильно? 0

@GbaLog- Любитель чаепитий 3742 / 1798 / 566 Регистрация: 24.08.2014 Сообщений: 6,016 Записей в блоге: 1
	21.10.2016, 15:06	38
	Сообщение от dreamn Что это означает? Нельзя инициализировать переменные-члены в самом классе, это надо сделать в конструкторе, например, или метод создать. Но вообще, можно просто С++11 включить. Добавлено через 48 секунд Сообщение от dreamn А вы какой компилятор используете? Тьфу, ошибся в слове. Я хотел сказать не компилятор, а среда разработки. Я использую для компиляции командную строку, запускаю оттуда же. 1

@dreamn 0 / 0 / 0 Регистрация: 25.09.2016 Сообщений: 27
	21.10.2016, 15:09 [ТС]	39
	Сообщение от GbaLog- Нельзя инициализировать переменные-члены в самом классе, это надо сделать в конструкторе, например, или метод создать. Исправим Добавлено через 14 секунд Сообщение от GbaLog- Тьфу, ошибся в слове. Я хотел сказать не компилятор, а среда разработки. Пользуюсь NetBeans 8.2 0