2017-07-14 6 views
1

Ajaxを使用したフォームで作業していましたが、これまでにエクストラのないバージョン(cssなど)で作業していました。それはすべて正常に動作し、データはデータベースに正常に挿入され、私は2つのdivを表示して非表示にすることができました。 私はこれまで作業していたフォームにそれを適用しました。これはPHPファイルからの "成功メッセージ"のように、まったく同じ(確かに、いくつかの名前を変更し、いくつかの入力を追加した)ので、突然すべてのデータがURLに表示されるので、現在のフォームは、次のものを隠して表示します。同じ構造ですが、Ajaxの成功レスポンスは以前のバージョンとは異なる動作をします

私は突然の行動の変化を理解することができない、間違いを見て、コードを比較したが、わからない。それは私がそれを見つけたり、何かが全体の建設に間違っているような小さな間違いのようです。

現在のファイルがある:

<html> 
<head> 
<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.2.1/jquery.min.js"></script> 
<? 
require 'config.php'; 
session_start(); 

// Check if user is logged in using the session variable 
if ($_SESSION['logged_in'] != 1) { 
    $_SESSION['message'] = "You must log in before viewing your profile page!"; 
    header("location: error.php");  
} 
else { 
    // Makes it easier to read 

    $id = $_SESSION['id']; 
    $name = $_SESSION['name']; 
    $email = $_SESSION['email']; 
    $active = $_SESSION['active']; 
    $hash = $_SESSION['hash']; 
} 


?> 


<script type="text/javascript"> 
function getState(val) { 
    $.ajax({ 
    type: "POST", 
    url: "demo_ajax.php", 
    data:'country_id='+val, 
    success: function(data){ 
    $("#region").html(data); 
    } 
    }); 
} 

$(document).ready(function(){ 
    $("#submit").click(function(){ 

    var size=$("#size").val(); 
    var industry=$("#industry").val(); 
    var country=$("#country").val(); 
    var region=$("#region").val(); 
    var url=$("#website").val();  
    var fb=$("#fb").val(); 
    var lkdn=$("#lkdn").val(); 

    $.ajax({ 
     type:"post", 
     url:"process2.php", 
     data:"size="+size+"&industry="+industry+"&country="+country+"&region="+region+"&url="+url+"&fb="+fb+"&lkdn="+lkdn, 
     success:function(data){ 
     $("#theform").hide(); 
     $("#info").html(data); 
     //$("#partone").css();  
     $("#partone").show(); 

     alert("Hello"); 
    } 
    }); 
    }); 
}); 




</script> 
    <?php include 'js/js.html'; ?> 
    <?php include 'css/css.html'; ?> 

</head> 

<body class="w3-blue r_login_corp_body"> 
<div id="info" style="color:white"></div> 
<div class="r_login_corp_body"></div> 

<div class="w3-content w3-white r_siu r_centered_div"> 
     <header class="w3-camo-black w3-container"> 
      <div class="w3-container "> 
       <span class="w3-xlarge r_caption">eRecruiter</span>   <span class="large">Corporate Login</span> 
      </div> 

      <div class="w3-black"> 
       <a href="javascript:void(0)" onclick="selectForm('register');"> 
        <div class="w3-half tablink w3-hover-text-yellow w3-padding w3-center w3-padding-16">Register</div> 
       </a> 
      </div>    
     </header> 

    <!-- Register --> 
    <div id="register" role="form" class="r_form_elements"> 
     <form name="formone" class="form" autocomplete="off"> 
      <div id="profed" class="w3-container w3-padding-16"> 
       <div class="alert alert-error"></div> 

       <label>Company Industry</label> 
       <input class="w3-input" name="industry" id="industry" type="text" placeholder="Your Industry" > 

       <label>Company Size</label> 
       <input class="w3-input" name="size" id="size" type="integer" placeholder="Your Company Size" > 

       <label >Country:</label> 
       <select name="country" id="country" class="demoInputBox" onChange="getState(this.value);" > 
        <option value="">Select Country</option> 
        <?php  
         $sql1="SELECT * FROM pentagonal_country"; 
          $results=$mysqli->query($sql1); 
         while($rs=$results->fetch_assoc()) { 
         ?> 
         <option value="<?php echo $rs["country_code"]; ?>"><?php echo $rs["country_name"]; ?></option> 
         <?php 
         } 
        ?> 
       </select> 

       <label>State:</label> 
       <select id="region" name="region" onKeyup="checkform()"> 
       <option value="">Select State</option> 
       </select> 

       <label>Website</label> 
       <input class="w3-input" name="website" id="website" type="url" placeholder="Your Website-Address" > 

       <label>Facebook</label> 
       <input class="w3-input" name="fb" id="fb" type="url" placeholder="https://facebook.com/" > 

       <label>Linkedin</label> 
       <input class="w3-input" name="lkdn" id="lkdn" type="url" placeholder="https://linkedin.com/in/">          
      </div> 
     <div class="w3-row"> 
     <button type="submit" id="submit" class="w3-button w3-black w3-half w3-hover-yellow" >Add</button> 
     <button class="w3-button w3-black w3-half w3-hover-pale-yellow">Forgot Password</button> 

     </div> 
    </form> 
    </div> 
    <!-- Register --> 
    <div id="partone" style="display:none"> 
     <form> 
       name : <input type="text" name="name" id="name"> 
       </br> 
       message : <input type="text" name="message" id="message"> 
       </br> 

       </br> 
       name : <input type="text" name="url" id="url"> 
       </br> 
       message : <input type="text" name="fb" id="fb"> 
       </br> 
       name : <input type="text" name="lkdn" id="lkdn"> 
       </br> 

       </br>            </br> 
       Send; 

     </form>  
</div> 
</div> 


</body> 
</html> 

とPHPファイルのデータを挿入するには、次のとおりです。

<?php 
$servername = "localhost"; 
$username = "root"; 
$password = ""; 
$dbname = "remotejobs"; 

session_start(); 

// Check if user is logged in using the session variable 
if ($_SESSION['logged_in'] != 1) { 
    $_SESSION['message'] = "You must log in before viewing your profile page!"; 
    header("location: error.php");  
} 
else { 
    // Makes it easier to read 

    $id = $_SESSION['id']; 
    $name = $_SESSION['name']; 
    $email = $_SESSION['email']; 
    $active = $_SESSION['active']; 
    $hash = $_SESSION['hash']; 
} 

// Create connection 
$conn = mysqli_connect($servername, $username, $password, $dbname); 
// Check connection 
if (!$conn) { 
    die("Connection failed: " . mysqli_connect_error()); 
} 

    $industry=$_POST["industry"]; 
    $size=$_POST["size"]; 
    $country=$_POST["country"]; 
    $region=$_POST["region"];  
    $website=$_POST["url"]; 
    $fb=$_POST["fb"]; 
    $lkdn=$_POST["lkdn"]; 
    $usrid=$id; 

$sql = "INSERT INTO corp_user_profile (id, industry, size, nation, region, url, facebook, linkedin) 
VALUES ('$usrid', '$industry','$size', '$country', '$region', '$website', '$fb', '$lkdn')"; 


if (mysqli_query($conn, $sql)) { 
    echo "New record created successfully"; 
} else { 
    echo "Error: " . $sql . "<br>" . mysqli_error($conn); 
} 

mysqli_close($conn); 
?> 

私は確かに私が一緒に働いてきた以前のファイルを操作するために使用バグを修正してから1週間後にはすべてのことが正しいということです。

誰かが問題がどこにあるのか教えていただけますか?おそらく、このような将来の問題を避けるのは間違いでしょうか?

+0

[ブラウザの開発者ツールでAJAX要求/応答を監視しましたか?プロジェクトにjQueryライブラリを含めましたか?エラーが報告されていますか?あなたはこれをWebサーバーで実行していますか?](http://jayblanchard.net/basics_of_jquery_ajax.html) –

+0

[Little Bobby](http://bobby-tables.com/)は*** [あなたのスクリプトはSQLインジェクション攻撃のリスク。](http://stackoverflow.com/questions/60174/how-can-i-prevent-sql-injection-in-php)*** [準備済み]について学ぶ(http:// en .wikipedia.org/wiki/Prepared_statement)の[MySQLi]に関する記述(http://php.net/manual/en/mysqli.quickstart.prepared-statements.php)を参照してください。 [文字列をエスケープする](http://stackoverflow.com/questions/5741187/sql-injection-that-gets-around-mysql-real-escape-string)でも安全ではありません! –

+0

セッション変数はどこで設定していますか? –

答えて

1

最も顕著なバグ(上記のSQLインジェクションスタッフ以外にも)は、 <button type="submit"は、スクリプトを使用して防止する場合を除き、ポストバックを介してフォームが正常にサブミットされることです。 「クリック」ハンドラの最初の行にevent.preventDefault()を追加します。フォームのマークアップで指定された他の方法はありませんので、とGETをやって(Ajaxは実行する機会を持つ前に)、およびGETフォームが正常に掲示されているため、あなたはURL内のデータを見ている

$("#submit").click(function(event){ 
    event.preventDefault(); //prevent default postback behaviour 
    var size=$("#size").val(); 
//...etc 

があります

+1

それは動作します!解決と説明のためにADysonに感謝します。今それは動作します。そして私は何かを学んだ。 – nucky

1

イベントをclick関数に渡してevent.preventDefault()を呼び出すことで、デフォルトの動作を防ぐことができます。

+0

"フォームを送信すると、相対URLを使用しているため、ajaxのURLが異なるようになるページのURLが変更されている可能性があります。これは無関係です。a)フォームが指定されていないため、アクションが指定されていないので、URLを同じにして、ポストバックします。b)ページをポストすると実行中のjavascriptが破棄されるため、ajaxは決して実行されません。その上に。次回の更新時に、同じスクリプトが復元されたとしても、実行する機会は決して決してありません。 – ADyson

+0

あなたがそのURLについて正しいので、その行を削除しました。私はちょうどテストケースを作ったし、Ajaxリクエストは間違いなく引き続き発生します。 – vi5ion

+0

これは、ページが_immediately_を返信しない場合には当てはまりますので、私の前のステートメントはあまりにも絶対的かもしれないと同意しましたが、完全なポストバックの前には決して返事を得ないでしょう。同じデータを2回送信することで、サーバー上のデータが壊れてしまうことはありません。いずれにしても、私たちはどちらも同じソリューションを思いつきましたが、私はそれを解決すると確信しています。 – ADyson

関連する問題