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.