php小白求教form display and update mysql的问题 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
yappa
V2EX    PHP

php小白求教form display and update mysql的问题

  •  
  •   yappa 2014-01-24 00:08:49 +08:00 3708 次点击
    这是一个创建于 4359 天前的主题,其中的信息可能已经有所发展或是发生改变。
    php小白,各种谷歌,也试了好多stackoverflow上的代码,都只能正常display,无法update。代码如下:
    <?php
    define('DB_NAME', 'form');
    define('DB_USER', 'root');
    define('DB_PASSWORD', '123456');
    define('DB_HOST', 'localhost');


    $link = mysql_connect(DB_HOST DB_USER, DB_PASSWORD);

    if (!$link)
    {
    die('could not connect: ' . mysql_error());
    }
    $db_selected = mysql_select_db(DB_NAME, $link);

    if (!$db_selected)
    {
    die('Can\'t use ' .DB_NAME .':' . mysql_error());
    }
    $query = "SELECT * FROM articles";
    $result = mysql_query($query) or die(mysql_error());
    ?>
    <div class="content-holder">
    <form action="" method="post">
    <table border='1' cellpadding='10' id='ViewTable'>
    <tr>
    <th>title</th>
    <th>id</th>
    </tr>
    <?php
    while($row = mysql_fetch_array($result))
    {
    $id = $row['id'];
    $title = $row['title'];
    ?>
    <tr>
    <td>
    <input type="textbox" class="TextAreaTitle" name="title" value="<?=$title?>"></input>
    <input type="hidden" name="id" value="<?=$title?>"></input>
    </td>
    <td><?=$id?></td>
    </tr>
    <?php
    }
    ?>
    </table>
    <input type="submit" name="update" class="submitlink" Value="update">
    </form>
    </div>

    <?php
    if(isset($_POST['update']))
    {
    for ($i=count($_POST['id']); $i--;)
    {
    $id = $_POST['title'][$i];
    $title = $_POST['id'][$i];
    mysql_query("UPDATE articles SET title= $title WHERE id= $id ");
    }
    }
    ?>

    4 条回复    1970-01-01 08:00:00 +08:00
    vibbow
        1
    vibbow  
       2014-01-24 05:33:00 +08:00   2
    槽点很多

    http://pic.vsean.net/di/MBB4/QQ截图20140123212941.png

    你的这个hidden input明明是id,为什么你要设置变量为value。
    其次Input标签是自闭和的。

    还有如果你的input有N多相同name的元素的话,那么name应该这样写:
    <input type="hidden" name="id[]" value="<?=$title?>" />
    这样PHP才会把这些name相同的元素识别成一个数组了,否则应该是后者替换前者的关系。

    最后你mysql语句update数值的时候不转移特殊字符么...

    看着太纠结了,给你重写一份出来吧。
    vibbow
        2
    vibbow  
       2014-01-24 06:30:38 +08:00   2
    重写完成
    http://pastebin.com/XPFa1EaS

    虽然我写的代码也很挫...
    vibbow
        3
    vibbow  
       2014-01-24 06:37:16 +08:00   2
    重写的时候又发现了代码的两个槽点:

    1. for loop写错了
    2. for loop没必要每次都去求一次数组的大小,太浪费性能了...

    LZ是在HTML中间去运行sql查询的,我重写的时候也这样写了。

    不过一般情况下我喜欢在开头把所有该查的东西都查完了,保存成数组,然后在HTML里直接输出。虽然会浪费点内存,反正这年头内存也不值钱。
    yappa
        4
    yappa  
    OP
       2014-01-24 08:34:32 +08:00
    @vibbow 感谢已发送
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     950 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 23ms UTC 22:36 PVG 06:36 LAX 14:36 JFK 17:36
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86