|
Hello everyone,
my project get data from dataGridView after that convert rows to Multidimensional Arrays
and i want to assign Multidimensional Arrays to IList Interface but the IList Interface fill last row in gridview every time
my code
public IList<InvoiceLine> invoiceLines { get; set; }
InvoiceLine invoiceline = new InvoiceLine();
string[,] datavalu = new string[dataGridView1.Rows.Count, dataGridView1.Columns.Count];
foreach (DataGridViewRow row in dataGridView1.Rows)
{
foreach (DataGridViewColumn col in dataGridView1.Columns)
{
datavalu[row.Index, col.Index] = dataGridView1.Rows[row.Index].Cells[col.Index].Value.ToString();
}
}
int i ;
for ( i = 0; i < datavalu.GetLength(0); i++)
{
invoiceline.description = datavalu[i, 0].ToString();
invoiceline.itemType = datavalu[i, 1].ToString();
invoiceline.itemCode = datavalu[i, 2].ToString();
invoiceline.internalCode = datavalu[i, 3].ToString();
invoiceline.unitType = datavalu[i, 4].ToString();
invoiceline.quantity = int.Parse(datavalu[i, 5].ToString());
invoicelines.Add(invoiceline);
}
my data
description itemType itemCode internalCode unitType quantity
Food EGS EG-538486562-1314 1314 EA 1
Food2 EGS EG-538486562-1315 1500 EA 5
my problem is that invoicelines return 2 rows same data in last row in the grid
Food2 EGS EG-538486562-1315 1500 EA 5
Can anyone tell me why please
Thanks in advance
|
|
|
|
|
My first question is why are you using a DGV to store data or transfer data between models? That's not what it's for and results in slow, buggy code that can distort your data.
The DGV should be used for the display of data in some data model. That avoids the entire block of code you're messing with now.
|
|
|
|
|
Thank you for your response. buz i load csv file in grid view after that i want to convert this data to multidimensional arrays
|
|
|
|
|
That's a bad way to go. Controls should NEVER be used as your storage for data.
Your CSV should be parsed to a data structure designed to hold that data. Fields can be setup to hold strings, integers, doubles, or whatever you need, and the CSV parser just places the values in the appropriate fields with no conversion. The DGV can then be used to display it if needed.
Using the DGV as your storage makes accessing that data slow because you're now doing a bunch of converting to sting or other types that you wouldn't need to do otherwise.
For example, this is garbage. You're converting a value in the DGV to a string and then parsing that string as an integer. This code will also fail if you've got no value in that cell.
invoiceline.quantity = int.Parse(datavalu[i, 5].ToString()); Putting the data directly into a datastructure designed to hold it from the CSV parser is a much better way to go as you get to validate the data before you use or even display it and handle that occurrence at a more appropriate time, when it's being loaded.
|
|
|
|
|
You're absolutely right but I'm trying code and my problem isn't with grid view but my problem is with
how to fill invoicelines right
i try another way to show invoicelines in listBox1 and run good
string[,] datavalu = new string[dataGridView1.Rows.Count, dataGridView1.Columns.Count];
foreach(DataGridViewRow row in dataGridView1.Rows)
{
foreach (DataGridViewColumn col in dataGridView1.Columns)
{
datavalu[row.Index, col.Index] = dataGridView1.Rows[row.Index].Cells[col.Index].Value.ToString();
}
}
int i = 0;
foreach(string ss in datavalu)
{
listBox1.Items.Add(ss);
i++;
}
rustle
after that i change code to this but run wrong
for ( i = 0; i < datavalu.GetLength(0); i++)
{
invoiceline2.description = datavalu[i, 0].ToString();
invoiceline2.itemType = datavalu[i, 1].ToString();
invoiceline2.itemCode = datavalu[i, 2].ToString();
invoiceline2.internalCode = datavalu[i, 3].ToString();
invoiceline2.unitType = datavalu[i, 4].ToString();
invoiceline2.quantity = int.Parse(datavalu[i, 5].ToString());
invoicelines.Add(invoiceline);
}
some thing i miss can you help me
|
|
|
|
|
Because you only have one instance of the c lass and you insert it multiple times into the list. And each time round the loop you overwrite the values you set in the previous instance.
Think of it like cars: You put your mobile phone in the glove box of my car. Then you open the glove box of your car - is your mobile in there? Of course not - they are different cars.
What your loop is doing is parking my car in different spaces in the car park: and expecting each spot to be filled with a car at the end!
Create a new instance each time round the loop.
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Thank you for the quick response.
Can you explain with code
|
|
|
|
|
He did explain it.
It's up to you to write the code. Griff is not going to write the code for you.
The difficult we do right away...
...the impossible takes slightly longer.
|
|
|
|
|
No need to - you already know how to create instances you even do it in that code fragment you originally showed us!
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Actually, his phone is also in his car ... he's creating multiple references to the same object ... which really means it's in neither car.
It was only in wine that he laid down no limit for himself, but he did not allow himself to be confused by it.
― Confucian Analects: Rules of Confucius about his food
|
|
|
|
|
public class InvoiceLine
{
public string description { get; set; }
public string itemType { get; set; }
public string itemCode { get; set; }
public string internalCode { get; set; }
public string unitType { get; set; }
public int quantity { get; set; }
public UnitValue unitValue { get; set; }
public int salesTotal { get; set; }
public double valueDifference { get; set; }
public double totalTaxableFees { get; set; }
public Discount discount { get; set; }
public int netTotal { get; set; }
public double itemsDiscount { get; set; }
public IList<TaxableItem> taxableItems { get; set; }
public int total { get; set; }
}
I'm sorry i forget to attach the class
i didn't understand your answer "Because you only have one instance of the class and you insert it multiple times into the list. And each time round the loop you overwrite the values you set in the previous instance."
if i have 10 records in grid it means 10 instance to insert one invoice sorry i can't under stand your example about phone and car
|
|
|
|
|
OK.
You have a car. It's a red Ford.
I have a car. It's a black Mercedes.
We go for a drive in my car.
What colour is it?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
|
Right. Obviously!
So you understand that "car" is a generic word, but that each vehicle that is a "car" is separate and distinct. "My car" is not the same as "your car" - it shares some generic characteristics such as the number of wheels, and that it has an engine (or engines) but most of it's attributes are specific to a particular vehicle.
In computing, we call "car" a class and a specific vehicle an instance of that class, with the properties of an instance being defined by the class but "filled in" by the instance:
class Car
{
Color Color {get; set;}
Manufacturer Manunfacturer {get; set;}
}
...
Car myCar = new Car() { Color = Colors.Black, Manufacturer = Manufacturers.Mercedes};
Car yourCar = new Car() { Color = Colors.Red, Manufacturer = Manufacturers.Ford};
If you respray yourCar blue, it doesn't affect the colour of myCar - you know that!
So if your code creates just one instance of an InvoiceLine, what happens if you set the properties over and over again?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
thank for you help
but i think it's hard create instance to each new line in invoice lines
first i used this code to see result
string[,] datavalu = new string[dataGridView1.Rows.Count, dataGridView1.Columns.Count];
foreach(DataGridViewRow row in dataGridView1.Rows)
{
foreach (DataGridViewColumn col in dataGridView1.Columns)
{
datavalu[row.Index, col.Index] = dataGridView1.Rows[row.Index].Cells[col.Index].Value.ToString();
}
}
int i = 0;
foreach(string ss in datavalu)
{
listBox1.Items.Add(ss);
i++;
}
I think I thought wrong from the start. Often there are simpler ways Do you know a better way
|
|
|
|
|
Quote: i think it's hard create instance to each new line in invoice lines
Why?
You do it pretty easily when you create the one you do use!
So why do you think it is going to be harder to create them when you actually need them?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
thanks a lot
for ( i = 0; i < datavalu.GetLength(0); i++)
{
InvoiceLine invoiceline2 = new InvoiceLine();
invoiceline2.description = datavalu[i, 0].ToString();
invoiceline2.itemType = datavalu[i, 1].ToString();
invoiceline2.itemCode = datavalu[i, 2].ToString();
invoiceline2.internalCode = datavalu[i, 3].ToString();
invoiceline2.unitType = datavalu[i, 4].ToString();
invoiceline2.quantity = int.Parse(datavalu[i, 5].ToString());
invoicelines.Add(invoiceline2);
}
my mistake was
InvoiceLine invoiceline = new InvoiceLine();
create instants before loop
|
|
|
|
|
That's the one.
Though I'd have just moved the original line into the loop! 
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Find whoever put "Food2" in there and stone them.
Bastard Programmer from Hell 
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
Hi
{
if (DataGridView2.Rows.Count != 0 && !string.IsNullOrEmpty(column_num.Text) && !string.IsNullOrEmpty(row_num.Text) && !string.IsNullOrEmpty(Equip_column.Text) && !string.IsNullOrEmpty(prog_num.Text))
{
for (int i = Convert.ToInt32(row_num.Text); i < DataGridView2.Rows.Count; i++)
{
if (!code_dic.ContainsKey(DataGridView2.Rows[i].Cells[Convert.ToInt32(column_num.Text)].Value.ToString()))
{
code_dic.Add(DataGridView2.Rows[i].Cells[Convert.ToInt32(column_num.Text)].Value.ToString(), string.Join(",", DataGridView2.Rows[i].Cells[Convert.ToInt32(Equip_column.Text)].Value.ToString(), prog_num.Text.ToString()));
}
}
}
foreach (KeyValuePair<string, string> pair in code_dic)
{
string[] my_arr = pair.ToString().Split(',');
my_table_code.Rows.Add(my_arr[0], my_arr[1], my_arr[2]);
DataGridView3.DataSource = my_table_code;
}
}
In the code above, dictionary puts information inside [ ] mark.
The output shows information like this:
[abc,def,ghi]
I want to omit [ ] marks. What to do?
|
|
|
|
|
Alex Dunlop wrote: In the code above, dictionary puts information inside [ ] mark.
Where does this happen, and how are you checking it?
|
|
|
|
|
The code puts Dictionary into a data table and then, datatable in brought into DataGridView.
The result is reflected in DataGridView. It has 3 cloumns. Column 1 and 3 has "[" and "]" marks.
Something like this:
column 1....columns 2....column 3
[abcd........efgj..........ijkl]
|
|
|
|
|
That does not make it much clearer and your code is almost impossible to understand. You are using strings as column numbers, which means you have to convert them back to integers. You call ToString on a KeyValuePair<string, string> and then Split on the result, instead of taking the returned items as is. Chances are that whatever you are doing is affecting the text values you are storing. A few minutes with the debugger will show you where.
|
|
|
|
|
Took me less than a minute to see that:
pair.ToString()
returns the key and value surrounded by square brackets:
KeyValuePair<string, string> foo = new KeyValuePair<string, string>("foo", "bar");
Console.WriteLine(foo.ToString());
Output:
[foo, bar]
|
|
|
|
|
Yeah, you're right.
Is there any way to avoid that or remove it?
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
