С Новым годом! Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 14, средняя оценка - 5.00
fors-dnet
3 / 3 / 0
Регистрация: 11.12.2011
Сообщений: 88
#1

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

16.02.2012, 23:31. Просмотров 1838. Ответов 26
Метки нет (Все метки)

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

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

Считывание происходит из простого файла 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++
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
#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 минут
ну хоть кто-то...
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
16.02.2012, 23:31
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Нужно произвести считывание пути к файлу с клавиатуры. Причём сделать это в конструкторе. Всё остальное корректно работает. (C++):

Нужно создать массив с случайных чисел всё остальное внутри - C++
нужно создать массив случайных чисел и найти в этом массиве его наименьший элемент и вывести его и его индекс! плз помогите оч нужно...

Считывание расположения (пути) файла с клавиатуры - C++
Нужно реализовать в консольной программе открытие файла, путь которого будет считан с клавиатуры. Если файл находится в директории...

Не показывает монитор, но всё остальное работает - Компьютерное железо
Здравствуйте. Собрал свой компьютер и столкнулся с проблемой. Мат плата: MSI H81M-E33 LGA 1150 Оперативка на две планки по 2...

Не запускается монитор, не пищит БИОС, всё остальное работает - BIOS
Искал пункт разгона в биосе и случайно поднял voltage и сохранил.При включении крутятся все кулеры, мигают лампочки, устройства...

Пагинация работает, а всё остальное без словаря не отображается - Python
Добрый день, подскажите пожалуйста. Проблемы с пагинацией. Во вью есть функция, в которой хранится словарь. Проблема в том, что если я...

Не добавляются новые товары, хотя всё остальное работает - Web
есть сайт www.potenciya.net, сделан на движке симпла, сегодня хотел добавить товар - не получается, ошибку не пишет просто страница...

26
DU
1484 / 1130 / 45
Регистрация: 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
DU
1484 / 1130 / 45
Регистрация: 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
NumBot
14 / 14 / 0
Регистрация: 16.02.2012
Сообщений: 59
17.02.2012, 00:49 #20
Цитата Сообщение от fors-dnet Посмотреть сообщение
Путь один для всех абонетов всегда. а как это сделать знаешь? Спасибо большое что помогаешь.
Сделай строку пути статическим членом класса, в конструкторе добавь условие, типа, если путь ещё не инициализирован, то делаем то-то.
C++
1
static char buf [255];
Способ кривой, но другое сложно придумать.
1
fors-dnet
3 / 3 / 0
Регистрация: 11.12.2011
Сообщений: 88
17.02.2012, 00:49  [ТС] #21
DU, просто не пойму как в функции можно в этой строке проверить на пустоту
C++
1
f=fopen(buf, "rt");
и после этого если не пустая опять туда вписать значение. Знаний катострафически не хватает
0
DU
1484 / 1130 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
17.02.2012, 00:50 #22
Проблема в непонимании ооп.
Есть объект класса А. Этот объект сам по себе. Он делжен знать, что где-то в коде сформирован массив из таких же объектов. Если массив нужно отсортировать, то этим должен заниматься не объект типа А, а тот, кто создал массив этих объектов.
Со временем понимание придет. Ну или может кто-нибудь другой доступно истолкует это. У меня нет таланта объяснять очевидные вещи.

buf - строка. пока ее не заполнили она имеет нулевую длинну. значит в конструкторе смотришь, если длинна строки равна нулю, значит ее еще не заполняли. заполняешь ее (запрашиваешь путь, который пишется в нее). когда позовутся другие конструкторы, они так же проверят длину строки. но она не будет равна нулю. поэтому они ее запрашивать не будут.
1
fors-dnet
3 / 3 / 0
Регистрация: 11.12.2011
Сообщений: 88
17.02.2012, 00:51  [ТС] #23
NumBot, уже сделал глобальным. Разницы вроде нету ведь.

конструкторе добавь условие, типа, если путь ещё не инициализирован, то делаем то-то.
хз как это тут сделать:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
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;
}
0
NumBot
14 / 14 / 0
Регистрация: 16.02.2012
Сообщений: 59
17.02.2012, 00:55 #24
хз как это тут сделать:
тебе нужно чтобы он 7 раз не просил тебя вводить путь до файла? если да то тебе надо вставить условие сюда
C++
1
2
3
4
5
6
Abonent :: Abonent ()
{
        setlocale(LC_ALL, "Rus");
        std::cout << "Введите имя файла : ";
        gets (buf);
}
1
fors-dnet
3 / 3 / 0
Регистрация: 11.12.2011
Сообщений: 88
17.02.2012, 00:56  [ТС] #25
DU, понял, проверять на заполненность надо в конструкторе, а не в функции лоад

Добавлено через 40 секунд
NumBot, спасибо. я просто тогда сообщение от ДУ ещё не прочитал
0
DU
1484 / 1130 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
17.02.2012, 00:58 #26
Вот конструктор. должно работать при условии, что глобальная (или статическая) переменная buf правильно проинициализирована нулями на старте программы.

C++
1
2
3
4
5
6
7
8
9
Abonent::Abonent()
{
  if (strlen(buf) == 0)
  {
    setlocale(LC_ALL, "Rus");
    std::cout << "Введите имя файла : ";
    gets(buf);
  }
}
1
fors-dnet
3 / 3 / 0
Регистрация: 11.12.2011
Сообщений: 88
17.02.2012, 01:03  [ТС] #27
DU, Спасибо большое, всё работает! вот итоговый 100% рабочий вариант (вдруг кому-то понадобится когда-то:

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
#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];
public:
        Abonent (void);
        int load (Abonent *arr);
        int sortost (Abonent *t);
        int sorttar ( Abonent *arr);
        void get(void);
        ~Abonent (void);
 }arr [5];
char buf [255];
 
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");
    if (strlen(buf)==0)
    {
    std::cout << "Введите имя файла : ";
    gets (buf);
    }
}
 
void main (void)
{
    Abonent Abonent_val;
    Abonent_val.get();
    system("pause");
}
0
17.02.2012, 01:03
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
17.02.2012, 01:03
Привет! Вот еще темы с ответами:

Не запускается монитор, не пищит БИОС, всё остальное работает - Компьютерное железо
И снова доброго времени суток, это опять я. У знакомых есть проблема, суть такова: существует компьютер: ОЗУ - 2 планки по 2 Гб,...

Есть видеокарта NVIDIA GeForce GTX260, нужно подобрать всё остальное. - Рабочая станция
здравствуите у меня есть видеокарта NVIDIA GeForce GTX260 скажите пожалуста какой процесор у меня поналобиться и с каой частоти , для...

Проверка пути к файлу введенного с клавиатуры - C#
String FileName = &quot;nulll&quot;; while (!File.Exists (FileName)) { Console.Writeline(&quot;Введите путь к файлу&quot;) ...

Не грузятся картинки в IE, а в других браузерах всё работает корректно - HTML, CSS
Всем привет! В Explorere на моём сайте не отображаются картинки на странице Галерея http://galadesign.org/photos.php и весь сайт съезжает...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
27
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.