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

@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 в продакшене: Лучшие практики для SRE Mr. Docker 19.03.2025 Когда сталкиваешься с запуском Kubernetes в продакшене, невольно задаешься вопросом: почему то, что так гладко работало в тестовой среде, вдруг начинает вызывать головную боль на боевых системах?. . .	Разработка продвинутого ИИ в Unity с использованием Behavior Graph GameUnited 19.03.2025 В разработке игр искусственный интеллект персонажей часто становится тем элементом, который превращает хорошую игру в выдающуюся. До недавнего времени разработчикам под Unity приходилось либо писать. . .	Словари в Python: методы работы, оптимизация, сериализация py-thonny 19.03.2025 Каждый хотя бы раз сталкивался с необходимостью хранить связанные данные, где важна не только сама информация, но и их взаимосвязь. В дебрях Python словари — это тот универсальный инструмент, который. . .	Реализация паттерна CQRS с Event Sourcing в PHP Jason-Webb 19.03.2025 CQRS (Command Query Responsibility Segregation) — это архитектурный паттерн, который разделяет операции чтения и записи данных в приложении. Если вы столкнулись с ситуацией, когда ваше PHP-приложение. . .	std::span в C++: Подпредставления и срезы NullReferenced 18.03.2025 Если вы когда-нибудь работали с большими объемами данных в C++, то наверняка сталкивались с необходимостью манипулировать отдельными частями массивов или контейнеров. Традиционные подходы часто. . .
std::span в C++: Доступ к элементам и итерирование NullReferenced 18.03.2025 В C++ каждый разработчик сталкивается с проблемой эффективного управления последовательностями данных. Представьте: вы работаете с массивом, передаете его в функцию, а затем в другую, и каждый раз. . .	Утечки памяти в C# UnmanagedCoder 18.03.2025 Когда мы говорим о разработке приложений на C#, то часто успокаиваем себя мыслью, что сборщик мусора решит все наши проблемы с памятью. "Память управляется автоматически" — эта мантра прочно засела в. . .	std::span в C++: Введение в невладеющее представление NullReferenced 18.03.2025 С появлением стандарта C++20 у нас появился новый инструмент — std::span, который представляет собой невладеющее представление для работы с последовательностями данных. std::span — это легковесный. . .	Введение в Dapr для разработчиков .NET UnmanagedCoder 18.03.2025 Разработка распределенных систем никогда не была настолько востребованной и одновременно такой сложной. Если вы . NET разработчик, то наверняка сталкивались с необходимостью жонглировать обнаружением. . .	Собеседование по Spring Boot: продвинутые вопросы и ответы Javaican 18.03.2025 Собеседования на позиции старших разработчиков и архитекторов требуют глубокого понимания внутренних механизмов Spring Boot, нюансов конфигурирования, подходов к оптимизации и построению сложных. . .

@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