Connect++ 0.5.0
A fast, readable connection prover for first-order logic.
Loading...
Searching...
No Matches
TermIndex Class Reference

Look after terms, (ideally) using hash consing to avoid storing copies of terms. More...

#include <TermIndex.hpp>

Collaboration diagram for TermIndex:

Public Member Functions

 TermIndex (const TermIndex &)=delete
 Don't allow copying as this is a terrible idea.
 
 TermIndex (const TermIndex &&)=delete
 
TermIndexoperator= (const TermIndex &)=delete
 
TermIndexoperator= (const TermIndex &&)=delete
 
size_t get_size () const
 Basic get method.
 
Termadd_variable_term (Variable *)
 Self-explanatory: add a Term containing a variable to the index.
 
Termadd_function_term (Function *, const vector< Term * > &)
 Self-explanatory: add a Term containing a function to the index.
 
Termreplace_variable_in_term (Variable *, Variable *, Term *)
 Replace a variable in a term with an alternative, maintaining the structure of the TermIndex.
 
Termreplace_variable_in_term_with_term (Term *, Variable *, Term *)
 Minor variation on replace_variable_in_term.
 

Private Member Functions

Termfind (const Term &)
 Find a term in the index.
 

Private Attributes

vector< Term * > term_pointers
 Hash table for storing pointers to where Terms are actually stored.
 

Friends

ostream & operator<< (ostream &, const TermIndex &)
 

Detailed Description

Look after terms, (ideally) using hash consing to avoid storing copies of terms.

Everything is implemented such that this ignores whether or not Variables are substituted. (Because to do otherwise isn't necessary.)

At present this is the best of two worlds. Unfortunately, use of unordered_map with Term and proper hash consing can result in strange errors, on some compilers but not others, and I'm not too sure where they come from, although there seems to be a problem with memory allocation/deallocation. Using the unordered_map means allowing various kinds of copying of Terms, which I'd rather not do, and that may be the place to look.

Despite this, early indications are that it's definitely worth using the full-blown hash table. If you allow HASHCONSTERMS to be defined in BasicTypes.hpp you will compile with full hash consing. Otherwise you'll get the alternative.

The alternative version has the same behaviour but is likely to be a little less efficient. The fact that it works just fine on every compiler I've tried suggests that something very obscure might be happening in the bowels of the hashed version to make memory errors show up. TODO...

You should only use this to make Terms. As long as you do that then this class takes all the responsibility for memory allocation and deallocation for Terms.

Definition at line 75 of file TermIndex.hpp.

Constructor & Destructor Documentation

◆ TermIndex() [1/2]

TermIndex::TermIndex ( )
inline

Definition at line 105 of file TermIndex.hpp.

105: term_pointers() {}
vector< Term * > term_pointers
Hash table for storing pointers to where Terms are actually stored.
Definition TermIndex.hpp:93

◆ ~TermIndex()

TermIndex::~TermIndex ( )

Definition at line 51 of file TermIndex.cpp.

51 {
52 for (size_t i = 0; i < term_pointers.size(); i++)
53 delete term_pointers[i];
54}

◆ TermIndex() [2/2]

TermIndex::TermIndex ( const TermIndex & )
delete

Don't allow copying as this is a terrible idea.

As usual, let the compiler be your friend.

Member Function Documentation

◆ add_function_term()

Term * TermIndex::add_function_term ( Function * fp,
const vector< Term * > & args )

Self-explanatory: add a Term containing a function to the index.

It's only actually added if it's not already present. If present, a pointer to the existing copy is returned.

Parameters
fpPointer to a Function for which an appropriate Term is to be added
argsVector of pointers to Terms constituting the arguments for the new function Term to be added.

Definition at line 73 of file TermIndex.cpp.

74 {
75 for (size_t i = 0; i < args.size(); i++) {
76 if (args[i]->is_subbed()) {
77 cerr << "Don't add substituted functions to the Term Index!" << endl;
78 }
79 }
80 Term t(fp, args);
81 Term* p = find(t);
82 if (p != nullptr) {
83 return p;
84 }
85 p = new Term(fp, args);
86#ifdef HASHCONSTERMS
87 index.insert(pair<Term, Term*>(t, p));
88#endif
89 term_pointers.push_back(p);
90 return p;
91}
General representation of terms.
Definition Term.hpp:62
Term * find(const Term &)
Find a term in the index.
Definition TermIndex.cpp:40

◆ add_variable_term()

Term * TermIndex::add_variable_term ( Variable * vp)

Self-explanatory: add a Term containing a variable to the index.

It's only actually added if it's not already present. If present, a pointer to the existing copy is returned.

Parameters
vpPointer to Variable for which an equivalent Term is to be added

Definition at line 56 of file TermIndex.cpp.

56 {
57 if (vp->is_subbed()) {
58 cerr << "Don't add substituted variables to the Term Index!" << endl;
59 }
60 Term t(vp);
61 Term* p = find(t);
62 if (p != nullptr) {
63 return p;
64 }
65 p = new Term(vp);
66#ifdef HASHCONSTERMS
67 index.insert(pair<Term, Term*>(t, p));
68#endif
69 term_pointers.push_back(p);
70 return p;
71}
bool is_subbed() const
Self-explanatory.
Definition Variable.hpp:111

◆ find()

Term * TermIndex::find ( const Term & t)
private

Find a term in the index.

Parameters
tThe term you want to look up.

Definition at line 40 of file TermIndex.cpp.

40 {
41 for (size_t i = 0; i < term_pointers.size(); i++) {
42 if (*(term_pointers[i]) == t) {
43 return term_pointers[i];
44 }
45 }
46 return nullptr;
47}

◆ get_size()

size_t TermIndex::get_size ( ) const
inline

Basic get method.

Definition at line 121 of file TermIndex.hpp.

121{ return term_pointers.size(); }

◆ replace_variable_in_term()

Term * TermIndex::replace_variable_in_term ( Variable * new_v,
Variable * old_v,
Term * t )

Replace a variable in a term with an alternative, maintaining the structure of the TermIndex.

The first two arguments are variables and the third a general term. Replace one variable with another while keeping the structure of the index correct. Replacement replaces any substitutions for the variable being replaced with those applied to the new one. You should probably not do that as ideally the index should only contain unsubstituted variables.

Parameters
new_vPointer to a Variable to use as replacement.
old_vPointer to a Variable to be replaced.
tPointer to Term to make the replacement for. This should already be in the index.

Definition at line 93 of file TermIndex.cpp.

95 {
96 if (t->is_variable()) {
97 if (old_v == t->get_v()) {
98 return add_variable_term(new_v);
99 }
100 else
101 return t;
102 }
103 else {
104 Function* f = t->get_f();
105 vector<Term*> new_args;
106 for (size_t i = 0; i < t->arity(); i++)
107 new_args.push_back(replace_variable_in_term(new_v, old_v, (*t)[i]));
108 return add_function_term(f, new_args);
109 }
110}
Basic representation of functions.
Definition Function.hpp:54
Variable * get_v() const
Self-explanatory access function.
Definition Term.hpp:106
Function * get_f() const
Self-explanatory access function.
Definition Term.hpp:110
bool is_variable() const
Self-explanatory.
Definition Term.hpp:114
Arity arity() const
Self-explanatory access function.
Definition Term.hpp:122
Term * add_function_term(Function *, const vector< Term * > &)
Self-explanatory: add a Term containing a function to the index.
Definition TermIndex.cpp:73
Term * add_variable_term(Variable *)
Self-explanatory: add a Term containing a variable to the index.
Definition TermIndex.cpp:56
Term * replace_variable_in_term(Variable *, Variable *, Term *)
Replace a variable in a term with an alternative, maintaining the structure of the TermIndex.
Definition TermIndex.cpp:93

◆ replace_variable_in_term_with_term()

Term * TermIndex::replace_variable_in_term_with_term ( Term * new_t,
Variable * old_v,
Term * t )

Minor variation on replace_variable_in_term.

ONLY use this if the new Term is in the index!

Parameters
new_tPointer to a Term to use as replacement.
old_vPointer to a Variable to be replaced.
tPointer to Term to make the replacement for. This should already be in the index.

Definition at line 114 of file TermIndex.cpp.

116 {
117 if (t->is_variable()) {
118 if (old_v == t->get_v()) {
119 return new_t;
120 }
121 else
122 return t;
123 }
124 else {
125 Function* f = t->get_f();
126 vector<Term*> new_args;
127 for (size_t i = 0; i < t->arity(); i++)
128 new_args.push_back(replace_variable_in_term_with_term(new_t, old_v, (*t)[i]));
129 return add_function_term(f, new_args);
130 }
131}
Term * replace_variable_in_term_with_term(Term *, Variable *, Term *)
Minor variation on replace_variable_in_term.

Friends And Related Symbol Documentation

◆ operator<<

ostream & operator<< ( ostream & out,
const TermIndex & t )
friend

Definition at line 133 of file TermIndex.cpp.

133 {
134 out << "Contents of term index:" << endl;
135 out << "-----------------------" << endl;
136 for (Term* p : t.term_pointers)
137 out << p << " : " << *p << endl;
138 return out;
139}

Member Data Documentation

◆ term_pointers

vector<Term*> TermIndex::term_pointers
private

Hash table for storing pointers to where Terms are actually stored.

The key is the Term itself. term_hash is defined in TermHash.hpp.

Used (1) by the alternative storage method, not doing full hash consing; and (2) to make destruction easy.

Definition at line 93 of file TermIndex.hpp.


The documentation for this class was generated from the following files: