FANDOM


 
Line 1: Line 1:
I know how to implement it, and have done so before with units that fit established categories in the existing code. Heck, I actually fixed one that was in an incorrect category and had to make a new one (Odette, 15 levels -> 16). There was precedent for that one. I could see how it had been implemented with similar units, and followed that convention. I added Claris to the event unit list because Hana was there, and she had the same circumstances. (Daily login bonus for multiple days in a row, event unit cost, shrine unit skill.)
+
I know how to implement it, and have done so before with units that fit established categories in the existing code (e.g., Shinomiya Komachi). Heck, I actually fixed one that was in an incorrect category and had to make a new one (Odette, 15 levels -> 16). There was precedent for both of them. I could see how similar units had been implemented, and followed that convention. I added Claris to the event unit list because Hana was there, and she had the same circumstances. (Daily login bonus for multiple days in a row, event unit cost, shrine unit skill.)
   
 
But Hana and Claris don't fit into those pre-existing categories with regard to skills, and there isn't precedent for them. Few things bother a programmer more than someone mucking up their nicely organized and elegant code. So, as professional/hobbyist courtesy, I leave the implementation of these two to the original author.
 
But Hana and Claris don't fit into those pre-existing categories with regard to skills, and there isn't precedent for them. Few things bother a programmer more than someone mucking up their nicely organized and elegant code. So, as professional/hobbyist courtesy, I leave the implementation of these two to the original author.

Latest revision as of 16:32, February 2, 2020

I know how to implement it, and have done so before with units that fit established categories in the existing code (e.g., Shinomiya Komachi). Heck, I actually fixed one that was in an incorrect category and had to make a new one (Odette, 15 levels -> 16). There was precedent for both of them. I could see how similar units had been implemented, and followed that convention. I added Claris to the event unit list because Hana was there, and she had the same circumstances. (Daily login bonus for multiple days in a row, event unit cost, shrine unit skill.)

But Hana and Claris don't fit into those pre-existing categories with regard to skills, and there isn't precedent for them. Few things bother a programmer more than someone mucking up their nicely organized and elegant code. So, as professional/hobbyist courtesy, I leave the implementation of these two to the original author.

Edit: Kkentauro, I have put thought into this. I know how I would do it, but not how you would do it. I'd prefer you to implement these two yourself first, so I know how you want it done in the future. If you want me to do it myself, though, or don't care as long as it works, just say so.

Community content is available under CC-BY-SA unless otherwise noted.