שלום לכם ושנה

אממ נתנו לנו לרשום תרגיל שהמשתמש מכניס מספר תלת ספרתי
ומספר והפלט הוא כמה פעמים מופיע המספר שהוא הכניס במספר התלת ספרתי..!
רשמתי את הקוד הבא:

קוד:

  
using System;
class Alon
{
	public static void Main()
	{
		int tnumber, number, achadot, asarot, meot;
		Console.WriteLine("enter number between 100-999");
		tnumber = int.Parse(Console.ReadLine());
		Console.WriteLine("enter a number");
		number = int.Parse(Console.ReadLine());
		achadot = tnumber % 10;
		asarot = tnumber % 100 / 10;
		meot = tnumber / 100;

		// אם המספר לא מופיע
		if (achadot !=  number) // אם המספר לא מופיע
		{
			if (asarot != number)
			{
				if (meot != number)
				{
					Console.WriteLine("0");
				}
			}
		}
		
		// אם המספר מופיע פעם אחת
							
		   if (achadot == number) //רק אם אחדות שווה למספר
			{
			if (asarot != number)
			{
				if (meot != number)
				{
					Console.WriteLine("1");
				}
			}
		}
				
		if (asarot == number) // רק אם עשרות שווה למספר
		{
			if (achadot != number)
			{
				if (meot != number)
				{
					Console.WriteLine("1");
				}
			}
		}

		if (meot == number) //רק אם מאות שווה למספר
		{
			if (asarot != number)
			{
				if (achadot != number)
				{
					Console.WriteLine("1");
				}
			}
		}
		// אם המספר מופיע פעמיים

		if (achadot==asarot==number) // אם המספר מופיע באחדות ובעשרות
		{
			 if (meot!=number)
			{
				Console.WriteLine("2");
			}
		}
		if (achadot==meot==number) // אם המספר מופיע באחדות ובמאות
		{
			if (asarot!=number)
			{
				Console.WriteLine("2");
			}
		}
		if (asarot==meot==number) // אם המספר מופיע בעשרות ובמאות
		{
			if (achadot!=number)
			{
				Console.WriteLine("2");
			}
		}
// אם המספר מופיע שלוש פעמים
		if (achadot==asarot==meot==number)
		{
			Console.WriteLine("3");
		}
			  
		
		Console.Read();
		

	}
}

אני לא מבין מה לא טוב בזה!!
מי שמבין ובא לו לעזור לי למצוא את הטעות שלי אז.. אממ.. תודה..!
חח שיהיה חג שמח ונטול שאלות מעצבנות..! :-\

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

שמע אני רק התחלתי ללמוד #C בביצפר..
ואין לי עוד את כל הידע [כמו שבטח כבר הבנת..]
אז אם תוכל לרשום לי את הדרך זה יהיה ממש נחמד..!

ציטוט:

במקור נכתב על ידי Alon:] שמע אני רק התחלתי ללמוד #C בביצפר.. ואין לי עוד את כל הידע [כמו שבטח כבר הבנת..] אז אם תוכל לרשום לי את הדרך זה יהיה ממש נחמד..!

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

קוד:

int num1 = int.Parse(Console.ReadLine()); 
 int num2 = int.Parse(Console.ReadLine());
 
int achadot = num1 % 10;
 int asarot = num1 % 100 / 10;
 int meot = num1 / 100;
 
int counter = 0;
 
if(num2 == achadot)
 counter++;
 
if(num2 == asarot)
 counter++;
 
if(num2 == meot)
 counter++;
 
Console.Write(counter);
 Console.Read();
 

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

קוד:

int num = int.Parse(Console.ReadLine()); 
int digit = int.Parse(Console.ReadLine());
int counter=0;
while (num){
if (num%10==digit) counter++;
num=num/10;
}
Console.Write(counter);
Console.Read();
 

Any sufficiently advanced bug is indistinguishable from a feature

ציטוט:

במקור נכתב על ידי ם_O אם אתה רוצה שזה יעבוד עבור כל מספר ולאו דווקא רק תלת ספרתי, תעשה ככה: קוד: int num = int.Parse(Console.ReadLine()); int digit = int.Parse(Console.ReadLine()); int counter=0; while (num){ if (num%10==digit) counter++; num=num/10; } Console.Write(counter); Console.Read();

אני מניח שזה היה אמור להיות התרגיל הבא שהם יקבלו =]

ה-While צריך להחזיר ערך בוליאני -

קוד:

 




while (num>0)

ציטוט:

במקור נכתב על ידי Joss ה-While צריך להחזיר ערך בוליאני - קוד: while (num>0)

יפה שטרחת לקרוא את הקוד

בC וCPP הwhile תקין ותקני. while (num) שקול ל while(num!=0) כי 0 זה FALSE ו1 זה TRUE כמובן.

Any sufficiently advanced bug is indistinguishable from a feature

ציטוט:

במקור נכתב על ידי ם_O בC וCPP הwhile תקין ותקני. while (num) שקול ל while(num!=0) כי 0 זה FALSE ו1 זה TRUE כמובן.

לא בדיוק.. כלומר, נכון ש true = "1" אבל מהעבודה שלי עם cpp אז הוא יתיחס לכל מספר בתור true.. אני לא חושב שיצאלי לבדוק את זה עם ערך של 0,
בכל מקרה.. אני לא חושב שזה כל כך תקני לעשות את ההמרה הזאת.. כלומר.. ערך בוליאני הוא בוליאני.. או 0 או 1. זה לא המקרה במספרים, והבדיקה שאם הערך הוא 0 או 1.. זה פשוט לא נראה לי בסדר
כמו שאם תשלח לו char של '0', זה לא משהו שהייתי רוצה לצפה לעבוד, גם יכול סתם לגרום לכל מיני טעויות מעצבנות..
ואם כבר.. אז שתצטרך לעשות explicit casting.. יהיה יותר הגיוני

// הוא יתיחס לכל מספר בתור true
לא, הוא התיחס לכל מספר ששונה מ-0 בתור true.

C pogrammers never die. They are just cast into void.

אמרתי ש0 זה FALSE ו1 זה TRUE,לא אמרתי שרק 1 זה TRUE, אז.. זה לא אומר שכל מספר שונה מ0 שאינו 1 הוא לא TRUE..
נהוג להחזיר מפונקציה שאמורה להיות בוליאנית 1 כTRUE ו0כFLASE והמהדר עצמו מתייחס ל0 כמספר היחיד שהוא FLASE.

Any sufficiently advanced bug is indistinguishable from a feature