אני משתגע כבר, עברתי על הקוד כל-כך הרבה פעמים, הרצתי ב-JAVA ועובד, ואין לי מושג למה זה לא עובד ב-PHP.
הפונקציה אמורה לקבל מחרוזת, ולבדוק אם היא שווה לאחת המחרוזות שבתוך מערך שהיא יוצרת:
אם כן, להחזיר True. ואם לא, להחזיר False.
זה הכל! O_O

הנה הפונקציה ב-PHP:

קוד PHP:

 function test($settings){
    
    $count = 0;
    $status = false;
    $array = array(0 => "morning", 1 => "afternoon", 2 => "evening");
    for($i = 0; $i < count($array); $i++){
        if($settings == $array[$i]){
            $count++;            
        }        
    }
    if($count == 1){
        $status = true;
    }
    
    return $status;    
            
} 


קלט:

קוד:

echo test("morning");
echo test("dsagsdgsdsdg");

פלט:

קוד:

1

הפונקציה ב-Java (לאחר המרה של הפונקציה לשפת Java):

קוד PHP:

 public static boolean test(String settings){

    int count = 0;
    boolean status = false;
        String array[] = {"morning", "afternoon", "evening"};
    for(int i = 0; i < array.length; i++){
        if(settings.equals( array[i])){
            count++;
        }
    }
    if(count == 1){
        status = true;
    }

    return status;

}

} 


קלט:

קוד:

		System.out.println(test("morning"));
		System.out.println(test("dsagsdgsdsdg"));

פלט:

קוד:

true
false

למישהו יש רעיון למה ב-PHP הוא מחזיר פלט כזה? וב-Java מחזיר פלט כמו שהוא אמור להיות?





תודה מראש,
orl2006. =]

משום שאתה לא יכול להדפיס את הערך הלוגי false בפקודת echo...

אני משער שהוא כן מנסה להדפיס את זה, ופשוט עושה cast ל string. כיוון ש 0 ב ascii שווה ל null, לא אמור להיות מודפס שום דבר חזותי (אבל פליטה לקובץ ובדיקה ב hex dump או סתם העברה ל hexdump עם pipe אם יש לך במערכת ההפעלה שלך תוכל לאמת טענה זו...).

הדרך הנכונה לבדוק ערך בוליאני שהוחזר מפונקציה היא להשתמש ב... if, לפחות בכל שפות התכנות שאני מכיר, ובהתאם ל if לעשות מה שרוצים (למשל להדפיס 1...).

בכל אופן, אם אתה מתעקש לעבוד כך, אם תעשה cast ל int, זה יעבוד...

קוד PHP:

 echo (int) test("dsagsdgsdsdg"); 


כמובן שכל הקוד שלך מבוסס על חשיבה ג'אווה-אית ולכן אתה מממש מחדש משהו ש PHP כבר יודעת לעשות בקוד הרבה יותר קצר...

הקוד הבא שקול לקוד שלך:

קוד PHP:

<?php

function test($settings)
 {   
    return (in_array($settings, array("morning", "afternoon", "evening")));
 }  

echo (int) test("morning");
echo (int) test("dsagsdgsdsdg");
?>

נמאס לכם לזכור סיסמאות? לחצו כאן!

למרות ששימי כבר הראה לך קוד מהיר/קצר יותר שעובד, אני חייב להעיר לך על משהו:

בקוד שלך אתה עובר על כול המערך, ובמידה ואתה מוצא את הערך המבוקש, אתה מעלה את count ב1, ובסוף הלולאה (שסיימנו לרוץ על המערך), אתה בודק אם count שווה ל-1.

כמה הצעות ייעול קטנות:

את הבדיקה אם count שווה ל1 תעשה בתוך הלולאה, ככה אם האיבר נמצא בתחילת המערך, זה יחסוך ממך לרוץ על כול המערך.

אומנם אם תעשה את מה שאמרתי, אז ההצעה הבאה לא רלוונטית כבר אבל בכו"ז:
אתה בודק אם count שווה ל1 אבל מה אם האיבר הופיע יותר מפעם אחת במערך?

בהצלחה.