שלום חבר'ה תגידו מה אומר הקוד הבא?

header("Location: index.php?$sn=$sid");

בדף כתוב לי שיש שגיאה בשורה הזו.

"כמים הפנים לפנים כך לב האדם לאדם"

הקוד הבא אומר לדפדפן לעבור לכתובת index.php עם הפרמטרים שכתובים שם (GET) ...
אם תגיד מה ה- ERROR אולי אני אוכל לעזור יותר.

<?php
include("../admin/config.php");
include("$include_path/functions.php");

include("$include_path/session.php");
include("$include_path/common.php");

sql == ""

select
*
from
$tb_users
where
username = '$UN'
and
password = password('$PW')
";

$query = sql_query($sql);

if(sql_num_rows($query) == 1){

if($array = sql_fetch_array($query)){

$username = $array["username"];

session_register("username");

$userid = $array["id"];

session_register("userid");

}
}

header("Location: index.php?$sn=$sid");
(זו השורה המדוברת משום מה השורות לא מסודרות נכון אבל זו השורה שיש בה בעיה שורה 12)

?>



השגיאה היא:

Parse error: syntax error, unexpected T_STRING in /login.php on line 12

אמרת שזה פעולת שליחה של הget.
איפה אני אמור לשים אותו??

"כמים הפנים לפנים כך לב האדם לאדם"

יש פה בעיות עם המרכאות שלך ... תבדוק את זה.

קוד PHP:

 sql == "" 
 
select 
*
from
$tb_users
where
username = '$UN'
and
password = password('$PW')
"; 


מה לעזאזל קורה כאן?!

• בהתחלה אתה בכלל שם אופרטור השמה (==) שבכלל אי אפשר לשים אותו ככה סתם.
• אחרי שהגדרת את המשתנה ככלום (לא באמת הגדרת, אבל זה בגלל טעות תחביר), אתה רושם את השאילתה סתם, כשהיא לא מוגדרת לשום משתנה.
• אני לא מכיר שום פונקצית SQL בשם password - אם זו פונקצית PHP אז צריך להיות מחוץ למרכאות - משורשר.
• ובנוסף לפונקציה יש גם להוציא את המשתנים מהמרכאות ולשרשר אותם.
• אתה רושם את כל פקודות ה-WHERE, AND) SQL וכו') באותיות קטנות, אני לא יודע אם זה עובד, אבל מה שכן זה בטוח לא מומלץ.
• זו בכלל לא השורה שאתה מדבר עליה, זה לא השורות מבולבלות, זה אתה.
• אף פעם לא הבנתי, למה אנשים אוהבים להגדיר את השאילתה למשתנה כשהם משתמשים בשאילתה הזו בדיוק פעם אחת.
  אין שום סיבה לשמור את השאילתה למשתנה אם השאילתה הזו תהיה בשימוש רק פעם אחת (רוב הפעמים) - עדיף לרשום אותה ישר לפונקציה.
• פעם הבאה תרשום את הקוד בתוך תגיות הPHP בשביל שיהיה לנו קצת יותר קל.
• פעם הבאה (או שמנהל כבר ימחוק את האשכול הזה) תרשום כותרת עניינית, הרי כולם פה צריכים עזרה בקוד PHP.

בקיצור - בלאגן אחד גדול.

EVERYTHING SHOULD BE MADE AS SIMPLE AS POSSIBLE, BUT NOT ONE BIT SIMPLER
ALBERT EINSTEIN

ככה צריך להראות ה- SQL שלך לפי מה שכתבת:

קוד:

 sql = "SELECT * FROM $tb_users WHERE username='$UN' AND password =" . password('$PW');

למרות שאם הסיסמה שלך היא לא מספר אז זה צריך להראות ככה:

קוד:

 sql = "SELECT * FROM $tb_users WHERE username='$UN' AND password ='" . password('$PW') . "'";

לא נכון, יש סיסמא ב- SQL שקוראים לה password...

זאת הצפנה חד-כיוונית דומה ל- MD5, וכבר לא ממש משתמשים בה, אבל היא קיימת!

אוקיי ... אז לא ידעתי, מצטער.

יותר נכון יהיה להגיד של-MySQL ישנה פונקציה שנקראת password שהיא פונקציית hash (בדומה ל-md5) ולכן ניתן לשלוח אליה מחרוזות על מנת לשמור ב-DB ערך שמאפשר בדיקת נכונות לסיסמא.
PostgreSQL למשל (עד כמה שזכור לי) לא מכילה את הפונקציה password, ולכן השאילתא המדוברת לא תעבוד בה.

בברכה,
דקל

"מתכנת זה אדם שפותר בעיה שעל קיומה לא ידעת, בדרך שאינך מבין"