2014-07-15 55 views
0

我正在开发一个QT窗口小部件的应用C++,用C的功能。我并不是说我在写C++,所以请避免说“这不是C++”。这是我的代码:拷贝计算器

#include <string.h> 
#include <stdio.h> 

---------------------------------------------------------- 

class MainWindow : public QMainWindow 
{ 

private: 

    char** NUM_ARRAY; 
    char* GUESS; 
    int  DIGIT_COUNT; 
    int  NUM_COUNT; 

    void fill_array(); 
    bool num_valid(char *NUM); 
}; 

---------------------------------------------------------- 

void MainWindow::fill_array() 
{ 
    int i, j = 0; 
    int START  = 1; 
    int END   = 9; 
    int CTR_START = 0; 
    int CTR_END  = 0; 

    NUM_COUNT = 1; 
    GUESS  = (char*)malloc(DIGIT_COUNT * sizeof(char)); 
    DIGIT_COUNT = ui->LE_DIGIT_COUNT->text().toInt(); 

    char* TEMP_NUM = (char*)malloc(DIGIT_COUNT * sizeof(char)); 

    for (i = 0; i < DIGIT_COUNT; i++) 
    { 
     NUM_COUNT *= END; 
     CTR_START = (CTR_START * 10) + (START++); 
     CTR_END = (CTR_END * 10) + (END--); 
    } 

    NUM_ARRAY = (char**)malloc(NUM_COUNT * sizeof(char*)); 

    for (i = CTR_START; i <= CTR_END; i++) 
    { 
     NUM_ARRAY[j] = (char*)malloc(DIGIT_COUNT * sizeof(char)); 

     sprintf(TEMP_NUM, "%d", i); 

     if (num_valid(TEMP_NUM)) 
      memcpy(NUM_ARRAY[j++], TEMP_NUM, DIGIT_COUNT * sizeof(char)); 
    } 

    memcpy(GUESS, NUM_ARRAY[0], DIGIT_COUNT * sizeof(char)); <----- ERROR 
    ui->L_GUESS->setText(GUESS); 
} 

我在做什么;填充由1234到9876的数字组成的字符串数组。这些数字不包括多于一次的任何0或同一个数字。我调试代码,一切都很好。我看不到NUM_ARRAY;它显示NUM_ARRAY "1234" char*,但我肯定数组填写正确,因为TEMP_NUMj显示正确的值。

在尖锐的线条上,发生了分段错误,我不知道为什么。我在这里错过了什么?请注意,我试过strcpy没有运气,并打开有关使用其他功能的建议。

+0

首先,如果您使用C++编程,为什么不使用'new'而不是'malloc'?或者更好的是,['std :: vector'](http://en.cppreference.com/w/cpp/container/vector)而不是处理你自己的内存? –

+0

感谢您的提示。 –

+1

当您为GUESS分配内存时,您正在使用未初始化的变量DIGIT_COUNT。这是对的吗? – sajas

回答

4

这可能是有问题的:

GUESS  = (char*)malloc(DIGIT_COUNT * sizeof(char)); 
DIGIT_COUNT = ui->LE_DIGIT_COUNT->text().toInt(); 

即:使用DIGIT_COUNT实际上初始化之前,所以它有一些垃圾数据。

另外:

http://qt-project.org/doc/qt-4.8/qstring.html#toInt提供了一种方法来实际检查,如果转换成功。使用它,它是为了您自己的利益。

+0

感谢您的回答 –

+0

不用费心给予额外的努力,我会接受您的答案,我只是在等待时间限制结束。 –

+0

我喜欢查看其他人的代码:P – fritzone

-2

我看不到您正在释放所使用的资源。

使用免费()...明智

例子..

free(TEMP_NUM); 
+0

感谢您的提示。 –

+4

虽然一个很好的评论,这实际上并不*回答* OP的问题。 –

+0

是的,我已经标记了它。 –