Описать структуру с именем Order

@Дмитрий12 · Регистрация: 19.02.2011

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

Всем Привет!

C++

#include <iostream>
#include <iomanip>
#include <algorithm>
 
struct Order {
    long payer;
    long receiver;
    double sum;
};
 
bool bySumDescendant(const Order & a, const Order & b){
    return a.sum > b.sum;
}
 
int main(){
    const int NUM_ORDERS = 8;
    Order orders[NUM_ORDERS], dummy;
 
    for ( int i = 0; i < NUM_ORDERS; ++i ){
        std::cout << "\nOrder #" << ( i + 1 ) << std::endl;
        std::cout << "Payer account: ";
        std::cin >> orders[i].payer;
        std::cout << "Receiver account: ";
        std::cin >> orders[i].receiver;
        std::cout << "Sum: ";
        std::cin >> orders[i].sum;
    }
 
    std::cout << "\nPayers account to calculate amount of pays: ";
    std::cin >> dummy.payer;
    dummy.sum = 0.0;
    for ( int i = 0; i < NUM_ORDERS; ++i )
        if ( dummy.payer == orders[i].payer )
            dummy.sum += orders[i].sum;
    std::cout << dummy.sum << " Ukrain peastro payd from this account." << std::endl;
 
    std::sort(orders, orders + NUM_ORDERS, bySumDescendant);
    std::cout << "\nAll orders by sum descendant:" << std::endl;
    std::cout << "Payer account       Receiver account    Sum" << std::endl;
    std::cout << std::fixed;
    for ( int i = 0; i < NUM_ORDERS; ++i )
        std::cout << std::left << std::setw(20) << orders[i].payer << std::setw(20) << orders[i].receiver << std::setprecision(2) << orders[i].sum << std::endl;
 
    return 0;
}

Сделал задачу в классе...Но не пойму ошибки в ней!!Вроде все праельно=)
Можете помочь с ошибками=!!!!

@ZloyVolkey · 07.05.2011, 12:45

Не плохо было бы описать суть программы, что она делает и какие ошибки.

@Дмитрий12 · 07.05.2011, 15:23 **[ТС]**

Описать структуру с именем Order содержащую следующие поля: расчётный счет плательщика, расчётный счёт получателя, перечисляемая сумма в гривнах. Написать программу, выполняющую следующие действия:
Ввод с клавиатуры данных в массив, состоящий из 8 структур типа Order, вывод на экран информации о сумме, снятой с расчётного счёта плательщика, введенного с клавиатуры. Упорядочить структуры по возрастанию перечисляемых сумм

@ZloyVolkey · 07.05.2011, 15:25

А какие ошибки?

@LedZeppelin · 07.05.2011, 15:55

Дмитрий12,
Попробуй создать структу вот так:

C++

struct Order {
    long payer;
    long receiver;
    double sum;
}a;

Тогда будет понятно что ты передаеш в функцию.
А вот эта строчка меня совсем удивила:

C++

1	return a.sum > b.sum;

что возвращает return мне лично непонятно

.

@Дмитрий12 · 07.05.2011, 16:09 **[ТС]**

Compiling...
1.cpp
c:\program files\microsoft visual studio\myprojects\h\1.cpp(32) : error C2374: 'i' : redefinition; multiple initialization
c:\program files\microsoft visual studio\myprojects\h\1.cpp(19) : see declaration of 'i'
c:\program files\microsoft visual studio\myprojects\h\1.cpp(41) : error C2374: 'i' : redefinition; multiple initialization
c:\program files\microsoft visual studio\myprojects\h\1.cpp(19) : see declaration of 'i'
Error executing cl.exe.

h.exe - 2 error(s), 0 warning(s)

Вот ошибки

@asics · 07.05.2011, 16:09

Сообщение от LedZeppelin

что возвращает return мне лично непонятно

Еквивалентно

C++

1	return a.sum > b.sum ? true : false;

теперь понятно ?

@LedZeppelin · 07.05.2011, 16:14

Если бы функция умела разговаривать она бы сказала "Я возвращаю тебе a.sum > b.sum;". Вот как бы Вы это поняли?. Я бы лично точно упал в обморок

.

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

Сообщение от asics

Еквивалентно

C++

1	return a.sum > b.sum ? true : false;

теперь понятно ?

Мне и до этого было понятно, непонятен был код

C++

1	return a.sum > b.sum;

@asics · 07.05.2011, 16:16

Сообщение от LedZeppelin

Вот как бы Вы это поняли?

Да что тут не понятного ?

Известно же, что оператор > возвращает значения типа bool.

Сообщение от LedZeppelin

Я бы лично точно упал в обморок

Мда...

@ZloyVolkey · 07.05.2011, 16:20

Попробуй описать переменную i в мейне 1 раз, а в циклах присваивать ей начальные значения.
Хотя код у меня и так запускается без ошибок и предупреждений.

@LedZeppelin · 07.05.2011, 16:22

А в этой строке

C++

1	std::sort(orders, orders + NUM_ORDERS, bySumDescendant);

помоему вы упустили скобки, т.е. по моему надо было писать так

C++

1	std::sort(orders, orders + NUM_ORDERS, bySumDescendant());

@Дмитрий12 · 07.05.2011, 16:25 **[ТС]**

Сообщение от LedZeppelin

А в этой строке

C++

1	std::sort(orders, orders + NUM_ORDERS, bySumDescendant);

помоему вы упустили скобки, т.е. по моему надо было писать так

C++

1	std::sort(orders, orders + NUM_ORDERS, bySumDescendant());

Теперь 4 ошибки выдает!=(

@LedZeppelin · 07.05.2011, 16:27

Дмитрий12,
Вообще попробуйте переписать код (он ведь не такой уж и большой).

@Дмитрий12 · 07.05.2011, 16:27 **[ТС]**

Сообщение от ZloyVolkey

Попробуй описать переменную i в мейне 1 раз, а в циклах присваивать ей начальные значения.
Хотя код у меня и так запускается без ошибок и предупреждений.

ммм!! Не совсем понял=)!!

@asics · 07.05.2011, 16:31

Сообщение от LedZeppelin

помоему вы упустили скобки, т.е. по моему надо было писать так

Не вводите людей в заблуждение, здесь не нужно скобок, потому что ми передаем не функциональный обект, а простой укзатель на функцию.

@ZloyVolkey · 07.05.2011, 16:32

C++

#include <iostream>
#include <iomanip>
#include <algorithm>
 
struct Order {
    long payer;
    long receiver;
    double sum;
};
 
bool bySumDescendant(const Order & a, const Order & b){
    return a.sum > b.sum;
}
 
int main(){
    int i;
    const int NUM_ORDERS = 8;
    Order orders[NUM_ORDERS], dummy;
 
    for ( i = 0; i < NUM_ORDERS; ++i ){
        std::cout << "\nOrder #" << ( i + 1 ) << std::endl;
        std::cout << "Payer account: ";
        std::cin >> orders[i].payer;
        std::cout << "Receiver account: ";
        std::cin >> orders[i].receiver;
        std::cout << "Sum: ";
        std::cin >> orders[i].sum;
    }
 
    std::cout << "\nPayers account to calculate amount of pays: ";
    std::cin >> dummy.payer;
    dummy.sum = 0.0;
    for ( i = 0; i < NUM_ORDERS; ++i )
        if ( dummy.payer == orders[i].payer )
            dummy.sum += orders[i].sum;
 
    std::cout << dummy.sum << " Ukrain peastro payd from this account." << std::endl;
 
    std::sort(orders, orders + NUM_ORDERS, bySumDescendant);
    std::cout << "\nAll orders by sum descendant:" << std::endl;
    std::cout << "Payer account       Receiver account    Sum" << std::endl;
    std::cout << std::fixed;
    for ( i = 0; i < NUM_ORDERS; ++i )
        std::cout << std::left << std::setw(20) << orders[i].payer << std::setw(20) << orders[i].receiver << std::setprecision(2) << orders[i].sum << std::endl;
 
    return 0;
}

@Дмитрий12 · 07.05.2011, 16:38 **[ТС]**

а еще вопросик!!!
А можно место std написать using namespace std; ????

@LedZeppelin · 07.05.2011, 16:38

Сообщение от asics

Не вводите людей в заблуждение, здесь не нужно скобок, потому что ми передаем не функциональный обект, а простой укзатель на функцию.

Я извиняюсь, а где описан этот указатель. Насколько я помню указатель нужно описывать так:

C++

1	bool (*bySumDescendant)(const Order, const Order);

, а потом уже использовать его О_о.

@ZloyVolkey · 07.05.2011, 16:39

Сообщение от Дмитрий12

а еще вопросик!!!
А можно место std написать using namespace std; ????

Можно. )

@Дмитрий12 · 07.05.2011, 16:40 **[ТС]**

Спасибо всем за помощь!!=)

@ZloyVolkey 27 / 27 / 7 Регистрация: 01.05.2011 Сообщений: 85
	07.05.2011, 16:20	10
	Попробуй описать переменную i в мейне 1 раз, а в циклах присваивать ей начальные значения. Хотя код у меня и так запускается без ошибок и предупреждений. 0

@LedZeppelin 7 / 7 / 1 Регистрация: 19.04.2011 Сообщений: 68
	07.05.2011, 16:27	13
	Дмитрий12, Вообще попробуйте переписать код (он ведь не такой уж и большой). 1

@Дмитрий12 1 / 1 / 2 Регистрация: 19.02.2011 Сообщений: 70
	07.05.2011, 16:38 [ТС]	17
	а еще вопросик!!! А можно место std написать using namespace std; ???? 0

@ZloyVolkey 27 / 27 / 7 Регистрация: 01.05.2011 Сообщений: 85
	07.05.2011, 12:45	2
	Не плохо было бы описать суть программы, что она делает и какие ошибки. 0

@Дмитрий12 1 / 1 / 2 Регистрация: 19.02.2011 Сообщений: 70
	07.05.2011, 15:23 [ТС]	3
	Описать структуру с именем Order содержащую следующие поля: расчётный счет плательщика, расчётный счёт получателя, перечисляемая сумма в гривнах. Написать программу, выполняющую следующие действия: Ввод с клавиатуры данных в массив, состоящий из 8 структур типа Order, вывод на экран информации о сумме, снятой с расчётного счёта плательщика, введенного с клавиатуры. Упорядочить структуры по возрастанию перечисляемых сумм 0

@ZloyVolkey 27 / 27 / 7 Регистрация: 01.05.2011 Сообщений: 85
	07.05.2011, 15:25	4
	А какие ошибки? 0

@Дмитрий12 1 / 1 / 2 Регистрация: 19.02.2011 Сообщений: 70
	07.05.2011, 16:09 [ТС]	6
	Compiling... 1.cpp c:\program files\microsoft visual studio\myprojects\h\1.cpp(32) : error C2374: 'i' : redefinition; multiple initialization c:\program files\microsoft visual studio\myprojects\h\1.cpp(19) : see declaration of 'i' c:\program files\microsoft visual studio\myprojects\h\1.cpp(41) : error C2374: 'i' : redefinition; multiple initialization c:\program files\microsoft visual studio\myprojects\h\1.cpp(19) : see declaration of 'i' Error executing cl.exe. h.exe - 2 error(s), 0 warning(s) Вот ошибки 0

@asics Freelance 2891 / 1826 / 356 Регистрация: 09.09.2010 Сообщений: 3,841
	07.05.2011, 16:16	9
	Сообщение от LedZeppelin Вот как бы Вы это поняли? Да что тут не понятного ? Известно же, что оператор > возвращает значения типа bool. Сообщение от LedZeppelin Я бы лично точно упал в обморок Мда... 1

@Дмитрий12 1 / 1 / 2 Регистрация: 19.02.2011 Сообщений: 70
	07.05.2011, 16:27 [ТС]	14
	Сообщение от ZloyVolkey Попробуй описать переменную i в мейне 1 раз, а в циклах присваивать ей начальные значения. Хотя код у меня и так запускается без ошибок и предупреждений. ммм!! Не совсем понял=)!! 0

@asics Freelance 2891 / 1826 / 356 Регистрация: 09.09.2010 Сообщений: 3,841
	07.05.2011, 16:31	15
	Сообщение от LedZeppelin помоему вы упустили скобки, т.е. по моему надо было писать так Не вводите людей в заблуждение, здесь не нужно скобок, потому что ми передаем не функциональный обект, а простой укзатель на функцию. 1

@ZloyVolkey 27 / 27 / 7 Регистрация: 01.05.2011 Сообщений: 85
	07.05.2011, 16:39	19
	Сообщение от Дмитрий12 а еще вопросик!!! А можно место std написать using namespace std; ???? Можно. ) 1

@Дмитрий12 1 / 1 / 2 Регистрация: 19.02.2011 Сообщений: 70
	07.05.2011, 16:40 [ТС]	20
	Спасибо всем за помощь!!=) 0