crossplatform.ru

Здравствуйте, гость ( Вход | Регистрация )

 
Ответить в данную темуНачать новую тему
> Проход по multimap
rp80
  опции профиля:
сообщение 7.11.2011, 18:15
Сообщение #1


Студент
*

Группа: Участник
Сообщений: 36
Регистрация: 10.9.2011
Пользователь №: 2860

Спасибо сказали: 0 раз(а)




Репутация:   0  


Вводим пары имя-значение. Нужно посчитать сумму и медиану значений по каждому ключу. Соответственно, нужно ходить по мультимапу.
Собственно, функцию я написал, но вопрос в том оптимальное ли это решение? Смущает эта возня с итераторами.. Буду рад выслушать ваши комментарии.

#include <iostream>
#include <map>

std::multimap<std::string,double> m;

template<typename F,typename S>
void sum(std::multimap<F,S>& m_)
{

    typename std::multimap<F,S>::iterator it;
    double sum=0;
    size_t count=0;

    for(it=m_.begin();it!=m_.end();++it)
    {
        sum=0;
        count=0;

        while(1)
        {
            sum+=(*it).second;
            ++count;
            typename std::multimap<F,S>::iterator oit=it;
            if(++it==m_.end())break;
            if((*oit).first!=(*(it)).first)
                break;
        }
        --it;
        std::cout<<"Sum of elements \""<<(*it).first<<"\"= "<<sum<<std::endl;
        std::cout<<"Median of elements \""<<(*it).first<<"\"= "<<sum/count<<std::endl;
    }
}

int main(int argc, char *argv[])
{
    std::string name;
    double value;
    while(std::cin>>name>>value)
        m.insert(std::pair<std::string,double>(name,value));
    sum(m);
}
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Iron Bug
  опции профиля:
сообщение 7.11.2011, 22:26
Сообщение #2


Профессионал
*****

Группа: Модератор
Сообщений: 1611
Регистрация: 6.2.2009
Из: Yekaterinburg
Пользователь №: 533

Спасибо сказали: 219 раз(а)




Репутация:   12  


зачем так усложнять? у мультимапа есть equal_range и он отсортирован по ключам.
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
rp80
  опции профиля:
сообщение 7.11.2011, 22:32
Сообщение #3


Студент
*

Группа: Участник
Сообщений: 36
Регистрация: 10.9.2011
Пользователь №: 2860

Спасибо сказали: 0 раз(а)




Репутация:   0  


Цитата(Iron Bug @ 7.11.2011, 22:26) *
зачем так усложнять? у мультимапа есть equal_range и он отсортирован по ключам.


Ну да. Я в курсе про equal_range. Но непонятно как его прикрутить к данной задаче. Тут же надо пройти по всем возможным значениям ключей. А какие они мы заранее не знаем.
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Iron Bug
  опции профиля:
сообщение 7.11.2011, 22:47
Сообщение #4


Профессионал
*****

Группа: Модератор
Сообщений: 1611
Регистрация: 6.2.2009
Из: Yekaterinburg
Пользователь №: 533

Спасибо сказали: 219 раз(а)




Репутация:   12  


берём первый ключ. ищем по нему интервал, считаем сумму. если последний элемент интервала не end(), то берём следующий элемент (который после конца интервала) и всё то же самое.

Сообщение отредактировал Iron Bug - 7.11.2011, 22:48
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
rp80
  опции профиля:
сообщение 7.11.2011, 22:56
Сообщение #5


Студент
*

Группа: Участник
Сообщений: 36
Регистрация: 10.9.2011
Пользователь №: 2860

Спасибо сказали: 0 раз(а)




Репутация:   0  


Цитата(Iron Bug @ 7.11.2011, 22:47) *
берём первый ключ. ищем по нему интервал, считаем сумму. если последний элемент интервала не end(), то берём следующий элемент (который после конца интервала) и всё то же самое.


Ага, примерно так и набросал щас. Но в общем те же яйца по моему, только в профиль. Но, тем не менее, спасибо.
    std::pair<typename std::multimap<F,S>::iterator,typename std::multimap<F,S>::iterator> ret;
    typename std::multimap<F,S>::iterator it,it_2;
    double sum;
    size_t count;

    for(it=m_.begin();it!=m_.end();++it)
    {
        sum=0;
        count=0;
        ret=m_.equal_range((*it).first);
        for(it_2=ret.first;it_2!=ret.second;++it_2)
        {
            sum+=(*it_2).second;
            ++count;
        }
         it=--it_2;
        std::cout<<"Sum of elements \""<<(*it).first<<"\"= "<<sum<<std::endl;
        std::cout<<"Median of elements \""<<(*it).first<<"\"= "<<sum/count<<std::endl;
    }
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Iron Bug
  опции профиля:
сообщение 7.11.2011, 23:15
Сообщение #6


Профессионал
*****

Группа: Модератор
Сообщений: 1611
Регистрация: 6.2.2009
Из: Yekaterinburg
Пользователь №: 533

Спасибо сказали: 219 раз(а)




Репутация:   12  


ну да. вроде верно.
только вместо неуклюжего (*it).first проще писать it->first.

на самом деле, второй вариант быстрее, потому что мультимап быстрее интервалы выдаёт, чем попарное сравнение ключей. впрочем, можешь ради эксперимента забить туда пару миллионов тестовых значений и посчитать время, затраченное на первый алгоритм и на второй. только вывод из цикла предварительно убрать, чтобы время считалось правильно.

Сообщение отредактировал Iron Bug - 7.11.2011, 23:22
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Iron Bug
  опции профиля:
сообщение 8.11.2011, 17:21
Сообщение #7


Профессионал
*****

Группа: Модератор
Сообщений: 1611
Регистрация: 6.2.2009
Из: Yekaterinburg
Пользователь №: 533

Спасибо сказали: 219 раз(а)




Репутация:   12  


самой стало интересно, насколько эффективность повысилась. написала быстро замер времени.
под линём получены следующие результаты сравнения данных двух реализаций:

На тесте 1000000 значений pair<string,int>("one",1):

без оптимизации:
через указатели - 134 мс
через equal_range - 66 мс

при оптимизации:
через указатели - 64 мс
через equal_range - 56 мс


результаты линейно зависят от количества значений. плюс первый вариант начинает существенно больше тормозить при длинных значениях ключа(увеличивается время на сравнение).

Сообщение отредактировал Iron Bug - 8.11.2011, 17:32
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
rp80
  опции профиля:
сообщение 8.11.2011, 22:01
Сообщение #8


Студент
*

Группа: Участник
Сообщений: 36
Регистрация: 10.9.2011
Пользователь №: 2860

Спасибо сказали: 0 раз(а)




Репутация:   0  


Цитата(Iron Bug @ 8.11.2011, 17:21) *
самой стало интересно, насколько эффективность повысилась. написала быстро замер времени.
под линём получены следующие результаты сравнения данных двух реализаций:

На тесте 1000000 значений pair<string,int>("one",1):

без оптимизации:
через указатели - 134 мс
через equal_range - 66 мс

при оптимизации:
через указатели - 64 мс
через equal_range - 56 мс


результаты линейно зависят от количества значений. плюс первый вариант начинает существенно больше тормозить при длинных значениях ключа(увеличивается время на сравнение).


Ну ещё одно доказательство преимущества STL над велосипедами своими.
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение
Iron Bug
  опции профиля:
сообщение 9.11.2011, 12:07
Сообщение #9


Профессионал
*****

Группа: Модератор
Сообщений: 1611
Регистрация: 6.2.2009
Из: Yekaterinburg
Пользователь №: 533

Спасибо сказали: 219 раз(а)




Репутация:   12  


Цитата(rp80 @ 9.11.2011, 0:01) *
Ну ещё одно доказательство преимущества STL над велосипедами своими.

ну, просто пример рационального использования STL.
свои-то велосипеды пошустрее STL будут. смотря сколько времени можно затратить на проект и смотря как шустро он должен работать. в большинстве юзерских задач скорости STL вполне хватает.
Перейти в начало страницы
 
Быстрая цитата+Цитировать сообщение

Быстрый ответОтветить в данную темуНачать новую тему
Теги
Нет тегов для показа


3 чел. читают эту тему (гостей: 3, скрытых пользователей: 0)
Пользователей: 0




RSS Текстовая версия Сейчас: 30.11.2024, 6:15