关于php的问题请高手帮忙!

来源:百度知道 编辑:UC知道 时间:2024/05/15 11:28:27
高手看这段代码:
if(is_uploaded_file($_FILES['userfile']['tmp_name']))
{
if($_FILES['userfile']['size']>$da)
{echo "你上传的文件超过规定大小!<br>\n";}
elseif
(($_FILES['userfile']['type']=="image/pjpeg") || ($_FILES['userfile']['type']=="image/jpeg"))
{
if($_POST['filename']=='')
{;}
elseif
(file_exists($lj . $_POST['filename'] . '.jpg'))
{echo "这个文件已经存在,请换个名字!<br>\n"; }
elseif
(!@move_uploaded_file($_FILES['userfile']['tmp_name'], $lj .$_POST['filename'] . '.jpg'))
{echo "上传失败,检查一下你的文件!<br>\n";}
else
{echo "上传成功!<br>\n"."文件名: ".$_POST['filename']."<br>\n"."文件容量: ".$_FILES['userfile']['size']." 字节<br>

一、可以光要if...{}没有else,而且if和else的语句如果只有一句可以不用{},PHP这些灵活的语法反而使得我们的程序像C一样费解,建议大家像PERL一样强制都加上{}。

二、不知道你是在什么软件里面编辑PHP文件,建议使用ULTRAEDIT等,能自动配对{}、()、<>的编辑器,这样你鼠标放在一个括号上,程序自动寻找并突出配对的那块,这样就非常方便。

三、你的程序我在ULTRAEDIT里面查看了一下,你的清理正确,逻辑合理,由于BAIDU会过滤空白,我把许多能移动到一行上的都移动到一行上,这样你在编辑器里面能更明白逻辑:

if(is_uploaded_file($_FILES['userfile']['tmp_name']))
{
if($_FILES['userfile']['size']>$da) {echo "你上传的文件超过规定大小!<br>\n";}
elseif (($_FILES['userfile']['type']=="image/pjpeg") || ($_FILES['userfile']['type']=="image/jpeg"))
{
if($_POST['filename']=='') {;}
elseif (file_exists($lj . $_POST['filename'] . '.jpg')) {echo "这个文件已经存在,请换个名字!<br>\n"; }
elseif (!@move_uploaded_file($_FILES['userfile']['tmp_name'], $lj .$_POST['filename'] . '.jpg')) {echo "上