значения массива считывается с файла, а повторные элементы этого массива выводятся во 2 файл C++

[carrie bradshaw]

привет, всем любителям программирования. помогите, пожалуйста, оценить программу (или возможно исправить её или предложить другой вариант). программа C++ (консольный режим работы)

задание:
написать программу в которой значения массива считывались с текстового файла, а повторные (одинаковые) элементы массива записать в другой файл.

код программы:

#pragma hdrstop
#pragma argsused
#include<iostream.h>
#include<fstream.h>
#include <tchar.h>
#include <stdio.h>
#include<conio.h>
int _tmain(int argc, _TCHAR* argv[])
{   std::ifstream ifs("d:/vzlom/login.txt");
	const int n = 4;
	int m[n], cnt = 0;

	for(int i = 0; i < 4; i++)
	{
            ifs >> m[i];
            std::cout << m[i] << ' ';
    }
	ifs.close();

    for(int i = 0; i < 4; i++)
    {
		for(int j  =0; j < i; j++)
        {
			if(m[i] == m[j])
            {
				cnt++;
            }
		}
	}

	std::ofstream ofs("d:/vzlom/login.txt");

	ofs << cnt;
	getch();
	return 0;
}

[Stilet]

Двойка.
Во-первых: визуальный режим работы - я тебя расстрою, эта программа консольная.
Во-вторых: const int n = 4; - тоже не порядок. Нужен динамический массив, или даже список, который будет наполняться из файла числами.
Третье: Для ускорения работы программы лучше было бы проверять наличие числа сразу при считке данных. Т.е.

	for(int i = 0; i < 4; i++)
	{
            ifs >> m[i];
            for(int j=i;j>0;j--) if(m[i]==m[j]) {ofs << m[i]; break;}
            std::cout << m[i] << ' ';
         }

И опять таки тут у меня нет проверки на повтор повтора, что тоже по хорошему должно быть учтено.
Короче твоя стратегия скорее провальна чем реальна.

[carrie bradshaw]

программа выдаёт ошибку
код программы:

#pragma hdrstop
#pragma argsused
#include<iostream.h>
#include<fstream.h>
#include <tchar.h>
#include <stdio.h>
#include<conio.h>
int _tmain(int argc, _TCHAR* argv[])
{   std::ifstream ifs("d:/vzlom/login.txt");
	const int n = 4;
	int m[n], cnt = 0;
    for(int i = 0; i < 4; i++)
	{
			ifs >> m[i];
			for(int j=i;j>0;j--) if(m[i]==m[j]) {ofs << m[i]; break;}
			std::cout << m[i] << ' ';

	{
	ifs >> m[i];
			std::cout << m[i] << ' ';
	}
	ifs.close();

    for(int i = 0; i < 4; i++)
	{
		for(int j  =0; j < i; j++)
		{
			if(m[i] == m[j])
            {
				cnt++;
			}
		}
	}

	std::ofstream ofs("d:/vzlom/login.txt");

	ofs << cnt;
	getch();
	return 0;
}

ошибка в строчке:

for(int j=i;j>0;j--) if(m[i]==m[j]) {ofs << m[i]; break;}

помогите, пожалуйста, убрать ошибку

[carrie bradshaw]

программа работает правильно. спасибо тому, кто написал совет, а также всем тем, кто читал мой пост.

помогите, пожалуйста, написать эту программу в визуальном режиме с помощью вкладки Dialogs.

[Streletz]

с помощью вкладки Dialogs

Если я правильно понял, речь здесь идёт о C++ Builder. В нём эта вкладка предназначена для того, чтобы программа могла использовать стандартные диалоговые окна, например диалоги открытия и сохранения файла. Создайте форму и в ней реализуйте алгоритм.

помогите, пожалуйста, написать эту программу в визуальном режиме

Возможно, я Вас удивлю, но данный алгоритм в визуальном режиме практически ни чем не будет отличатся от своего консольного варианта. Отличия и общий принцип реализации в визуальном режиме можно прочитать, например, здесь: Вывести в порядке возрастания все целые числа которые находятся между "А" и "В" С++ (визуальный режим).

[Stilet]

написать эту программу в визуальном режиме

У меня Билдера нет, соболезную.