SPOILER: No. No, he isn’t.
As every dedicated Harry Potter fan knows, the morality of Severus Snape is one of the most important and continuous debates of our time, and possibly in the history of forever.
Warner Bros.
Many fans argue that, because Snape died fighting against Voldemort on behalf of the wizarding world, he was actually one of the series' greatest heroes.
His unconditional love for Harry's mother Lily – after her death – motivated him to double-cross Voldemort and work with the Order of the Phoenix in order to ~defeat evil~.
Warner Bros.
Others, however, say that Snape simply used his unrequited love for Lily Potter as an excuse to act abusively towards his students. Not to mention the fact that he actually decided to become a Death Eater in the first place.
Snape actively abused Harry for no reason other than he looked like his father, whom Snape hated. He was such an awful person that, when Neville came to face the Boggart, it took the form of Snape because – in a world where his parents were tortured by Death Eaters to the extent that they were hospitalised for the rest of their lives – Snape is what Neville feared the most.
Warner Bros.