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

קוד:

if(a==1){
   if(b==1){
	  //...
   }else{
	  //...
   }
}else{
   if(b==1){
	  //...
   }else{
	  //...
   }
}

ואל תגידו לי שזה קוד יפה..

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

קוד:

		switch(   (a==1)
			+ .1*
			    (b==1)	 ){
			case 1.1:
				//....
				break;
			case 1.0:
				//....
				break;
			case 0.1:
				//....
				break;
			case 0.0:
				//....
				break;
		}

יש רעיונות טובים יותר?

אפשר גם ככה הרבה יותר פשוט
אבל נראה לי שswitch
יותר יעיל וגם נראה יותר טוב
{ }

קוד PHP:

 if((a==1) && (b==1))
              {     }
   elseif ((a==1) && (b<>1))
                {     }
     elseif ((a<>1)&&(b==1))
                  {     }
       else
       { } 


כן, אבל כמו שאמרת זה לא נראה טוב.

נ.ב תוך כדי חיפוש מצאתי את הטריק הזה

קוד PHP:

 switch (true) {
                case (cnt >= 10 && cnt <= 20):
                    alert('10');
                    break;
                case (cnt >= 21 && cnt <= 30):
                   alert('21');
                    break;
                case (cnt >= 31 && cnt <= 40):
                    break;

} 


לא בטוח שזה יותר יפה מ
if....else if
הרגיל, אבל זה יצירתי...

הנושא מתייחס לפן תכנותי טהור, ולכן אינו קשור לפורום בניית אתרים.
להבא, פתח את האשכול בפורום הנכון

הפורום הועבר לפורום תיכנות.

קח רעיון גאוני פשוט !
http://www.ketacode.com/post.asp?s=140

בהצלחה.

יקיר.
מילים לשירים חסידיים.

וואלה! התלהבתי

איך זה רעיון טוב?
יש שם do וwhile שלא משמשים כלולאה, איך זה הופך את הקוד ליותר קריא?

הגלריה שלי בפליקר

לא הצלחתי להבין את תגובתך, אתה בטוח שקראת את הודעתי?

וואו... עוד רעיון "גאוני" מבית מייקרסופט..
רעיון גאוני שלא משנה... כלום!
בכל מקרה אם יידרש תנאי, על תנאי, על תנאי יהיה בלאגן..
אין שום טעם בלולאה, בשביל זה בדיוק יש ELSE IF.

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

בקיצור אין טריקים, כאשר יש תנאים מורכבים(תנאים שאשכרה יושבים על תנאים אחרים, הקוד לא יראה יפה...)

לא הבנתי מה הבעיה, אתה צריך לבדוק 6-7 דברים ולהודיע הודעה אם אחד מהם משתבש, או להודיע שהכל תקין. יש לך רעיון קריא יותר מזה?

למה להשתמש בצורה כזאת קלוקלת בלולאה וב-break? אם כבר להשתמש ב-goto אבל גם זה לא צריך.
אפשרות א':

קוד:

if (someObject == null)
{
	printf("null pointer");
}
else if (checkSomething() < 5)
{
	printf("less than 5");
}
else if (checkSomethingElse() < 5)
{
	printf("also less than 5");
}
else if (initFunction() != RESULT_OK)
{
	printf("failed to init");
}
else if (doSomething() != RESULT_OK)
{
	printf("failed to do");
}
else
{
	printf("Success!");
}

אפשרות ב' (אם רוצים שהמהלך התקין יהיה יותר דומיננטי):

קוד:

if (check())
{
	printf("Success!");
}
...

check()
{
	if (someObject == null)
	{
		printf("null pointer");
		return 0;
	}
	
	if (checkSomething() < 5)
	{
		printf("less than 5");
		return 0;
	}
	
	if (checkSomethingElse() < 5)
	{
		printf("also less than 5");
		return 0;
	}
	
	if (initFunction() != RESULT_OK)
	{
		printf("failed to init");
		return 0;
	}
	
	if (doSomething() != RESULT_OK)
	{
		printf("failed to do");
		return 0;
	}
	
	return 1;
}

לא לא!
בוא נהיה מגנבים! נעשה לולאה...

את האופציה השנייה הוא גם הציע

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

קוד:

if (x && y) {
    ...
    return;
}
if (!x && y) {
    ...
    return;
}
if (x && !y) {
    ...
    return;
}
if (!x && !y) {
    ...
    return;
}

ככה גם מאוד ברור מה כל תנאי עושה ולא צריך לנחש (if-else יכול להיות מאוד מבלבל) ולהתחיל לעשות תרשימים כדי להבין בדיוק למה מתייחס התנאי.
בנוסף גם אין את הבעיה של שמרוב הזחות הקוד מתחיל באמצע המסך (כמו למשל בדוגמא של boynett).

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

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

ציטוט:

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

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

ציטוט:

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

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

קוד:

if (A || B)
{
  if (A && B)
  {
	 // BOTH
  }
  elseif (A)
  {
	// A
  }
  else
  {
	 // B
  }
}
else
{
  // NOTHING
}

נ.ב:
תמיד יותר קל לקרוא וגם לכתוב תנאים בצורה חיובית, כלומר ללא NOT
ובנוגע להבנת ה"טריק" שמאחורי, מספר הערות פשוטות, כמו שכדא שיהיו במקרים כאלו סוגרות את הפינה בקלות.

ציטוט:

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

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

בדיקת תנאי? אתה באמת חושב שזה מה שישפיע על היעילות של התוכנית שלך? (בהנחה שאתה לא עושה משהו מטורף בבדיקה הזאת...)
בכל מקרה, גם עם הערות, תכתוב בצורה שתיצור כמה שפחות בלוקים בתוך בלוקים. אולי בדוגמא שציינת יש לך רק 2 בלוקים אחד בתוך השני, אבל תחשוב על מצב שתצטרך עוד כמה כאלה בתוך אותם בלוקים, ותוכל לשכוח מקוד ברור למישהו שנתקל בו בפעם הראשונה.

ציטוט:

במקור נכתב על ידי gogofo נ.ב: תמיד יותר קל לקרוא וגם לכתוב תנאים בצורה חיובית, כלומר ללא NOT

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

אם לדוגמא יש לך משתנה דגלון שמסמל האם איזה אובייקט כלשהו נמצא, ההגיון יהיה לכנות אותו בשם found או משהו דומה, ואז ברור שהביטוי ‎!found משמעותו "לא נמצא".
או לדוגמא פונקציה הבודקת אם מספר טבעי הוא ראשוני, אז בכינוי המתאים, לדעתי לפחות, די ברור שביטוי כדוגמת ‎!isPrime(n)‎ משמעותו "n אינו ראשוני".

תוספים לפורום
קורס לטינית בלעדי לפרש, בפורום שפות ובלשנות.

כן, אבל ברור שעדיף

קוד:

if(isPrime(n) ){
 
}else{

}

מאשר

קוד:

if(!isPrime(n) ){
 
}else{
    
}

כן, רק שפה נתנו דוגמא לשימוש בשניהם ביחד.

כגון:

קוד:

var n_is_prime = isPrime(n);

if ( found && n_is_prime ) {
    // command set 1;
}

else if ( !found && n_is_prime ) {
    // command set 2;
}

else if ( found && !n_is_prime ) {
    // command set 3;
}

else {
    // command set 4;
}

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

תוספים לפורום
קורס לטינית בלעדי לפרש, בפורום שפות ובלשנות.

ציטוט:

במקור נכתב על ידי ShoobyD מצטער, לא מסכים. פה בדיוק נכנסת החשיבות שבמתן שמות בעלי משמעות למשתנים/פונקציות.. אם לדוגמא יש לך משתנה דגלון שמסמל האם איזה אובייקט כלשהו נמצא, ההגיון יהיה לכנות אותו בשם found או משהו דומה, ואז ברור שהביטוי ‎!found משמעותו "לא נמצא". או לדוגמא פונקציה הבודקת אם מספר טבעי הוא ראשוני, אז בכינוי המתאים, לדעתי לפחות, די ברור שביטוי כדוגמת ‎!isPrime(n)‎ משמעותו "n אינו ראשוני".

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

אם כבר, יותר קריא לטעמי:

קוד:

enum TwoTermsFlags { None = 0, First = 1, Second = 2, Both = 3 };
#define TwoTerms(term1, term2) (TwoTermsFlags)((term1) | (term2) << 1)
...
switch(TwoTerms(term1, term2))
{
	case None:
		...
		break;
	case First:
		...
		break;
	case Second:
		...
		break;
	case Both:
		...
		break;
}