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

@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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
Осваиваем Kubernetes: Подробная шпаргалка Mr. Docker 15.03.2025 Kubernetes — это открытая платформа для автоматизации развертывания, масштабирования и управления контейнеризированными приложениями. Он был создан для решения проблем, с которыми сталкиваются. . .	Лучшие PHP REST API фреймворки Jason-Webb 15.03.2025 Современные PHP REST API фреймворки предлагают большой набор функциональности: от автоматической валидации данных и управления маршрутизацией до генерации документации и интеграции с различными. . .	Многопоточность в Java с Project Loom: виртуальные или обычные потоки Javaican 15.03.2025 Многопоточность всегда была одноим из основных элементов в разработке современного программного обеспечения. Она позволяет приложениям обрабатывать несколько задач одновременно, что критично для. . .	Что нового в Swift 6 и особенности миграции mobDevWorks 15.03.2025 Swift 6 — это новый крупный релиз языка программирования от Apple, анонсированный на WWDC 2024. Если вы следили за эволюцией Swift, то наверняка заметили, что многие значимые возможности, которые. . .	Вопросы на собеседовании по 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 хороши для повседневной разработки, но совершенно недостаточны для высоконагруженных систем. Представьте, что вы запускаете финансовую платформу, обрабатывающую. . .

@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