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

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

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

Новые блоги и статьи Все статьи Все блоги /
Циклы for в Python py-thonny 17.03.2025 Существует множество ситуаций, когда нам нужно выполнить одно и то же действие несколько раз. Цикл for в Python — настоящий рабочий конь для большинства программистов. Если вам нужно пройтись по всем. . .	Предсказание ветвлений - путь к высокопроизводительному C++ NullReferenced 17.03.2025 В высокопроизводительном программировании на C++ каждый такт процессора на счету. Когда речь заходит о разработке систем с низкой задержкой — будь то высокочастотная торговля, обработка потоковых. . .	Паттерн CQRS в C# UnmanagedCoder 17.03.2025 Создание сложных корпоративных приложений часто требует нестандартных подходов к архитектуре. Один из таких подходов — паттерн CQRS (Command Query Responsibility Segregation), предлагающий простую,. . .	Паттерн Цепочка ответственности в C# UnmanagedCoder 17.03.2025 Цепочка ответственности — это поведенческий паттерн проектирования, который позволяет передавать запросы последовательно по цепочке потенциальных обработчиков, пока один из них не обработает запрос. . . .	Создаем микросервисы с NestJS, TCP и Typescript run.dev 17.03.2025 NestJS — фреймворк, который значительно упрощает создание серверных приложений на Node. js. Его прелесть в том, что он комбинирует концепции ООП, функционального программирования и предлагает. . .
Гексагональная архитектура со Spring Boot Javaican 17.03.2025 Если вы когда-нибудь сталкивались с ситуацией, когда внесение простых изменений в базу данных или пользовательский интерфейс заставляло вас переписывать весь код, то вы точно оцените элегантность. . .	Позиционирование Kafka Consumer и Seek-операции Javaican 17.03.2025 Что же такое Consumer Seek в Kafka? По сути, это API-метод, который позволяет программно указать, с какой позиции (offset) Consumer должен начать или продолжить чтение данных из партиции. Без этого. . .	Python NumPy: Лучшие практики и примеры py-thonny 17.03.2025 NumPy (Numerical Python) — одна из ключевых библиотек для научных вычислений в Python. Она превращает Python из просто удобного языка общего назначения в среду для проведения сложных математических. . .	Java Micronaut в Docker: контейнеризация с Maven и Jib Javaican 16.03.2025 Когда речь заходит о микросервисной архитектуре на Java, фреймворк Micronaut выделяется среди конкурентов. Он создан с учётом особенностей облачных сред и контейнеров, что делает его идеальным. . .	Управление зависимостями в Java: Сравнение Spring, Guice и Dagger 2 Javaican 16.03.2025 Инъекция зависимостей (Dependency Injection, DI) — один из фундаментальных паттернов проектирования, который радикально меняет подход к созданию гибких и тестируемых Java-приложений. Суть этого. . .

@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