自学蠢货!各位大婶,你们觉得这段代码,有更简洁,执行效率更高的办法吗
自学蠢材!各位大婶,你们觉得这段代码,有更简洁,执行效率更高的办法吗?
我总感觉这段代码写的傻傻的,纯粹是为了实现功能而已。应该有更好的解决方案。如果是你你会怎么写?
------解决方案--------------------
用sql join查询
并且不要用*
那个地方也没必要用switch,if就够了
------解决方案--------------------
select * from xxx where xxx in(select * from XXX where UserId=....) and IP不为空
------解决方案--------------------
你这个写得非常好,千穿万穿 马屁不穿。
------解决方案--------------------
中规中矩的代码比优化过头的代码更友善
------解决方案--------------------
确实可以优化。
至少不要这么多遍的去连接数据库查询。
------解决方案--------------------
用switch的地方都可以用委托。
------解决方案--------------------
for里面做查库,呵呵,一般,最好,不要这样做。你把xxx表在for前面就先拿出来。and 登录一般只有成功和失败,用if就行了。 if(login != 0) continue;button8.Enabled = false;webForm.postsAdd(账号ID,客户ID,文章标题,文章内容);
------解决方案--------------------
你在查找数据的时候不能用一个左连接吗?
------解决方案--------------------
//查找新的任务表,根据任务数,根据任务数找出匹配的用户数,循环执行发布文章
string sql = "select * from XXX where UserId='" + Id + "' and 执行状态=否";
DataTable dt = DBClass.ExecSel(sql);
for (int i = 0; i < dt.Rows.Count; i++)
{
aTitle = dt.Rows[i]["Title"].ToString();
aContent = dt.Rows[i]["Content"].ToString();
//找出新的任务及数量后,查找需要执行用户执行任务数最少的用户及用户数量
string _exsql = "select * from xxx where 数量 and IP不为空";
DataTable _dt = DBClass.ExecSel(_exsql);
for (int j = 0; j <= dt.Rows.Count; j++)//根据任务数找出用户数
{
string 账号 = _dt.Rows[j]["账号"].ToString();
string 密码 = _dt.Rows[j]["密码"].ToString();
//找出后登录.....
int login = webForm.LoginIn(账号, 密码);
switch (login)
{
case 0:
button8.Enabled = false;
//执行发布文章任务
webForm.postsAdd(账号ID,客户ID,文章标题,文章内容);
break;
default:
break;
//goto end;
}
}
我总感觉这段代码写的傻傻的,纯粹是为了实现功能而已。应该有更好的解决方案。如果是你你会怎么写?
------解决方案--------------------
用sql join查询
并且不要用*
那个地方也没必要用switch,if就够了
------解决方案--------------------
select * from xxx where xxx in(select * from XXX where UserId=....) and IP不为空
------解决方案--------------------
你这个写得非常好,千穿万穿 马屁不穿。
------解决方案--------------------
中规中矩的代码比优化过头的代码更友善
------解决方案--------------------
确实可以优化。
至少不要这么多遍的去连接数据库查询。
------解决方案--------------------
用switch的地方都可以用委托。
------解决方案--------------------
for里面做查库,呵呵,一般,最好,不要这样做。你把xxx表在for前面就先拿出来。and 登录一般只有成功和失败,用if就行了。 if(login != 0) continue;button8.Enabled = false;webForm.postsAdd(账号ID,客户ID,文章标题,文章内容);
------解决方案--------------------
你在查找数据的时候不能用一个左连接吗?
------解决方案--------------------