Create nicely tabbed text output in C#

Question

0.00/5 (No votes)

See more:

I managed to change my code up and now it is running better, my only issue is that i cant use day/week twice.
I want it to output looking like this: http://i.stack.imgur.com/LZu7u.jpg[^]

Here is my code:

C#

int[,] toys = new int[5, 4];
            for (int week = 0; week <= 3; week++)
            {
                for (int day = 0; day <= 4; day++)
                {
                    int tempVal = 0;
                    if (int.TryParse(Microsoft.VisualBasic.Interaction.InputBox("Please enter value for Day " + (day + 1) + " in week " + (week + 1) + "."), out tempVal))
                    {
                        toys[day, week] = tempVal;
                    }
                    else
                    {
                        MessageBox.Show("Invalid entry");
                        day--;
                        continue;
                    }

txtOutput.Text += "\t\t" + "Mon" + "\t" + "Tue" + "\t" + "Wed" + "\t" + "Thu" + "\t" + "Fri" + "\t" + "\r\n";
     //here it says "A local variable named 'week' cannot be declared in this scope because it would give a different meaning to 'week', which is already used in a 'parent or current' scope to denote something else" For day & week
              
for (int week = 0; week <= 3; week++)//foreach week
                   {
                       //construct the line of text which represents the week's data
                       txtOutput.Text += "\tWeek " + (week + 1) + "\t";
                       for (int day = 0; day <= 4; day++)
                       {
                           txtOutput.Text += toys[day, week];
                           if (day != 4)
                           {
                               //so long as it is not the last day, then it tabs over
                               txtOutput.Text += "\t";
                           }
                       }
                       //moving over to the next line
                       txtOutput.Text += "\r\n\r\n";
                   }

Posted 23-Jan-14 14:53pm

Member 10540766

Add a Solution

1 solution

Add a Solution

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

Ron Beyer · Accepted Answer · 2014-01-23T15:01:00

Solution 1

Take a look at string.Format...

Here's an easy version of it:

C#

string headerFormat = "\t\tMon\tTue\tWed\tThur\tFri\n";
string lineFormat = "Week {0}\t{1}\t{2}\t{3}\t{4}\t{5}\n";

textbox1.Text = headerFormat;
for (int week = 0; week < 4; week++)
{
    textbox1.AppendText(string.Format(lineFormat,
        week, toys[week, 0], toys[week, 1], toys[week, 2] toys[week, 3]
        toys[week, 4]));
}

Thats the easiest way to do it, you can play around with the format strings for each field to make them line up if they don't.

Posted 23-Jan-14 15:01pm

Ron Beyer

Updated 23-Jan-14 15:02pm

v2

Comments

Member 10540766 23-Jan-14 21:15pm

Yeah checking it out now, but this gives me the same weird message "A local variable named 'week' cannot be declared in this scope because it would give a different meaning to 'week', which is already used in a 'parent or current' scope to denote something else" in the for int week part

Ron Beyer 23-Jan-14 21:17pm

Just remove the "int" part in front of week in the for loop.

Member 10540766 23-Jan-14 21:33pm

Didn't think it was that obvious. :)

Btw, with week 0,1,2,3,4. Does that output it as Week 0, Week 1 ... etc?
Because techincally i have Week 1-4 only in the array

Ron Beyer 23-Jan-14 21:36pm

Just change week to week+1 inside the for loop and that will fix it.

Like : ...string..Format(lineFormat,
week + 1, toys[week, 0] ...

Member 10540766 23-Jan-14 21:47pm

That seems to fix that issue, now i tried mixing up the for loop to change things up.

If i run ur code it gives me "Index was outside the bounds of the array."
If i run this it gives me "Index (zero based) must be greater than or equal to zero and less than the size of the argument list."
for (week = 0; week < 3; week++)
{
txtOutput.AppendText(string.Format(lineFormat, week + 1, toys[week, 0], toys[week, 1], toys[week, 2], toys[week, 3]));

Im trying to make it all make sense

Ron Beyer 23-Jan-14 21:52pm

You are missing toys[week, 4] after toys[week, 3] in the format string.

Member 10540766 23-Jan-14 22:01pm

That also comes up with outside the bounds of the array.
Its weird because the array is like this up top:

int[,] toys = new int[5, 4];
for (int week = 0; week <= 3; week++)
{

so when i tried ur code it came up with outside bounds :/

Ron Beyer 23-Jan-14 22:11pm

Ahh, you have days and weeks reversed in the definition, it should be int[,] toys = new int[4,5], not sure how you aren't getting the out of bounds error in the first loop...

Member 10540766 23-Jan-14 22:15pm

Doesn't 5,4 come out like this.
http://i.stack.imgur.com/LZu7u.jpg

Ron Beyer 23-Jan-14 22:17pm

You are using the array in the loops like [week, day] so you need to define it the same way. Since there are 5 days and 4 weeks, it should be int[5, 4]. Otherwise in all your loops you have to swap the indicies for week and day so that it is toys[day, week].

Member 10540766 23-Jan-14 23:13pm

So basically to keep it 5,4 i have to change the top for loop to day/week in all loops.
Otherwise is must be 4,5 and week/day ?

Thanks for all the help so far Ron!

Ron Beyer 23-Jan-14 23:23pm

The only thing you have to change is the declaration of toys, it should read:

int[] toys = new int[4, 5]

That is the easiest way to fix it, I wouldn't monkey around with the loops since they are in a good logical order.

Member 10540766 23-Jan-14 23:43pm

I keep getting errors, going to prob re-do like all the code again from scratch..This stuff doeesnt work at the moment.

int[,] toys = new int[4, 5];
for (int week = 0; week <= 3; week++)
{
for (int day = 0; day <= 4; day++)
{
int tempVal = 0;
if (int.TryParse(Microsoft.VisualBasic.Interaction.InputBox("Please enter value for Day " + (day + 1) + " in week " + (week + 1) + "."), out tempVal))
{
toys[day, week] = tempVal;
}
else
{
MessageBox.Show("Invalid entry");
day--;
continue;
}
string headerFormat = "\t\tMon\tTue\tWed\tThur\tFri\n";
string lineFormat = "Week {0}\t{1}\t{2}\t{3}\t{4}\t{5}\n";

txtOutput.Text = headerFormat;
for (week = 0; week < 4; week++)
{
txtOutput.AppendText(string.Format(lineFormat, week + 1, toys[week, 0], toys[week, 1], toys[week, 2], toys[week, 3], toys[week, 4]));
}

The tempval stuff causes it to crash atm

BillWoodruff 23-Jan-14 22:17pm

+5 gold-star for patience awarded :)

Ron Beyer 23-Jan-14 22:18pm

Thanks Bill! :)

Create nicely tabbed text output in C#

1 solution

Solution 1

Add your solution here

Preview 0

Create nicely tabbed text output in C#

1 solution

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us