Hide some menu from MenuStrip Winfoms

Rose Rouge

I have a MenuStrip in my Windows Form Application which Contains MenuItems.

I want to check some condition and disable the visibility of MenuItems in the MenuStrip bar. Suppose Normal User is accessing the application ,then Some MenuItems will be hidden and if Admin user is accessing the application then all MenuItems should be visible.

I try This code but it doesn't work :

public Visite(string username)
{
InitializeComponent();
label1.Text = username;
using (SqlConnection con = new SqlConnection("Data Source=ADMIN-PC\\SQLEXPRESS;Initial Catalog=timar;Integrated Security=True"))
{
con.Open();
bool UserIsAdmin = true;
using (SqlCommand cmd = new SqlCommand("select * from [User] where Role =@Role", con))
{
cmd.Parameters.AddWithValue("@Role", "Admin");
UserIsAdmin = (int)cmd.ExecuteScalar() > 0;
}
if (UserIsAdmin == false)
{
utilisateurToolStripMenuItem.Visible = false;
}
else
{
utilisateurToolStripMenuItem.Visible= true;
}
con.Close();
}
}

Can someBody Have any idea ,please ??

7 Replies

Know the answer? Post it — somebody with the same question will find it here.

Ranjit Powar10y ago
Accepted answer
I think you have to check username in query like below
public Visite(string username)
{
InitializeComponent();
label1.Text = username;

using (SqlConnection con = new SqlConnection("Data Source=ADMIN-PC\\SQLEXPRESS;Initial Catalog=timar;Integrated Security=True"))
{
con.Open();
stringUserRole = "";
using (SqlCommand cmd = new SqlCommand("select Role from [User] where UserName=@UserName", con))
{
cmd.Parameters.AddWithValue("@UserName", username);

UserRole = cmd.ExecuteScalar().ToString();
}
if (UserRole == "Admin")
{
utilisateurToolStripMenuItem.Visible = true;
}
else
{
utilisateurToolStripMenuItem.Visible= false;
}
con.Close();
}

}
Rose Rouge10y ago
I edit my code , The result was it show me menu for all types of Users (both Normal User and Admin ) , what I want is only for ADMIN
Ranjit Powar10y ago
1. public Visite(string username)
2. {
3. InitializeComponent();
4. label1.Text = username;
6. using (SqlConnection con = new SqlConnection("Data Source=ADMIN-PC\\SQLEXPRESS;Initial Catalog=timar;Integrated Security=True"))
7. {
8. con.Open();
9. bool UserIsAdmin = true;
10. using (SqlCommand cmd = new SqlCommand("select Count(*) from [User] where Role =@Role", con))
11. {
12. cmd.Parameters.AddWithValue("@Role", "Admin");
14. UserIsAdmin = (int)cmd.ExecuteScalar();
15. }
16. if (UserIsAdmin == false)
17. {
18. utilisateurToolStripMenuItem.Visible = false;
19. }
20. else
21. {
22. utilisateurToolStripMenuItem.Visible= true;
23. }
24. con.Close();
25. }
27. }
Rose Rouge10y ago
My essentiel problem is that I want to hide menu( utilisateurToolStripMenuItem)for NORMAL USER and to show it for ADMIN.
Since this exemple doesn't work , Have you any other idea to do it ?
Ranjit Powar10y ago
Which condition do you want to check? And what is the table structure
Rose Rouge10y ago
In this case , What should I do please??
Ranjit Powar10y ago
I think there is problem in your query. Use select all columns from table. But you are using ExecuteScalar() method, this method used when your query return only one value. In above case ExecuteScalar() method returns the value in first row and first column.