How to draw pattern like a diamond C#

Question

1.00/5 (3 votes)

See more:

Write a program in C# to display the diamond pattern shown below. The user provides the number of the row that is the middle of the diamond. Note that the number of stars in the rows are are 1, 3, 5, etc respectively and then downwards ….5, 3, 1.

   *   
  *** 
 ***** 
******* 
 ***** 
  *** 
   *

Please note all stars are centered.

What I have tried:

C#

if (radioButton2.Checked) //full diamond
            {
                int i, j, k, n;
                //Convert Text Box TOINT32
                n = Convert.ToInt32(textBox1.Text);  
                string line = ""; // for individual output

                for (i = 1; i <= n; i++)//Upper Diamond
                {
                    
                    for (k = 1; k <= n-i; k++) 
                    {
                        line = "";
                    }
                    for (j = 1; j <= i; j++)
                    {
                        line +=  "* " ;
                    }
                    richTextBox1.SelectionAlignment = HorizontalAlignment.Center;
                    richTextBox1.Text += line + "\n";
                }
                for (i = 1; i <= n - 1; i++) //Bottom Diamond
                {
                    for (k = 1; k <= i; k++)
                    {
                        line = "";
                    }
                    for (j = n - 1; j >= i; j--)
                    {
                        line += "* ";
                    }
                    richTextBox1.SelectionAlignment = HorizontalAlignment.Center;
                    richTextBox1.Text += line + "\n";
                }

            }

My results is

      *  
     * * 
    * * * 
* * * * * * * 
    * * * 
     * * 
      *

Please note all stars are centered

my diamond comes out wrong in a 1 2 3 7 and 3 2 1 formart. please help

Posted 14-May-17 15:41pm

Member 13200081

Updated 15-May-17 7:39am

PIEBALDconsult

v2

Add a Solution

3 solutions

Solution 3

I would start off by analysing what is going on. As I see it, there are a number of fixed-length lines. The length of each line is equal to the total number of lines. The number of lines needs to be an odd number and >1 so that you have a middle line to give the diamond effect. Initially there is a central ‘*’ character and then a new * is added to the left and right of the existing stars as the line number increases. This carries on until the middle line has been printed, then the process is reversed and the most recently added * to the left and the right is then replaced by a space.

Although a string is char array, it is read only. You can, however, use a char[], manipulate that and then convert it to a string before printing it. Something like

C#

var currentLine = new string(' ', lines); //fill line with spaces
char[] characters = currentLine.ToCharArray();

You can then add ‘*’ to the characters array by indexing into it. You will need a leftposition and rightposition variable along with an increment variable. The increment is set to -1 after the middle line is printed.

C#

characters[lpos] = charToPaste;
characters[rpos] = charToPaste;
 Console.WriteLine(new string(characters));
lpos -= increment;
rpos += increment;

You need only a single loop to do this.

Posted 15-May-17 7:38am

George Swan

Solution 1

Check carefully where you clear the line variable.

When you don't understand what your code is doing or why it does what it does, the answer is debugger.
Use the debugger to see what your code is doing. Just set a breakpoint and see your code performing, the debugger allow you to execute lines 1 by 1 and to inspect variables as it execute, it is an incredible learning tool.

Debugger - Wikipedia, the free encyclopedia[^]

Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]
Basic Debugging with Visual Studio 2010 - YouTube[^]
The debugger is here to show you what your code is doing and your task is to compare with what it should do.
There is no magic in the debugger, it don't find bugs, it just help you to. When the code don't do what is expected, you are close to a bug.

Posted 14-May-17 16:02pm

Patrice T

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)

Mehedi Shams · Accepted Answer · 2017-05-14T16:35:00

Hi ,
There is a mistake in this line:

for (i = 1; i <= n; i++)//Upper Diamond

It should be:

for (i = 1; i < n; i+=2)//Upper Diamond - change in comparison and increment

Same applies to the bottom diamond. There needs to be a couple of adjustments for the bottom diamond; I leave that up to you as I can only guide you to the solution, but you need to do the homework.

Plus line cleaning involved in the for loop is unnecessary.

for (k = 1; k <= n-i; k++) 
{
   line = "";
}

This can simply be a single line; the FOR loop is unnecessarily punishing the compiler to do the same thing for a couple of times.

line = "";