دالة member من تصميمي

[alsohba]

يا إخوان أحتاج إلى مساعدة من الإخوة المبرمجين .. آنة سويت دالة member التي تتأكد من تسجيل دخول المستخدم .. ولكن المشكلة فيها خلل بسيط .. و آنة ما عرفته .. أحتاج مساعدتكم في تصحيح هذا الخطأ .. و التشييك على الثغرات الأمنية .. و شكرا

function member($username, $password, $page)
{
$user = trim($username);
$pass = trim($password);

    if(!$user && !$pass)
        {
        echo "Your Username or Password is not correctly. Please, Go back and try again 1";
        }
    else
        {
        $query="SELECT user, pass FROM stud WHERE user = '$user' AND pass = password('$pass')";
        $result=mysql_query($query);

        if(!$result)
            {
            echo"Your Username or Password is not correctly. Please, Go back and try again 2";
            }
        else
            {
            $_SESSION['user'] = $_POST[user];
            $_SESSION['pass'] = $_POST[pass];
            echo $page;
            }
        }
} 

هذا هو الكود أرجو المساعدة في أقرب وقت ممكن ..

[محمود حسين]

السلام عليكم ورحمة الله وبركاته...

بالنسبة للكود فلديك خطأ في هذا الجزء

$_SESSION['user'] = $_POST[user]; 
$_SESSION['pass'] = $_POST[pass]; 

والذي يجب استبداله بهذا الجزء:

$_SESSION['user'] = $result['user']; 
$_SESSION['pass'] = $result['pass']; 

جرب وأخبرني بالنتيجة

تحياتي

[na3im]

كما قال اخي محمود

وهناك ملاحظة أخرى:

يمكن استبدال:

function member($username, $password, $page) 
{ 
$user = trim($username); 
$pass = trim($password); 

    if(!$user && !$pass) 
        { 
        echo "Your Username or Password is not correctly. Please, Go back and try again 1"; 
        } 

اقتراح:


قبل استدعاء الدالة :

$username = strip_tags($_POST['username']); 
$password = strip_tags($_POST['password']); 


الدالة:
function member($username, $password, $page) 
{
$user = trim($username); 
$pass = trim($password); 

    if(!isset($user) OR !isset($pass))
        { 
        echo "Your Username or Password is not correctly. Please, Go back and try again 1"; 
        } 

[alsohba]

شكرا على مروركم .. ولكن المشكلة حسب علمي ليست فيما ذكرتما !!!
المشكلة في هذه النقطة ..

          }
    else
        {
        $query="SELECT user, pass FROM stud WHERE user = '$user' AND pass = password('$pass')";
        $result=mysql_query($query);

        if(!$result)
            {
            echo"Your Username or Password is not correctly. Please, Go back and try again 2";
            } 

لأني كلما سجلت الدخول باسم غير مسجل بقاعدة البيانات .. يعرض لي المتغير $page مباشرة !!
أما إذا لم أدخل شيئاً فإنه يعرض لي الخطأ الذي أعطيته أنا الرقم 1 ..
فعلى حد علمي القاصر أن if(!$user OR !$pass) قامت بعملها على أكمل وجه .. ولكن المشكلة في else على حد علمي و ما بعدها ..

[alsohba]

أما عن $query فهو فعال .. لأني جربته في phpmyadmin وكان فعالا ..

[na3im]

الدالة التي كتبتها بها أخطاء كثيرة...

لم تستخرج البيانات من القاعدة...

اضف هذه الاكواد:

else
    {
        $sql = mysql_query("SELECT user, pass FROM stud WHERE user = '$user' AND pass = '$pass'")
        or die(mysql_error());
        $num = mysql_num_rows($sql);
        
        
        if($num ==0)
         {
           echo 'Your Username or Password is not correctly. Please, Go back and try again';
         }
         else
         {
         $result = mysql_fetch_object($sql)
         $user = $result->user;
         $pass = $result->pass;
        
          $_SESSION['user'] = $user;
          $_SESSION['pass'] = $pass;
        
        }
        mysql_free_result($sql);

} 

[alsohba]

طيس أخي الغالي نعيم ..
ممكن تشرح لي الإضافات التي قمت بإضافتها ..

أختصر لك الموضوع أكثر ..
mysql_fetch_object($sql)
$user = $result->user;
$pass = $result->pass;
mysql_free_result($sql);

that's all thank you >>

[na3im]

من عيوني اخي.

mysql_fetch_object($sql)
$user = $result->user;
$pass = $result->pass;
mysql_free_result($sql); 

كما تفصلت انت تجرب الاستعلام في phpmyadmin ...
لكن اذا صممت صفحة ويب بالطيع ستحتاج لاستعمال دوال تقوم بهذه العملية اي استخراج نتائج بعد تنفيذ الاستعلام..
لهذه المهمة هناك طرق عدة..

انا استعملت دالة mysql_fetch_object اعتقد انها الافضل...

لم استخدم اي حلقة تكرار لان الهدف استخراج عضو وحيد لذلك كتبنا
mysql_fetch_object($sql) على كود $sql
بعدها نختار متغيرين جديدين وهما مثلا

$user و $pass 

نحفظ فيهما القيم التي استخرجناها من قاعدة البيانات عن طريق الاسناد:

$result->user 

و

$result->$pass 

ارجو ان تكون الفكرة وصلت

[na3im]

دالة

mysql_free_result($sql); 

لافراغ الذاكرة من الاستعلام

[alsohba]

هل تقصد أخي الحبيب أن $result->$pass تغنينا عن كتابة
$row['pass'] ..

أخي أنا أركز على -> لأني ما أعرف ايش هو عملها .. إذا كان في درس عربي يعلمنا كيف نستخدمها و كذلك أين تستخدمها أرجوا كتابة الرابط حتى تعم الفائدة .. ولأني ضعيف جداً باللغة الإنجليزية ..

أو أني أستميحك عذرا .. بأن تكتب لي شرح أكثر تفصيلاً عنها ..

[na3im]

اهلا اخي

تستطيع استعمال دالة mysql_fetch_array في هذه الحالة سيكون النتائج على شكل مصفوفة..
وتستعمل هذه الطريقة:

$sql = mysql_query('SELECT .....');
$row = mysql_fetch_array($sql);

$newUser = $row['user'];
$newPass = $row['pass'];
........
unset($sql); 

اذا اردت استعمال الدالة mysql_fetch_object

فالطريقة تختلف قليلا

$sql = mysql_query('SELECT .....');
$row = mysql_fetch_Object($sql);

$newUser = $row->user;
$newPass = $row->pass;

unset($sql); 

-> هذا الرمز يستعمل كثيرا في البرمجة بالكائنات OOP وظيفته اسناد قيمة لمتغير

ارجو ان تكون الصورة قد توضحت

[alsohba]

أكن إن الإنسان بالفعل كلما تعلم شيء .. عرف إنه جاهل .. هههه .. و أنا أول مرة أسمع بالــ OOP هذي ..

إذا أمكن اندور شرح مع بعض لها .. حتى اتعم الفائدة

[na3im]

يقول الشاعر العربي:
كلما ازددت علما --- زادني علما بجهلي.

لا يمكن ان تجد انسانا عاقلا يقول انه يعلم كل شيء سواء في البرمجة او في شيء آخر...


تستطيع اخي الاطلاع على الدروس من بعض المواقع اذا احببت اضعها لك

وشكرا مجددا