高手帮我看看　我这样的设计结构好吗？

我在设计三层架构的时候是这样设计的
1.连接字符串加密之后放在web.config文件中
2.连接数据库的时候读出连接字符串，解密，连接数据库
3.所有的SQL（select * from tableName）语句放在一个单独的类里面，做成静态变量
4.数据更新操作全部用存储过程，在过程中判断操作是否成功，操作失败就回滚我这样做行吗？