How to arrange dynamic button

Question

1.00/5 (1 vote)

See more:

Hello Guys,I hopeyou will be fine guys i have facing a logical problems that it
i want buttons like this in m[^]
in my form but i can't do it please help me how can i do it
Thanks

What I have tried:

Button button = sender as Button;
string buttonText = button.Text;
MessageBox.Show(buttonText);
int CatId = OperationAccess.GetId("SELECT catID from tbl_Catagary where name='" + button.Text + "'");
List buttons = new List();
List<string> All_Catagary = OperationAccess.GetName("SELECT name FROM tbl_product where catID=" + CatId + "");
int repetition = 0;
for (int i = 0; i < All_Catagary.Count; i++)
{
Button Productbtn = new Button();
Productbtn.Text = All_Catagary[i].ToString();
string Command = "SELECT [image] FROM tbl_product where name ='" + Productbtn.Text + "'";
string path = OperationAccess.GetImage(Command);
string appPath = Path.GetDirectoryName(Application.ExecutablePath) + @"\ProductImage\";
string TotalPath = appPath + path;
Productbtn.Image = System.Drawing.Image.FromFile(TotalPath);
Productbtn.Visible = true;
Productbtn.Location = new Point(3 + repetition * 107, 3);
Productbtn.Height = 54;
Productbtn.Width = 110;

Productbtn.BackColor = Color.FromArgb(40, 40, 40);
Productbtn.ForeColor = Color.White;
Productbtn.Top =10;
Productbtn.Font = new Font("Lucida Console", 16);
buttons.Add(Productbtn);
repetition++;
this.panel9.Controls.Add(Productbtn);
Productbtn.Click += new EventHandler(Productbtn_Click);

Posted 18-Jun-17 5:01am

Member 9983063

Updated 25-Jul-17 22:45pm

Add a Solution

Comments

Richard Deeming 19-Jun-17 12:24pm

Your code is vulnerable to SQL Injection[^]. NEVER use string concatenation to build a SQL query. ALWAYS use a parameterized query.

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[^]

Member 9983063 19-Jun-17 13:55pm

well i just want to solution of this question after this i will never use concatenation please bro help me how can i solve this issue

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

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

Ravi Bhavnani · Answer 1 · 2017-06-18T05:17:00

Solution 1

There are several issues with your code. But perhaps most importantly, you're simply adding the created buttons to a list, and not the form. Instead of doing:

buttons.Add(Productbtn);

try:

this.Controls.Add(Productbtn);

/ravi

Posted 18-Jun-17 5:17am

Ravi Bhavnani

Simon_Whale · Answer 2 · 2017-06-19T01:04:00

I see a couple of issues.

1. like Ravi pointed out you need to add the button to the panel only.
2. you need to set the top and left properties of the button to position the button to the screen where you want it

at the moment you are setting all of them to 10, I would do something similar to the below (admittedly I am only setting 5 in a row but it should give you an idea)

C#

int leftPosition = 10;
for(int i =0; i< 5;i++)
{
  Button myButton = new Button();
  myButton.Name = "Button" + i.ToString();
  myButton.Text = i.ToString();
  myButton.width = 110;
  myButton.Height = 58;
  myButton.top = 10;
  myButton.Left = leftPosition;
  panel1.Controls.Add(myButton);

  leftPosition += 120;
}

Member 9983063 · Answer 3 · 2017-06-20T17:30:00

Solution 3

Well i have solved this issue my self with this link .net - Dynamic button creation & placing them in a predefined order using c# - Stack Overflow[^]
Thank you guys for my help.

Posted 20-Jun-17 17:30pm