Сразу 3 косяка.
1. Сразу после show фрагмент ещё не готов, он ещё находится в процессе показывания.
2. findViewById может вернуть null, если ничего не найдено. Но вы ему говорите: а мне пофиг!!
Используйте !! только в самых крайних случаях, когда вы точно понимаете, какие будут последствия.
3. Нельзя просто так снаружи фрагмента вызывать его методы (свойства устанавливать, листенеры и т.п.)