You do a lot of calls like:
which, looking at the method, seems to be checking the field isn't empty.
However you do nothing with the boolean result.
In fact, the check done inside there seems to be incorrect (and EditText will never be equal to a test
string for example).
It looks to me like you should
be correcting that method so it checks the values actually exist, and then do something with the return result.
That is, if it returns false then
pop up the toast and exit. If true, then continue to check the rest of the fields. If all true then (and only then
) deal with the data.
Currently you only validate after
you start using the data.