Нужно произвести считывание пути к файлу с клавиатуры. Причём сделать это в конструкторе. Всё остальное корректно работает.

@fors-dnet · Регистрация: 11.12.2011

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

Заранее благодарен всем кто поможет.

Здравстуйте, была полностью рабочая программа которая считывает данные с файла, потом сортирует их и сохраняет.

Считывание происходит из простого файла txt вот такого содержания:

09342598623 56,30 life
05085458748 23,10 mtc
09765874598 11,20 kyivstar
09345874122 89,65 life
09645874114 11,50 mtc

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

) Я создал переменную в классе char buf [255] и в конструкторе сделал считывание пути к файлу с клавиатуры в эту переменную. После этого программа перестала работать. Эрроров нет.

Предлагает раз шесть ввести путь к файлу, а потом всё равно не может его открыть

Помогите пожалуйста у кого какие идеи

C++

#include<iostream>
#include <conio.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <locale.h>
 
class Abonent
{
private:   
        char nomer[11];
        double ostatok;
        char tarif[15];
        char buf [255];
public:
        Abonent (void);
        int load (Abonent *arr);
        int sortost (Abonent *t);
        int sorttar ( Abonent *arr);
        void get(void);
        ~Abonent (void);
 }arr [5];
 
int Abonent:: load (Abonent *arr)
{
    int i=0;
    FILE *f=NULL; 
    f=fopen("buf", "rt");
    if (f==NULL)
    {
        std::cout<< "No file!!!\n";
        exit (0);
    }
        
    for(i=0;i<5;i++)
    {
        fscanf(f,"%s", &arr[i].nomer);
        fscanf(f,"%lf", &arr[i].ostatok);
        fscanf(f,"%s", &arr[i].tarif);
    }
    fclose (f);
 
    return 0;
}
 
 int Abonent:: sortost (class Abonent *t)
 {
        class Abonent x;
        int i=0, j=0;   
        for(i=0;i<5;i++)
        {
                for(j=0; j<4; j++)
                {
                        if( t[j].ostatok > t[j+1].ostatok )
                        {
                                x=t[j];
                                t[j]=t[j+1];
                                t[j+1]=x;
                        }
                        
                }
        }
 
        return 0;       
 }
int Abonent:: sorttar (class Abonent *arr)
{
    int i=0, j=0;
    struct Abonent temp;
    for (i=0; i<5; i++)
    {
        for (j=0; j<4; j++)
        {
            if (strcmp(arr[j].tarif, arr [j+1].tarif) >= 0)
            {
                temp=arr[j];
                arr[j]=arr[j+1];
                arr[j+1]=temp;
            }
        }
    }
 
    return 0;
}
 
Abonent :: ~Abonent ()
{
    int i=0;
    FILE *f=NULL;
    f=fopen ("D:\Result.txt", "wt");
    if (f==NULL)
    {
        std::cout<<"No file!!!\n";
        exit (0);
    }
   
    fprintf (f, "Spisok abonentov:\n\n");
    for(i=0;i<5;i++)
    {
        fprintf(f,"%s\t", arr[i].nomer);
        fprintf(f,"%lf\t", arr[i].ostatok);
        fprintf(f,"%s\t", arr[i].tarif);
        fprintf(f,"\n");
    }
    fclose (f);
 
}
 
void Abonent:: get(void)
{
    setlocale(LC_ALL, "Rus");
    int n=0, i=0;
    while(1)
    {
        std::cout<<"1.Загрузка данных\n2.Сортировка данных по остатку\n3.Сортировка данных по тарифу\n4.Выход (данные будут сохранены автоматически)\n\n";
        std::cout<<"Введите номер выбранного действия: ";
        std::cin>>n;
        std::cout<<std::endl;
 
        switch (n)
        {
            case 1 : load(arr);
                         break;
 
            case 2 : sortost (arr);
                             break;
 
            case 3 : sorttar (arr);
                             break;
 
            case 4 : exit (0);
                        break;
        }
        for (i=0;i<5;i++)
        std::cout<<arr[i].nomer<<'\t'<<arr[i].ostatok<<'\t'<<arr[i].tarif<<std::endl;
        std::cout<<std::endl;
        } 
}
 
Abonent :: Abonent ()
{
    setlocale(LC_ALL, "Rus");
    std::cout << "Введите имя файла : ";
    gets (buf);
}
 
void main (void)
{
    Abonent path ();
    class Abonent Abonent_val;
    Abonent_val.get();
    system("pause");
}

Добавлено через 27 минут
ну что вообще никак?

Добавлено через 36 минут
ну хоть кто-то...

@DU · 16.02.2012, 23:45

строка 28

C++

f=fopen("buf", "rt"); // это нормально?
 
// может быть надо
f=fopen(buf, "rt");

вот еще до кучи строка 90

C++

1 2	f=fopen ("D:\Result.txt", "wt"); // кривой путь f=fopen ("D:\\Result.txt", "wt"); // правильный путь

@BRcr · 16.02.2012, 23:47

Сообщение от fors-dnet

C++

1	f=fopen("buf", "rt");

Сообщение от fors-dnet

C++

1	f=fopen ("D:\Result.txt", "wt");

Сообщение от fors-dnet

C++

void main (void)
{
Abonent path ();
class Abonent Abonent_val;
Abonent_val.get();
system("pause");
}

Такое ощущение, что вы вообще не знаете, что делаете... а программа так никогда и не работала

@fors-dnet · 16.02.2012, 23:55 **[ТС]**

DU, мда, это был мой эпический провал. Проблему считывания это решило (теперь всё корректно считывает из файла, но всё равно предлагает 6 раз ввести путь, для каждого элемента массива. А можно как-то сделать так чтоб только один раз вводить?.

Спасибо, что потратил время, нашел ошибку, я чёт жестко завтыкал

Добавлено через 4 минуты
BRcr, насчёт первого, то завтыкал жестко.
насчёт второго - в чём проблема? открываем поток для записи отсортированных данных в файл
насчет третьего - опять же в чем проблема?

насчёт того что не работала - пока не стал прописывать через конструктор путь к файлу вручную всё работало. в смысле в функции лоад задавал путь автоматически

@DU · 16.02.2012, 23:56

никак. я смотрю в массиве 5 элементов. для каждого вызывается конструктор. в каждом конструкторе идет запрос пути к файлу. хотел сделать запрос в конструкторе, вот и получай эти запросы при каждом создании объекта.

@fors-dnet · 16.02.2012, 23:58 **[ТС]**

BRcr, а насчёт того что не знаю, я и написал сверху что не знаю как задание сделать

. Но после того как
DU, помог всё зароботало, ток путь требует 6 раз ввести, что не есть хорошо

Добавлено через 1 минуту
DU, может по другому как-то можно реализовать? не через переменную char buf [255]? У меня просто задание сделать так чтоб путь к файлу пользователь вводил сам и через конструктор. Дальше свободный полёт. Помоги если есть идеи

@DU · 16.02.2012, 23:59

есть глобальный массив из 5ти элементов типа Abonent и один в main(). глобальный массив нужно убрать наверно.

C++

class Abonent
{
private:   
        char nomer[11];
        double ostatok;
        char tarif[15];
                char buf [255];
public:
                Abonent (void);
        int load (Abonent *arr);
        int sortost (Abonent *t);
        int sorttar ( Abonent *arr);
        void get(void);
                ~Abonent (void);
 }arr [5];

arr[5] нужно удалить, после '}' оставить точку с запятой.

C++

void main (void)
{
        // это тут зачем? если не используется, то не надо создавать. кстати раз запросов 6, а не 7, значит 
        // компилятор воспринимает это как декларацию функции.
        Abonent path ();
 
     // слово класс - лишнее
     class Abonent Abonent_val;
    Abonent_val.get();
    system("pause");
}

@fors-dnet · 17.02.2012, 00:02 **[ТС]**

но ведь это как бы объект класса. Куда тогда считывание данных будет происходить?

@DU · 17.02.2012, 00:05

C++

1	Abonent Abonent_val;

вот это строка - это объявление переменной типа Abonent (объект класса по твоему). Для нее позовется конструктор. в конструкторе запросится путь до файла. В этом же объекте и будут всякие счытывания и прочая работа.

A вот вызов метода у этого объекта:

C++

1	Abonent_val.get();

@fors-dnet · 17.02.2012, 00:08 **[ТС]**

Abonent path (); // это хз зачем так назвал переделал на Abonent (); вроде всё так же работает, ток 7 раз стало запускаться =) Тут у меня как бы конструктор этой строчкой запускается( по крайней мере по моей задумке должно было выйти именно так)

class Abonent Abonent_val; \\ по поводу этого переделал на Abonent Abonent_val - без изменений, это вопрос синтаксиса, привык на Си чистом что там ничего упускать нельзя просто)

@retmas · 17.02.2012, 00:09

fors-dnet, в С++ при объявлении переменных необязательно писать ключевые слова class, struct, enum, union

@fors-dnet · 17.02.2012, 00:13 **[ТС]**

Abonent () значит это лишнее? просто нужно содержимое в Abonent Abonent_val переместить (описать его отдельно)?

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

@DU · 17.02.2012, 00:15

C++

int main()
{
  Abonent Abonent_val; // для Abonent_val позовется конструктор без параметров, в котором запросистя путь.
  Abonent_val.get(); // тут что-то делается с абонентом.
  system("pause");
  return 0;
}

Остальное не трогайте, если работает.

@retmas · 17.02.2012, 00:16

Сообщение от fors-dnet

Abonent path ();
Тут у меня как бы конструктор этой строчкой запускается( по крайней мере по моей задумке должно было выйти именно так)

это объявление ф-ии path(), возвращающей Abonent. при записи

C++

1	Abonent path;

к-р по умолчанию вызывается без твоего вмешательства

@fors-dnet · 17.02.2012, 00:21 **[ТС]**

C++

#include<iostream>
#include <conio.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <locale.h>
 
class Abonent
{
private:   
        char nomer[11];
        double ostatok;
        char tarif[15];
        char buf [255];
public:
        Abonent (void);
        int load (Abonent *arr);
        int sortost (Abonent *t);
        int sorttar ( Abonent *arr);
        void get(void);
        ~Abonent (void);
 }arr [5];
 
int Abonent:: load (Abonent *arr)
{
    int i=0;
    FILE *f=NULL; 
    f=fopen(buf, "rt");
    if (f==NULL)
    {
        std::cout<< "No file!!!\n";
        exit (0);
    }
        
    for(i=0;i<5;i++)
    {
        fscanf(f,"%s", &arr[i].nomer);
        fscanf(f,"%lf", &arr[i].ostatok);
        fscanf(f,"%s", &arr[i].tarif);
    }
    fclose (f);
 
    return 0;
}
 
 int Abonent:: sortost (class Abonent *t)
 {
        class Abonent x;
        int i=0, j=0;   
        for(i=0;i<5;i++)
        {
                for(j=0; j<4; j++)
                {
                        if( t[j].ostatok > t[j+1].ostatok )
                        {
                                x=t[j];
                                t[j]=t[j+1];
                                t[j+1]=x;
                        }
                        
                }
        }
 
        return 0;       
 }
int Abonent:: sorttar (class Abonent *arr)
{
    int i=0, j=0;
    struct Abonent temp;
    for (i=0; i<5; i++)
    {
        for (j=0; j<4; j++)
        {
            if (strcmp(arr[j].tarif, arr [j+1].tarif) >= 0)
            {
                temp=arr[j];
                arr[j]=arr[j+1];
                arr[j+1]=temp;
            }
        }
    }
 
    return 0;
}
 
Abonent :: ~Abonent ()
{
    int i=0;
    FILE *f=NULL;
    f=fopen ("D:\Result.txt", "wt");
    if (f==NULL)
    {
        std::cout<<"No file!!!\n";
        exit (0);
    }
   
    fprintf (f, "Spisok abonentov:\n\n");
    for(i=0;i<5;i++)
    {
        fprintf(f,"%s\t", arr[i].nomer);
        fprintf(f,"%lf\t", arr[i].ostatok);
        fprintf(f,"%s\t", arr[i].tarif);
        fprintf(f,"\n");
    }
    fclose (f);
 
}
 
void Abonent:: get(void)
{
    setlocale(LC_ALL, "Rus");
    int n=0, i=0;
    while(1)
    {
        std::cout<<"1.Загрузка данных\n2.Сортировка данных по остатку\n3.Сортировка данных по тарифу\n4.Выход (данные будут сохранены автоматически)\n\n";
        std::cout<<"Введите номер выбранного действия: ";
        std::cin>>n;
        std::cout<<std::endl;
 
        switch (n)
        {
            case 1 : load(arr);
                         break;
 
            case 2 : sortost (arr);
                             break;
 
            case 3 : sorttar (arr);
                             break;
 
            case 4 : exit (0);
                        break;
        }
        std::cout<<std::endl;
        for (i=0;i<5;i++)
        std::cout<<arr[i].nomer<<'\t'<<arr[i].ostatok<<'\t'<<arr[i].tarif<<std::endl;
        std::cout<<std::endl;
        } 
}
 
Abonent :: Abonent ()
{
    setlocale(LC_ALL, "Rus");
    std::cout << "Введите имя файла : ";
    gets (buf);
}
 
void main (void)
{
    Abonent Abonent_val;
    Abonent_val.get();
    system("pause");
}

DU,
Вот так всё работает (сделал мэйн как ты посоветовал) на 5+, если не считать того, что опять таки 7 раз вводить путь к файлу это плохо. Нету никаких других способов? А то например если в массиве 30 абонентов - 30 раз что-ли вводить.

Получается задание некорректно поставлено?

@DU · 17.02.2012, 00:27

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

@fors-dnet · 17.02.2012, 00:31 **[ТС]**

Путь один для всех абонетов всегда.

то эту информацию нужно расшарить между всеми абонентами

а как это сделать знаешь? Спасибо большое что помогаешь.

@DU · 17.02.2012, 00:34

Вообще программа кривая. Качество кода и все такое. Но вот что мне "понравилось" больше всего.
Создается глобальный массив из объектов типа Абонент. А в методах Абонента идет обращение к этому глобальному массиву, к каждому его элементу и что-то там делается. Эти обращения идут из методов объекта, который в main.

Программу нужно конечно же переписывать. Но если хочется по быстрому, то просто массив buf делаешь глобальным. В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь.

@fors-dnet · 17.02.2012, 00:47 **[ТС]**

DU, это моя первая программа на с++

Создается глобальный массив из объектов типа Абонент. А в методах Абонента идет обращение к этому глобальному массиву, к каждому его элементу и что-то там делается. Эти обращения идут из методов объекта, который в main.

так что не пойму в чём проблема тут

.

если хочется по быстрому, то просто массив buf делаешь глобальным. В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь.

сейчас буду пробывать

Добавлено через 6 минут
DU,

В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь.

а как это реализовать?

@NumBot · 17.02.2012, 00:49

Сообщение от fors-dnet

Путь один для всех абонетов всегда. а как это сделать знаешь? Спасибо большое что помогаешь.

Сделай строку пути статическим членом класса, в конструкторе добавь условие, типа, если путь ещё не инициализирован, то делаем то-то.

C++

1	static char buf [255];

Способ кривой, но другое сложно придумать.

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	17.02.2012, 00:34	18
	Вообще программа кривая. Качество кода и все такое. Но вот что мне "понравилось" больше всего. Создается глобальный массив из объектов типа Абонент. А в методах Абонента идет обращение к этому глобальному массиву, к каждому его элементу и что-то там делается. Эти обращения идут из методов объекта, который в main. Программу нужно конечно же переписывать. Но если хочется по быстрому, то просто массив buf делаешь глобальным. В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь. 1

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	17.02.2012, 00:47 [ТС]	19
	DU, это моя первая программа на с++ Создается глобальный массив из объектов типа Абонент. А в методах Абонента идет обращение к этому глобальному массиву, к каждому его элементу и что-то там делается. Эти обращения идут из методов объекта, который в main. так что не пойму в чём проблема тут . если хочется по быстрому, то просто массив buf делаешь глобальным. В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь. сейчас буду пробывать Добавлено через 6 минут DU, В конструкторе смотришь, пустая ли это строка или нет. Если пустая - запрашиваешь путь. Если не пустая - используешь. а как это реализовать? 0

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	16.02.2012, 23:55 [ТС]	4
	DU, мда, это был мой эпический провал. Проблему считывания это решило (теперь всё корректно считывает из файла, но всё равно предлагает 6 раз ввести путь, для каждого элемента массива. А можно как-то сделать так чтоб только один раз вводить?. Спасибо, что потратил время, нашел ошибку, я чёт жестко завтыкал Добавлено через 4 минуты BRcr, насчёт первого, то завтыкал жестко. насчёт второго - в чём проблема? открываем поток для записи отсортированных данных в файл насчет третьего - опять же в чем проблема? насчёт того что не работала - пока не стал прописывать через конструктор путь к файлу вручную всё работало. в смысле в функции лоад задавал путь автоматически 0

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	16.02.2012, 23:56	5
	никак. я смотрю в массиве 5 элементов. для каждого вызывается конструктор. в каждом конструкторе идет запрос пути к файлу. хотел сделать запрос в конструкторе, вот и получай эти запросы при каждом создании объекта. 1

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	16.02.2012, 23:58 [ТС]	6
	BRcr, а насчёт того что не знаю, я и написал сверху что не знаю как задание сделать . Но после того как DU, помог всё зароботало, ток путь требует 6 раз ввести, что не есть хорошо Добавлено через 1 минуту DU, может по другому как-то можно реализовать? не через переменную char buf [255]? У меня просто задание сделать так чтоб путь к файлу пользователь вводил сам и через конструктор. Дальше свободный полёт. Помоги если есть идеи 0

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	17.02.2012, 00:02 [ТС]	8
	но ведь это как бы объект класса. Куда тогда считывание данных будет происходить? 0

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	17.02.2012, 00:08 [ТС]	10
	Abonent path (); // это хз зачем так назвал переделал на Abonent (); вроде всё так же работает, ток 7 раз стало запускаться =) Тут у меня как бы конструктор этой строчкой запускается( по крайней мере по моей задумке должно было выйти именно так) class Abonent Abonent_val; \\ по поводу этого переделал на Abonent Abonent_val - без изменений, это вопрос синтаксиса, привык на Си чистом что там ничего упускать нельзя просто) 0

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	17.02.2012, 00:09	11
	fors-dnet, в С++ при объявлении переменных необязательно писать ключевые слова class, struct, enum, union 1

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	17.02.2012, 00:13 [ТС]	12
	Abonent () значит это лишнее? просто нужно содержимое в Abonent Abonent_val переместить (описать его отдельно)? Добавлено через 2 минуты я запутался чёт совсем 0

@DU 1500 / 1146 / 165 Регистрация: 05.12.2011 Сообщений: 2,279
	17.02.2012, 00:27	16
	Если условие такое: В конструкторе всегда и при любых условиях запрашивать путь, то других вариантов нет. для массива из n абонентов путь запросится n-раз. Если же этот путь один на всех абонентов, и его достаточно запросить лишь один раз, то эту информацию нужно расшарить между всеми абонентами. Когда первый будет создаваться, он поймет, что путь еще не запрашивался и запросит его и сохранит где-то этот путь так, чтобы остальные абоненты смогли понять при своем создании, что путь уже запрашивался и еще раз его запрашивать не надо, а надо лишь использовать тот, что уже есть. Но тогда делать запрос пути в конструкторе абонента - это та еще кривизна. Он в таком случае должен запросится отдельно от создания абонентов и передаваться каждому абоненту. В общем задание кривое. 1

@fors-dnet 3 / 3 / 0 Регистрация: 11.12.2011 Сообщений: 88
	17.02.2012, 00:31 [ТС]	17
	Путь один для всех абонетов всегда. то эту информацию нужно расшарить между всеми абонентами а как это сделать знаешь? Спасибо большое что помогаешь. 0