Ann W.

Simply, I am not satisfied with your work. first of all, you are very late. You accepted job with "3 days" proposal but delivered first version much later. Additionaly, it was very buggy and did not met requirements at all, about 40%? Quality of source code: a) I gave your code to another developer to check how it is well formed and is based on actual dev standards he said that you are coding like student beginner Here his and my notes 1. you are making copyblocks and due to that project development in later stages will be much more difficult 1.1 for example instantiating one thing on many places, for example tinyDB - why do not you use singleton and static property? 2. you are storing single values for ins, outs etc.... you should have all settings in one data class and work only with it! One shift = one class. 3. you absolutely do not check for errors, for example I said you to display message if sms is NOT sent.... You did not do it - simply try...catch would be ok 3.1 but you do not use try..catch almost nowhere so application is about to hang easily... 4. bad OOP at all - for example you solve logging in setOnClickListener directly, without making some TryLogin method 5. you have old unused code in codebase, eg. old approach to machines - arrays with 10 items - machine names and zeros 6. you store numbers as string in tinyDB, it's unacceptable to have numbers as strings.... 7. using "eval" for formula counting is bad 8. bad string formatting etc....