🚀 go-pugleaf

Dit is vreemde code. Heb je dit geschreven om
je systeem zwaar te belasten (voor testen) ?

> void main ()
> {
> //************declaraties***********************
>  string tempstring, te_zoeken,stringbuffer;
>  char bestandsnaam [75],schrijfnaar[75],x,buffer [10000],inlezen;
>  int aantal=0,atl_lijn_teller=0;
>  long tijd;
>  FILE * inleesbestand;
>  FILE * resultaatbestand;
> 
>  cout<<"Geef de directory en de bestandsnaam in van het bestand: ";
>  cin>>bestandsnaam;
> 
> //***********inputcontrole***********************
>  while ((inleesbestand = fopen (bestandsnaam, "r")) == NULL)
>    {
>         cout<<"Sorry, het opgegeven bestand kan niet gevonden worden"<<endl;
>      cout<<"Geef opnieuw je bestandsnaam op: " ;
>      cin>>bestandsnaam;
> 
>    }
> 
>  cout<<"Geef op waarnaar het programma moet zoeken: \n";
>  getline(cin,te_zoeken);
>  cout<<"naar welk bestand moet geschreven worden?: \n";
>  cin>>schrijfnaar;
> 
> //**************start tijdsopname na correcte invoer***********************
>  tijd=time(0);
>  cout<<"Bezig met verwerken van het bestand..."<<endl;
> 
> //*************verwerking van het aantal karakters***********************
>  inleesbestand = fopen (bestandsnaam, "r");
>  for ( ;getc (inleesbestand) != EOF; aantal++);
>  fclose (inleesbestand);

Je zou hier ook al eens de tijd moeten afprinten, dan weet je
beter welke operatie het langste duurt.
Je kan trouwens beter gewoon de filesize opvragen om 
het aantal karakters te kennen ...

> 
> //*************opzoeken van de string**********************************
>  int teller=0;
>  inleesbestand = fopen (bestandsnaam, "r");
>  FILE *output= fopen(schrijfnaar, "w");
>  char zinneke[1000];
> 
>  while (fgets(zinneke,10000, inleesbestand))

Bugje: dit moet fgets(zinneke,1000, inleesbestand)
       zijn.

>  { string stringzinneke;
>    atl_lijn_teller++;           //verwerking atl lijnen
>   for(int x=0; x<1000 && zinneke[x]!='\n';x++)
>   {
>    stringzinneke= stringzinneke+ zinneke[x];
>   }

Zeer inefficiente code, die tot gevol heeft dat 
er zeer veel memory allocaties moeten gebeuren.


> 
>  if(stringzinneke.find(te_zoeken) != -1)
>    {
>     stringzinneke.append(1,'\n');
>     fwrite(zinneke,stringzinneke.length(),1,output);
>     teller++;
>    }
>  }

Ipv String.find() te gebruiken, kan je veel beter
simpelweg strstr(zinneke, te_zoeken) gebruiken,
dan wordt alles een stuk eenvoudiger:

  while (fgets(zinneke,1000, inleesbestand))
  {
    const char* ptr = strstr(zinneke, te_zoeken);
    if (ptr)
    {
	// resulaten wegschrijven
	// Ik weet niet juist wat jouw string class
	// doet, maar ik zou die code toch ook maar
	// eens nakijken.
	... 
    }
  }

>  fclose(inleesbestand);
>  fclose(output);
> 
> //****************uitvoer van de resultaten****************************
>  cout<< "Bestand verwerkt in "<<time(0)-tijd<<" seconden \n";
>  cout<<"Het bestand "<<bestandsnaam<<" bevat "<<aantal<<" karakters en "<<atl_lijn_teller<<" lijnen"<<endl;
>  cout<<te_zoeken<<" komt in "<<teller<<" zinnen voor"<<endl;
> 

Probeer het eens met deze veranderingen. Volgens mij
zal het op windows EN op linux veel sneller gaan.

Ronny