Thread: Max Line Size and Structs

Heres what I'm trying to do

for each item in N file:
for each item in M file:
see if this N list item is the same as this M list item

Now to do this I'm using this code and have a few problems :

Code:

void DupRemover( const string& dontcall)
{
	ifstream in("tested.all");
 	ofstream trial("newfile.all");
	ifstream dnc("dnc.all");
	ofstream nodup( "dup.all");
	string line;
	string number;
	set<string, compare> strSet;
	
	while (getline(in, line))	 
		{
                     strSet.insert(line);/*When I put this in the file trial>>"newfile.all"  it shortens the line to 64-65 characters which screws up my code from then on.  The line size is 501 characters per  row/*
                  }
	set<string, compare>::iterator tic;
copy(strSet.begin(),strSet.end(),ostream_iterator<string>(trial,"/n"));

while (getline(dnc,number,'\n'))//;tic != strSet.end())	
		{	  
		
		tic = find_if(strSet.begin(),strSet.end(),Search(number));
		if( tic != strSet.end() )
			{
			trial<<*tic <<endl;
			strSet.erase(tic);
                        }


        	}
	copy(strSet.begin(),strSet.end(),ostream_iterator<string>(nodup,"\n"));
}

Do you have any ideas, the program doesn't really out put anything and it seems to hang in spots. Is the strSet.erase actually erasing the data at position tic when the data matches up?

Thanks again for all your future and past help.

This is my adjusted Search code
This is what I think it should do
Tested is the main file the phone number is out on the 259th char on the row and the search uses the row(called number from dnc file)
to check if it is found in tested. In the dnc file the phone number is the first thing hence; a.compare(258,12,value,0,12)

Code:

struct Search : unary_function<string, bool>
{
    string value;
    Search(const string& val) : value (val) {}
    bool operator()(const string& a)
    {
        return a.compare(258,12,value,0,12) == 0;
    }
};

IN this section I want it to write to Nodup("dup.all") at the end without the numbers in dnc to be there. I kept code the way I had it for you to see what is wrong with it .

I get a row from dnc.all and put it in number which goes into search the if statement != it is suppose to write to trial and erase tic, then copy the set to Nodup. IT seems that the if statement is never true because nothing is ever sent to Trial. if I set the if to == the program errors out at runtime.

Code:

{
	ifstream in("tested.all");
 	ofstream trial("newfile.all");
	ifstream dnc("dnc.all");
	ofstream nodup( "dup.all");
	string line;
	string number;
	set<string, compare> strSet;
	
	while (getline(in, line))	 
		strSet.insert(line);
	set<string, compare>::iterator tic;
	while (getline(dnc,number,'\n'))
		{	  
		
		tic = find_if(strSet.begin(),strSet.end(),Search(number));
		if( tic != strSet.end() )
			{
			trial<<*tic <<endl;
			strSet.erase(tic);
			}
		}
	copy(strSet.begin(),strSet.end(),ostream_iterator<string>(nodup,"\n"));
}

IF you need anything else let me know I'll now got do some cout testing

Quick question why do I get a can not read error when I try to
cout<<*tic<<endl; I put this in between the find if and the if statement

Code:

tic = find_if(strSet.begin(),strSet.end(),Search(number));
	here
if( tic != strSet.end() )
			{
			trial<<*tic <<endl;
			strSet.erase(tic);
			}

Cool thanks for all of your help, your explanation really helped me to understand.
It seems to make sense now thinking about all this makes me think it has to be the lengths and offsets because I added dnc numbers into the file and it still doesn't pass that if so I'll mess with that for awhile.

Thanks again

I have a separate issue now... Before I reduce the main file of the duplicates and do not calls, I need to group the file by id key code. How would I remove the duplicate part of this program to just search for duplicates of the same string and put them in a file. My code so far for this is:

Code:

#include "stdafx.h"
#include <iostream>
#include <fstream>
#include <functional>
#include <set>
#include <string>
#include <algorithm>


using namespace std;


int i=0;


struct Search : unary_function<string, bool>
{
    string value;
    Search(const string& val) : value (val) {}
    bool operator()(const string& a)
    {
        
		return a.compare(17,10,value,0,4) == 0;
    }
};

struct compare: binary_function<string, string, bool> {
  bool operator()(const string& a, const string& b)
  {
	
	  return a.compare(18,10,b,18,10)>0;
	
  }
};




void Annotate ( const string& inSource2)
/* Searches given file with user entered keycodes and appends
to the user given file with the extension of .all*/


{
	
	string line;
	string filename="box";
	string keyCode;
	int knum=2;
	set<string,compare> strSet;
	set<string,compare>::iterator it;
	ifstream in(inSource2.c_str());
	
	cout<<"Using "<<inSource2<<" as master file."<<endl;
	
        //cout<< "Enter your new file name: ";
	//getline(cin,filename);          hard coded for testing
	
        filename+=".all";
	cout << "Your File Name is " << filename <<endl;
	
        //cout<<"Enter number of key codes: ";
	//cin>>knum;            Hard coded for testing
	
        cout<<"Thinking..."<<endl;
	
	
	
	ofstream out(filename.c_str());
	
	
	
	while (getline(in,line))
		strSet.insert(line);
	while (knum!=0)
	{
                cout<<"Enter a Key Code: ";
		getline(cin,keyCode);
		cin.ignore(1,'\n');
		it = find_if(strSet.begin(),strSet.end(),Search(keyCode));
		if (it != strSet.end())
			{
			
			}
		knum--;
	}	
	copy(strSet.begin(),strSet.end(),(out,
}



int main (void)
{
	int Groupnum;
	cout<<"How many files? ";
	cin>> (Groupnum);
	cout<<"Your number is "<<Groupnum<<endl<<endl;
	while (Groupnum !=0)//allows you to enter multiple files 
	{
	Annotate("tested.all");
	--Groupnum;
	}
	return(0);



}

My last program I didn't have a problem with the get line function after I fixed the string bug and using the fix in the stdafx.h file. But in this one it does some funky things. It will at times automatically press enter without allowing the user to enter the filename(when not hard coded) Program should allow you to pick multiple files to make and in each of those files to add multiple keycodes to such file. pulling from the tested.all file. I changed the offset but I need to get rid of the duplicate killer? Any suggestions...