jquery 自动改变div高度 怎么不行啊

Question

function g(o) {
return document.getElementById (o);
}
function sleep(n)
    {
        var start=new Date().getTime();
        while(true)  if(new Date().getTime()-start> n) break;
    }function ShowDiv(n){
sleep(50);
g('m_c_'+n).style.display='block';
$('#m_c_'+n).slideDown(8000,callback);
g(n).className = "item fore hover";
}function HiddenDiv(n){
    sleep(50);
g('m_c_'+n).style.display='none';
document.getElementById(n).className = "item fore";
}
function P_ShowDiv(n, id){
sleep(50);
g('m_c_'+n).style.display='block'
document.getElementById(n).className = "item fore hover";
$('#'+id).hide();
}
function R_ShowDiv(n){
sleep(50);
g('m_c_'+n).style.display='block'

document.getElementById(n).className = "citem";
}function R_HiddenDiv(n){
    sleep(50);
g('m_c_'+n).style.display='none';
document.getElementById(n).className = "";
}
function MM_showHideLayers() {
  var i,p,v,obj,args=MM_showHideLayers.arguments;
  for (i=0; i<(args.length-2); i+=3)
  with (document) if (getElementById && ((obj=getElementById(args[i]))!=null)) { v=args[i+2];
    if (obj.style) { obj=obj.style; v=(v=='show')?'visible':(v=='hide')?'hidden':v; }
    obj.visibility=v; }
}
var speed=10000;
     function scrollit()
     {        var obj=document.getElementById("ccc");
        obj.style.posTop=obj.style.posTop-501;
        var offH=obj.style.posTop+obj.offsetHeight;
        if (offH<0)
        {
            obj.style.posTop=0;
        }
     }
  function scrollitb()
     {        var obj=document.getElementById("bbb");
        obj.style.posTop=obj.style.posTop-501;
        var offH=obj.style.posTop+obj.offsetHeight;
        if (offH<0)
        {
            obj.style.posTop=0;
        }
     }
 function scrollitg()
     {        var obj=document.getElementById("ggg");
        obj.style.posTop=obj.style.posTop-501;
        var offH=obj.style.posTop+obj.offsetHeight;
        if (offH<0)
        {
            obj.style.posTop=0;
        }
     }
这个还是不行啊求高手帮小弟解决下，效果是div随着内容变化改变高度

Answer 1

哇塞，你这个代码看着让人痛苦，发帖的时候使用那个代码格式，那样看起来会爽很多！
而且代码的命名也很有问题，你的代码方式需要去改进，如果这样写代码的话，以后会养成不好的习惯，建议你去看一下clean code！
以下是我的建议，如果你觉得我说的不对，你可以当作我神马也没说
建议：
1、代码最好命名要清晰；
2、方法名要统一；
3、看完这么多，我只看到了N多个函数，而且每个函数我看着很蛋疼，不知道你想干嘛，因此加点必要的注释；
4、变量名最后命名要有意义，比如说你的function g(o)｛｝这个函数你完全可以使用如下替代：function getDomById(var domId){
   //code for function
}这样看起来就舒服多啦！
5、funtion最好有输入输出，如果没有输入是当然可以啦，但是输出如果没有这是不应该的，即使是完成一个
动作，那你也需要加入一个操作是否成功之类的！