+8
- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
public float Ratio
{
get { return ratio; }
set
{
float newValue = value;
if (value < 0.002f)
{
newValue = 0.002f;
}
else if (value > 0.983f)
{
newValue = 0.983f;
}
if (ratio != newValue)
{
ratio = value;
InvalidateMeasure();
}
}
}
Мой "любимый" телерик опять пишет говно. Хорошо, что я скачал исходники и теперь сам могу править баги, не дожидаясь реакции этих слоупоков, которые после отправки бага с гифом (!), воспроизводящими исходниками (!!) и, блядь, патчем (!!!) исправляли два месяца сраный NRE.
Догадайтесь, что здесь не так. Про magic numbers уже молчу.
Запостил: kerman,
01 Августа 2016
3_dar 01.08.2016 22:15 # +18
kerman 01.08.2016 22:27 # +15
Для начала, если уж про говнокод, то этим обизянам надо бы рассказать, что value в сеттере можно изменять, что исключает целый класс ошибок, включая эту.
Далее про бессмысленный codestyle. Я решарпером вырезал все приставки "this.", получилось около полмегабайта с 25 мб исходников. И это только четыре библиотеки. И это только "this". К слову, 25 мб получилось из 29, путём смени начальных пробелов на табы (цыц, холивариоры!), что говорит о плотности символов на строку кода. Походу, им за строчки платят.
Soul_re@ver 01.08.2016 22:29 # +16
Хм. А сколько нужно найти багов чтобы им пришлось доплачивать за продукт?
kerman 01.08.2016 22:30 # +15
bormand 01.08.2016 22:31 # +13
kerman 01.08.2016 22:37 # +13
bormand 01.08.2016 22:42 # +14
kerman 01.08.2016 23:35 # +14
guesto 01.08.2016 22:39 # +15
ЁБАНЫЙ ПИЗДЕЦ АХУЕТЬ
kerman 01.08.2016 22:42 # +17
guesto 01.08.2016 22:50 # +15
kerman 01.08.2016 22:58 # +17
guesto 01.08.2016 23:05 # +14
Или визуальная студия, и она генерит код для всех платформ?
kerman 01.08.2016 23:08 # +13
guesto 01.08.2016 23:12 # +14
kerman 01.08.2016 23:19 # +15
3_14dar 01.08.2016 22:42 # +18
Объясни это питонистам теперь.
3_dar 01.08.2016 22:51 # +14
kerman 01.08.2016 23:00 # +23
3_14dar 02.08.2016 16:24 # +12
santa_microbe 02.08.2016 11:44 # +17
Судя по коду у них это специально было сделано, что бы отличать локальные переменные от полей класса. После твоей чистки решарпером, глядя на переменную нельзя будет визуально определить локальная она или нет, если метод чуть больше будет
kerman 02.08.2016 12:09 # +12
Lokich 02.08.2016 15:20 # +12
да и вообще, чем плох this?
или это при декомпиляции он сам ставит this.. но в любом случае, в этом нет ничего плохого.
inkanus-gray 02.08.2016 16:36 # +13
Lokich 02.08.2016 17:32 # +12
чего рябит? больше конкретики, код яснее
nihau 02.08.2016 19:06 # +14
gost 03.08.2016 02:00 # +12
Dummy00001 03.08.2016 12:32 # +13
К this можно привыкнуть. И это часть языка (которую временами приходится держать в голове в случае оптимизации производительность).
К тридцать-третьей вариации "правильных имен" - заябало уже давно. Не говоря о вечных нескончаемых спорах, чья вореция "правильных имен" даёт лучшую кобенацию читабельного кода.
inkanus-gray 02.08.2016 20:19 # +11
bormand 02.08.2016 20:29 # +13
3_dar 02.08.2016 18:12 # +13
kegdan 03.08.2016 02:50 # +11
bormand 03.08.2016 06:28 # +11
kegdan 03.08.2016 08:47 # +10
Soul_re@ver 03.08.2016 10:24 # +14
Fuck the police
nihau 03.08.2016 12:46 # +11
3_14dar 02.08.2016 16:25 # +10
inkanus-gray 02.08.2016 11:55 # +11
kerman 02.08.2016 12:08 # +11
Да и вообще, мне табы удобнее. Бесит четыре раза стрелку нажимать, чтобы на один табстоп перейти.
wvxvw 02.08.2016 15:49 # +10
3_14dar 02.08.2016 16:25 # +10
wvxvw 02.08.2016 18:05 # +10
3_14dar 02.08.2016 19:46 # +12
>M-m
А теперь на нормальный язык переведи.
bormand 02.08.2016 18:19 # +11
А если там было x = this.x или this.x = x?
3_14dar 02.08.2016 19:47 # +14
kegdan 03.08.2016 02:58 # +11
Lokich 03.08.2016 14:48 # +10
Assignment made to same variable; did you mean to assign something else?
мне честно говоря как-то не очень нравится идея, писать в конструкторе firstName. как бы хуевый нейминг.
да и вообще, когда тебе нужно вдруг обратиться к свойствам, или методам данного класса, то когда ты пишешь this., то студия сама тебе предложит вариант выбора. то студия мне выводит ворнинг:
1024-- 03.08.2016 14:59 # +10
То ли дело C++...
nihau 03.08.2016 15:01 # +10
https://msdn.microsoft.com/en-us/library/ab6a8y1b(v=vs.71).aspx
Use camel case for parameter names.
Да в чем твоя проблема?
santa_microbe 03.08.2016 18:26 # +12
bormand 03.08.2016 18:32 # +11
kegdan 03.08.2016 18:40 # +10
А в ангуляре все с маленькой
bormand 03.08.2016 18:40 # +10
kegdan 03.08.2016 18:41 # +10
bormand 03.08.2016 18:42 # +10
kegdan 03.08.2016 18:44 # +11
inkanus-gray 03.08.2016 18:58 # +10
Источник: http://www.unicode.org/Public/UCD/latest/ucd/extracted/DerivedGeneralCategory.txt
Sc = Currency_Symbol.
Действительно, не буква.
santa_microbe 03.08.2016 20:16 # +10
inkanus-gray 03.08.2016 20:24 # +13
3oJloTou_xyu 03.08.2016 20:25 # +11
santa_microbe 04.08.2016 13:42 # +11
bormand 08.08.2016 22:27 # +10
bormand 01.08.2016 22:26 # +12
Причём числа то интересные, не банальные 0.002 и 0.998. Интересно, какое у них обоснование?
kerman 01.08.2016 22:34 # +12
Fike 02.08.2016 07:01 # +10
3_14dar 01.08.2016 22:43 # +13
bormand 01.08.2016 22:43 # +11
kerman 01.08.2016 22:46 # +14
guesto 01.08.2016 22:54 # +11
kerman 01.08.2016 23:02 # +10
kerman 01.08.2016 23:03 # +12
Lokich 02.08.2016 15:23 # +10
3_14dar 02.08.2016 16:26 # +10
Lokich 02.08.2016 17:31 # +10
nihau 02.08.2016 17:52 # +15