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

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> if continue 
V
    Опции темы
DenWPF
Дата 18.4.2011, 00:25 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


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

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



Есть какие то основания сказать, что нужно использовать continue. В данном примере?
Вариант 1

Код

for (int i = 0; i < m_list.Count; i++)
{
    if (m_list[i].m_title != temp) continue;
    key = m_list[i].m_pathImage;
    break;
}



Вариант 2
Код

for (int i = 0; i < m_list.Count; i++)
{
    if (m_list[i].m_title == temp)
    {
        key = m_list[i].m_pathImage;
        break;
    }
}


Это сообщение отредактировал(а) DenWPF - 18.4.2011, 17:31
PM MAIL   Вверх
Экскалупатор
Дата 18.4.2011, 00:38 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1746
Регистрация: 1.4.2009
Где: г. Минск

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



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


Опытный
**


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

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



1. первый вариант ужасен - две управляющие инструкции совершенно неоправданно заставляют дольше времени потртить на понимание работы цикла
2. в наши дни такие задачи обычно симпатичнее решаются с linq

а вообще, я бы большее внимание уделил другим вещам
1. нейминг хреновый, что за переменная m_list, к примеру? нихрена из её имени не ясно
2. код стайл ужасен, ничуть не боясь гнилых томатов, скажу, что для читаемости кода большинством разработчиков, правильный код стайл должен быть максимально приближен к майкрософтовскому варианту, см. StyleCop, к примеру

и именно второй паре пунктов стоит уделить внимание в первую очерень, а не использовать var, или нет


Это сообщение отредактировал(а) mihryak - 18.4.2011, 19:16
PM MAIL ICQ   Вверх
DenWPF
Дата 18.4.2011, 19:22 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


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

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



m_list был переименован специально до этого было m_listЧототам
PM MAIL   Вверх
mihryak
Дата 18.4.2011, 19:56 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


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

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



я к тому, что большинство современных гвайдов советуют вообще избегать указания типов в имени переменной
PM MAIL ICQ   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
Прежде чем создать тему, посмотрите сюда:
Partizan
PashaPash

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


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

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


 




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


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

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