Function not running correctly (keep getting #VALUE error)

[Delgrath]

I am writing a very simple function to calculate pressure drop along a rough pipe, the code is below

Please Login or Register  to view this content.

From my understand of the way I've written the function, I should be able to execute it by simply writing =pdrop() into any cell in my worksheet.. this is not the case however. If I do this I simply get a #VALUE error, and I know the input data I'm using works because I've tested it independently elsewhere. Any ideas as to what could be the problem?

NB: This is the first function I've had to write so if I've done anything stupid don't hesistate to let me know as I need to pick VBA up quite quickly.

[sweep]

Probably better written like so,

Please Login or Register  to view this content.

and used like this:

Please Login or Register  to view this content.

[Delgrath]

Thank you sweep that worked perfectly. With regards to assigning cell values to the variables what was the problem? Was it because I listed the variables in the function both as inputs AND as separate variables within the code? (I take it this was the wrong thing to do).

Thanks again.

[sweep]

The error was being raised because there were insufficient arguments in the function call, in much the same way as if you entered =SUM() into a cell. You could (although it's foolish to do so) enter something along the lines of =pdrop(1,2,3,4,5) into a cell, which will pass the correct number of arguments to the function code, which will immediately overwrite those values with the contents of the cells (specified in your original code).

To use specific cells as the arguments in your function, your opening line should have been

Please Login or Register  to view this content.

followed by dimensioning of variables, and assigning values.


For my money, if you're writing a function, you're almost certainly at some time going to want to change the cells that contain its arguments, so I would stick with my interpretation of your code.

I hope that's clear - let me know if not.

[Delgrath]

No that makes perfect sense, I agree that your way makes the function more versatile. Many thanks!