Android HTTPPOST执行两次PHP脚本

时间:2015-02-05 04:52:58

标签: java php android http-post

我正在开发基本的Android APK,它将成为我运行的网站的基本移动版本。 APK的一部分使人们能够注册用户帐户(基本注册)。我正在使用HTTPPOST将用户表单信息发送到PHP脚本。我所有的错误检查工作(密码匹配,长度等),但我从有效提交得到的响应,我可以告诉它尝试运行PHP两次。 APK将收到一条错误消息,并告诉用户用户已输入用户名和密码,但这是因为php获取运行的,插入数据,然后由于某种原因再次运行并报告错误捕获而不仅仅是在第一次运行时报告成功。如果我拿出错误catch来查找重复的用户名和电子邮件,我可以在我的数据库中看到两个插入。为什么PHP页面会运行两次?

步骤进行:

  1. 清除用户数据库。
  2. 插入有效的注册信息(不抛出错误)。
  3. 结果

    APK显示错误消息,说明已使用用户名和密码。 SQL数据库显示已插入表单中的信息。

    文件

    我将登录设置在我的APK中,似乎只调用HTTPPOST一次。下面是JAVA文件和PHP页面。任何煽动这一点将不胜感激。

    JAVA文件

    Register.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Log.d("Registration: ", "Button Pressed");
                dialog = ProgressDialog.show(Register.this, "",
                        "Creating account...", true);
                new Thread(new Runnable() {
                    public void run() {
                        new RegisterUser().execute("");
                    }
                }).start();
            }
        });
    }
    
    private class RegisterUser extends AsyncTask<String, Void, String> {
    
        @Override
        protected String doInBackground(String... arg0) {
            try {
    
    
                httpclient = new DefaultHttpClient();
                // This is the page that will get all the information to create the account
                HTTPPOST = new HttpPost("http://www.linetomyphppage.php");
    
                NAMEVALUEPAIRS = new ArrayList<NameValuePair>();
    
                NAMEVALUEPAIRS.add(new BasicNameValuePair("username",username.getText().toString().trim()));  // $Edittext_value = $_POST['Edittext_value'];
                NAMEVALUEPAIRS.add(new BasicNameValuePair("userEmail",emailaddress.getText().toString().trim()));
                NAMEVALUEPAIRS.add(new BasicNameValuePair("password1",password1.getText().toString().trim()));
                NAMEVALUEPAIRS.add(new BasicNameValuePair("password2",password2.getText().toString().trim()));
                NAMEVALUEPAIRS.add(new BasicNameValuePair("newsletter",subscribe));
    
                HTTPPOST.setEntity(new UrlEncodedFormEntity(NAMEVALUEPAIRS));
                HTTPRESPONSE = httpclient.execute(HTTPPOST);
                ResponseHandler<String> responseHandler = new BasicResponseHandler();
                final String response = httpclient.execute(HTTPPOST, responseHandler);
                Log.d("Registration: ", "Executing Post");
                errorcode = response;
    
                // Redirect the user depending on the response
                if (response.equalsIgnoreCase("success")) {
                   // Toast.makeText(Register.this, "Registration successful", Toast.LENGTH_SHORT).show();
                    Log.d("Registration: ", "Registration successful");
                    valid = "Successful";
                } else {
                    Log.d("Registration: ", "Registration failed");
                    valid = "Invalid";
                  //  Toast.makeText(Register.this, "Registration failed: " + response, Toast.LENGTH_SHORT).show();
                }
    
    
            } catch (Exception e) {
                System.out.println("Exception here : " + e);
            }
            dialog.dismiss();
            return "Executed";
        }
        @Override
        protected void onPostExecute(String result){
    
            if(valid.equalsIgnoreCase("Invalid")){
                Log.d("Registration: ", "Error generated");
                String code = errorcode.replace("Invalid","");
                Toast.makeText(Register.this, "Registration failed: " + code, Toast.LENGTH_SHORT).show();
            }else{
                Toast.makeText(Register.this, "Registration successful", Toast.LENGTH_SHORT).show();
                startActivity(new Intent(Register.this, HomeScreen.class));
                finish();
            }
        }
    }
    

    PHP Side

    <?php 
    // Connecting to database
    $connect = $_SERVER['DOCUMENT_ROOT'];
    $connect .= "pathtoconnect.php";
    include_once($connect);
    
    // Get all the variables being passed from the APK
    $username = mysql_real_escape_string($_POST['username']);
    $password1 = mysql_real_escape_string($_POST['password1']);
    $password2 = mysql_real_escape_string($_POST['password2']);
    $email = mysql_real_escape_string($_POST['userEmail']);
    $newsletter = mysql_real_escape_string($_POST['newsletter']);
    $pass = 1;
    $error = false;
    
    // Check if the username is valid. 
    if (preg_match('/[^0-9a-z-_]/i', $username) == 1) {
        $error = 'You can not use spaces or strange characters in a usermame.';
        $pass = 0;      
    }
    if(strlen($username) < 5) { // Ensure the length is at least 5
        $error = 'Username must be at least 5 characters.';
        $pass= 0 ;
    }
    if(strlen($password1) < 5) { // Ensure the length is at least 5
        $error = 'Passwords have to be at least slightly challenging (5+ characters).';
        $pass= 0 ;
    }
    
    if($password1 != $password2) { // Check for passwords to match
        $error = "Your passwords did not match. Please try again.";
        $pass= 0 ;  
    }
    
    if (!filter_var($email, FILTER_VALIDATE_EMAIL)) { // Check to make sure email address is valid
        $error = "The email address you entered is not valid. Double check it.";
        $pass= 0 ;
    }
    
    // check if username or email is already in use
    $checking = mysql_query("SELECT COUNT(user_id) AS total FROM users WHERE username='$username' OR email='$email'");
    $checking = mysql_fetch_array($checking);
    if($checking['total'] > 0) {
        $error = 'The username or email you are trying to use is already in use. Please go to the website if you forgot your password to reset it.';    
        $pass= 0 ;
    }
    if(!$error) {
        echo 'success';
        $time = time();
        mysql_query("INSERT INTO users (username, password, email, date_created, subscription) VALUES ('$username', md5('$password1'), '$email',  '$time', '$subscribe')"); 
    
    
        }
    else {
        echo 'Invalid'.$error;  
    }
    

    &GT;

    设备正在报告重复错误,正如我所提到的,数据会被插入,因此它的“成功”就不会得到回应。 下面是来自JAVA文件的日志,它显示它只获得失败的响应。

    02-04 22:31:37.442  26159-26159/com.demo D/Registration:﹕ Button Pressed
    02-04 22:31:37.862  26159-26559/com.demo D/Registration:﹕ Executing Post
    02-04 22:31:37.862  26159-26559/com.demo D/Registration:﹕ Registration failed
    02-04 22:31:37.872  26159-26159/com.demo D/Registration:﹕ Error generated
    

    我构建了一个基本的html表单来传递数据,并且该方法运行良好,执行php onces所以我不知所措。 Android / JAVA对我来说是新手,所以这是完成此任务的错误途径吗?

    由于

3 个答案:

答案 0 :(得分:2)

你两次调用“httpclient.execute(..)”:

HTTPRESPONSE = httpclient.execute(HTTPPOST);
ResponseHandler<String> responseHandler = new BasicResponseHandler();
final String response = httpclient.execute(HTTPPOST, responseHandler);

删除其中一个应解决您的问题:)

答案 1 :(得分:0)

你不需要将你的AsynTask子类(RegisterUser)调用到一个Thread中,因为它是一个Thread itselt,它运行在主IU线程之外。只需调用execute方法。

而不是:

new Thread(new Runnable() {
            public void run() {
                new RegisterUser().execute("");
            }
        }).start();

做:

                new RegisterUser().execute("");

可能可以解决问题。

此外,您从DefaultHttpClient对象,第一行和第三行调用execute方法两次:

HTTPRESPONSE = httpclient.execute(HTTPPOST);
ResponseHandler<String> responseHandler = new BasicResponseHandler();
final String response = httpclient.execute(HTTPPOST, responseHandler);

答案 2 :(得分:0)

它不是一个真正的答案,但我将代码转换为使用JSON请求而不是httppost方法,它的工作效果要好得多。

Log.d("JSON", "Running the JSON script");
            JSONObject json = jsonParser.makeHttpRequest(url_register, "POST", params);

            //Getting specific Response
            try {
                RESPONSE_STATUS = json.getInt("success");
                RESPONSE_MESSAGE = json.getString("message");

            } catch (JSONException e) {
                RESPONSE_MESSAGE = "An error has occured during the registration";
                e.printStackTrace();
            }
            Log.d("JSON", json.toString());
            Log.d("Registration: ", "Response Status: " + RESPONSE_STATUS);
            Log.d("Registration: ", "Response: " + RESPONSE_MESSAGE);

            // Redirect the user depending on the response
            if (RESPONSE_STATUS == 1) {
                Log.d("Registration: ", "Registration successful");
                valid = "Successful";
            } else {
                Log.d("Registration: ", "Registration failed");
                valid = "Invalid";
            }

这确实需要完全重写php以将响应代码更改为数组。

同样,这不能解决我原来的问题,但它是一种可行的替代方法。