Ошибка в функции

@Mr. Pyatachok · Регистрация: 30.06.2011

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

Здравствуйте, уважаемые знатоки!
Итак, ящик в студию! Внимательно прочитайте код и найдите ошибки идиота!

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
#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <math.h>
#include <string>
 
char bufRus[256];
char* Rus(const char* text)
{
    CharToOem(text,bufRus);
    return bufRus;
}
 
class Money
{
        private:
    long rub;           //long int
    unsigned char cop;
        public:
    friend class Goods;
};
 
class Goods
{
        private:
    char product_name[100]; //наименование товара
    int day,month,year;     //дата оформления товара
    int number_of_goods;    //количество единиц товара
    int tracking_number;    //номер накладной
        public:
    friend class Money;
    Money money;
    void ReadGoods();
    float TransferPriceMoney();
    void ChangePriceGoods();
};
 
void Goods::ReadGoods()
{
    while(true)
    {
        cout<<Rus("Введите наименование товара: ");
        cin>>product_name;
        cout<<Rus("Введите цену товара: ")<<endl;
        cout<<Rus("Введите рубли:   ");
        Goods goods;
        cin>>goods.money.rub;
        cout<<Rus("Введите копейки: ");
        cin>>goods.money.cop;
        cout<<Rus("Введите дату оформления товара: ");
        cin>>day>>month>>year;
        cout<<Rus("Введите количество товара: ");
        cin>>tracking_number;
        if(cin.peek()=='\n'&&(goods.money.rub>=0)&&(1>=day&&day<=31)&&(1>=month&&month<=12)&&(year>=2012)&&(tracking_number>=0))
        {
            cin.get();
            break;
        }
        else
        {
            cout<<Rus("Ошибка! Проверьте вводимое значение.")<<endl<<endl;
            cin.clear();
            while(cin.get()!='\n') {  }
        }
    }
    cout<<Rus("Значения введены верно.")<<endl;
}
 
float Goods::TransferPriceMoney()
{
    Goods goods;
    float price;
    price=goods.money.rub+goods.money.cop/100.0;
    return price;
}
 
void Goods::ChangePriceGoods()
{
    float summa;
    cout<<Rus("Введите сумму на которую необходимо изменить текущую цену: ");
    cin>>summa;
    cout<<Rus("При увеличении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()+summa<<endl;
    cout<<Rus("При уменьшении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()-summa<<endl;
}
 
int main()
{
    Goods goods;
    goods.ReadGoods();
    goods.ChangePriceGoods();
 
 
    getch();
    return 0;
}

проверьте пожалуйста ChangePriceGoods()

Добавлено через 19 минут
Заранее спасибо

@A555 · 08.03.2012, 23:51

объясняй логику что делает прога вообще надём ошибки сейчас

@Mr. Pyatachok · 08.03.2012, 23:56 **[ТС]**

Необходимо создать 2 класса: первый-основной, второй-вспомогательный.
Цена задана таким вот уродством long rub; unsigned char cop; Цену же необходимо соединить в тип float и затем изменить

@A555 · 08.03.2012, 23:58

C++    
        
    Скопировано
1
  if(cin.peek()=='\n'&&(goods.money.rub>=0)&&(1>=day&&day<=31)&&(1>=month&&month<=12)&&(year>=2012)&&(tracking_number>=0))

вот это объясни а то не вводиться ничего

@Mr. Pyatachok · 09.03.2012, 00:01 **[ТС]**

извиняюсь за этот бред, тут я хотел написать проверку на ошибки)

C++    
        
    Скопировано
1
if(cin.peek()=='\n'&&(goods.money.rub>=0)&&(day>=1&&day<=31)&&(month>=1&&month<=12)&&(year>=2012)&&(tracking_number>=0))

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

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
#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <math.h>
#include <string>
 
char bufRus[256];
char* Rus(const char* text)
{
    CharToOem(text,bufRus);
    return bufRus;
}
 
class Money
{
        private:
    long rub;           //long int
    unsigned char cop;
        public:
    friend class Goods;
};
 
class Goods
{
        private:
    char product_name[100]; //наименование товара
    int day,month,year;     //дата оформления товара
    int number_of_goods;    //количество единиц товара
    int tracking_number;    //номер накладной
        public:
    friend class Money;
    Money money;
    void ReadGoods();
    float TransferPriceMoney();
    void ChangePriceGoods();
};
 
void Goods::ReadGoods()
{
    while(true)
    {
        cout<<Rus("Введите наименование товара: ");
        cin>>product_name;
        cout<<Rus("Введите цену товара: ")<<endl;
        cout<<Rus("Введите рубли:   ");
        Goods goods;
        cin>>goods.money.rub;
        cout<<Rus("Введите копейки: ");
        cin>>goods.money.cop;
        cout<<Rus("Введите дату оформления товара: ");
        cin>>day>>month>>year;
        cout<<Rus("Введите количество товара: ");
        cin>>tracking_number;
        if(cin.peek()=='\n'&&(goods.money.rub>=0)&&(day>=1&&day<=31)&&(month>=1&&month<=12)&&(year>=2012)&&(tracking_number>=0))
        {
            cin.get();
            break;
        }
        else
        {
            cout<<Rus("Ошибка! Проверьте вводимое значение.")<<endl<<endl;
            cin.clear();
            while(cin.get()!='\n') {  }
        }
    }
    cout<<Rus("Значения введены верно.")<<endl;
}
 
float Goods::TransferPriceMoney()
{
    Goods goods;
    float price;
    price=goods.money.rub+goods.money.cop/100.0f;
    cout<<price;
    return price;
}
 
void Goods::ChangePriceGoods()
{
    float summa;
    cout<<Rus("Введите сумму на которую необходимо изменить текущую цену: ");
    cin>>summa;
    cout.setf(ios_base::floatfield,ios_base::fixed);
    cout<<Rus("При увеличении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()+summa<<endl;
    cout<<Rus("При уменьшении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()-summa<<endl;
    TransferPriceMoney();
    }
 
int main()
{
    Goods goods;
    goods.ReadGoods();
    goods.ChangePriceGoods();
 
 
    getch();
    return 0;
}

Вопрос тот же)

@A555 · 09.03.2012, 00:02

так прости меня дай комбинацию с которой прокатит воод всех данных

@Mr. Pyatachok · 09.03.2012, 00:03 **[ТС]**

Извините, но что не вводится?

@A555 · 09.03.2012, 00:04

ну ввожу данные посмотрел проверку должны пройти а мн всешда выводит

C++    
        
    Скопировано
1
cout<<Rus("Ошибка! Проверьте вводимое значение.")<<endl<<endl;

прости разобрался увидел ошибку сейчас посмотрим

@Mr. Pyatachok · 09.03.2012, 00:05 **[ТС]**

А что вводите, а то может чего не учел?

@A555 · 09.03.2012, 00:12

так кароч

C++    
        
            
                
                
                
                
            
        
    Скопировано
1
2
3
4
5
6
7
8
float Goods::TransferPriceMoney()
{
        Goods goods;
        float price;
        price=goods.money.rub+goods.money.cop/100.0f;
        cout<<price;
        return price;
}

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

C++    
        
    Скопировано
1
goods.money.cop

просто пустая строка а

C++    
        
    Скопировано
1
goods.money.rub

в неё забито неизвестно число
а так принципе остальное корректно
а какие именно объекты ты там хочешь использовать?

@Mr. Pyatachok · 09.03.2012, 00:22 **[ТС]**

а почему мусором то заполнены? как подправить?
Заранее спасибо

Добавлено через 5 минут
да желательно money

Добавлено через 37 секунд
хотя какие угодно, просто не понимаю почему мусор

@A555 · 09.03.2012, 00:28

почему мусором потому что ты создал новый объект и сразу вызвался конструктор блин как сказать стандартный пустой и значения этих переменных не вврдят

@Mr. Pyatachok · 09.03.2012, 00:31 **[ТС]**

а поможете исправить мое безобразие
напишите пожалуйста

@A555 · 09.03.2012, 00:36

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
#include <iostream.h>
#include <conio.h>
#include <windows.h>
#include <math.h>
#include <string>
 
 
 
char bufRus[256];
char* Rus(const char* text)
{
        CharToOem(text,bufRus);
        return bufRus;
}
 
class Money
{
                private:
        long rub;                       //long int
        unsigned char cop;
                public:
        friend class Goods;
};
 
class Goods
{
                private:
        char product_name[100]; //наименование товара
        int day,month,year;     //дата оформления товара
        int number_of_goods;    //количество единиц товара
        int tracking_number;    //номер накладной
                public:
        friend class Money;
        Money money;
        void ReadGoods();
        float TransferPriceMoney();
        void ChangePriceGoods();
};
 
void Goods::ReadGoods()
{
        while(true)
        {
                cout<<Rus("Введите наименование товара: ");
                cin>>product_name;
        cout<<Rus("Введите цену товара: ")<<endl;
                cout<<Rus("Введите рубли:   ");
                cin>>money.rub;
                cout<<Rus("Введите копейки: ");
                cin>>money.cop;
                cout<<Rus("Введите дату оформления товара: ");
                cin>>day>>month>>year;
                cout<<Rus("Введите количество товара: ");
                cin>>tracking_number;
                if(cin.peek()=='\n'&&(money.rub>=0)&&(day>=1&&day<=31)&&(month>=1&&month<=12)&&(year>=2012)&&(tracking_number>=0))
                {
                        cin.get();
                        break;
                }
                else
                {
                        cout<<Rus("Ошибка! Проверьте вводимое значение.")<<endl<<endl;
                        cin.clear();
                        while(cin.get()!='\n') {  }
                }
        }
        cout<<Rus("Значения введены верно.")<<endl;
}
 
float Goods::TransferPriceMoney()
{
        float price;
        price=money.rub+money.cop/100.0;
        cout<<price;
        return price;
}
 
void Goods::ChangePriceGoods()
{
        float summa;
        cout<<Rus("Введите сумму на которую необходимо изменить текущую цену: ");
        cin>>summa;
 
        cout<<Rus("При увеличении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()+summa<<endl;
        cout<<Rus("При уменьшении на заданную сумму текущая цена составит: ")<<TransferPriceMoney()-summa<<endl;
        TransferPriceMoney();
        }
 
int main()
{
        Goods goods;
        goods.ReadGoods();
        goods.ChangePriceGoods();
 
 
        getch();
        return 0;
}

смотри теперь работает всё ты просто напутал ты используя объект класса создавал в функции новый объект и часть данных вводил в свой объект а часть в ново созданный

@Mr. Pyatachok · 09.03.2012, 00:41 **[ТС]**

Я нашел, можно было и не париться через goods, затупил, а за исправление спасибо

Добавлено через 1 минуту
Сбросьте конечно, посмотрю, может интересного чего найду

@A555 · 09.03.2012, 00:49

а что сбрасывать то? я весь исправленный код сбросил уже

@Mr. Pyatachok · 09.03.2012, 00:57 **[ТС]**

Спасибо, не обновил страницу просто, спасибо, вопрос решен

Новые блоги и статьи Все статьи Все блоги /
Вопросы на собеседовании по Android mobDevWorks 14.03.2025 По данным статистики, Android занимает более 70% мирового рынка мобильных операционных систем, что делает платформу привлекательной как для начинающих разработчиков, так и для опытных профессионалов. . . .	Лучшие игровые движки для Python py-thonny 14.03.2025 Python обеспечивает разработчиков игр мощными движками и фреймворками, которые позволяют воплотить практически любую идею — от простой аркады до визуального романа с разветвленным сюжетом. Главное. . .	Бессерверный JavaScript: Разработка масштабируемых API с AWS Lambda run.dev 14.03.2025 Но что такое бессерверные вычисления на самом деле? По сути, это модель облачных вычислений, где разработчик фокусируется исключительно на создании бизнес-логики, не тратя время на настройку. . .	Безопасность кода в C++26: Менеджеры ресурсов и висячие ссылки NullReferenced 14.03.2025 C++ всегда был языком, предоставляющим разработчикам большие возможности и гибкость, но вместе с тем требующим ответственности. Одной из самых коварных проблем даже для опытных программистов остаются. . .	smart-agent proper interface settings (2025) jigi33 14.03.2025 Smart-agent proper interface settings (mart 2025). (see screenshots to look at "Etalon" ARM)
Продвинутые настройки JVM Javaican 14.03.2025 Стандартные параметры запуска JVM хороши для повседневной разработки, но совершенно недостаточны для высоконагруженных систем. Представьте, что вы запускаете финансовую платформу, обрабатывающую. . .	CI/CD для приложений Java с Azure DevOps и Docker Mr. Docker 14.03.2025 Разработка современных Java-приложений немыслима без системы непрерывной интеграции и доставки (CI/ CD). Azure DevOps в сочетании с Docker предоставляет мощный инструментарий для создания таких. . .	Разработка на PHP и интернет вещей (IoT) Jason-Webb 14.03.2025 Интернет вещей (IoT) произвел настоящую революцию в способах взаимодействия устройств с окружающим миром. В эпоху, когда холодильники сами заказывают молоко, а термостаты учатся вашим привычкам,. . .	Node.js 20: Новые возможности и улучшения производительности Reangularity 14.03.2025 Что же принёс нам релиз Node. js 20? В первую очередь, это существенные улучшения в производительности. Движок V8 получил серьёзные оптимизации, благодаря чему JavaScript-код выполняется заметно. . .	Безопасность кластеров Apache Kafka Javaican 14.03.2025 Apache Kafka стал одним из ключевых компонентов современных архитектур, обрабатывающих потоки данных в режиме реального времени. Его используют тысячи компаний от стартапов до технологических. . .

@A555 51 / 51 / 4 Регистрация: 04.04.2011 Сообщений: 209
	08.03.2012, 23:51	2
	объясняй логику что делает прога вообще надём ошибки сейчас 1

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	08.03.2012, 23:56 [ТС]	3
	Необходимо создать 2 класса: первый-основной, второй-вспомогательный. Цена задана таким вот уродством long rub; unsigned char cop; Цену же необходимо соединить в тип float и затем изменить 0

@A555 51 / 51 / 4 Регистрация: 04.04.2011 Сообщений: 209
	09.03.2012, 00:02	6
	так прости меня дай комбинацию с которой прокатит воод всех данных 1

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:03 [ТС]	7
	Извините, но что не вводится? 0

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:05 [ТС]	9
	А что вводите, а то может чего не учел? 0

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:22 [ТС]	11
	а почему мусором то заполнены? как подправить? Заранее спасибо Добавлено через 5 минут да желательно money Добавлено через 37 секунд хотя какие угодно, просто не понимаю почему мусор 0

@A555 51 / 51 / 4 Регистрация: 04.04.2011 Сообщений: 209
	09.03.2012, 00:28	12
	почему мусором потому что ты создал новый объект и сразу вызвался конструктор блин как сказать стандартный пустой и значения этих переменных не вврдят 1

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:31 [ТС]	13
	а поможете исправить мое безобразие напишите пожалуйста 0

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:41 [ТС]	15
	Я нашел, можно было и не париться через goods, затупил, а за исправление спасибо Добавлено через 1 минуту Сбросьте конечно, посмотрю, может интересного чего найду 0

@A555 51 / 51 / 4 Регистрация: 04.04.2011 Сообщений: 209
	09.03.2012, 00:49	16
	а что сбрасывать то? я весь исправленный код сбросил уже 1

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	09.03.2012, 00:57 [ТС]	17
	Спасибо, не обновил страницу просто, спасибо, вопрос решен 0