перлы code review
От: Svjat Украина  
Дата: 12.07.07 15:52
Оценка: -5 :))) :))) :))) :))) :))
// вариант метода, кот. выгребает всю таблицу из базы
public List<DownloadCompleteInfo> Query();


public List<DownloadCompleteInfo> Query( int count )
{
    List<DownloadCompleteInfo> list = Query();            // !!!
    list.Sort();

    // а теперь берем, и вырезаем сколько нужно - вуаля! 
    if ( list.Count > count ) 
       list.RemoveRange( count, list.Count - count );     
    return list;
}
Re: перлы code review
От: _Jane_ Украина  
Дата: 12.07.07 16:35
Оценка:
У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.

S>
S>// вариант метода, кот. выгребает всю таблицу из базы
S>public List<DownloadCompleteInfo> Query();


S>public List<DownloadCompleteInfo> Query( int count )
S>{
S>    List<DownloadCompleteInfo> list = Query();            // !!!
S>    list.Sort();

S>    // а теперь берем, и вырезаем сколько нужно - вуаля! 
S>    if ( list.Count > count ) 
S>       list.RemoveRange( count, list.Count - count );     
S>    return list;
S>}
S>
Jane
Re[2]: перлы code review
От: dr.Chaos Россия Украшения HandMade
Дата: 12.07.07 16:43
Оценка:
Здравствуйте, _Jane_, Вы писали:


_J_>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


Ну что вы, что вы это же стресс тест.
Побеждающий других — силен,
Побеждающий себя — Могущественен.
Лао Цзы
Re[2]: перлы code review
От: Aviant  
Дата: 13.07.07 06:30
Оценка: :)
Здравствуйте, _Jane_, Вы писали:


_J_>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


S>>
S>>// вариант метода, кот. выгребает всю таблицу из базы
S>>public List<DownloadCompleteInfo> Query();
S>>public List<DownloadCompleteInfo> Query( int count )
S>>{
<....>
S>>}
S>>

1 Порочна сама идея "хочу получить первые 7 записей из таблицы"
2 За то, что новичек догадался отсортировать выборку хоть как-то ему жирный плюс
Re[3]: перлы code review
От: _Jane_ Украина  
Дата: 13.07.07 08:07
Оценка:
Здравствуйте, Aviant, Вы писали:

_J_>>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


S>>>
S>>>// вариант метода, кот. выгребает всю таблицу из базы
S>>>public List<DownloadCompleteInfo> Query();
S>>>public List<DownloadCompleteInfo> Query( int count )
S>>>{
A><....>
S>>>}
S>>>

A>1 Порочна сама идея "хочу получить первые 7 записей из таблицы"

Это может зависеть от необходимостей пользователя. А и выборка по каким-то критериям тоже может быть достаточно объемной, чтобы делить ее на куски. Это может быть нужно по разным причинам.

A>2 За то, что новичек догадался отсортировать выборку хоть как-то ему жирный плюс


Не знаю, новичок ли писал этот код, но лучше бы он сделал поиск по ключу
Jane
Re[3]: перлы code review
От: Svjat Украина  
Дата: 13.07.07 11:35
Оценка:
Здравствуйте, Aviant, Вы писали:

A>Здравствуйте, _Jane_, Вы писали:



_J_>>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


S>>>
S>>>// вариант метода, кот. выгребает всю таблицу из базы
S>>>public List<DownloadCompleteInfo> Query();
S>>>public List<DownloadCompleteInfo> Query( int count )
S>>>{
A><....>
S>>>}
S>>>

A>1 Порочна сама идея "хочу получить первые 7 записей из таблицы"

в данном случае — это "top 100"


A>2 За то, что новичек догадался отсортировать выборку хоть как-то ему жирный плюс


в контексте задачи это неверно.
это уже не в юмор, просто ошибка.
Re[4]: перлы code review
От: Aviant  
Дата: 13.07.07 11:56
Оценка:
Здравствуйте, Svjat, Вы писали:

A>>1 Порочна сама идея "хочу получить первые 7 записей из таблицы"


S>в данном случае — это "top 100"


Порочна сама идея получать top 100 без Order by

A>>2 За то, что новичек догадался отсортировать выборку хоть как-то ему жирный плюс


S>в контексте задачи это неверно.

S>это уже не в юмор, просто ошибка.

Может и ошибка, я просто хотел сказать что новичек молодец, что догадался как-то отсортировать массив, а не вывалить первые n элементов
Re[5]: перлы code review
От: Svjat Украина  
Дата: 13.07.07 12:05
Оценка:
Здравствуйте, Aviant, Вы писали:

A>Здравствуйте, Svjat, Вы писали:


A>>>1 Порочна сама идея "хочу получить первые 7 записей из таблицы"


S>>в данном случае — это "top 100"


A>Порочна сама идея получать top 100 без Order by


кто сказал, что его нет? это все уже есть в самом запросе
зачем еще сортировать?

A>>>2 За то, что новичек догадался отсортировать выборку хоть как-то ему жирный плюс


S>>в контексте задачи это неверно.

S>>это уже не в юмор, просто ошибка.

A>Может и ошибка, я просто хотел сказать что новичек молодец, что догадался как-то отсортировать массив, а не вывалить первые n элементов


мдя...... ай маладэц какой!

нужно ему было еще GC.Collect() после обрезания парочки лишних млн. записей вызывать, тогда еще один + , да?
Re[3]: перлы code review
От: olexandr Новороссия http://demotivation.me/images/20140818/lxz0l278b9ep.jpg
Дата: 13.07.07 15:26
Оценка: :)
Здравствуйте, dr.Chaos, Вы писали:

DC>Здравствуйте, _Jane_, Вы писали:



_J_>>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


DC>Ну что вы, что вы это же стресс тест.


Мне лично понравилось что в этой ветке все 3 сообщения из Днепропетровска (:
Некроссплатформенность маловероятна (c) Sheridan
...трава никак не влияет, разве что срывает покровы барьеров... (с) мыщъх
Re[4]: перлы code review
От: dr.Chaos Россия Украшения HandMade
Дата: 13.07.07 15:48
Оценка:
Здравствуйте, olexandr, Вы писали:

O>Здравствуйте, dr.Chaos, Вы писали:


DC>>Здравствуйте, _Jane_, Вы писали:



_J_>>>У меня каждый второй новичок-студент нечто подобное предлагает так что типичный "прием", к сожалению.


DC>>Ну что вы, что вы это же стресс тест.


O>Мне лично понравилось что в этой ветке все 3 сообщения из Днепропетровска (:


ААААА. Спалился !!
Побеждающий других — силен,
Побеждающий себя — Могущественен.
Лао Цзы
Re[4]: перлы code review
От: int13h Украина  
Дата: 13.07.07 17:57
Оценка:
Здравствуйте, olexandr, Вы писали:
-=[ skipped ]=-
O>Мне лично понравилось что в этой ветке все 3 сообщения из Днепропетровска (:
Ну вот, уже 4...
Re: перлы code review
От: Кодт Россия  
Дата: 20.07.07 11:28
Оценка: :)))
Здравствуйте, Svjat, Вы писали:

Не раскрыта тема реализации списка. Может, он ленивый? Тогда нормально.
Для хаскелла это типичный приём программирования.
... << RSDN@Home 1.2.0 alpha rev. 655>>
Перекуём баги на фичи!
Re[5]: перлы code review
От: _ks_  
Дата: 20.07.07 12:28
Оценка:
O>>Мне лично понравилось что в этой ветке все 3 сообщения из Днепропетровска (:
I>Ну вот, уже 4...

Щас пятый будет.

У меня тут один мега программист забодяжил...
Было вот такое:
void Run()
{
    [здесь 100 строчек кода, обрабатывающих объект]
    [и последняя 101-я строчка отсылает объект]
}


Надо было ему сделать тоже самое , но для нескольких объектов. Что он делает:
void Run(int count)
{
  if (count == 1)
  {
    [здесь 100 строчек кода, обрабатывающих объект]
    [и последняя 101-я строчка отсылает объект]
  }
  else
  {
    for(...по count...)
    {
      [здесь 100 строчек кода, обрабатывающих объект]
    }
    [и еще одна строчка, отсылающая все обработаные объекты объект]
  }
}


Вот такой звиздец.
Re[6]: перлы code review
От: int13h Украина  
Дата: 20.07.07 16:03
Оценка:
Здравствуйте, _ks_, Вы писали:

-=[ скипануто ]=-

__>Вот такой звиздец.


Може это так жара сказывается?
Re[7]: перлы code review
От: _ks_  
Дата: 20.07.07 16:12
Оценка:
Здравствуйте, int13h, Вы писали:

I>Здравствуйте, _ks_, Вы писали:


I>-=[ скипануто ]=-


__>>Вот такой звиздец.


I>Може это так жара сказывается?


Это было весной. Так что не летня жара, а весенний ... эээ ... хм ... гармоны.
Re: перлы code review
От: trophim Россия  
Дата: 22.07.07 23:08
Оценка: :)
Фи... Вот я видел... Знаете как подсчитать длину файла на Си? Ключевые слова: getc, переменная, цикл...
[EOF]
Let it be! — Давайте есть пчелу!
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.