Модераторы: Partizan, gambit

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> То, что профи используют незадумываясь, Негласные соглашения 
:(
    Опции темы
Antenka
Дата 1.4.2008, 20:49 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Getting Nowhere ...



Профиль
Группа: Участник
Сообщений: 15
Регистрация: 14.11.2007

Репутация: нет
Всего: нет



Заранее извиняюсь, если где-то чего-то недонашла или недочитала...

Все когда-то слышали такие слова, как "негласные соглашения" или что-либо в этом роде. Некоторые из них частенько можно услышать, к примеру "называть классы лучше с заглавной буквы, а методы и переменные - с маленькой". Такие мелочи, как эта (по крайней мере я так думаю) являются одной из ступенек к написанию хорошего, понятного кода. Напишите, кто чего помнит и знает (если не сложно smile ).

P.S. Создавая эту тему я надеялась собрать как можно больше этих соглашений в одном месте. Заранее всем спасибо smile
PM MAIL   Вверх
Optimus
Дата 1.4.2008, 21:36 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Бывалый
*


Профиль
Группа: Участник
Сообщений: 186
Регистрация: 1.9.2007

Репутация: 1
Всего: 14



--------------------
"постановка задачи наполовину решает саму задачу"
PM MAIL   Вверх
PashaPash
Дата 1.4.2008, 23:09 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1233
Регистрация: 3.1.2008

Репутация: 9
Всего: 49





--------------------
PM MAIL WWW   Вверх
ivashkanet
Дата 2.4.2008, 08:16 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодю потиху
****


Профиль
Группа: Участник Клуба
Сообщений: 3684
Регистрация: 23.2.2006
Где: Гомель, Беларусь

Репутация: 3
Всего: 149



Я использую стандарты описанные здесь: cоглашения по оформлению кода команды RSDN

Как к этому пришел -- соответствует стандартам MS да еще и на русском.
PM MAIL WWW ICQ   Вверх
werqwrt
Дата 3.4.2008, 08:32 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 63
Регистрация: 23.3.2008

Репутация: нет
Всего: 1



Во многих исходниках видел такое
Код

  chaar ch = 'c';
if(IsDigit(c) == false)
{
  //
}


Я делаю так
Код

char ch = 'c';
if(!IsDigit(c))
{
  //
}

можно узнать какой подход лучше и читабельней
PM MAIL   Вверх
ivashkanet
Дата 3.4.2008, 08:53 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодю потиху
****


Профиль
Группа: Участник Клуба
Сообщений: 3684
Регистрация: 23.2.2006
Где: Гомель, Беларусь

Репутация: 3
Всего: 149



werqwrt, чисто субъективно первый.

С одной стороны: восклицательный знак отрицания не так сильно бросается в глаза, как == false
C другой: == false заставляет задуматься (а что/зачем это?), в то время как !bool очевидно (если его заметишь, конечно).
С третьей: если "== false" оговорено стандартами кодирования, то никаких пробем с осмыслением не возникает, кроме того остается "заметность".
PM MAIL WWW ICQ   Вверх
mr.DUDA
Дата 3.4.2008, 08:53 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


3D-маньяк
****


Профиль
Группа: Экс. модератор
Сообщений: 8244
Регистрация: 27.7.2003
Где: город-герой Минск

Репутация: 8
Всего: 232



Второй, конечно. Представьте себе что сравнений в условии не одно а несколько, как "расползётся" код от ненужных == false или == true...


--------------------
user posted image
PM MAIL WWW   Вверх
ivashkanet
Дата 3.4.2008, 08:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодю потиху
****


Профиль
Группа: Участник Клуба
Сообщений: 3684
Регистрация: 23.2.2006
Где: Гомель, Беларусь

Репутация: 3
Всего: 149



mr.DUDA,  smile // Сколько людей, столько мнений
PM MAIL WWW ICQ   Вверх
mr.DUDA
Дата 3.4.2008, 08:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


3D-маньяк
****


Профиль
Группа: Экс. модератор
Сообщений: 8244
Регистрация: 27.7.2003
Где: город-герой Минск

Репутация: 8
Всего: 232



ivashkanet,  smile 

Ну честно, нигде не видел чтобы кто-то писал if (что-то == false).  smile 


--------------------
user posted image
PM MAIL WWW   Вверх
ivashkanet
Дата 3.4.2008, 09:02 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодю потиху
****


Профиль
Группа: Участник Клуба
Сообщений: 3684
Регистрация: 23.2.2006
Где: Гомель, Беларусь

Репутация: 3
Всего: 149



mr.DUDA, так и я не пишу (пока на такие грабли не наступал), просто перечислил преимущества ;-)

Добавлено @ 09:09
И вообще в "правилах хорошего тона" (Совершенный кодне рекомендуют использовать отрицания в проверках условия.

А такой подход акцентирует внимание на отрицании условия (если программисты с этим подходом знакомы, то он ничем не хуже "!").

"Исходный код чаще читается чем пишется", поэтому, ИМХО, стоит сделать все скользкие места выделяющимися.

Добавлено через 10 минут и 52 секунды
Цитата(mr.DUDA @  3.4.2008,  08:53 Найти цитируемый пост)
Представьте себе что сравнений в условии не одно а несколько, как "расползётся" код от ненужных == false или == true... 

Что-то сразу не заметил Тогда "!" явно предпочтительней. 
Но ИМХО, тут дело в одиночном отрицании

Это сообщение отредактировал(а) ivashkanet - 3.4.2008, 09:10
PM MAIL WWW ICQ   Вверх
werqwrt
Дата 3.4.2008, 09:17 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 63
Регистрация: 23.3.2008

Репутация: нет
Всего: 1



Думаю что надо сделать подсветку для "!" чтобы както выделялось, и тогда одназначно второй вариант
на пример красным на желтом фоне

Это сообщение отредактировал(а) werqwrt - 3.4.2008, 09:19
PM MAIL   Вверх
VisualProgrammerNET
Дата 3.4.2008, 23:54 (ссылка) |    (голосов:1) Загрузка ... Загрузка ... Быстрая цитата Цитата


Почётный халявщик
**


Профиль
Группа: Участник
Сообщений: 461
Регистрация: 30.10.2005
Где: Зеленоград

Репутация: 1
Всего: 6



Я щас себя приучаю к использованию конструкций типа ?:, ?? и подобных. Код выглядит очень красиво, но неопытному взгляду совершенно непонятен =)


--------------------
3 ГОДА НА user posted image 
PM MAIL ICQ   Вверх
mr.DUDA
Дата 4.4.2008, 07:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


3D-маньяк
****


Профиль
Группа: Экс. модератор
Сообщений: 8244
Регистрация: 27.7.2003
Где: город-герой Минск

Репутация: 8
Всего: 232



Цитата(VisualProgrammerNET @  3.4.2008,  23:54 Найти цитируемый пост)
Я щас себя приучаю к использованию конструкций типа ?:, ?? и подобных. Код выглядит очень красиво, но неопытному взгляду совершенно непонятен =)

?? - очень полезная штука, кстати. А тернарная операция вообще азы  smile 


--------------------
user posted image
PM MAIL WWW   Вверх
Gelis
Дата 5.4.2008, 17:41 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 145
Регистрация: 26.10.2006
Где: Брест

Репутация: нет
Всего: 4



Цитата(mr.DUDA @  4.4.2008,  07:26 Найти цитируемый пост)
тернарная операция вообще азы   

Только некоторые ей так увлекаются, что иногда встречаешь код типа
Код

bool withOutXml = ...;
.................................
return (withOutXml)?ToObject(true):ToObject(false);


Про стандарты кодирования. Обычно перед началом большого проекта во время предварительного проектирования создается документик "Coding standarts". Который и описывает все правила именования для данного проекта. Новичку на проекте он дается к прочтению, а затем еще и более опытный товарищ вычитывает за новичком код и делает ему некоторые замечания.

PM MAIL   Вверх
NightmareZ
Дата 5.4.2008, 19:44 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


[хакер]
**


Профиль
Группа: Участник
Сообщений: 699
Регистрация: 10.8.2006

Репутация: 1
Всего: 13



Цитата(werqwrt @  3.4.2008,  08:32 Найти цитируемый пост)

Во многих исходниках видел такое
Код

  chaar ch = 'c';
if(IsDigit(c) == false)
{
  //
}


Это вообще бред. Метод IsDigit сам по себе позвращает булево значение. Сравнивать его с true или false - бессмысленно.

Цитата(Gelis @  5.4.2008,  17:41 Найти цитируемый пост)

Цитата(mr.DUDA @  4.4.2008,  07:26 Найти цитируемый пост)
тернарная операция вообще азы   

Только некоторые ей так увлекаются, что иногда встречаешь код типа
Код

bool withOutXml = ...;
.................................
return (withOutXml)?ToObject(true):ToObject(false);


Непрофессионализм. И не более того.


Это сообщение отредактировал(а) NightmareZ - 5.4.2008, 19:47


--------------------
NightmareZ.net - мой блог и сайт, мои проекты и прочий трэш
Ely-Art.ru - наша маленькая домашняя арт-студия
mugcraft.ru - кружки на любой вкус
PM WWW ICQ Skype GTalk AOL YIM   Вверх
Ответ в темуСоздание новой темы Создание опроса
Прежде чем создать тему, посмотрите сюда:
Partizan
PashaPash

Используйте теги [code=csharp][/code] для подсветки кода. Используйтe чекбокс "транслит" если у Вас нет русских шрифтов.
Что делать если Вам помогли, но отблагодарить помощника плюсом в репутацию Вы не можете(не хватает сообщений)? Пишите сюда, или отправляйте репорт. Поставим :)
Так же не забывайте отмечать свой вопрос решенным, если он таковым является :)


Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, mr.DUDA, Partizan, PashaPash.

 
1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | .NET для новичков | Следующая тема »


 




[ Время генерации скрипта: 0.0877 ]   [ Использовано запросов: 21 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.