通过const_cast进行的const初始化是否具有未定义的行为?

时间:2010-11-27 10:53:48

标签: c++ initialization undefined undefined-behavior const-cast

根据我的小测试,这段代码可行。但是,它有未定义的行为吗?通过使用const_cast修改const对象导致我之前测试中的运行时访问违规,但我不记得它们是如何不同的。那么,这里根本不存在问题吗?

// test.h
#pragma once
#include <boost/array.hpp>

typedef boost::array<int,100000> bigLut_t;
extern const bigLut_t constBigLut;

// test.cpp
#include "test.h"

bigLut_t& initializeConstBigLut()
{
    bigLut_t* pBigLut = const_cast<bigLut_t*>( &constBigLut );

    for(int i = 0; i < 100000; ++i) {
        pBigLut->at(i) = i;
    }
    return const_cast<bigLut_t&>(constBigLut);
}

const bigLut_t constBigLut = initializeConstBigLut();

// const_test.cpp
#include <iostream>
#include "test.h"

void main()
{
    for(int i = 0; i < 100; ++i) {
        std::cout << constBigLut[i] << std::endl;
    }
    system("pause");
}

(请注意,sizeof(bigLut_t)太多而无法放入堆栈。)

编辑: 我实际上喜欢ybungalobill的小评论中的想法,最好是初始化这些大对象的方法:

// test.h
#pragma once
#include <boost/array.hpp>

extern const struct BigLut : public boost::array<int,100000> {
    BigLut();
} constBigLut;

// test.cpp
#include "test.h"

const BigLut constBigLut;
BigLut::BigLut()
{
    for(int i = 0; i < 100000; ++i) {
        this->at(i) = i;
    }
}

3 个答案:

答案 0 :(得分:6)

修改定义为const的对象。无论何时进行,无论是否在初始化期间,它仍然是未定义的行为。仅当const指针是在某个早期阶段从该对象的非const指针获取时,才定义使用const_cast删除constness。那不是你的情况。

你能做的最好的事情是

const bigLut_t& initializeConstBigLut()
{
    static bigLut_t bigLot;

    for(int i = 0; i < 100000; ++i) {
        bigLut.at(i) = i;
    }
    return bigLut;
}

const bigLut_t constBigLut = initializeConstBigLut();

希望编译器能够优化静态临时。

答案 1 :(得分:3)

您滥用const_cast运算符是不幸的,并且在这种情况下会生成未定义的行为...您可以通过调用其隐式复制构造函数来为constBigLut使用动态初始化程序(假设boost::array是与std::array)相同的概念:

struct bigLut_tinit  {  
  bigLut_t BigLut; 

  bigLut_tinit() {
    for(int i = 0; i < 100000; ++i) {  
        BigLut[i] = i;  
    }
  }
};

const bigLut_tinit constBigLut;

编辑:似乎VC ++ 10完美地应用了RVO,因此临时将直接移动到静态持续时间对象中。所以imho不需要声明本地静态或对tempraries的引用......

编辑2:是的,我错过了尺寸问题。建议使用上面的构造函数包装成非平凡类型...

答案 2 :(得分:1)

它是一个UB,因为该数组可以存储在ROM中。

你可以这样做:

// test.h
#include <boost/array.hpp>

typedef boost::array<int,100000> bigLut_t;
const bigLut_t& Lut();


// test.cpp
#include "test.h"

bool initialized=false;

const bigLut_t& Lut()
{
  static bigLut_t lut;

  if (!initialized)
  {
    for(int i = 0; i < 100000; ++i) {
        lut.at(i) = i;
    }
  }
    return lut;
}