Действительно ли приемлемо сделать цель внешних ссылок = “_blank”?

Question

Действительно ли приемлемо сделать цель внешних ссылок = “_blank”?

Для.NET 2.0, вот хороший бит кода, я записал, что это делает точно, что Вы хотите, и работы для любого свойства на Control:

private delegate void SetControlPropertyThreadSafeDelegate(
    Control control, 
    string propertyName, 
    object propertyValue);

public static void SetControlPropertyThreadSafe(
    Control control, 
    string propertyName, 
    object propertyValue)
{
  if (control.InvokeRequired)
  {
    control.Invoke(new SetControlPropertyThreadSafeDelegate               
    (SetControlPropertyThreadSafe), 
    new object[] { control, propertyName, propertyValue });
  }
  else
  {
    control.GetType().InvokeMember(
        propertyName, 
        BindingFlags.SetProperty, 
        null, 
        control, 
        new object[] { propertyValue });
  }
}

Вызов это как это:

// thread-safe equivalent of
// myLabel.Text = status;
SetControlPropertyThreadSafe(myLabel, "Text", status);

при использовании.NET 3.0 или выше Вы могли бы переписать вышеупомянутый метод как дополнительный метод Control класс, который тогда упростит вызов до:

myLabel.SetPropertyThreadSafe("Text", status);

ОБНОВЛЕНИЕ 10.05.2010:

Для.NET 3.0 необходимо использовать этот код:

private delegate void SetPropertyThreadSafeDelegate<TResult>(
    Control @this, 
    Expression<Func<TResult>> property, 
    TResult value);

public static void SetPropertyThreadSafe<TResult>(
    this Control @this, 
    Expression<Func<TResult>> property, 
    TResult value)
{
  var propertyInfo = (property.Body as MemberExpression).Member 
      as PropertyInfo;

  if (propertyInfo == null ||
      !@this.GetType().IsSubclassOf(propertyInfo.ReflectedType) ||
      @this.GetType().GetProperty(
          propertyInfo.Name, 
          propertyInfo.PropertyType) == null)
  {
    throw new ArgumentException("The lambda expression 'property' must reference a valid property on this Control.");
  }

  if (@this.InvokeRequired)
  {
      @this.Invoke(new SetPropertyThreadSafeDelegate<TResult> 
      (SetPropertyThreadSafe), 
      new object[] { @this, property, value });
  }
  else
  {
      @this.GetType().InvokeMember(
          propertyInfo.Name, 
          BindingFlags.SetProperty, 
          null, 
          @this, 
          new object[] { value });
  }
}

, который использует LINQ и лямбда-выражения для разрешения намного более чистого, более простого и более безопасного синтаксиса:

myLabel.SetPropertyThreadSafe(() => myLabel.Text, status); // status has to be a string or this will fail to compile

Не только имя свойства, теперь проверенное во время компиляции, тип свойства также, таким образом, невозможно (например), присвоить строковое значение булево свойству и следовательно вызвать исключение на этапе выполнения.

, К сожалению, это не мешает никому делать глупые вещи, такие как передача в другом Control свойство и значение, таким образом, следующее счастливо скомпилирует:

myLabel.SetPropertyThreadSafe(() => aForm.ShowIcon, false);

Следовательно я добавил проверки на этапе выполнения, чтобы гарантировать, что переданный - в свойстве действительно на самом деле принадлежит Control что то, что метод был обращенным. Не прекрасный, но еще намного лучше, чем.NET 2,0 версии.

, Если у кого-либо есть дальнейшие предложения о том, как улучшить этот код для безопасности времени компиляции, прокомментируйте!

5

hyperlink

задан Robert Greiner 28 August 2009 в 20:00

12 ответов

Most people will say to leave it out, all for very good reasons.

Most people will not tell you this: watch how different people browse using their computer. Sit down for an hour with an average user: a teacher, a piano player, a retired Vet, a stay-at-home mother, or a mechanic. Watch how people who are disabled browse.

You will discover that you should not:

Open new windows (it is highly confusing)
Have menus disappear immediately after leaving them (requires fine motor control)
Use excessive numbers of links on a page (also confusing)
Provide too many features and options
Use Flash animation
Be overly clever

You will discover that you should:

Be clear and concise
Be simple
Keep the user on the same page
Avoid frustrating the end user at any cost
Create an extremely responsive website

And much more.

10