What ELISHEVA said. I'd go further and say yes, they are bad; not always but more often than not. As samtregar mentioned, they often lead to action at a distance and that leads to debugging difficultly which leads to 100 wasted hours over time because the original dev wanted to save 1 or 2 up front by using a global or a singleton instead of something reusable.