CA1301: Avoid duplicate accelerators
| Item | Value |
|---|---|
| RuleId | CA1301 |
| Category | Microsoft.Globalization |
| Breaking change | Non-breaking |
Cause
A type extends System.Windows.Forms.Control and contains two or more top-level controls that have identical access keys that are stored in a resource file.
Rule description
An access key, also known as an accelerator, enables keyboard access to a control by using the Alt key. When multiple controls have the same access key, the behavior of the access key is not well defined. The user might not be able to access the intended control by using the access key, and a control other than the one that is intended might be enabled.
The current implementation of this rule ignores menu items. However, menu items in the same submenu should not have identical access keys.
How to fix violations
To fix a violation of this rule, define unique access keys for all controls.
When to suppress warnings
Do not suppress a warning from this rule.
Example
The following example shows a minimal form that contains two controls that have identical access keys. The keys are stored in a resource file, which is not shown. However, their values appear in the commented out checkBox.Text lines. The behavior of duplicate accelerators can be examined by exchanging the checkBox.Text lines with their commented out counterparts. However, in this case, the example will not generate a warning from the rule.
using System;
using System.Drawing;
using System.Resources;
using System.Windows.Forms;
namespace GlobalizationLibrary
{
public class DuplicateAccelerators : Form
{
[STAThread]
public static void Main()
{
DuplicateAccelerators accelerators = new DuplicateAccelerators();
Application.Run(accelerators);
}
private CheckBox checkBox1;
private CheckBox checkBox2;
public DuplicateAccelerators()
{
ResourceManager resources =
new ResourceManager(typeof(DuplicateAccelerators));
checkBox1 = new CheckBox();
checkBox1.Location = new Point(8, 16);
// checkBox1.Text = "&checkBox1";
checkBox1.Text = resources.GetString("checkBox1.Text");
checkBox2 = new CheckBox();
checkBox2.Location = new Point(8, 56);
// checkBox2.Text = "&checkBox2";
checkBox2.Text = resources.GetString("checkBox2.Text");
Controls.Add(checkBox1);
Controls.Add(checkBox2);
}
}
}