Запись из одного текстового файла в другой не работает. Что делать?

Question

[Протопоп Метапоп]

По идее, программа должна посимвольно сравнивать содержание двух текстовых файлов (txt1 и txt2) и одинаковые элементы записывать в третий текстовый файл(txt3).

#include <iostream>
#include <fstream>
#include <string>
int main(){
//переменные, отвечающие за посимвольный перебор//
    char sim1, sim2,n;
//строки, на которые мы ориентируемся//
    std::string line;
    std::string line2;
//открытие двух материнских файлов для чтения//
    std::ifstream txt1("txt.txt");
    std::ifstream txt2("txt2.txt");
    if (txt1.is_open())
    {
        if (txt2.is_open())
            {
            std::cout<<"Файлы txt1 и txt2 открыты"<<std::endl;
//посимвольно сравниваем два файла//
            while (txt1.get(sim1),feof)
                while (txt2.get(sim2),feof)
//одинаковые элементы записываем в третий//
                    if (sim1==sim2) {
                        std::ofstream txt3("txt3.txt");
                        std::cout<<sim1;
                        txt3.close();}              
            }
    }
//закрываем файлы//
    txt1.close();
    txt2.close();
    std::cout << "End of program" << std::endl;
    return 0;

};

Программа выводит только строчку что оба файла открыты. Что я делаю не так?

Comments

[Dmitry]

while (txt1.get(sim1),feof)
                while (txt2.get(sim2),feof)

Для чего здесь нужны вложенные циклы?

[Протопоп Метапоп]

Dmitry, мы прогоняем один файл и параллельно прогоняем =ещё и второй.

[jcmvbkbc]

мы прогоняем один файл и параллельно прогоняем =ещё и второй.

Протопоп Метапоп, неа, не параллельно. Эта конструкция прогоняет весь второй файл для первого символа первого файла. И пытается прогонять весь второй файл для второго символа первого файла, но второй файл на этом месте уже кончился и не был открыт заново.

Answer 1

[Иван Соломенников]

#include <iostream>
#include <fstream>

int main() {

    // переменные, отвечающие за посимвольный перебор
    char sim1, sim2, n;

    // abbreviation: not identical
    bool ni = false;

    // открытие двух материнских файлов для чтения
    std::ifstream txt1("txt1.txt");
    std::ifstream txt2("txt2.txt");

    // открытие файла для записи
    std::ofstream txt3("txt3.txt");

    if (!txt1.is_open() || !txt2.is_open()) {
        std::cout << "I can't open a file for reading" << std::endl;

        return 1;
    }

    if(!txt3.is_open()) {
        std::cout << "I can't open a file for writing" << std::endl;

        return 1;
    }

    std::cout << "txt1 and txt2 files are open" << std::endl;

    // посимвольное сравниваем два файла
    while(!txt1.eof() && !txt2.eof()) {

        txt1.get(sim1);
        txt2.get(sim2);

        if(sim1 == sim2) {
            txt3.put(sim1);
        } else {
            ni = true;
        }
    }

    if(!ni && txt1.eof() && txt2.eof()) {
        // сообщение о успехе
        std::cout << "The file contents are identical" << std::endl;
    } else {
		// Если посимвольное сравнение не увенчалось успехом
        std::cout << "The contents of the files are not identical" << std::endl;
	}

    // закрываем файлы
    txt1.close();
    txt2.close();
    txt3.close();

    std::cout << "End of program" << std::endl;

    return 0;
}

Comments

[Протопоп Метапоп]

Как я понимаю, программа будет выполняться до первого НЕ одинакового символа.
А как сделать так чтобы она такие пропускала и работала дальше?

[Иван Соломенников]

Протопоп Метапоп, Если более подробно объясните что нужно именно. Правильно понимаю, что если, символы не равны, то все равно нужно писать в выходной файл из первого открытого файла("txt1.txt")? И какое необходимо поведение, если допустим, первый файл прочитан, а второй ещё нет?!

[Протопоп Метапоп]

Иван Соломенников, если символы на одинаковых позициях не равны- в txt3 они не записываются. Если хотя бы один из файлов не открыт программа должна завершиться(т.к недостаточно данных для сравнения).

[Иван Соломенников]

Протопоп Метапоп, Поправил код, работает следующим образом: если символы не равны, чтение из файлов продолжается до тех пор пока один из файлов не будет прочитан.

Answer 2

[Даниил Демидко]

Что я делаю не так?

Не читаете документацию к используемым абстракциям.
У вас в цикле, на каждой итерации вот здесь затирается и заново открывается на запись файл txt3.txt

if (sim1==sim2) {
  std::ofstream txt3("txt3.txt");
  std::cout<<sim1;
  txt3.close();}              
}

Вам нужно один раз открыть его там же где открываете файлы на чтение. По умолчанию при создании std::ofstream, стирается все что есть в файле и новые данные заново записываются.

Comments

[Протопоп Метапоп]

Исправил, но программа не заработала.
В третий файл ничего не записывается(

#include <iostream>
#include <fstream>
#include <string>
int main(){
    int i,j;
    char sim1, sim2,n;
    std::string line;
    std::string line2;
    std::ifstream txt1("txt.txt");
    std::ifstream txt2("txt2.txt");
    std::ofstream txt3("txt3.txt");
    if (txt1.is_open())
    {
        if (txt2.is_open())
            {
            std::cout<<"Файлы txt1 и txt2 открыты"<<std::endl; 

            txt1.seekg(0,std::ios::end);
            txt2.seekg(0,std::ios::end);

            long len1 = txt1.tellg();
            long len2 = txt2.tellg();
            char *buf1 = new char [len1 +1];
            char *buf2 = new char [len2 +1];
            std::ofstream("/txt3.txt");
            if (buf1[i]==buf2[i])
             std::cout<<buf1[i];
            }
    }

    txt1.close();
    txt2.close();
    txt3.close();

    std::cout << "End of program" << std::endl;
    return 0;

};

[Александр Ананьев]

Протопоп Метапоп, наверно нужен цикл потпеременной i.

[Антон Жилин]

char *buf1 = new char [len1 +1];
char *buf2 = new char [len2 +1];
std::ofstream("/txt3.txt");
if (buf1[i]==buf2[i]) std::cout<<buf1[i];

Во-первых — ААА! — выкинь char*, new и замени на std::string. Во-вторых, buf1 и buf2 никак не заполняются.