Programming Forums > Application Development > C++

angry_asian · Jun 28th, 2006, 8:32 PM

what things can one shorten in this area program

#include <iostream.h>
int area(unsigned short int x, unsigned short int y)
{
	return (x * y);
}

int main()
{
unsigned short int x, y, z;
cout << "Enter width and height (with a space in between):";
cin >> x;
cin >> y;
z=area(x,y);
cout << "Area=" << z << "\n";
return 0;
}

i've already shortend it from what the book had :banana:

Jimbo · Jun 28th, 2006, 8:37 PM

newlines... yep, they've got to go... Seriously though, shorter code is not necessarily better code. Often it's the opposite. Obfuscated code is only good for confusing people (hence the name).

By the way, you're using a deprecated header. Use #include<iostream> and look up namespace std.

Yarvin · Jun 28th, 2006, 8:37 PM

uhm... none that I can see.....

Ooble · Jun 28th, 2006, 8:42 PM

There are a couple of ways to shorten this without making it any less readable:

(Toggle Plain Text)

#include <iostream>

int main()
{
    unsigned int x, y;
    std::cout << "Enter width and height (with a space in between):";
    std::cin >> x >> y;
    std::cout << "Area = " << x * y << "\n";
    
    return 0;
}

Seriously though, I don't know why you'd bother. DaWei has a link to an excellent article on optimisation and how to determine whether it's necessary - if I wasn't so sleepy, I'd point you to it. I think it's in his signature. More important would be error-checking.

angry_asian · Jun 28th, 2006, 8:49 PM

how do i look up namespace std...?
instead of
#include<iostream.h>
is it
#include<iostream>
using namespace std;
just like that or what exactly (with right punctuation and spelling) is it?

Kilo · Jun 28th, 2006, 9:05 PM

(without offense): [b]how do i look up namespace std...?[b] Aren't you supposed to be some super asain? genius? I mean I have asked some pretty (IMO) stupid questions but you just blew me out of the water!

using the following is stating that the entire program is using the standard namespace.

(Toggle Plain Text)

using namespace std;

without using the above use the following to state the namespace in a single instance

(Toggle Plain Text)

std::cout << "(IMO) this one is better" << std::endl;

snipertomcat · Jun 28th, 2006, 9:12 PM

ouch...

Infinite Recursion · Jun 28th, 2006, 9:30 PM

you could optimize a bit within the inline keyword...

(Toggle Plain Text)

inline int area(unsigned short int x, unsigned short int y)
{  return (x * y);  }

Prm753 · Jun 28th, 2006, 9:46 PM

Kilo, why did you make a reference to him being Asian? That has nothing to do with the OP's question.

I think Ooble about shortened it up with it still being readable. Of course, you could still do this:

(Toggle Plain Text)

#include <iostream>
int main() { int x, y; std::cout << "Enter x and y values (with space): "; std::cin >> x >> y; std::cout << "Area: " << x * y << std::endl; return 0; }

That would be an awful way to put it, though.

angry_asian · Jun 28th, 2006, 9:54 PM

Quote:

Originally Posted by Kilo

(without offense): [b]how do i look up namespace std...?[b] Aren't you supposed to be some super asain? genius? I mean I have asked some pretty (IMO) stupid questions but you just blew me out of the water!

using the following is stating that the entire program is using the standard namespace.

(Toggle Plain Text)

using namespace std;

without using the above use the following to state the namespace in a single instance

(Toggle Plain Text)

std::cout << "(IMO) this one is better" << std::endl;

well i did think that if u read my second post fully i said "i think it is "using namespace std;"" so i was just making sure
but still, thanks especially for the second way fo defining without doing it at the beginningm thats what ooble did and it confused me but now i get it
ps- i am a genius, just not at programming

Jun 28th, 2006, 8:32 PM	#1
angry_asian Hobbyist Programmer Join Date: Jun 2006 Location: at my computer desk Posts: 138 Rep Power: 3	how short can you make this what things can one shorten in this area program (Toggle Plain Text) #include <iostream.h> int area(unsigned short int x, unsigned short int y) { return (x * y); } int main() { unsigned short int x, y, z; cout << "Enter width and height (with a space in between):"; cin >> x; cin >> y; z=area(x,y); cout << "Area=" << z << "\n"; return 0; } #include <iostream.h> int area(unsigned short int x, unsigned short int y) { return (x * y); } int main() { unsigned short int x, y, z; cout << "Enter width and height (with a space in between):"; cin >> x; cin >> y; z=area(x,y); cout << "Area=" << z << "\n"; return 0; } i've already shortend it from what the book had :banana:

Jun 28th, 2006, 8:42 PM	#4
Ooble I eat cake for breakfast. Join Date: Jul 2004 Location: In my box. Posts: 4,434 Rep Power: 9	There are a couple of ways to shorten this without making it any less readable: (Toggle Plain Text) #include <iostream> int main() { unsigned int x, y; std::cout << "Enter width and height (with a space in between):"; std::cin >> x >> y; std::cout << "Area = " << x * y << "\n"; return 0; } #include <iostream> int main() { unsigned int x, y; std::cout << "Enter width and height (with a space in between):"; std::cin >> x >> y; std::cout << "Area = " << x * y << "\n"; return 0; } Seriously though, I don't know why you'd bother. DaWei has a link to an excellent article on optimisation and how to determine whether it's necessary - if I wasn't so sleepy, I'd point you to it. I think it's in his signature. More important would be error-checking. __________________ Me :: You :: Them

Jun 28th, 2006, 9:05 PM	#6
Kilo Expert Programmer Join Date: Nov 2005 Location: In Pink Clam? Posts: 542 Rep Power: 0	(without offense): [b]how do i look up namespace std...?[b] Aren't you supposed to be some super asain? genius? I mean I have asked some pretty (IMO) stupid questions but you just blew me out of the water! using the following is stating that the entire program is using the standard namespace. (Toggle Plain Text) using namespace std; using namespace std; without using the above use the following to state the namespace in a single instance (Toggle Plain Text) std::cout << "(IMO) this one is better" << std::endl; std::cout << "(IMO) this one is better" << std::endl; __________________ "When in Rome, Do as the Romans Do" "Beauty is in the eye of the BEER holder" "Save your breath your going to need it for your blow up doll later" SearchLores.org

Jun 28th, 2006, 9:12 PM	#7
snipertomcat Programmer Join Date: Nov 2005 Location: Spring Valley, CA Posts: 52 Rep Power: 3	ouch... __________________ if (u=an_asshole) then GOTO (hell)

Jun 28th, 2006, 9:30 PM	#8
Infinite Recursion Programming Guru Join Date: Jul 2004 Location: United States Posts: 3,467 Rep Power: 8	you could optimize a bit within the inline keyword... (Toggle Plain Text) inline int area(unsigned short int x, unsigned short int y) { return (x * y); } inline int area(unsigned short int x, unsigned short int y) { return (x * y); } __________________ http://jasonpowers.net "There are a thousand hacking at the branches of evil to one who is striking at the root."

Jun 28th, 2006, 8:37 PM	#2
Jimbo Battle Programmer Join Date: Feb 2006 Location: Bellevue, WA, USA Posts: 763 Rep Power: 3	newlines... yep, they've got to go... Seriously though, shorter code is not necessarily better code. Often it's the opposite. Obfuscated code is only good for confusing people (hence the name). By the way, you're using a deprecated header. Use #include<iostream> and look up namespace std.

Jun 28th, 2006, 8:37 PM	#3
Yarvin Programmer Join Date: Sep 2005 Location: Anchorage, Alaska Posts: 37 Rep Power: 0	uhm... none that I can see.....

Jun 28th, 2006, 8:49 PM	#5
angry_asian Hobbyist Programmer Join Date: Jun 2006 Location: at my computer desk Posts: 138 Rep Power: 3	how do i look up namespace std...? instead of #include<iostream.h> is it #include<iostream> using namespace std; just like that or what exactly (with right punctuation and spelling) is it?

Jun 28th, 2006, 9:46 PM	#9
Prm753 Professional Programmer Join Date: Oct 2005 Location: United States Posts: 447 Rep Power: 4	Kilo, why did you make a reference to him being Asian? That has nothing to do with the OP's question. I think Ooble about shortened it up with it still being readable. Of course, you could still do this: (Toggle Plain Text) #include <iostream> int main() { int x, y; std::cout << "Enter x and y values (with space): "; std::cin >> x >> y; std::cout << "Area: " << x * y << std::endl; return 0; } #include <iostream> int main() { int x, y; std::cout << "Enter x and y values (with space): "; std::cin >> x >> y; std::cout << "Area: " << x * y << std::endl; return 0; } That would be an awful way to put it, though. __________________ The world's first athletic computer geek! The home of PrProgramsStudios How not to post a question: <-- Please don't reply

Currently Active Users Viewing This Thread: 1 (0 members and 1 guests)

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode

Programming Forums > Application Development > C++

how short can you make this