Re^2: What to test in a new module

It's a case of TIMTOTWDI, and I have not looked at the original code, but two comments:

No need for this: $self->{'error'} = '';. It'll be autovivified if needed.

It will autovivify as undef if it is not assigned to before it is accessed. That could cause issues later on with other code.

There's no need for a return; at the end of a sub if nothing's being returned (unless you intentionally are returning undef.

It's safer to explicitly return as otherwise the value of the last expression is returned. FWIW, this is covered in PBP on p197 under Implicit Returns, with a follow-on under Returning Failure on p199 (although that also encourages the Contextual::Return module which I don't use).

Comment on Re^2: What to test in a new module

Replies are listed 'Best First'.
Re^3: What to test in a new module by jdporter (Chancellor) on Jul 07, 2023 at 13:50 UTC
It's safer to explicitly return as otherwise the value of the last expression is returned. He did say "unless you intentionally are returning undef". I agree with stevieb on that point; however, in my code if I am intentionally returning undef I make that explicit with `return();` I never use `return;` in my code unless I am returning early from a sub which is not expected to return a value. (If the caller insists on doing something with the returned value from such a sub, TNMP.)	[reply] [d/l] [select]
Re^4: What to test in a new module by swl (Parson) on Jul 08, 2023 at 05:56 UTC
Thanks for the clarification. Two other points, which you are probably well aware of but some readers might not be: `return;` and `return();` are the same, although the brackets do make it more visually distinct and perhaps that's your point. `return;` in list context returns the empty list. This means it is effectively the same as `return wantarray ? () : undef;` unless it is called in void context. Your use of it when not expecting a return value implies void context, though, so perhaps this point is moot.	[reply] [d/l] [select]
Re^5: What to test in a new module by jdporter (Chancellor) on Jul 10, 2023 at 14:14 UTC
the brackets do make it more visually distinct and perhaps that's your point. That is precisely my point. It isn't obvious that `return;` returns undef, so I don't depend on that. That is, I don't use that construction when that's what I want to do.	[reply] [d/l]
Re^6: What to test in a new module by choroba (Cardinal) on Jul 11, 2023 at 18:22 UTC
Re^7: What to test in a new module by jdporter (Chancellor) on Jul 11, 2023 at 18:43 UTC


more useful options
	PerlMonks