England Forum - UK Forum
 

Go Back   England Forum - UK Forum > Computers and Technology > Developers Forum > C++ Forum

 

 


Reply
 
LinkBack Thread Tools Display Modes
  #1 (permalink)  
Old 10-02-2008
Peon
 
Join Date: Aug 2008
Posts: 2
Default what's wrong with this c++ code?

include<iostream>
using namespace std;

int main ()
{
int number;
cout<<"Enter Number"<<"\n"; cin>>number; cout<<"\n";

int count=0;int sum=0;
while(count<=number){
cout<<count<<"\n";
sum = sum + ((number+1)*number) - count;
count++;}
cout<<sum<<"\n";
return 0;
}

say 5 was entered by a user the code should produce the following result: (5+4+3+2+1+0=15)
Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
Sponsored Links

  #2 (permalink)  
Old 10-02-2008
Peon
 
Join Date: Aug 2008
Posts: 2
Default

include<iostream>

is wrong. It should be

#include <iostream>

Also try separating the declarations of count and sum onto two different lines, and the cin and cout (just below the line where number is declared) onto two different lines. It's better style and it might be necessary for some, if not all, compilers.

I don't see why you use this, though:

sum = sum + ((number+1)*number) - count;

You should just replace that with:

sum = sum + count;

Finally, add this just before the 'return 0;' line:

cin.get();

Compile the code with all the above changes and run it. If it closes the console window automatically before you can see the output, add another cin.get(); below the first one, then compile and run again. It should work.
Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #3 (permalink)  
Old 10-02-2008
Peon
 
Join Date: Oct 2008
Posts: 1
Default

first off, your code is very messy. You should always put new statements on new lines.

include<iostream>
using namespace std;

int main ()
{
int number;
cout<<"Enter Number"<<"\n";
cin>>number; cout<<"\n";

int count=0;
int sum=0;

while(count<=number){
cout<<count<<"\n";
sum = sum + ((number+1)*number) - count;
count++;}

cout<<sum<<"\n";
return 0;

Much easier to read.

Now some changes to make.

include<iostream>
using namespace std;

int main ()
{
int number;

cout<<"Enter Number"<<"\n"; // instead of using \n you can use the 'endl' object. Or if you wanted to use \n, then use it the correct way. You should do cout << "Enter Number\n"; instead.

cin>>number;

cout<<"\n"; // once again. cout << endl;

int count=0;
int sum=0;

while(count<=number){
cout<<count<<"\n";
sum = sum + ((number+1)*number) - count;
count++;} // Here you want to add 5 and repeat that decreasing by 1 each time. Your code is a bit sloppy. There is an easier way. You should put 'sum += number-count;'. Much easier isn't it? The += operator is just adding the value of 'number-count' to sum.

cout<<sum<<"\n"; // cout << sum << endl;
return 0;
Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #4 (permalink)  
Old 10-02-2008
Peon
 
Join Date: Oct 2008
Posts: 1
Default

int count=0;int sum=0;
while(count<=number)
{
cout<<number<<"+";
sum = sum + number - count;
count++;
}
cout<<"="<<sum<<"\n";
return 0;
}
Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On


All times are GMT. The time now is 04:04 PM.


Powered by vBulletin® Version 3.6.10
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.
Search Engine Optimization by vBSEO 3.2.0 RC7
Sedo - Buy and Sell Domain Names and Websites project info: englanddebate.co.uk Statistics for project englanddebate.co.uk etracker® web controlling instead of log file analysis

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 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183