- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 29
- 30
- 31
- 32
public string GetValue(string value)
{
var displayOptions = (DisplayOptions)ViewBag.Options;
var defaultCulture = ContentHelper.GetDefaultCulture();
var currentCulture = UserHelper.GetSessionCulture() ?? defaultCulture;
var domainValue = Model.Domain.Values.FirstOrDefault(v => v.Value.ToLower() == value.ToLower());
var text = domainValue == null ? value : (domainValue?.Translations.Where(t => currentCulture.Equals(t.Culture)).Select(t => t.Text).FirstOrDefault() ?? domainValue.Text);
var result = text;
if (!string.IsNullOrEmpty(Model.Domain.DisplayPrefix))
{
result = string.Format("{0} {1}", Model.Domain.DisplayPrefix, text);
}
if (!string.IsNullOrEmpty(Model.Domain.DisplayPrefix) && !string.IsNullOrEmpty(Model.Domain.DisplaySuffix))
{
result = string.Format("{0} {1} {2}", Model.Domain.DisplayPrefix, text, Model.Domain.DisplaySuffix);
}
if (!string.IsNullOrEmpty(Model.Domain.DisplaySuffix))
{
result = string.Format("{0} {1} {2}", Model.Domain.DisplayPrefix, text, Model.Domain.DisplaySuffix);
}
return String.Format("{0}{1}{2}{3}"
, displayOptions.StartTag != null && displayOptions.EndTag != null ? (displayOptions.StartTag ?? string.Empty) : string.Empty
, displayOptions.ShowDomainAlternative ? @Html.DisplayForDomainAlternative(domainValue) : new MvcHtmlString(string.Empty)
, displayOptions.ShowDomainText ? result : string.Empty
, displayOptions.StartTag != null && displayOptions.EndTag != null ? (displayOptions.EndTag ?? string.Empty) : string.Empty);
}
Так безжалостно получали result чтобы в конце заменить его на пустую сроку. Не говоря уже про общее построение метода и как себя ненавидел автор, добавляя еще одну строку кода в String.Format.
dm_fomenok 27.01.2017 18:51 # −4
roman-kashitsyn 27.01.2017 19:53 # +5
gost 31.01.2017 10:30 # 0
dm_fomenok 28.01.2017 16:31 # −11
WTF? domainValue в блоке else тернарного оператора не может быть null!
guestinho 28.01.2017 18:16 # +3
inkanus-gray 28.01.2017 18:24 # +3