Null check after dereference
Review this Java entitlement check used to gate a premium feature.
What a strong answer looks like
Separate real bugs from style. Rank issues by severity, point at the root cause rather than the symptom, and suggest a concrete fix, specific and kind.
0:00 of about 15 min
Mark a line and say what kind of problem it is.0 findings
1boolean isActivePremium(User user) {
2 if (user.getSubscription().isActive() && user != null) {
3 return user.getSubscription().getTier().equals("premium");
4 }
5 return false;
6}
7
8void render(User user) {
9 if (isActivePremium(user)) {
10 showPremiumDashboard(user);
11 } else {
12 showUpgradePrompt(user); // also reached for signed-out (null) users
13 }
14}
Which questions mattered is sealed until you submit. Telling you now would just be handing over the edge cases.
Run or narrate your approach, then ask the coach.