We value your feedback.
Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!
Solved
C# -- remove comma when reading TXT file ?
Posted on 2011-03-04
The below code reads in my below "John Doe" variance.txt correctly.
NAME,FILE#,SSN,CHECKVIEWPA
"John Doe",229999,9263,12/06/201
"Jane Doe",181,6331,12/06/2010,4
"John Smith",1041,1207,12/06/201
--------------------------
How can I change the code to remove the , or replacing it with
a space or something so it reads in the below "John, Doe"
as one column ?
NAME,FILE#,SSN,CHECKVIEWPA
"John, Doe",229999,9263,12/06/201
"Jane, Doe",181,6331,12/06/2010,4
"John, Smith",1041,1207,12/06/201
--------------------------
private DataSet ReadFile(string filePath)
{
OleDbConnection conn = new OleDbConnection("Provider=
conn.Open();
OleDbDataAdapter adapter = new OleDbDataAdapter("SELECT * FROM " + System.IO.Path.GetFileName
DataSet ds = new DataSet("Temp");
adapter.Fill(ds);
return ds;
}
--------------------------
public bool ImportFileVariance()
{
string filePath = @"C:\import\variance.txt";
DataSet ds = ReadFile(filePath);
return dbAccess.ImportVarianceRec
}
--------------------------
NAME,FILE#,SSN,CHECKVIEWPA
"John Doe",229999,9263,12/06/201
"Jane Doe",181,6331,12/06/2010,4
"John Smith",1041,1207,12/06/201
--------------------------
How can I change the code to remove the , or replacing it with
a space or something so it reads in the below "John, Doe"
as one column ?
NAME,FILE#,SSN,CHECKVIEWPA
"John, Doe",229999,9263,12/06/201
"Jane, Doe",181,6331,12/06/2010,4
"John, Smith",1041,1207,12/06/201
--------------------------
private DataSet ReadFile(string filePath)
{
OleDbConnection conn = new OleDbConnection("Provider=
conn.Open();
OleDbDataAdapter adapter = new OleDbDataAdapter("SELECT * FROM " + System.IO.Path.GetFileName
DataSet ds = new DataSet("Temp");
adapter.Fill(ds);
return ds;
}
--------------------------
public bool ImportFileVariance()
{
string filePath = @"C:\import\variance.txt";
DataSet ds = ReadFile(filePath);
return dbAccess.ImportVarianceRec
}
--------------------------
[X]
Welcome to Experts Exchange
Add your voice to the tech community where 5M+ people just like you are talking about what matters.
- Help others & share knowledge
- Earn cash & points
- Learn & ask questions
8
3-
2
- +3
17 Comments
Read the text into a string.
string txtFileContents = "";
// execute the code to open the file and serialize it to a string.
//Now use .Replace()
txtFileContents = txtFileContents.Replace(",", "_"); // the underscore can be a space or anything you want.
Active today
Hi finance_teacher;
This code snippet should give you the results you want.
Fernando
This code snippet should give you the results you want.
// Reading data from a text file.
string data = System.IO.File.ReadAllText( "TestData.txt" );
// Remove all commas that are in between quotes
data = System.Text.RegularExpressions.Regex.Replace( data, @"(\x22.*)(,)(.*\x22)", "$1$3" );
Fernando
You could also use a Split() function.
Basically, do something like this:
string[] parts = curLine.Split(',');
That should give you an array of every field. Then you can manipulate your fields the way you want to.
Basically, do something like this:
string[] parts = curLine.Split(',');
That should give you an array of every field. Then you can manipulate your fields the way you want to.
Active today
How can I get the below "data" string into my "ds" DataSet ?
public bool ImportFileVariance()
{
string filePath = @"C:\import\variance.txt";
string data = System.IO.File.ReadAllText
//below works, replacing , with space
data = System.Text.RegularExpress
//errors when running below step
DataSet ds = ReadFile(data);
return dbAccess.ImportVarianceRec
}
public bool ImportFileVariance()
{
string filePath = @"C:\import\variance.txt";
string data = System.IO.File.ReadAllText
//below works, replacing , with space
data = System.Text.RegularExpress
//errors when running below step
DataSet ds = ReadFile(data);
return dbAccess.ImportVarianceRec
}
Active today
What does your original CSV text file look like (your question is a little unclear):
Doe,John,1234,123-45-6789,
John,Doe,1234,123-45-6789,
John Doe,1234,123-45-6789, etc
"Doe", "John",1234,123-45-6789, etc
"Doe, John",1234,123-45-6789, etc
"John", "Doe",1234,123-45-6789, etc
"John, Doe",1234,123-45-6789, etc
And to be clear you want to "transform" it like below?
"John Doe",1234,123-45-6789, etc
Doe,John,1234,123-45-6789,
John,Doe,1234,123-45-6789,
John Doe,1234,123-45-6789, etc
"Doe", "John",1234,123-45-6789, etc
"Doe, John",1234,123-45-6789, etc
"John", "Doe",1234,123-45-6789, etc
"John, Doe",1234,123-45-6789, etc
And to be clear you want to "transform" it like below?
"John Doe",1234,123-45-6789, etc
Active today
Ahh, sorry - didn't notice you were using the OleDb provider to read the source file. Try this ReadFile() method:
static DataSet ReadFile(string filePath)
{
string connectionString = String.Format(
@"Provider=Microsoft.Jet.OleDb.4.0;Data Source={0};Extended Properties=""Text;HDR=Yes;FMT=Delimited""",
System.IO.Path.GetDirectoryName(filePath));
using (OleDbConnection conn = new OleDbConnection(connectionString))
{
string command = String.Format(
@"SELECT * FROM {0}", System.IO.Path.GetFileName(filePath));
using (OleDbDataAdapter adapter = new OleDbDataAdapter(command, conn))
{
DataSet ds = new DataSet("Temp");
adapter.Fill(ds);
// Loop through the data table and change names like "Doe,John" to "John Doe"
foreach (DataRow row in ds.Tables[0].Rows)
row[0] = Regex.Replace((string)row[0], @"([^,]*),([^,]*)", "$2 $1");
return ds;
}
}
Active today
In my quick test I was using it as a static method, you don't necessarily have to do that in your project - change "static" to "private" to keep it the way you had it (I also missed the closing curly-brace when I copy/pasted).
Active today
And I didn't show it, but you'll need a using System.Text.RegularExpress
Active today
Hi finance_teacher;
To your last question. You have a couple of options, 1 save the modified data to the file system and then use your code. 2 create the data table in code as shown in the code snippet below.
Fernando
To your last question. You have a couple of options, 1 save the modified data to the file system and then use your code. 2 create the data table in code as shown in the code snippet below.
private void button1_Click( object sender, EventArgs e )
{
string data = System.IO.File.ReadAllText("TestData.txt" );
data = System.Text.RegularExpressions.Regex.Replace( data, @"(\x22.*)(,)(.*\x22)", "$1$3" );
DataTable dt = CtreateTable( data );
dataGridView1.DataSource = dt;
}
private DataTable CtreateTable( string data )
{
string[] dataRows = data.Split( "\r\n".ToCharArray( ), StringSplitOptions.RemoveEmptyEntries );
DataTable dt = new DataTable( );
// First line has the column headders
foreach( string headder in dataRows[ 0 ].Split( ",".ToCharArray( ) ) )
{
dt.Columns.Add( new DataColumn( headder ) );
}
// Load the data
for( int idx = 1; idx < dataRows.Length; idx++)
{
string[] fields = dataRows[ idx ].Split( ",".ToCharArray( ) );
int column = 0;
DataRow dr = dt.NewRow( );
foreach( string field in fields )
{
dr[ column ] = field.Trim("\"".ToCharArray());
column++;
}
dt.Rows.Add( dr );
}
return dt;
}
Fernando
Active today
I think the below line would incorrectly split "Doe,John" into two fields:
If you wanted to go that route I think I would use StreamReader.ReadLine (as opposed to looping through a string[]), and I think you would need to use Regex.Replace(dataRows[idx
string[] fields = dataRows[ idx ].Split( ",".ToCharArray( ) );
If you wanted to go that route I think I would use StreamReader.ReadLine (as opposed to looping through a string[]), and I think you would need to use Regex.Replace(dataRows[idx
Active today
@ tgerbert;
This is the code that tales care of the commas in "Doe, John" field.
this code correctly splits the filed in each line. At this point "Doe, John" is "Doe John" without comma.
Fernando
This is the code that tales care of the commas in "Doe, John" field.
data = System.Text.RegularExpressions.Regex.Replace( data, @"(\x22.*)(,)(.*\x22)", "$1$3" )
this code correctly splits the filed in each line. At this point "Doe, John" is "Doe John" without comma.
string[] fields = dataRows[ idx ].Split( ",".ToCharArray( ) );
Fernando
Active today
Ah, I missed that it was in the other method.
I still would use Regex.Replace(data, @"[""]([^""]*),([^""]*)[""
I still would use Regex.Replace(data, @"[""]([^""]*),([^""]*)[""
Following code should help -
public System.Data.DataSet LoadData()
{
string path = @"C:\a.txt";
string fileContents = System.IO.File.ReadAllText(path);
fileContents = System.Text.RegularExpressions.Regex.Replace(fileContents, @"(\x22.*)(,)(.*\x22)", "$1$3");
System.IO.File.WriteAllText(path,fileContents.ToString());
System.Data.OleDb.OleDbConnection conn = new System.Data.OleDb.OleDbConnection("Provider=Microsoft.Jet.OleDb.4.0; Data Source = " + System.IO.Path.GetDirectoryName(path) + "; Extended Properties = \"Text;HDR=YES;FMT=Delimited\"");
conn.Open();
System.Data.OleDb.OleDbDataAdapter adapter = new System.Data.OleDb.OleDbDataAdapter("SELECT * FROM " + System.IO.Path.GetFileName(path), conn);
System.Data.DataSet ds = new System.Data.DataSet("Temp");
adapter.Fill(ds);
return ds;
}
Active today
That Regex statement above leaves the name as DoeJohn instead of John Doe. That approach also requires a read from disk (line 5), write to disk (line 8) and another read from disk (line 14).
In my comment above http:#35039195 I'm only performing one disk operation, which is to read the file with the adapter.Fill(ds) statement, subsequent operations are done on in-memory data.
I suppose both approaches have their merits, depending on the desired end result - but it's not necessarily uncommon to see text data files up to the tens of thousands of records; and reading, writing then reading a file that big will be noticeably slower than just reading it. With small files, to the hundreds of lines, I would imagine there wouldn't be a perceivable impact on performance one way or the other.
In my comment above http:#35039195 I'm only performing one disk operation, which is to read the file with the adapter.Fill(ds) statement, subsequent operations are done on in-memory data.
I suppose both approaches have their merits, depending on the desired end result - but it's not necessarily uncommon to see text data files up to the tens of thousands of records; and reading, writing then reading a file that big will be noticeably slower than just reading it. With small files, to the hundreds of lines, I would imagine there wouldn't be a perceivable impact on performance one way or the other.
Thats true, it is just sample code. if somebody is loading data from txt file and processing million records then ETL tool is best but not big fat .Net dataset. I assume here the data set is for showing in grid in this context expected to have hundreds of records.
the sample code illustrates solution for the given problem but not the best pattern, there could be tons of patterns to follow depends on the requirement.
the sample code illustrates solution for the given problem but not the best pattern, there could be tons of patterns to follow depends on the requirement.
Active today
Like I said, both approaches have their pros and cons, but the regular expression is still wrong regardless. The asker specified "LastName,FirstName" should become "FirstName LastName" (http:#a35039194). Given an input string of "LastName,FirstName" the statement Regex.Replace(fileContents
string sample = @"""Name"",""Street Address""";
Regex.Replace(sample, @"(\x22.*)(,)(.*\x22)", "$1$3"); // "Name""Street Address"
Regex.Replace(sample, @"[""]([^""]*),([^""]*)[""']($|,)", "$2 $1$3"); // "Name","Street Address"
Featured Post
Windows’ budding partnership with Canonical has certainly led to some great improvements. One of them being the ability to use Bash on your Windows machine without third party applications! This might be one of the greatest things a cloud engineer in a Windows environment can do!
Question has a verified solution.
If you are experiencing a similar issue, please ask a related question
Calculating holidays and working days is a function that is often needed yet it is not one found within the Framework. This article presents one approach to building a working-day calculator for use in .NET.
Real-time is more about the business, not the technology. In day-to-day life, to make real-time decisions like buying or investing, business needs the latest information(e.g. Gold Rate/Stock Rate). Unlike traditional days, you need not wait for a fe…
Add bar graphs to Access queries using Unicode block characters. Graphs appear on every record in the color you want. Give life to numbers.
Hopes this gives you ideas on visualizing your data in new ways ~
Create a calculated field in a query:
…
In response to a need for security and privacy, and to continue fostering an environment members can turn to for support, solutions, and education, Experts Exchange has created anonymous question capabilities. This new feature is available to our Pr…
Suggested Courses
Course of the Month7 days, 15 hours left to enroll
715 members asked questions and received personalized solutions in the past 7 days.
Join the community of 500,000 technology professionals and ask your questions.