Problems parsing a file

[Eric Lilja]

Hello, I'm creating a small utility for an online game. It involves
parsing a text file of "tradesskil l recipes" and inserting these
recipes in a gui tree widget (similar to gui file browsers if you know
what I mean).
Here's an example of a recipe as it appears in the text file:
* Cashew Pie (lvl 39, 5h 3 min, + max power)
- Candied Cashew (level 30)
-- Cashew
-- Refined Honey (level 30)
--- Raw Honey
--- Liquid
- Dough
- Refined Honey (level 30)
-- Raw Honey
-- Liquid

* denotes new recipe (a new root node in the tree)
-/--/--- denotes the depth of the child node.
One node per line.

The gui code is out of scope for the ng, but the parser is written in
standard C++, and I'm having some problems with it. The code:
#include <cassert>
#include <cctype>
#include <cstdlib>
#include <fstream>
#include <iostream>
#include <stdexcept>
#include <string>

using std::cerr;
using std::cout;
using std::endl;
using std::ifstream;
using std::isspace;
using std::runtime_er ror;
using std::string;

enum depth_t { NO_DEPTH = 0, FIRST, SECOND, THIRD };

static depth_t get_depth(const string &);

int
main()
{
ifstream food_recipes_fi le("recipes_foo d.txt");

if(!food_recipe s_file)
{
cerr << "Failed to open food recipe file." << endl;

return EXIT_FAILURE;
}

string line;
string recipe_root_nod e;
depth_t previous_depth = NO_DEPTH;

while(getline(f ood_recipes_fil e, line))
{
if(line[0] == '*')
{
assert(line[1] == ' ');

recipe_root_nod e = line.substr(2, line.length());
cout << "Inserting new recipe " << line << endl;

previous_depth = NO_DEPTH;
}
else if(isspace(line[0]))
; /* Simply skip lines beginning with a blank. */
else /* We're not dealing with a new recipe */
{
assert(line[0] == '-');

depth_t depth = get_depth(line) ;

cout << line << " at depth " << endl;
}
}

food_recipes_fi le.close();

return EXIT_SUCCESS;
}

/* get_depth() assumes the first char in line is a - */
static depth_t
get_depth(const string& line)
{
if(line[1] == ' ')
return FIRST;

/* If the second char is not a ' ', it must be a '-'. */
assert(line[1] == '-');

if(line[2] == ' ')
return SECOND;

/* If the third char is not a ' ', it must be a '-'. */
assert(line[2] == '-');

if(line[3] == ' ')
return THIRD;

throw runtime_error(" Couldn't identify depth!");
}

This compiles without warning on my compiler (with warning level set to
max, all extension turned off), but the output is really messed up:
$ ./create_tree.exe
Inserting new recipe * Cashew Pie (lvl 39, 5h 3 min, + max power)
at depth Cashew (level 30)
at depth
at depth Honey (level 30)
at depth ney
at depth
at depth
at depth Honey (level 30)
at depth ey
at depth

Any obvious errors?

Thanks for any replies!

/ E

[Eric Lilja]

Eric Lilja wrote:
[snip]

Well, turns out the recipe file was in dos style line breaks and the
executable was expecting unix style line breaks. I converted the text
file to unix style line breaks and everything works. But I might be
back with further questions. =)

/ E

[Mike Wahler]

"Eric Lilja" <mi********@gma il.com> wrote in message
news:11******** **************@ f14g2000cwb.goo glegroups.com.. .

Hello, I'm creating a small utility for an online game. It involves
parsing a text file of "tradesskil l recipes" and inserting these
recipes in a gui tree widget (similar to gui file browsers if you know
what I mean).
Here's an example of a recipe as it appears in the text file:
* Cashew Pie (lvl 39, 5h 3 min, + max power)
- Candied Cashew (level 30)
-- Cashew
-- Refined Honey (level 30)
--- Raw Honey
--- Liquid
- Dough
- Refined Honey (level 30)
-- Raw Honey
-- Liquid

* denotes new recipe (a new root node in the tree)
-/--/--- denotes the depth of the child node.
One node per line.

The gui code is out of scope for the ng, but the parser is written in
standard C++, and I'm having some problems with it. The code:
#include <cassert>
#include <cctype>
#include <cstdlib>
#include <fstream>
#include <iostream>
#include <stdexcept>
#include <string>

using std::cerr;
using std::cout;
using std::endl;
using std::ifstream;
using std::isspace;
using std::runtime_er ror;
using std::string;

enum depth_t { NO_DEPTH = 0, FIRST, SECOND, THIRD };

static depth_t get_depth(const string &);

int
main()
{
ifstream food_recipes_fi le("recipes_foo d.txt");

if(!food_recipe s_file)
{
cerr << "Failed to open food recipe file." << endl;

return EXIT_FAILURE;
}

string line;
string recipe_root_nod e;
depth_t previous_depth = NO_DEPTH;

while(getline(f ood_recipes_fil e, line))
{
if(line[0] == '*')
{
assert(line[1] == ' ');

recipe_root_nod e = line.substr(2, line.length());
cout << "Inserting new recipe " << line << endl;

previous_depth = NO_DEPTH;
}
else if(isspace(line[0]))
; /* Simply skip lines beginning with a blank. */
else /* We're not dealing with a new recipe */
{
assert(line[0] == '-');

depth_t depth = get_depth(line) ;

cout << line << " at depth " << endl;
}
}

food_recipes_fi le.close();

return EXIT_SUCCESS;
}

/* get_depth() assumes the first char in line is a - */
static depth_t
get_depth(const string& line)
{
if(line[1] == ' ')
return FIRST;

/* If the second char is not a ' ', it must be a '-'. */
assert(line[1] == '-');

if(line[2] == ' ')
return SECOND;

/* If the third char is not a ' ', it must be a '-'. */
assert(line[2] == '-');

if(line[3] == ' ')
return THIRD;

throw runtime_error(" Couldn't identify depth!");
}

This compiles without warning on my compiler (with warning level set to
max, all extension turned off), but the output is really messed up:
$ ./create_tree.exe
Inserting new recipe * Cashew Pie (lvl 39, 5h 3 min, + max power)
at depth Cashew (level 30)
at depth
at depth Honey (level 30)
at depth ney
at depth
at depth
at depth Honey (level 30)
at depth ey
at depth

Any obvious errors?

Thanks for any replies!

After fixing the one error that caused your code to
fail to compile:

using std::getline;

and creating an input file with your sample data,
it compiled and linked OK for me (MSVC 6.0 SP6),
and gave the following output:

Inserting new recipe * Cashew Pie (lvl 39, 5h 3 min, + max power)
- Candied Cashew (level 30) at depth
-- Cashew at depth
-- Refined Honey (level 30) at depth
--- Raw Honey at depth
--- Liquid at depth
- Dough at depth
- Refined Honey (level 30) at depth
-- Raw Honey at depth
-- Liquid at depth
(I don't know if it's part of what you called 'messed up',
but the reason there's nothing after 'at depth' is because
the code makes no attempt to output that value.)

-Mike

[Eric Lilja]

Mike Wahler wrote:

"Eric Lilja" <mi********@gma il.com> wrote in message
news:11******** **************@ f14g2000cwb.goo glegroups.com.. .

Hello, I'm creating a small utility for an online game. It involves
parsing a text file of "tradesskil l recipes" and inserting these
recipes in a gui tree widget (similar to gui file browsers if you know
what I mean).
Here's an example of a recipe as it appears in the text file:
* Cashew Pie (lvl 39, 5h 3 min, + max power)
- Candied Cashew (level 30)
-- Cashew
-- Refined Honey (level 30)
--- Raw Honey
--- Liquid
- Dough
- Refined Honey (level 30)
-- Raw Honey
-- Liquid

* denotes new recipe (a new root node in the tree)
-/--/--- denotes the depth of the child node.
One node per line.

The gui code is out of scope for the ng, but the parser is written in
standard C++, and I'm having some problems with it. The code:
#include <cassert>
#include <cctype>
#include <cstdlib>
#include <fstream>
#include <iostream>
#include <stdexcept>
#include <string>

using std::cerr;
using std::cout;
using std::endl;
using std::ifstream;
using std::isspace;
using std::runtime_er ror;
using std::string;

enum depth_t { NO_DEPTH = 0, FIRST, SECOND, THIRD };

static depth_t get_depth(const string &);

int
main()
{
ifstream food_recipes_fi le("recipes_foo d.txt");

if(!food_recipe s_file)
{
cerr << "Failed to open food recipe file." << endl;

return EXIT_FAILURE;
}

string line;
string recipe_root_nod e;
depth_t previous_depth = NO_DEPTH;

while(getline(f ood_recipes_fil e, line))
{
if(line[0] == '*')
{
assert(line[1] == ' ');

recipe_root_nod e = line.substr(2, line.length());
cout << "Inserting new recipe " << line << endl;

previous_depth = NO_DEPTH;
}
else if(isspace(line[0]))
; /* Simply skip lines beginning with a blank. */
else /* We're not dealing with a new recipe */
{
assert(line[0] == '-');

depth_t depth = get_depth(line) ;

cout << line << " at depth " << endl;
}
}

food_recipes_fi le.close();

return EXIT_SUCCESS;
}

/* get_depth() assumes the first char in line is a - */
static depth_t
get_depth(const string& line)
{
if(line[1] == ' ')
return FIRST;

/* If the second char is not a ' ', it must be a '-'. */
assert(line[1] == '-');

if(line[2] == ' ')
return SECOND;

/* If the third char is not a ' ', it must be a '-'. */
assert(line[2] == '-');

if(line[3] == ' ')
return THIRD;

throw runtime_error(" Couldn't identify depth!");
}

This compiles without warning on my compiler (with warning level set to
max, all extension turned off), but the output is really messed up:
$ ./create_tree.exe
Inserting new recipe * Cashew Pie (lvl 39, 5h 3 min, + max power)
at depth Cashew (level 30)
at depth
at depth Honey (level 30)
at depth ney
at depth
at depth
at depth Honey (level 30)
at depth ey
at depth

Any obvious errors?

Thanks for any replies!
After fixing the one error that caused your code to
fail to compile:

using std::getline;

I was under the impression that I didn't have to qualify getline() in
this case because of Koenig lookup?

and creating an input file with your sample data,
it compiled and linked OK for me (MSVC 6.0 SP6),
and gave the following output:

Inserting new recipe * Cashew Pie (lvl 39, 5h 3 min, + max power)
- Candied Cashew (level 30) at depth
-- Cashew at depth
-- Refined Honey (level 30) at depth
--- Raw Honey at depth
--- Liquid at depth
- Dough at depth
- Refined Honey (level 30) at depth
-- Raw Honey at depth
-- Liquid at depth
(I don't know if it's part of what you called 'messed up',
but the reason there's nothing after 'at depth' is because
the code makes no attempt to output that value.)
Thanks for taking the time to test this, Mike, but did you read my
reply to myself? It was a silly line-break mess-up. =( I removed the
output of the depth variable when trying to track down the error.

-Mike

/ E