Tips n Tricks. StockSharp

Tips n Tricks

1/20/2011

Постоянно смотрю на изменяющийся код. Весь поток контролировать не могу, поэтому выбираю некоторые места по принципу Медведь на рыбалке в период нереста. Далее, привожу что увидел и как бы изменил. Рекомендую смотреть всем (даже если свой код не увидели), чтобы и самому учиться и не допустить в будущем:

ODE0� Тут лучше использовать ключевое слово var

foreach (var pair in _sections)

ODE2� С помощью Ecng.Common пишется короче

pair.Value.Join(Environment.NewLine)

ODE4� Тут или область видимости должна быть private или название должно быть SectionName.
ODE5� С помощью Ecng.Common пишется короче

"{0}={1}".Put(key, value)

ODE7� Лучше проверить через Contains.
ODE8� Старайтесь минимизироваться перехват исключений. В том случае это решается через простую проверку на существование в коллекции.
ODE9� Я бы заменил на ArgumentException. Он как то более осмысленный.
Не пишите Decimal, Int64, String или Single. Понятно, что это межъязыковое название. Но лучше использовать то, что родное для C#.
ODE10� Стиль именования как в Win32 API. Последний умер де факто много лет назад, наследие его живет и по сей день.[biggrin]
Добавляйте xml комментарии к самим классам, а не только его членам (видел в некоторых местах через // что есть совсем не то). И не забывайте про русский язык. В конце предложения ставится точка.
Пользуйтесь R#. Он показывает допущенные ошибки и предупреждает заранее о неправильном коде.

Tags:

Plaza 2

Thanks:

< 1 2

Mikhail Sukhov

Date: 5/8/2011

Подниму топик парочкой советов:

internal на методы, свойста и поля (но не типа) в C# признак плохого дизайна. Говорит о том, что приосходит размытость границ логики.
ODE0� Плох тем, что не важно, есть необходимый уровень трассировки или нет, строчка будет собираться всегда. Если такая конструкция встречается в коде, который вызывается часто, может снизить производительность.
Любите readonly. Тоесть, когда создаете поля, сразу их делайте readonly. Снять атрибут можно всегда. Но он предотвратит от нежелательной логики перетирания значения.
Два кода абсолютно эквивалентны:

public class RootClass
{
  private class NestedClass
  {
     internal int Prop { get; set; }
  }
}

public class RootClass
{
  private class NestedClass
  {
     public int Prop { get; set; }
  }
}

Thanks:

aspirant

Date: 5/8/2011

Mikhail Sukhov: Подниму топик парочкой советов:

internal на методы, свойста и поля (но не типа) в C# признак плохого дизайна. Говорит о том, что приосходит размытость границ логики.

ODE0� Плох тем, что не важно, есть необходимый уровень трассировки или нет, строчка будет собираться всегда. Если такая конструкция встречается в коде, который вызывается часто, может снизить производительность.

Сделал криво, знаю. Уже думал над тем, чтобы переделать + поменять internal на public. В релиз не будет попадать вообще: выделю в отдельный метод с аттрибутом [Conditional("DEBUG")]

Thanks:

anothar

Date: 5/9/2011

Добрый день. Если верить http://www.codeproject.com/KB/cs/WeakEvents.aspx, кот. ссылается на Ecma cпецификацию, то для обеспечения потокобезопасности делегатов необходимо еще и локирование:


EventHandler eh;
lock (this) { eh = MyEvent; }
if (eh != null) eh(this, EventArgs.Empty);

Thanks:

aspirant

Date: 5/27/2011

@esper, может выставишь у себя tab'ы (Tools \ Options \ Text Editor \ Tab \ Keep tabs)?

Thanks:

esper

Date: 5/28/2011

aspirant: @esper, может выставишь у себя tab'ы (Tools \ Options \ Text Editor \ Tab \ Keep tabs)? Сделал[smile]

Thanks:

< 1 2

Attach files by dragging & dropping, , or pasting from the clipboard.