2013-05-15 72 views
0

我很惊讶,在C++中没有内置的方法来投射字符串和字符。这是我的问题:我要更新一个文本文件,这样一行:字符串和字符不兼容

#include <stdlib.h> 
#include <fstream> 
#include <windows.h> 
#include <iostream> 
#include <stdio.h> 

using namespace std; 

int main(void) { 

    char l1 [12]; 
    char l2 [15]; 
    char l3 [15]; 
    char l4 [15]; 
    char md [16]; 

    FILE * myfile2 = fopen("state.txt", "r+"); 
    fgets(l1,12,myfile2); 
    fgets(l2,12,myfile2); 
    fgets(l3,12,myfile2); 
    fgets(l4,12,myfile2); 
    fseek (myfile2 , 0 , SEEK_SET); 

    strcpy(md , "detectoroff"); 
    fputs (md,myfile2); 

    strcpy(md,l2); 
    fputs (md,myfile2); 
    strcpy(md,l3); 
    fputs (md,myfile2); 
    strcpy(md,l4); 
    fputs (md,myfile2); 
    Sleep(1000); 
    fclose(myfile2); 
    return EXIT_SUCCESS; 
} 

Inicially文本文件具有

a 
    b 
    c 
    d 

时运行的代码的输出是

detectoroffb 
    c 
    d 

和如果我更换线

strcpy(md , "detectoroff"); 

这个

strcpy(md , "detectoroff\n"); 

现在的结果是正确的,但如果我运行此两次,将出现一个空行下的第一个,另一个,如果是跑三圈等我怎样才能解决这个问题?

+5

无关使用C++和C++ 11(删除此标记)。这个代码是纯粹的C,除了无用的C++包含。 –

+4

这与铸造字符串和字符有什么关系?它与'std :: string'有什么关系? – Barmar

+0

也许我应该使用“转换”而不是“转换”这个词,将字符串转换为字符是我进一步需要的。 – JoeCoolman

回答

0

不要害怕使用几千字节。

#include <cstdlib> 
#include <cstdio> 
int main(void) { 
    using namespace std; 
    static const int BUFFERSIZE = 4096; 
    char line_1[BUFFERSIZE]; 
    char line_2[BUFFERSIZE]; 
    char line_3[BUFFERSIZE]; 
    char line_4[BUFFERSIZE]; 
    char md [] = "detectoroff\n"; 

    FILE * myfile2 = fopen("state.txt", "r+"); 

    fgets(line_1, BUFFERSIZE, myfile2); 
    fgets(line_2, BUFFERSIZE, myfile2); 
    fgets(line_3, BUFFERSIZE, myfile2); 
    fgets(line_4, BUFFERSIZE, myfile2); 

    fseek(myfile2, 0, SEEK_SET); 

    fputs(md, myfile2); 
    fputs(line_2, myfile2); 
    fputs(line_3, myfile2); 
    fputs(line_4, myfile2); 

    fclose(myfile2); 
    return EXIT_SUCCESS; 
} 

更妙的是用C编写这个程序,而不是++与C风格的:

#include <fstream> 
#include <string> 
int main(void) { 
    using namespace std; 
    string line_1, line_2, line_3, line_4; 
    string md("detectoroff"); 
    fstream myfile2("state.txt", fstream::in|fstream::out); 

    getline(myfile2, line_1); 
    getline(myfile2, line_2); 
    getline(myfile2, line_3); 
    getline(myfile2, line_4); 

    myfile2.clear(); 
    myfile2.seekg(0, ios::beg); 

    myfile2 
    << md << '\n' 
    << line_2 << '\n' 
    << line_3 << '\n' 
    << line_4 << '\n'; 
    myfile2.close(); 
    return 0; 
} 
+0

这工作得很好@Hal Canary!,与千字节没有问题,但为什么要定义line_1,2,3,4的长度过长呢?谢谢。 – JoeCoolman

3

变化l1的声明:

char l1 [13]; 

detectoroff是11个字符。您需要另一个字符作为换行符,另一个字符表示字符串结尾的空字节,总共为13个字符。由于您只声明l1有12个字符,因此fgets()正在写入数组边界之外。这导致未定义的行为。

使用std::string和C++流方法,你不会遇到这样的问题。

+0

未经测试的样本:http://ideone.com/s7CpX8 –

+0

对于Sleep()的抱歉,我没有将它从原始代码中删除。 – JoeCoolman

+0

我没有对Sleep()做任何说明,这是Mooing Duck样本中的一条评论。 – Barmar