vyhledávat binární vyhledávací strom

Question

vyhledávat binární vyhledávací strom

hlasů

0

Snažím se najít jméno v rámci klíče. Myslím, že je to načítání v pořádku. Nicméně, jeho příchod jako nebyl nalezen. Možná můj kód je špatně někde?

if (database.retrieve(name, aData))  // both contain the match

v main()

static void retrieveItem(char *name, data& aData)
{
cout << >>> retrieve  << name << endl << endl;
if (database.retrieve(name, aData))            // name and aData both contain the match
    cout << aData << endl;
else
    cout << not found\n;
cout << endl;
     }

     static void removeItem(char *name)
    {
cout << >>> remove  << name << endl << endl;
if (database.remove(name))
    cout << name <<  removed\n;
else
    cout << name <<  not found\n;
cout << endl;
    }

   int main()
   {
   #ifdef _WIN32
// request memory leak report in Output Window after main returns
_CrtSetDbgFlag ( _CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF );
   #endif

data    aData;


     << Database Of Great Computer Scientists\n\n;

database.insert(data(Ralston, Anthony));
database.insert(data(Liang, Li));
database.insert(data(Jones, Doug));
database.insert(data(Goble, Colin));
database.insert(data(Knuth, Donald));
database.insert(data(Kay, Alan));
database.insert(data(Von Neumann, John));
database.insert(data(Trigoboff, Michael));
database.insert(data(Turing, Alan));
displayDatabase(true);
retrieveItem(Trigoboff, Michael, aData);
retrieveItem(Kaye, Danny, aData);

removeItem(Ralston, Anthony);
displayDatabase(true);

načtení funkce ...

bool BST::retrieve(const char *key, data &aData, int parent) const
 {

for(int index=0; index < maxsize+1; index++)
{

    if (!items[index].empty) 
    {


        if ( items[index].instanceData == key )
        {
            aData.setName(key);
            return true;                   // doesn't return right away
        }


    }

}


 }

a je definován v data.cpp

bool operator== (const data& d1, const data& d2)
{

return strcmp(d1.getName(), d2.getName()) == 0;

}

takže tento kus kódu uvnitř main () je místo, kde se říká, že nebyl nalezen, když si myslím, že by mělo být správně. jak jméno a ADATA obsahovat správný název, který byl nalezen ..

static void retrieveItem(char *name, data& aData)
{
cout << >>> retrieve  << name << endl << endl;
if (database.retrieve(name, aData))            // name and aData both contain the match
    cout << aData << endl;
else
    cout << not found\n;
cout << endl;
     }

c++binary-search-tree

Položena 05/12/2009 v 19:54
zdroj uživatelem Steller

V jiných jazycích...

3 odpovědí

hlasů

0

Nejsem žádný expert C ++, ale je váš operátor == skutečně hodnoceny? Je to znamenalo užít dvě const datové reference, ale ty se zdají být porovnávání bez ohledu na typ items[index].instanceDataje a char*.

Navrhuji vám dát nějaké přihlášení do obsluhy a uvidíme, jestli je to skutečně volal - obávám se, že to není.

Jednou z možností, aby operátor == z rovnice by být dočasně, aby se srovnání explicitní:

 if (strcmp(items[index].instanceData.getName(), key) == 0)
 {
     ...
 }

Na jiném místě, nevidím, jak se to vlastně dělá binární vyhledávání vůbec. Připadá mi, že je to jen obyčejný list - děláte lineární hledání v rámci retrievenamísto srovnávání klíče a jít doleva nebo doprava dolů stromu (nebo vrácení „nalezen“) v závislosti na výsledku.

Odpovězeno 05/12/2009 v 20:03
zdroj uživatelem Jon Skeet

hlasů

0

Nemůžu říct s jistotou, aniž by viděl kód pro BST, ale to vypadá špatně:

for(int index=0; index < maxsize+1; index++)

S tradičními konvencemi, by mělo být:

for(int index=0; index < maxsize; index++)

Mimo to, že se také zdá, svou funkci buď vrátí true nebo nějakou nedefinovanou boolean. Pravděpodobně byste měli mít return false;na konci BST :: načíst.

Odpovězeno 05/12/2009 v 21:58
zdroj uživatelem Joh

dellco · Accepted Answer · 2009-12-05 23:32

Ty by měly být pomocí BST procházet strom - ne smyčky nad každou položku v poli, stejně jako jiní uvedli. Zkuste něco jako:

bool retrieve(key, aData)
  retrieve(key, aData, parent)
  if (key == aData)
    return true
  else
    return false

bool retrieve(key, aData, parent)
  if (key == items[parent].name)
    aData.setName(key)
  else if (key < items[parent].name)
    retrieve(key, aData, 2*parent+1)
  else
    retrieve(key, aData, 2*parent+2)

To by mělo fungovat dobře! :)