تحسين كود PHP لتخزين بيانات استمارة التسجيل من HTML

In order to address the issue you’re facing with your PHP code for handling form registration data, let’s carefully examine the provided code snippets and identify potential areas for improvement.

Firstly, in your HTML form, you are trying to send data to the server using the GET method, and you’ve included parameters in the action URL. However, in your PHP code on the other server, you are attempting to access the values using incorrect keys. Specifically, in the PHP code, you are using $_GET["fname"] and $_GET["lname"], but in the HTML form, the input names are “firstname” and “lastname”.

Here’s a corrected version of your PHP code:

phpCopy code
 
$txt = "reg.log";  

if (isset($_GET["log"]) && isset($_GET["firstname"]) && isset($_GET["lastname"])) { 
    $firstname = $_GET["firstname"]; 
    $lastname = $_GET["lastname"]; 

    // Additional data you might want to capture
    $email = isset($_GET["email"]) ? $_GET["email"] : "";
    $password = isset($_GET["password"]) ? $_GET["password"] : "";

    // Output the values for testing
    echo $firstname . PHP_EOL . $lastname . PHP_EOL . $email . PHP_EOL . $password;

    // Open the file in append mode
    $fh = fopen($txt, 'a');  

    // Write information to the file
    fwrite($fh, $firstname . ' ' . $lastname . ' ' . $email . ' ' . $password . PHP_EOL);

    // Close the file
    fclose($fh);
} 
?>

In this corrected version, I’ve adjusted the variable names to match the ones used in the HTML form, and I’ve included optional checks for the email and password parameters. Additionally, I’ve modified the fwrite function to concatenate the values and include a line break (PHP_EOL) after each set of information.

This should help you capture the information from the URL parameters and write it to your log file accurately. Please ensure that the file has the necessary write permissions for the PHP script to update it.

I hope this provides the clarity you need. If you have any further questions or encounter issues, feel free to ask for assistance.

بالطبع، دعنا نقوم بتوسيع النقاش حول المشكلة التي تواجهك في تنفيذ عملية تسجيل البيانات من النموذج HTML إلى السيرفر الآخر باستخدام لغة PHP.

أولاً وقبل الدخول في تفاصيل أكثر، يُفضل دائماً في تطوير البرمجيات الويب استخدام آليات أمان صحيحة. في الشيفرة التي قدمتها، لاحظت أنك تقوم بكتابة البيانات المُدخلة مباشرة إلى ملف نصي (reg.log). من الأفضل دائماً فحص وتنقية البيانات المُدخلة لتجنب هجمات الحقن.

يمكنك تحسين ذلك باستخدام دوال مثل htmlspecialchars() لتحويل الرموز غير الآمنة إلى أكواد HTML، وذلك لضمان سلامة البيانات المُدخلة.

فيما يخص قراءة القيم من URL، يجب عليك التحقق من وجود هذه القيم في العنوان بشكل صحيح. يمكن أن يؤدي وجود متغيرات غير مُعرفة في عنوان URL إلى أمور غير متوقعة. استخدم دائماً الدوال مثل isset() للتحقق من وجود قيم قبل استخدامها.

اليك تحسينات إضافية للشيفرة:

phpCopy code
 
$txt = "reg.log";  

if (isset($_GET["log"], $_GET["firstname"], $_GET["lastname"])) { 
    $firstname = htmlspecialchars($_GET["firstname"]);
    $lastname = htmlspecialchars($_GET["lastname"]);

    // Additional data you might want to capture
    $email = isset($_GET["email"]) ? htmlspecialchars($_GET["email"]) : "";
    $password = isset($_GET["password"]) ? htmlspecialchars($_GET["password"]) : "";

    // Output the values for testing
    echo $firstname . PHP_EOL . $lastname . PHP_EOL . $email . PHP_EOL . $password;

    // Open the file in append mode
    $fh = fopen($txt, 'a');  

    // Write information to the file
    fwrite($fh, $firstname . ' ' . $lastname . ' ' . $email . ' ' . $password . PHP_EOL);

    // Close the file
    fclose($fh);
} 
?>

تأكد من أن هناك صلاحيات كتابة للملف reg.log وأن الشيفرة تعمل في سياق الأمان الملائم لتطبيق الويب الخاص بك.

إذا كنت بحاجة إلى المزيد من التوضيح أو لديك أي أسئلة إضافية، فلا تتردد في طرحها.