[Tomer]

נכתב במקור על ידי MasterT

קטע קטע, נתחיל מהיעילות וסידור הקוד.

PHP קוד:

if($a == 1){
$b = str_replace("1","ראשון",$a);
}
if($a == 2){
$b = str_replace("2","שני",$a);
}
if($a == 3){
$b = str_replace("3","שלישי",$a);
}
if($a == 4){
$b = str_replace("4","רביעי",$a);
}
if($a == 5){
$b = str_replace("5","חמישי",$a);
}
if($a == 6){
$b = str_replace("6","שישי",$a);
}
if($a == 7){
$b = str_replace("7","שבת",$a);
} 


גם בזבוז של משאבים, ויש דרך הרבה יותר טובה.

PHP קוד:

$days = array("ראשון","שני","שלישי","רביעי","חמישי","שישי","שבת");
if(intval($a) == $a && $a > 0 && $a < 8) {
$b = $days[$a - 1];
} 


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

בשביל קוד כזה מחלקות? אתה רציני?