Solved

How to recode a FOR Loop so that the variables in the FOR Loop are assigned to a StringBuilder object that is created outside of the FOR Loop?

Posted on 2014-01-20
7
437 Views
Last Modified: 2014-01-22
I am trying to rewrite a C# console application to employ StringBuilder.

Do you know how I can recode the contents of my FOR loop so that the variables within the FOR loop can be assigned to the StrinBuilder object created outside of the FOR Loop?

My code is in the attached file.
stringbuilder.txt
0
Comment
Question by:zimmer9
  • 3
  • 2
  • 2
7 Comments
 
LVL 62

Expert Comment

by:Fernando Soto
ID: 39795970
Well if you want to write the contents of the StringBuilder to the StreamWriter you need to change this:

sw.WriteLine(stringb);

To this:

sw.WriteLine(stringb.ToString());


To prepare the StringBuilder for the next iteration through the loop you need to clear the context of the StringBuilder by doing this:

stringb.Length = 0;

That should do it.
0
 

Author Comment

by:zimmer9
ID: 39795985
I don't seem to be generating any of the variable values in my output, all the values are BLANK:

For ex:       stringb.Append(checknum);
                  stringb.Append(routingtransit);
                  stringb.Append(bankacct);
             

COMMENT: specify code page of the index date
CODEPAGE:819
COMMENT: CHECK NUMBER #0
GROUP_FIELD_NAME:CheckNumber
GROUP_FIELD_VALUE:                            <-------    checknum
GROUP_FIELD_NAME:RoutingTransit
GROUP_FIELD_VALUE:                            <-------    routingtransit  
GROUP_FIELD_NAME:BankName
GROUP_FIELD_VALUE:BANK
GROUP_FIELD_NAME:BankAccountNo
GROUP_FIELD_VALUE:                            <-------    bankacct
GROUP_FIELD_NAME:CheckAmount
GROUP_FIELD_VALUE:
GROUP_FIELD_NAME:CpcsNo
GROUP_FIELD_VALUE:00000000
GROUP_FIELD_NAME:CheckPaidDate
GROUP_FIELD_VALUE:
GROUP_FIELD_NAME:OfficeNo
GROUP_FIELD_VALUE:
0
 
LVL 62

Accepted Solution

by:
Fernando Soto earned 500 total points
ID: 39796005
Hi zimmer9;

That is because from what I see you write to the StringBuilder at the begining of the subroutine where all the values of the variables have been initialized to an empty string or in the case of CheckOrderNum to the value of 0. What you need to do is move the individual StringBuilder Appends to where you set the value of the variables. For example you would need to move this line from the top of the code :

stringb.Append(CheckOrderNum);

to after this line in your code:

CheckOrderNum = CheckOrderNum + 1;
stringb.Append(CheckOrderNum);

So first place all the column header text into StringBuilder and write that out to the StreamWriter then after you moved the other StringBuilder statement to the right locations the loop will write the data underneath it.
0
Master Your Team's Linux and Cloud Stack!

The average business loses $13.5M per year to ineffective training (per 1,000 employees). Keep ahead of the competition and combine in-person quality with online cost and flexibility by training with Linux Academy.

 

Author Comment

by:zimmer9
ID: 39796056
Then the way I understand your observation is that I cannot build the entire StringBuilder structure beforehand due to the variable values.

Only the constant header values can be contained in the StringBuilder before entering the FOR LOOP.

As you can see, virtually every other record contains a variable value, so how could I place all the FIXED column header text into the StringBuilder before entering the FOR LOOP while leaving alternate records to be appended inside the FOR LOOP.  

For ex: It seems to me that I can only place the following 2 lines in the StringBuilder and then the rest of the records would be appended inside the FOR LOOP.


2 lines in the StringBuilder:
StringBuilder stringb = new StringBuilder();  
stringb.Append("COMMENT: CHECK NUMBER #");
----------------------------------------------------------------------------

then in the FOR LOOP
----------------------------
CheckOrderNum = CheckOrderNum + 1;
stringb.Append(CheckOrderNum);
stringb.Append(Environment.NewLine);

stringb.Append("GROUP_FIELD_NAME:CheckNumber");
stringb.Append(Environment.NewLine);
stringb.Append("GROUP_FIELD_VALUE:");
checknum = s.Substring(246, 6);

stringb.Append(checknum);
0
 
LVL 12

Expert Comment

by:deanvanrooyen
ID: 39796295
What is the source of the data is it a single database row? If it is and you are iterating through the columns and know the column names but for some reason can't get the column names at runtime eg you are using ordinal position, you could use string builder to build a string pattern e.g
X.appendline("name:{0}")

Then once you have all the lines just do some thing like this
String.format(X.tostring(), row[0], row[1])

This does seem a bit clunky though your better being more oo create an object like a view model approach with matching properties and override to string with something injected so you can unit test it easily and the data isn't couple to your view or string representation, think about if this is plain text for and email what happens I you are asked to make it look good and send it as HTML?

Another route is your dealing with key values here? Why not use a collection iterate like so
Full collection from data source column name as key, field as value
Foreach key in collection. Keys
X.appendline(key + collection[key])
0
 

Author Comment

by:zimmer9
ID: 39796997
I am reading a binary check image file and I know the field values that I need to retrieve based on their displacement after a specific record type field value. For each check, there is another record of this specific record type.
0
 
LVL 12

Expert Comment

by:deanvanrooyen
ID: 39802226
I would create an object that contains those fields with the exact names, populate the object and you could be clever using reflection and output the property name and value in an overriden tostring
0

Featured Post

Master Your Team's Linux and Cloud Stack

Come see why top tech companies like Mailchimp and Media Temple use Linux Academy to build their employee training programs.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Introduction This article shows how to use the open source plupload control to upload multiple images. The images are resized on the client side before uploading and the upload is done in chunks. Background I had to provide a way for user…
This article shows how to deploy dynamic backgrounds to computers depending on the aspect ratio of display
This Micro Tutorial will give you a basic overview how to record your screen with Microsoft Expression Encoder. This program is still free and open for the public to download. This will be demonstrated using Microsoft Expression Encoder 4.
Email security requires an ever evolving service that stays up to date with counter-evolving threats. The Email Laundry perform Research and Development to ensure their email security service evolves faster than cyber criminals. We apply our Threat…

805 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question