Объединение int с пустотой * - PullRequest
0 голосов
/ 12 февраля 2012

Мне нужно объединить int с пустотой *. Я думал о том, чтобы сделать это как:

long int lint = ((int)integer<<(sizeof(void*)*8)) | ((void*)ptr);

но , согласно моему предыдущему сообщению , длинного int будет недостаточно.

Кто-нибудь может подсказать, как это сделать?

PS. Некоторые люди спрашивают, с какой стати я могу захотеть сделать это. Ну, я разрабатываю общую систему событий (рассылка / прослушивание), которая обслуживает бесчисленное множество различных случаев. Я отправлю код, когда он будет готов ...

Ответы [ 4 ]

3 голосов
/ 12 февраля 2012

Единственный ответ - использовать struct

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

Структура здесь лучше, потому что:

  1. Это портативный
  2. Это безопасно
  3. Это идеология с ++.
  4. Это быстрее .

Позвольте мне разрушить ваши мифы о сравнении скорости long long и struct. Как вы знаете, все способы оптимизации вашего кода начинаются с профилирования. Давайте сделаем простую программу и измеряем сравнение скорости векторов long long и struct S:

#include <iostream>
#include <string>
#include <vector>
#include <windows.h>

struct S
{
    unsigned int a;
    void* b;

    bool operator==(const S& other)  const
    {
        return a == other.a && b == other.b;
    }
};

template <typename Iterator>
int count_eq(Iterator begin, Iterator end)
{
    int result = 0;
    for (Iterator i  = begin; i != end; ++i) {
        for (Iterator j  = i + 1; j != end; ++j) {
            result += *i == *j;
        }
    }
    return result;
}

template <typename Iterator>
void mesure(Iterator begin, Iterator end)
{
    long long t0 = GetTickCount();
    int res = count_eq(begin, end);
    long long t1 = GetTickCount();
    std::cout << "result: " << res <<"; Time: "<<(t1-t0)<<"\n";
}

int main()
{
    const unsigned int Size = 20000;
    std::vector<unsigned long long> l;
    for (int i = 0; i < Size; i++) {
        l.push_back(i% (Size/10));
    }

    std::vector<S> s;
    for (int j = 0; j < Size; j++) {
        S el;
        el.a = j% (Size/10);
        el.b = 0;
        s.push_back(el);
    }

    mesure(l.begin(), l.end());
    mesure(s.begin(), s.end()); 
}

Позволяет проверить результаты:

>g++ src.cpp -O3
>a
result: 90000; Time: 327
result: 90000; Time: 188

Да, struct с пользовательскими operator == в 1,5 раза быстрее.

1 голос
/ 12 февраля 2012

Хмм, не должно ли это быть что-то вроде:

long long res = ((long long)integer<<(sizeof(void*))) | ((long long)ptr);

Это все еще будет работать только для 32-битных указателей и не будет работать для 64-битных. Нет встроенного типа, который бы подходил для такой арифметики.

0 голосов
/ 12 февраля 2012

Если вам нужна структура данных, которая может хранить указатель типа int или void, используйте union . Объединение будет иметь размер наибольшего значения, которое оно содержит. Вы можете сделать что-то вроде этого:

typedef union {
    void *data,
    int *value
} myUnion_t;

myUnion_t storage;
int num = 10;

// if you want to store a void pointer
storage.data = &num;

// if you want to store an int
storage.value = num;

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

0 голосов
/ 12 февраля 2012

вы должны использовать что-то вроде uint_least64_t, доступное из stdint.h, конечно, оно не может содержать указатели из 64-битных систем, для этого вам нужно будет использовать структуру, как многие люди упоминали в комментариях.

Хотя то, как вы используете указатели, кажется странным, похоже, что вам даже не нужно использовать void* для того, что вы делаете, не говоря уже о том, что вы можете сделать то же самое с полиморфизмомсистема, которую вы описали).

...