Hashing implementation with collision resolution using chaining #92
Conversation
Hashing/Chaining.cpp
Outdated
| cout<<"5. Exit."<<endl; | ||
| cin>>c; | ||
| switch(c){ | ||
| case 1:{ |
There was a problem hiding this comment.
Remove the curly braces by the case branches.
Hashing/Chaining.cpp
Outdated
| cout<<"Element not found"; | ||
| return; | ||
| } | ||
| while(1){ |
There was a problem hiding this comment.
Instead you can write
while(temp->data != x && temp->next)
temp=temp->next;
Hashing/Chaining.cpp
Outdated
| #include<iostream> | ||
| #include<math.h> | ||
| using namespace std; | ||
| struct node{ |
There was a problem hiding this comment.
Please use upper case.
Hashing/Chaining.cpp
Outdated
| } | ||
|
|
||
| void add(int x,int h){ | ||
| struct node *temp = new node; |
There was a problem hiding this comment.
Instead of node write Node
Hashing/Chaining.cpp
Outdated
| struct node *temp; | ||
| int i; | ||
| for(i=0;i<mod;i++){ | ||
| if(head[i]==NULL){ |
There was a problem hiding this comment.
Instead of if(head[i]==NULL){ write if(head[i]) {
Hashing/Chaining.cpp
Outdated
| #include<iostream> | ||
| #include<math.h> | ||
| using namespace std; | ||
| struct node{ |
There was a problem hiding this comment.
Please leave a space between the curly brace and node.
| @@ -0,0 +1,132 @@ | |||
| #include<iostream> | |||
| #include<math.h> | |||
| using namespace std; | |||
There was a problem hiding this comment.
After this line write a empty line.
Hashing/Chaining.cpp
Outdated
| int c,x,mod; | ||
| cout<<"Enter the size of Hash Table. = "; | ||
| cin>>mod; | ||
| while(1){ |
There was a problem hiding this comment.
Instead of while(1){ write while(loop) . For a boolean variable loop. Then you write bool loop = true;
Hashing/Chaining.cpp
Outdated
| case 4:{ | ||
| display(mod); | ||
| } | ||
| default:{ |
There was a problem hiding this comment.
For the case 5 you turn of the loop flag loop = false
Hashing/Chaining.cpp
Outdated
| break; | ||
| } | ||
| } | ||
| if(c>4) |
There was a problem hiding this comment.
Avoid this if . see my comment above.
|
Thanks for reviewing my pull request @christianbender :) |
|
Hello @christianbender, |
Relates to Issue #24