<div dir="ltr">I will write the Readme and look at getting rid of accessor methods except where values are changed. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jun 25, 2023 at 7:17 PM Vikriti D'Vita via PLUG-discuss <<a href="mailto:plug-discuss@lists.phxlinux.org">plug-discuss@lists.phxlinux.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Yeah a usage readme would be helpful. Quick glance through the code:<br>
Why are `add_currying` and `multiply_currying` their on functions?<br>
The only place their used is in `test_hdr`.<br>
Too much abstraction makes it harder to understand. I'd have just used <br>
the lambda right there so its obvious what you're testing for without <br>
having to jump between files.<br>
<br>
Also, this is specific to python but, in general, it is discouraged to <br>
have getters and setters in python like `def get_die_name(self) -> str:`.<br>
A classes attributes are considered public by default and should be <br>
directly used. If you need a private attribute, you prepend it with two <br>
underscores. like `__some_var`. While this is mostly convention, python <br>
renamed them to `_CLASSNAME__some_var` when accessing from outside the <br>
class and subclasses.<br>
If later you decide that you want to do some work in the getter, you can <br>
make a method with the same name as the attribite, make the attribute <br>
private, and add the `@property` decorator to the getter. No change in <br>
your class's interface.<br>
<br>
Otherwise looks great :)<br>
<br>
On 6/20/23 10:14, trent shipley via PLUG-discuss wrote:<br>
> I just did a thing.<br>
> <br>
> And I could really use some feedback.  If it is lite the feedback I'd <br>
> get from my last coding job it will go something like: your code works, <br>
> but I don't understand it, there isn't enough documentation, and what <br>
> documentation there is, doesn't help.<br>
> <br>
> <a href="https://github.com/trent-shipley/hackable_dice_roller" rel="noreferrer" target="_blank">https://github.com/trent-shipley/hackable_dice_roller</a> <br>
> <<a href="https://github.com/trent-shipley/hackable_dice_roller" rel="noreferrer" target="_blank">https://github.com/trent-shipley/hackable_dice_roller</a>><br>
> <br>
> <br>
> ---------------------------------------------------<br>
> PLUG-discuss mailing list: <a href="mailto:PLUG-discuss@lists.phxlinux.org" target="_blank">PLUG-discuss@lists.phxlinux.org</a><br>
> To subscribe, unsubscribe, or to change your mail settings:<br>
> <a href="https://lists.phxlinux.org/mailman/listinfo/plug-discuss" rel="noreferrer" target="_blank">https://lists.phxlinux.org/mailman/listinfo/plug-discuss</a><br>
<br>
-- <br>
Vikriti D'Vita<br>
<a href="mailto:v@lokegaonkar.in" target="_blank">v@lokegaonkar.in</a><br>
---------------------------------------------------<br>
PLUG-discuss mailing list: <a href="mailto:PLUG-discuss@lists.phxlinux.org" target="_blank">PLUG-discuss@lists.phxlinux.org</a><br>
To subscribe, unsubscribe, or to change your mail settings:<br>
<a href="https://lists.phxlinux.org/mailman/listinfo/plug-discuss" rel="noreferrer" target="_blank">https://lists.phxlinux.org/mailman/listinfo/plug-discuss</a><br>
</blockquote></div>