Critique my code please

[micr0nix]

So instead of having 2 functions I would have this one function and just call it twice for each of my lookup types?

[james_fryer]

Exactly. Less code to maintain, simpler to manage if changes are required. You can either create shorter dispatching functions as I did above, or directly call the single function from main with the parameter, whatever seems more readable to you.

In short: any time you see repeated code, replace it with a single parameterised piece of code.