การแปลงตัวชี้เป็นจำนวนเต็ม

ฉันกำลังพยายามปรับรหัสที่มีอยู่ให้เป็นเครื่อง 64 บิต ปัญหาหลักคือในฟังก์ชันหนึ่ง coder ก่อนหน้านี้ใช้อาร์กิวเมนต์ void * ที่ถูกแปลงเป็นประเภทที่เหมาะสมในฟังก์ชันนั้นเอง ตัวอย่างสั้น ๆ :

void function(MESSAGE_ID id, void* param)
{
    if(id == FOO) {
        int real_param = (int)param;
        // ...
    }
}

แน่นอนในเครื่อง 64 บิตฉันได้รับข้อผิดพลาด:

error: cast from 'void*' to 'int' loses precision

ฉันต้องการแก้ไขสิ่งนี้เพื่อให้มันยังคงทำงานบนเครื่อง 32 บิตและสมบูรณ์ที่สุด ความคิดใด ๆ ?

ใช้intptr_tและuintptr_t.

เพื่อให้แน่ใจว่ามีการกำหนดแบบพกพาคุณสามารถใช้โค้ดดังนี้:

#if defined(__BORLANDC__)
    typedef unsigned char uint8_t;
    typedef __int64 int64_t;
    typedef unsigned long uintptr_t;
#elif defined(_MSC_VER)
    typedef unsigned char uint8_t;
    typedef __int64 int64_t;
#else
    #include <stdint.h>
#endif

เพียงวางไว้ในไฟล์. h และรวมไว้ที่ใดก็ได้ที่คุณต้องการ

หรือคุณสามารถดาวน์โหลดรุ่นไมโครซอฟท์ของstdint.hไฟล์จากที่นี่หรือใช้แบบพกพาจากที่นี่

ฉันจะบอกว่านี่เป็นวิธี C ++ สมัยใหม่

#include <cstdint>
void *p;
auto i = reinterpret_cast<std::uintptr_t>(p);

แก้ไข :

ประเภทที่ถูกต้องสำหรับจำนวนเต็ม

ดังนั้นวิธีที่ถูกต้องในการจัดเก็บตัวชี้เป็นจำนวนเต็มคือการใช้uintptr_tหรือintptr_tประเภท (ดูเพิ่มเติมในประเภทจำนวนเต็ม cppreference สำหรับ C99 )

ประเภทเหล่านี้กำหนดไว้<stdint.h>สำหรับ C99 และในเนมสเปซstdสำหรับ C ++ 11 ใน<cstdint>(ดูประเภทจำนวนเต็มสำหรับ C ++ )

C ++ 11 (และเป็นต้นไป) เวอร์ชัน

#include <cstdint>
std::uintptr_t i;

C ++ 03 เวอร์ชัน

extern "C" {
#include <stdint.h>
}

uintptr_t i;

เวอร์ชัน C99

#include <stdint.h>
uintptr_t i;

ตัวดำเนินการหล่อที่ถูกต้อง

ใน C มีการร่ายเพียงครั้งเดียวและการใช้ C cast ใน C ++ นั้นถูกขมวดคิ้ว (ดังนั้นอย่าใช้ใน C ++) ใน C ++ มีการร่ายที่แตกต่างกัน reinterpret_castคือการแคสต์ที่ถูกต้องสำหรับการแปลงนี้ (ดูเพิ่มเติมที่นี่ )

C ++ 11 เวอร์ชัน

auto i = reinterpret_cast<std::uintptr_t>(p);

C ++ 03 เวอร์ชัน

uintptr_t i = reinterpret_cast<uintptr_t>(p);

เวอร์ชัน C

uintptr_t i = (uintptr_t)p; // C Version

คำถามที่เกี่ยวข้อง

• ชนิดข้อมูล uintptr_t คืออะไร

"size_t" และ "ptrdiff_t" จะต้องตรงกับสถาปัตยกรรมของคุณ (ไม่ว่าจะเป็นอะไรก็ตาม) ดังนั้นฉันคิดว่าแทนที่จะใช้ 'int' คุณควรจะใช้ 'size_t' ได้ซึ่งในระบบ 64 บิตควรเป็นประเภท 64 บิต

การสนทนาที่ไม่ได้ลงนาม int vs size_t นี้จะกล่าวถึงรายละเอียดเพิ่มเติมอีกเล็กน้อย

ใช้uintptr_tเป็นประเภทจำนวนเต็มของคุณ

คำตอบหลายคำชี้ให้เห็นuintptr_tและ#include <stdint.h>เป็น 'ทางออก' นั่นคือฉันขอแนะนำเป็นส่วนหนึ่งของคำตอบ แต่ไม่ใช่คำตอบทั้งหมด คุณต้องดูด้วยว่าฟังก์ชันถูกเรียกใช้ด้วยรหัสข้อความของ FOO ที่ใด

พิจารณารหัสนี้และการรวบรวม:

$ cat kk.c
#include <stdio.h>
static void function(int n, void *p)
{
    unsigned long z = *(unsigned long *)p;
    printf("%d - %lu\n", n, z);
}

int main(void)
{
    function(1, 2);
    return(0);
}
$ rmk kk
        gcc -m64 -g -O -std=c99 -pedantic -Wall -Wshadow -Wpointer-arith \
            -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes \
            -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE kk.c -o kk 
kk.c: In function 'main':
kk.c:10: warning: passing argument 2 of 'func' makes pointer from integer without a cast
$

คุณจะสังเกตเห็นว่ามีปัญหาในตำแหน่งที่เรียก (ในmain()) - การแปลงจำนวนเต็มเป็นตัวชี้โดยไม่มีการโยน คุณจะต้องวิเคราะห์ของคุณfunction()ในการใช้งานทั้งหมดเพื่อดูว่าค่าถูกส่งผ่านไปอย่างไร รหัสภายในfunction()จะใช้งานได้หากมีการเขียนการโทร:

unsigned long i = 0x2341;
function(1, &i);

เนื่องจากของคุณอาจเขียนไม่เหมือนกันคุณจึงต้องตรวจสอบจุดที่เรียกใช้ฟังก์ชันเพื่อให้แน่ใจว่าการใช้ค่าดังที่แสดงนั้นสมเหตุสมผล อย่าลืมว่าคุณอาจกำลังพบจุดบกพร่องแฝงอยู่

นอกจากนี้หากคุณจะจัดรูปแบบค่าของvoid *พารามิเตอร์ (ตามที่แปลงแล้ว) ให้ดู<inttypes.h>ส่วนหัวอย่างละเอียด(แทนstdint.h- inttypes.hให้บริการstdint.hซึ่งผิดปกติ แต่มาตรฐาน C99 ระบุว่า[t] ส่วนหัว<inttypes.h>มีส่วนหัว<stdint.h>และ ขยายด้วยสิ่งอำนวยความสะดวกเพิ่มเติมที่จัดเตรียมโดยการใช้งานโฮสต์ ) และใช้มาโคร PRIxxx ในสตริงรูปแบบของคุณ

นอกจากนี้ความคิดเห็นของฉันยังใช้ได้กับ C มากกว่า C ++ อย่างเคร่งครัด แต่รหัสของคุณอยู่ในส่วนย่อยของ C ++ ที่พกพาได้ระหว่าง C และ C ++ มีโอกาสที่จะดีที่ความคิดเห็นของฉันนำไปใช้

1. #include <stdint.h>
2. ใช้uintptr_tประเภทมาตรฐานที่กำหนดไว้ในไฟล์ส่วนหัวมาตรฐานที่รวมอยู่

ฉันเจอคำถามนี้ขณะศึกษาซอร์สโค้ดของSQLiteข้อมูล SQLite

ในsqliteInt.hมีย่อหน้าของโค้ดที่กำหนดมาโครแปลงระหว่างจำนวนเต็มและตัวชี้ ผู้เขียนได้กล่าวคำแถลงที่ดีมากก่อนโดยชี้ให้เห็นว่าควรเป็นปัญหาที่ขึ้นอยู่กับคอมไพเลอร์จากนั้นจึงใช้วิธีแก้ปัญหาสำหรับคอมไพเลอร์ยอดนิยมส่วนใหญ่

#if defined(__PTRDIFF_TYPE__)  /* This case should work for GCC */
# define SQLITE_INT_TO_PTR(X)  ((void*)(__PTRDIFF_TYPE__)(X))
# define SQLITE_PTR_TO_INT(X)  ((int)(__PTRDIFF_TYPE__)(X))
#elif !defined(__GNUC__)       /* Works for compilers other than LLVM */
# define SQLITE_INT_TO_PTR(X)  ((void*)&((char*)0)[X])
# define SQLITE_PTR_TO_INT(X)  ((int)(((char*)X)-(char*)0))
#elif defined(HAVE_STDINT_H)   /* Use this case if we have ANSI headers */
# define SQLITE_INT_TO_PTR(X)  ((void*)(intptr_t)(X))
# define SQLITE_PTR_TO_INT(X)  ((int)(intptr_t)(X))
#else                          /* Generates a warning - but it always works     */
# define SQLITE_INT_TO_PTR(X)  ((void*)(X))
# define SQLITE_PTR_TO_INT(X)  ((int)(X))
#endif

และนี่คือคำพูดของความคิดเห็นสำหรับรายละเอียดเพิ่มเติม:

/*
** The following macros are used to cast pointers to integers and
** integers to pointers.  The way you do this varies from one compiler
** to the next, so we have developed the following set of #if statements
** to generate appropriate macros for a wide range of compilers.
**
** The correct "ANSI" way to do this is to use the intptr_t type.
** Unfortunately, that typedef is not available on all compilers, or
** if it is available, it requires an #include of specific headers
** that vary from one machine to the next.
**
** Ticket #3860:  The llvm-gcc-4.2 compiler from Apple chokes on
** the ((void*)&((char*)0)[X]) construct.  But MSVC chokes on ((void*)(X)).
** So we have to define the macros in different ways depending on the
** compiler.
*/

เครดิตไปที่คอมมิทเตอร์

สิ่งที่ดีที่สุดที่ควรทำคือหลีกเลี่ยงการแปลงจากประเภทตัวชี้เป็นประเภทที่ไม่ใช่ตัวชี้ อย่างไรก็ตามสิ่งนี้ไม่สามารถทำได้อย่างชัดเจนในกรณีของคุณ

อย่างที่ทุกคนพูด uintptr_t คือสิ่งที่คุณควรใช้

ลิงค์นี้มีข้อมูลที่ดีเกี่ยวกับการแปลงเป็นรหัส 64 บิต

นอกจากนี้ยังมีการอภิปรายที่ดีเกี่ยวกับเรื่องนี้ในcomp.std.c

ฉันคิดว่า "ความหมาย" ของ void * ในกรณีนี้เป็นแฮนเดิลทั่วไป ไม่ใช่ตัวชี้ไปที่ค่า แต่เป็นค่านั้นเอง (นี่เป็นเพียงการใช้ void * โดยโปรแกรมเมอร์ C และ C ++)

ถ้ามันถือค่าจำนวนเต็มก็ควรอยู่ในช่วงจำนวนเต็ม!

นี่คือการแสดงผลเป็นจำนวนเต็มอย่างง่ายดาย:

int x = (char*)p - (char*)0;

ควรให้คำเตือนเท่านั้น

เนื่องจากuintptr_tจะไม่ได้รับประกันว่าจะมีใน C ++ / C ++ 11ถ้าเป็นแปลงวิธีหนึ่งที่คุณสามารถพิจารณากำหนดเสมอในuintmax_t<cstdint>

auto real_param = reinterpret_cast<uintmax_t>(param);

ในการเล่นอย่างปลอดภัยเราสามารถเพิ่มที่ใดก็ได้ในรหัสเพื่อยืนยัน:

static_assert(sizeof (uintmax_t) >= sizeof (void *) ,
              "No suitable integer type for conversion from pointer type");