Why do I get a syntax error at line 22 of this C# code?

Question

0.00/5 (No votes)

See more:

I am very new to C# and I have been learning off of a Brackeys tutorial and I am trying to the print the average of three numbers that are selected by the user. My code is shown below.

Does anyone know why line 22 gets a syntax error? If so, please respond. The error I get is

C:\Users\perry\Desktop\My Awesome Program\Program.cs(22,42): error CS1003: Syntax error, ',' expected [C:\Users\perry\Desktop\My Awesome Program\My Awesome Program.csproj]

Thanks
-P.M

What I have tried:

C#

  1  using System;
  2  
  3  namespace My_Awesome_Program
  4  {
  5      class Program
  6      {
  7          static void Main(string[] args)
  8          {
  9              
 10           int num1;
 11           int num2;
 12           int num3;
 13  
 14           Console.Write("Enter your first number: ");
 15           num1 = Convert.ToInt32(Console.ReadLine());
 16           Console.Write("Enter your second number: ");
 17           num2 = Convert.ToInt32(Console.ReadLine());
 18           Console.Write("Enter your third letter: ");
 19           num3 = Convert.ToInt32(Console.ReadLine());
 20  
 21           int avgRes = num1 + num2 + num3 / 3;
 22           Console.Write("The average of " + num1 + num2 + num3 + " is " + avgRes);
 23           // Console.Write("The average of " , num1 , num2 , num3 , " is " , avgRes);
 24  
 25             //Wait before closing
 26             
 27              Console.ReadKey();
 28          }
 29      }
 30  }

Posted 4-May-22 0:06am

Perry Melenhorst

Updated 4-May-22 16:26pm

Sean Ewington

v6

Add a Solution

Comments

Richard Deeming 4-May-22 6:18am

Neither version of the code you've posted produces the syntax error you describe.

However, your "average" calculation does nothing of the sort. Aside from the fact that you're performing integer division, which will truncate the result, you're also only dividing the third number by the count, rather than dividing the sum of the numbers by the count.

Eg: Enter 1, 1, 1, and your code will tell you that the average is 2, which is clearly wrong.

double avgRes = (num1 + num2 + num3) / 3D;
Console.WriteLine("The average of ({0}, {1}, {2}) is {3}", num1, num2, num3, avgRes);

Graeme_Grant 4-May-22 6:44am

Or

Console.WriteLine($"The average of ({num1}, {num2}, {num3} is {avgRes}");

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

M Imran Ansari · Answer 1 · 2022-05-04T16:26:00

In your problem statement, you are taking inputs as integer and can take avgRes as Integer but the error is you need to enclosed three variables in parenthesis "()" due to operator presidency. Without parenthesis "/" have high presidency and will calculate before others. Secondly, you can also use String.Format() method to align your values.

C#

int avgRes = (num1 + num2 + num3) / 3;
Console.Write(String.Format("The Average of {0}, {1}, {2} is {3}", num1, num2, num3, avgRes));

OriginalGriff · Answer 2 · 2022-05-04T00:50:00

As Richard has said, that code does not generate a compilation error, not does it calculate an average properly.

I'd suggest that you start using string interpolation though, it makes your code easier to read:

Console.WriteLine($"The average of {num1 + num2 + num3} is {avgRes}");

And produces the same result.

But don't ever use Convert methods on user input: they fail if he miskeys. Instead, use the TryParse methods:

C#

if (!int.TryParse(Console.ReadLine(), out num1))
   {
   ... report input problem to the user ...
   return;
   }

With only three inputs it's not a major problem - but if you've just typed 20 values in and miskey the last one ... if the app crashes the user is not going to be a happy bunny ... Write yourself a method to get an integer and loop until the user enters a valid number.

Why do I get a syntax error at line 22 of this C# code?

2 solutions

Solution 2

Solution 1

Add your solution here

Preview 0