don't hide your experience in this ... plz

[mohammed zakaria]

hi all ,
i want to extract word by word in a text file to use this in other application so i wrote that code but there is some problems in the way these words apperes(ie. one line doesn't appear till it's end) and so on

the code is

#include <iostream>
#include <fstream>
#include <string.h>
using namespace std;
void main ()
{
char word[15];
system ("D:\\Program Files\\Microsoft Visual Studio\\sakr.txt");
char str[100];
ifstream in("sakr.txt");
if(!in)
cout<<"not found\n";
while(in.getline(str,100))
{
int count=0;
for(int i=0;i<100;i++)
{
if (str[i]!=' ')
{
word[count]=str[i];
count++;
}
else
{

for(int k=count;k<15;k++)
word[k]=0;
count=0;
cout<<word<<endl;
}

}
}

in.close();
}



can u figure out wats wronge ..... or suggest any other solution

thanx in advance

[Soneji]

Not sure if this will be any help, but I'll try...

What are you trying to accomplish with the 'system' call?
If it's to point to the text file, that won't work.

System is used to run a command in the console.
( For example: "cls" to clear the screen, or "exit" to exit the console. )

You can put the path to the file directly into the 'ifstream' constructor:

ifstream in("D:\\Program Files\\Microsoft Visual Studio\\sakr.txt");
 

Let's try something... Try this code, and see if it does what you are trying to get your code to do:

#include <iostream>
#include <string>
#include <fstream>
#include <vector>
using namespace std;
 
int main()
{  
  string input;
  vector<string> words;
 
  ifstream in( "D:\\Program Files\\Microsoft Visual Studio\\sakr.txt" );
 
  if( !in.is_open() )
  {
    cout << "Not found...\n";
    exit( 1 );
  }
 
  while( !in.eof() )
  {
    in >> input;
    words.push_back( input );
  }
 
  in.close();
  in.clear(); 
 
  for( int i = 0; i < words.size(); i++ )
  {
    cout << words[i] << '\n';
  }
// If you want all the words on one line with spaces, change the  line above to: cout << words[i] << ' ';
// If you want all the words on one line with NO spaces, change the line above to: cout << words[i];
 
  return 0;
}
 

( BTW: I added the 'clear' on line 28 because the 'while' loop above it will most likely set the 'fail', and 'bad' bits.
The 'clear' will reset to 'goodbit' so you can open the file again without hassle. )

If this is what you are trying to get out of your code, then I may be able to fix it.

I'm working on it now. I'm sure some of the experts here can fix it, if I can't.
Lord knows, they've helped me a few times! :)

I'm off to work on your original code. Let me know if this code does what you want.

[weaknessforcats]

You are making this too hard.

Use the >> operator. It skips whitespace.

Given this file:

Mary had a little lamb
It's fleece was white as snow

you extract the words with this code:

ifstream input("C:\\scratch\\instructor\\test.txt");
string str;
while (!input.eof())
{
  input >> str;
  cout << str << endl;
}
 

Part of the trouble here is writing in C when you should be writing in C++.

[Soneji]

I am making it too hard, or Mohammed is?

:)

I only added the vector because he wanted to take the words from a file, and use them in another application.
I figured the vector was an easy way to store those words.

I was going to use a few strings, but I think vectors are so cute. LOL.

Hmm... This leads into a question. Be back later, I need to post this question...

Lates!

-Soneji

[niskin]

char read[1024];
ifstream chat ( "C:\\file.txt" );
chat>> read;
chat.close();
 

Do what you want with read once you have put the text in there.

Or if you want to do it word by word then:

chat>> reada >> readb >> readc >> readd >> reade >> readf >> readg >> readh >> readi >> readj >> readk >> readl >> readm >> readn >> reado >> readp >> readq >> readr >> reads >> readt;
chat.close();
cout<< reada <<" " << readb <<" " <<  readc <<" " << readd <<" " << reade <<" " << readf <<" " << readg <<" " << readh <<" " << readi <<" " << readj <<" " << readk <<" " << readl <<" " << readm <<" " << readn <<" " << reado <<" " << readp <<" " << readq <<" " << readr <<" " << reads <<" " << readt;