Re: Copy a selected gridview row to the same gridview - C# Discussion Boards

OriginalGriff24-Mar-21 23:38

OriginalGriff

24-Mar-21 23:38

You don't show the code of your loop body, but from the error message you are either adding or removing a control / form from the collection fc inside the loop. You can't do that: modifying the source of an iterator while the iterator is in use because if you did, it would not be clear what the next iteration value should be!

You could change to a for loop, but you may need to run it backwards to avoid getting a similar problem in your own code.

"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!

Alex Dunlop25-Mar-21 0:08

Alex Dunlop

25-Mar-21 0:08

I forgot to provide my code. I added it to the main question.
I repeat it again:

FormCollection fc = Application.OpenForms;
            foreach (Form frm in fc)
            {
                if (frm.Name == "Chart")
                {
                    MessageBox.Show("The form is open","error",MessageBoxButtons.OK,MessageBoxIcon.Error);
                }
                else
                {
                    Chart myCharts = new Chart();
                    myCharts.Show();
                }
            }

OriginalGriff25-Mar-21 0:37

OriginalGriff

25-Mar-21 0:37

Yes - and you are adding a form to the collection.
When you call Show, it adds the new form to the Application.OpenForms collection, and since fc is a reference to that collection, you are modifying the collection which you are iterating over it. Worse, you are adding it every time an existing form is not teh one you are looking for!

You can't do that.

The way I'd do it is simpler:

if (!Application.OpenForms.Cast<Form>().Any(frm => frm.Name == "Chart"))
    {
    ... add your new form ...
    }

Copy a selected gridview row to the same gridview

Richard Deeming25-Mar-21 2:51

Richard Deeming

25-Mar-21 2:51

NB: In your current code, you will open another instance of the chart form for every non-chart form you have open, regardless of how many instances of the chart form are already open.

Try:

ChartForm myCharts = Application.OpenForms.OfType<Chart>().FirstOrDefault();
if (myCharts == null)
{
    myCharts = new Chart();
    myCharts.Show();
}
else
{
    myCharts.Activate();
}

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

Member 1493285423-Mar-21 2:41

Member 14932854

23-Mar-21 2:41

Please help me to create a code for copy a selected gridview row to the same gridview in asp.net c#

Re: Copy a selected gridview row to the same gridview

Gerry Schmitz23-Mar-21 7:08

Gerry Schmitz

23-Mar-21 7:08

You create a new row; you then copy the selected row to the new row.

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

Re: Copy a selected gridview row to the same gridview

Nathan Minier24-Mar-21 1:20

Nathan Minier

24-Mar-21 1:20

Now now, isn't suggesting problem decomposition a bit unfair? That requires more thought than "cOdEz PlZ!"

"Never attribute to malice that which can be explained by stupidity."
- Hanlon's Razor

Re: Copy a selected gridview row to the same gridview

jsc4224-Mar-21 1:25

jsc42

24-Mar-21 1:25

Slightly off topic:

Thanks for your sig
"Never attribute to malice that which can be explained by stupidity."
- Hanlon's Razor

I've been using the quote for a long time (and I'd misremembered it as "Never attribute to malevolence that which can be explained by stupidity.") and had not found out where it had originated.

Re: Copy a selected gridview row to the same gridview

Nathan Minier24-Mar-21 1:38

Nathan Minier

24-Mar-21 1:38

Eh, you may well have it right; it's existed in many forms over the years. I think the original went with "incompetence" as well, but I have admittedly picked a version that reflects my own feelings.

At any rate, it's an absolutely essential attitude when working in cybersecurity.

"Never attribute to malice that which can be explained by stupidity."
- Hanlon's Razor

Re: Copy a selected gridview row to the same gridview

jsc4225-Mar-21 3:54

jsc42

25-Mar-21 3:54

Nathan Minier wrote:
At any rate, it's an absolutely essential attitude when working in cybersecurity.

or when getting edicts from management!

How to able to have the information of the particular image button clicked?

Member 1508474922-Mar-21 10:50

Member 15084749

22-Mar-21 10:50

I am creating a book site for that, I have created dynamic buttons so that if on a particular category is clicked then that row will be fetched from the database and required information will be displayed. Now I want that if the user clicks on any particular book then the data of that book should be displayed in another panel and the current panel should hide.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.UI;
using System.Web.UI.WebControls;
using System.Data;
using System.Data.SqlClient;
using System.Configuration;
using System.Web.UI.HtmlControls;

namespace WebApplication1
{

<pre>
public partial class Genre : System.Web.UI.Page
{

    private int num;
    SqlConnection con = new SqlConnection(ConfigurationManager.ConnectionStrings["ConnectionString"].ConnectionString);<br />
    protected void Page_Load(object sender, EventArgs e)
    {
        Label1.Text = "Romance";
        Panel2.Visible = false;

        con.Open();
        String SQL = "Select * from Book_List where Book_category='" + Label1.Text + "'";
        SqlDataAdapter da = new SqlDataAdapter(SQL, con);
        DataSet ds = new DataSet();
        DataTable dt = new DataTable();
        da.Fill(ds);
        num = ds.Tables[0].Rows.Count;
        HtmlGenericControl Book = new HtmlGenericControl("div");
        Book.Attributes.Add("class", "books");
        Panel1.Controls.Add(Book);

        for (int i = 1; i < num; i++)
        {
            HtmlGenericControl myDiv = new HtmlGenericControl("div");
            myDiv.Attributes.Add("class", "myDiv");
            ImageButton image = new ImageButton();
            image.ImageUrl = ds.Tables[0].Rows[i]["Book_image"].ToString();
            image.CssClass = "Img";
            image.Click += new ImageClickEventHandler(this.image_Click);
            HtmlGenericControl content = new HtmlGenericControl("div");
            content.Attributes.Add("class", "content");
            Label name = new Label();
            name.CssClass = "name";
            name.Text = ds.Tables[0].Rows[i]["Book_name"].ToString();
            Label cost = new Label();
            cost.CssClass = "cost";
            cost.Text = "<br/> Rs " + ds.Tables[0].Rows[i]["Book_cost"].ToString();
            Button wishlist = new Button();
            wishlist.CssClass = "wishlist";
            wishlist.Text = "ADD TO WISHLIST";
            Book.Controls.Add(myDiv);
            myDiv.Controls.Add(image);
            myDiv.Controls.Add(content);
            content.Controls.Add(name);
            content.Controls.Add(cost);
            content.Controls.Add(wishlist);

        }

    }
    protected void image_Click(object sender, ImageClickEventArgs e)
    {
        ImageButton image = sender as ImageButton;
        Panel2.Visible = true;
        Panel1.Visible = false;            
    }
   </pre>

Re: How to able to have the information of the particular image button clicked?

OriginalGriff22-Mar-21 11:18

OriginalGriff

22-Mar-21 11:18

Every control in .NET has a Tag property, which is an object so it can store any instance of any class. If you create a class to hold the book information, you could set the Tag of the button to that class instance when you create the Button.
Then in the Click handler you can get the Button instance from the sender parameter, and use that to get the book information from it's Tag.

Then you can do with it what you wish!

Re: How to able to have the information of the particular image button clicked?

Richard Deeming22-Mar-21 22:25

Richard Deeming

22-Mar-21 22:25

Not in a WebForms app. Smile | :)

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

Re: How to able to have the information of the particular image button clicked?

Richard Deeming22-Mar-21 22:33

Richard Deeming

22-Mar-21 22:33

Member 15084749 wrote:
String SQL = "Select * from Book_List where Book_category='" + Label1.Text + "'";

Don't do it like that!

Although this specific case is OK since you're using a constant value, using string concatenation to build a SQL query can and will leave you vulnerable to SQL Injection[^].

NEVER use string concatenation to build a SQL query. ALWAYS use a parameterized query.

const string SQL = "Select * from Book_List where Book_category = @BookCategory";
SqlDataAdapter da = new SqlDataAdapter(SQL, con);
da.SelectCommand.Parameters.AddWithValue("@BookCategory", Label1.Text);

Everything you wanted to know about SQL injection (but were afraid to ask) | Troy Hunt[^]
How can I explain SQL injection without technical jargon? | Information Security Stack Exchange[^]
Query Parameterization Cheat Sheet | OWASP[^]

Also, don't store the database connection in a class-level field. Instead, create it where necessary, and wrap it in a using block to ensure that it's always disposed of properly.

private static SqlConnection CreateConnection()
{
    return new SqlConnection(ConfigurationManager.ConnectionStrings["ConnectionString"].ConnectionString);
}

...

DataSet ds = new DataSet();
using (SqlConnection con = CreateConnection())
{
    const string SQL = "Select * from Book_List where Book_category = @BookCategory";
    SqlDataAdapter da = new SqlDataAdapter(SQL, con);
    da.SelectCommand.Parameters.AddWithValue("@BookCategory", Label1.Text);
    da.Fill(ds);
}

DataTable dt = ds.Tables[0];
...

And do yourself a favour and give your controls meaningful IDs, rather than accepting the default values suggested by the Visual Studio designer. Sure, you might remember what Label1 refers to now, but when you come back to your code in three weeks, you'll have forgotten.

As to your question, Page_Load is too late to initialize your dynamic controls. Do it in Page_Init instead.

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

Re: How to able to have the information of the particular image button clicked?

DerekT-P23-Mar-21 0:46

DerekT-P

23-Mar-21 0:46

ooh, I hope one of the categories is

SQL Injection - how to do it';drop table book_list;'

Let me know the URL when it's live Big Grin | :-D

Seriously, parameterise that.

C# Excel test specific cells in a column

Steve Rudd22-Mar-21 5:59

22-Mar-21 5:59

Hello All,
I am still pretty new to C# and have run into a little issue.
I have program which passes data into Excel and can be used on the same workbook many times. Therefore, I need to check to see if cell 7,1 (for instance) already contains data from a previous pass and if so, move to cell 11,1, do the same test and if that has data, move to cell 15,1 etc. etc. I think I possibly require some sort of Switch statement where we keep falling through until the specified cell is empty and therefore ready to receive data, i.e. look at 7,1 > 11,1 > 15,1 > 19,1 right down to row number 55 (the cell rows increase by multiples of 4) but I am struggling with it.
Can anyone provide the best practice for doing this?
Thanks,
Steve

modified 22-Mar-21 12:44pm.

Gerry Schmitz22-Mar-21 7:40

Gerry Schmitz

22-Mar-21 7:40

The whole thing sounds pretty fragile; and without some "what for", a pointless mental exercise.

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

Steve Rudd22-Mar-21 9:31

22-Mar-21 9:31

OK Gerry, here is the full scenario.
My program functions inside of AutoCAD and is intended to export some product data to a "Packing List"
which is a spreadsheet in Excel.
This is the workflow:
1. The draughtsman is prompted to make a selection from a number of products (AutoCAD entities) on the drawing which will comprise of the items to be added to the Packing List.
2. He then allocates the selection to a Bundle number, which he is prompted for and defines himself.
3. The program then looks for an excel book with the same name as the drawing file and if it does not exist, creates a new one from a defined template.
4. It then iterates through the selection set and extracts all of the part numbers and builds a string of the numbers to be written out i.e. 1, 2, 3, 4, 5 etc.
5. If we are in a new workbook, the string is written into the first cell at row 7.
6. The workbook is then saved and closed.
7. When this process is finished, the draughtsman can then start over again and make another selection which he then allocates to a different Bundle number.
8. At this point, the program finds the existing workbook and opens it, but now needs to start at the first cell on row 11 as cell 7, 1 already holds data.

This workflow can carry-on upto 13 times, when the last row will be reached.

Hope this is clearer.

Steve

ps I can't see how to include a snapshot of the packing list spread sheet.

Richard MacCutchan22-Mar-21 22:56

Richard MacCutchan

22-Mar-21 22:56

It is a simple loop:

bool found = false;
for (int row = 7; row < 56; row += 4)
{
    if (cell[row, 1] is empty // not sure how you test for this
    {
        // store information in the cell
        found = true;
        break:
    }
}
if (!found)
{
    // did not find an empty cell
}

Steve Rudd23-Mar-21 0:13

23-Mar-21 0:13

Hi Richard,
Thanks for your suggestion. I will try this later today and let you know how I get on.
Steve

Steve Rudd23-Mar-21 1:32

23-Mar-21 1:32

Hi Richard,

Thanks for pointing me in the right direction.
I changed your suggestion as below...it works a treat!

for (int row = 7; row < 56; row += 4)
{
    if (workSheet.Cells[row, 1] == null || workSheet.Cells[row, 1].Value2 == null || workSheet.Cells[row, 1].Value2.ToString() == "")
    {
        workSheet.Cells[row, 1] = (string)fillinData.GetValue(0);//PANEL NOS.
        workSheet.Cells[row + 1, 18] = (string)fillinData.GetValue(1);//TOTAL WGHT
        workSheet.Cells[row - 1, 10] = (string)fillinData.GetValue(8);//BUNDLE NO.
        break;
    }
}

Thanks again,

Steve