שלום,

אני בונה איזה משהו ועשיתי לו מין מודולים... בכל מודל יש משהו אחר. (לדוגמא: חדשות, הורדות וכו..)

כדי לבדוק מה המודל שהמשתמש בחר יש לי את הקוד הזה:

קוד PHP:

 $act = array(
news => "news"
); 
if($_GET["act"] == "")
{
$act = "news";
}
require "./modules/{$act}.php"; 


מתואר בקוד שאם לא נבחר מודל ( ACT ) אז המודל שהוא בוחר זה מודל news..
לפי הקוד, הכתובת שאמורה להיות אם בוחרים במודל news היא: index.php?act=news
עד לפה הכל בסדר.. אבל נגיד שמישהו בוחר לראות את הכתבה המורחבת שהכתובת שלה היא: index.php?act=news&op=full&id=1 אז הוא כבר מציג את השגיאה הזאת:

ציטוט:

Warning: main(./modules/Array.php): failed to open stream: No such file or directory in /home/*******/domains/*******/public_html/portal/index.php on line 94 Warning: main(./modules/Array.php): failed to open stream: No such file or directory in /home/*******/domains/*******/public_html/portal/index.php on line 94 Fatal error: main(): Failed opening required './modules/Array.php' (include_path='.:/usr/local/lib/php') in /home/*******/domains/*******/public_html/portal/index.php on line 94

לפי קובץ index.php, שורה 94 היא:

קוד PHP:

 require "./modules/{$act}.php"; 


מקווה שתעזרו לי, תודה.

טוב, בלבלת בין שני דברים שונים, דבר ראשון, למה:

קוד PHP:

 $act = array(
news => "news"
); 


??
הגדרת על המשתנה כמערך, עם תדפיס את הערך שלו זה יחזיר array, ובגלל זה רק

קוד PHP:

 $act['news'] 


יהיה את הערך news, בגלל זה, אם אין כלום ב-

קוד PHP:

 $_GET['act'] 


זה יעבוד כי הערך של המשתנה act$ יהיה news, אם לא התבצעה הפעולה, הערך שלו יהיה array, ולכן, שאתה כותב:

קוד PHP:

 require "./modules/{$act}.php"; 


הנתיב הוא modules/Array.php, אין קובץ שקוראים לו Array.php בתקיית modules, ובגלל זה זה לא פועל, עצה שלי, תלמד מה זה מערכים ואז תכתוב את כל הקובץ מחדש

אוקי, תודה על העצה.. בניתי קוד חדש:

קוד PHP:

 $act = $_GET["act"];
$choice = array(
'news' => "news"
);
if (!isset($choice[$act])) {
$act = "news";
}
require "./modules/{$choice[$act]}.php"; 


אך זה עדיין לא עובד

תודה

מה הודעת השגיאה שאתה מקבל עכשיו?
לפי דעתי, יותר נכון לעבוד כך:

קוד PHP:

 // Const modules
$incFiles = array(
    'default' => 'news.php',
    'news' => 'news.php',
    'cal' => 'events.php'
)
.
.


$action = isset($_GET['act']) ? $_GET['act'] : 'default';

if (!isset($incFiles[$action]))
    $action = 'default';

require_once 'modules/' . $incFiles[$action]; 


ניסיתי את הקוד שלך וגם הוא לא עובד

זה כאילו מריץ את הכל בסדר אבל כשמנסים לבחור באפשרות FULL זה משאיר אותו על בחירת המחדל יעני על דף הבית ולא משתנה

ואני שואל: למה להסתבך עם מערכים?
אתה יכול פשוט לעשות משהו כזה:

קוד PHP:

<?php
if(empty($_GET['act']))
{
    //הגדרת ברירת מחדל
    $_GET['act'] = 'news';
}
else
{
    //בהנחה שהצפינו את המשתנה, מפענחים אותו
    $_GET['act'] = base64_decode($_GET['act'])
}
include("modules/".$_GET['act'].".php");
?>

אל מנת לשפר את האבטחה, אפשר לעשות דברים כאלו:

קוד PHP:

 if(!file_exists("modules/".$_GET['act'].".php"))
    die("file is not found"); 


או לחשוב על משהו אחר לגמרי, למשל: בשביל מודולים לפעמים צריך יותר מקובץ אחד, בגלל זה, אפשר לעשות משהו כזה:

קוד PHP:

 include("modules/".$_GET['act']."/main.php"); 


כך שהמשתנה בעצם מכיל ספריה, שימושי לאבטחה

בשביל זה שמתי מערך, למנוע הרצה של קבצים לא רצויים (אלא רק מוגדרים מראש).

ואני אומר: יש דרך יותר יעילה

בגלל ה"הצפנה"?

זאת דרך יותר נוחה ומסודרת, אם רוצים להוסיף מודול אז צריך להכין אותו, לשים אותו במקום המתאים, להוסיף למערך עוד דברים, לאט לאט המערך מתחיל להתנפח ומתחיל להיות לא ריאלי, בדרך שלי צריך להכין את המודול, לשים אותו אותו במקום המתאים, להוסיף קישור וזהו...

ואיך תמנע ממנו להריץ קבצים לא רצויים?

נתתי לזה פתרון, שם הקובץ הוא קבוע, מה שמשתנה זה הספרייה

base64 היא לא הצפנה!
אני ממליץ לך לקרוא בויקיפדיה על base64.
ואם אתה רוצה לדעת איזה דרך הכי טוב להצפין את זה, אני ישאל למה להצפין את זה בכלל?

אני עובד בצורה אחרת, המטרה ב"הפרדה" הזו בשבילי היא יותר בשביל ההגיון, ובגלל שאני עובד עם דיזיין של MVC הact שלי הוא בעצם הView, אני עובד עם xml ולתגית action יש name וfile ועוד כמה מאפיינים כמו מחלקה לשימוש הMVC.
נ.ב אפשר לחסום את דף הxml על ידי .htaccess

אוקיייייי

תודה לכולם!! באמת ממש תודה!!

הסתדרתי

אגב, חבל להשתמש יותר מדי ב-if , בדיוק בשביל זה יש את המבנה switch-case