Sandforce today is not nearly as bad as it used to be in the past.
Sandforce is not a cool controller though. It has more complexity than other SSD controllers because it employs deduplication, which requires a separate deduptable to store the data. While other SSDs have just one table: the mapping table, Sandforce SSDs have two. This also means they have two points of failure regarding consistency, instead of just one.
Sandforce was designed to fool popular benchmarks at the time, like IOmeter and ATTO. This allowed OCZ to sell SSDs that do 75MB/s write as 255MB/s (SATA/300) or even 520MB/s (SATA/600). In the end, consumers got a slower SSD that was very error prone and paid more money than a 'proper' SSD.
Today, Sandforce has stabilised its firmware and its a decent SSD. But not very cool since all the tricks are not necessary any longer. Normal SSDs already peak their SATA/600 interface without resorting to dirty tricks. So in the end, all those complexity was for nothing. Only with PCI-express/NVMe Sandforce will live its glory again. They can claim 2GB/s write while in fact the device only does 650MB/s or so.
@hoknikb: i agree in general about your statement. But if you dislike Sandforce so much, why recommend a Phison controller? Sure it's cheaper, but i would never recommend such an SSD. Phison controllers are found most in USB sticks. You'll be getting an inferior product lacking the sophisticated design of Crucial/Samsung for example. It could fail in various ways the modern SSDs have protection against.