You can store the value of player:getEffectiveSkillLevel(SKILL_FISHING) in a variable instead of calling it twice.