תרגיל ב-C#

[gshhar]

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


לדוגמא אם אני מכניס את הטקסט:

test abcd$rrr.tyu ver
asd dfg

אז אני מקבל את המילים מסודרות ככה:

test
abcd
rrr
tyu
ver
asd
dfg

התחלתי לכתוב משהו אבל אני מסתבך:

קוד: בחירת הכל

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace PrintWordWordFromText
{
    class Program
    {
        static void Main(string[] args)
        {
            //define temp 
            string temp = "";

            int WordIndex = 0;
            
            //read text from user
            Console.Write("Enter Text: ");
            string line = Console.ReadLine();

            //define array
            string[] Words = new string[100];

            while (line.Length != 0)
            {
                for &#40;int i = 0; i < line.Length; i++&#41;
                &#123;
                    //We have a regular character - part of a word 
                    if &#40;line&#91;i&#93; != ' ' & line&#91;i&#93; != ' ' & line&#91;i&#93; != ',' & line&#91;i&#93; != '.' & line&#91;i&#93; != '$'&#41;
                    &#123;
                        temp += line&#91;i&#93;;
                    &#125;

                    //special characters 
                    else
                    &#123;
                        Words&#91;WordIndex&#93; = temp;
                        WordIndex++;
                        temp = "";
                    &#125;
                &#125;
            &#125;

            for &#40;int i = 0; i < WordIndex; i++&#41;
              &#123;
                  Console.Write&#40;Words&#91;i&#93;&#41;;
              &#125;
            
            Console.ReadLine&#40;&#41;;

        &#125;
    &#125;
&#125;

[Itsik_G]

אתה יכול להשתמש ב StringCollection?
אם כן, נסה את זה:

קוד: בחירת הכל

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Collections.Specialized;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            StringCollection sc = new StringCollection();
            bool done = false;
            string[] seperators = { ",", " ", ".", "$" };
            while (!done)
            {
                string lineInput = Console.ReadLine();
                if (lineInput == "")
                    done = true;
                else
                {
                    foreach (string seperator in seperators)
                    {
                        lineInput = lineInput.replace(seperator, "|");
                    }
                    string[] arrItems = lineInput.Split('|');
                    foreach(string item in arrItems)
                        sc.Add(item);
                }
            }
            DisplayItems(sc);
        }
        static void DisplayItems(StringCollection items)
        {
            foreach (string item in items)
                Console.WriteLine(item);
        }
    }
}

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

[gshhar]

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

איך אני יכול למשל לדעת שהיוזר הכניס שורה ריקה ז"א הכניס טקסט, לחץ אנטר ובשורה השנייה לא הכניס טקסט אלא לחץ שוב אנטר ואז התרגיל מתחיל ?

אני יכול לרשום מה שרשמתי ?

while (line.Length != 0)

[bg1987]

התרגיל הזה צועק regular expressions.

http://www.radsoftware.com.au/articles/ ... yntax.aspx

תחפש בMSDN על REGEX, כי לC# יש כמה סימנים מיוחדים שלא נפוצים בשאר השפות.

[Itsik_G]

איך אני יכול למשל לדעת שהיוזר הכניס שורה ריקה ז"א הכניס טקסט, לחץ אנטר ובשורה השנייה לא הכניס טקסט אלא לחץ שוב אנטר ואז התרגיל מתחיל ?

אני יכול לרשום מה שרשמתי ?

while (line.Length != 0)

...

כן, זה ייתן אותה תוצאה.

[יוני_א]

אפשר גם להשתמש ב console.ReadKey במקום ב RealLine

[bg1987]

אם אתה לא יכול לעשות REGEX, אז בIF שאתה בודק את האות שהיא רגילה,
תשתמש ב isLetter של CHAR (וזה האובייקט שאתה מקבל כשפונים לאינדקס בסטרינג)
http://msdn.microsoft.com/en-us/library ... s.71).aspx

בנוסף
יש לך פוטנציאל לבאג, בגלל שכל פעם שאתה נתקל במילה שהיא לא אות, אתה מקדם את האינדקס במערך הפתרונות, עבור קלט
"hello$$world"
המערך של המילים יראה
[0] "hello"
[1]
[2] "world"

ובהתאם תהיה לך שורה ריקה בפלט.

[gshhar]

וואי וואי אתם זורקים לי פה מושגים ואין לי מושג מה הם אומרים כי רק התחלתי ללמוד C#.

אני אנסה צעד צעד:

איך אני יוצר לולאה שתקבל את כל הטקסטים ותעצר מתי שיש שורה ריקה ?

[bg1987]

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

הלולאה שלך של ה line.length != 0 תעבוד.
רק בשביל שזה יקרא את השורה מחדש, צריך להכניס בסוף הלולאה (לפני שאתה סוגר אותה)
line = console.readLine() כדי שהמשתנה יתעדכן.

אז הקוד הבא:

קוד: בחירת הכל

String line = Console.ReadLine();
while (line.length != 0{
	Console.WriteLine(line);
	line = Console.ReadLine();
}
Console.WriteLine("end");

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

[gshhar]

מסלול של פיתוח תוכנה

[bg1987]

מכללה? או קורס של חברה כמו ג'ון ברייס?

[gshhar]

כן.

איפה אני מכניס את הקוד שנתת לי ?
מה עם הלולאה שלי:

קוד: בחירת הכל

                    //We have a regular character - part of a word 
                    if (line[i] != ' ' & line[i] != ' ' & line[i] != ',' & line[i] != '.' & line[i] != '$')
                    {
                        temp += line[i];
                    }
                    //special characters 
                    else
                    {
                        Words[WordIndex] = temp;
                        WordIndex++;
                        temp = ""; 
                    }

[bg1987]

קוד: בחירת הכל

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 

namespace PrintWordWordFromText 
{ 
    class Program 
    { 
        static void Main(string[] args) 
        { 
            //define temp 
            string temp = ""; 

            int WordIndex = 0; 
            
            //read text from user 
            Console.Write("Enter Text: "); 
            string line = Console.ReadLine(); 

            //define array 
            string[] Words = new string[100]; 

            while (line.Length != 0) 
            { 
                for &#40;int i = 0; i < line.Length; i++&#41; 
                &#123; 
                    //We have a regular character - part of a word 
                    if &#40;line&#91;i&#93; != ' ' & line&#91;i&#93; != ' ' & line&#91;i&#93; != ',' & line&#91;i&#93; != '.' & line&#91;i&#93; != '$'&#41; 
                    &#123; 
                        temp += line&#91;i&#93;; 
                    &#125; 

                    //special characters 
                    else 
                    &#123; 
                        Words&#91;WordIndex&#93; = temp; 
                        WordIndex++; 
                        temp = ""; 
                    &#125; 
                &#125;
				//If this line isn't here the program will be stuck in an infinite loop.	
				line = Console.ReadLine&#40;&#41;;	
				////////////////////////////////////////////////////////////////////////
            &#125; 

            for &#40;int i = 0; i < WordIndex; i++&#41; 
              &#123; 
                  Console.Write&#40;Words&#91;i&#93;&#41;; 
              &#125; 
            
            Console.ReadLine&#40;&#41;; 

        &#125; 
    &#125; 
&#125;

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

[gshhar]

יש פה עדיין בעיה זה לא עובד הוא משום מה לא נכנס ל-ELSE:

קוד: בחירת הכל

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace PrintWordWordFromText
{
    class Program
    {
        static void Main(string[] args)
        {
            //read text from user
            Console.Write("Enter Text: ");
            string line = Console.ReadLine();

            //define array
            string[] Words = new string[100];

            //define temp 
            string temp = "";

            int WordIndex = 0;

            while (line.Length != 0)
            {                
                for &#40;int i = 0; i < line.Length; i++&#41;
                &#123;
                    //We have a regular character - part of a word 
                    if &#40;line&#91;i&#93; != ' ' & line&#91;i&#93; != ' ' & line&#91;i&#93; != ',' & line&#91;i&#93; != '.' & line&#91;i&#93; != '$'&#41;
                    &#123;
                        temp += line&#91;i&#93;;
                    &#125;
                    //special characters 
                    else
                    &#123;
                        Words&#91;WordIndex&#93; = temp;
                        WordIndex++;
                        temp = ""; 
                    &#125;                      
                &#125;

                line = Console.ReadLine&#40;&#41;;
            &#125;

            for &#40;int i = 0; i < WordIndex; i++&#41;
            &#123;
                Console.Write&#40;Words&#91;i&#93;&#41;;
            &#125; 

            Console.ReadLine&#40;&#41;;

        &#125;
    &#125;
&#125;

[bg1987]

קוד: בחירת הכל

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Text; 

namespace PrintWordWordFromText 
{ 
    class Program 
    { 
        static void Main(string[] args) 
        { 
            //read text from user 
            Console.Write("Enter Text: "); 
            string line = Console.ReadLine(); 

            //define array 
            string[] Words = new string[100]; 

            //define temp 
            string temp = ""; 

            int WordIndex = 0; 

            while (line.Length != 0) 
            {                
                for &#40;int i = 0; i < line.Length; i++&#41; 
                &#123; 
					///////////////DEBUG/////////////////////////
					Console.WriteLine&#40;i&#41;;
					Console.WriteLine&#40;line&#91;i&#93;&#41;;
					/////////////////////////////////////////////
                    //We have a regular character - part of a word 
                    if &#40;line&#91;i&#93; != ' ' & line&#91;i&#93; != ' ' & line&#91;i&#93; != ',' & line&#91;i&#93; != '.' & line&#91;i&#93; != '$'&#41; 
                    &#123; 
                        temp += line&#91;i&#93;; 
                    &#125; 
                    //special characters 
                    else 
                    &#123; 
                        Words&#91;WordIndex&#93; = temp; 
                        WordIndex++; 
                        temp = ""; 
                    &#125;                      
                &#125; 

                line = Console.ReadLine&#40;&#41;; 
            &#125; 

            for &#40;int i = 0; i < WordIndex; i++&#41; 
            &#123; 
                Console.Write&#40;Words&#91;i&#93;&#41;; 
            &#125; 

            Console.ReadLine&#40;&#41;; 

        &#125; 
    &#125; 
&#125; 

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

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

[shayze]

אני הייתי עושה משהו אחר קצת.
במקום לבדוק כל מיני סוגי סימנים שונים, הייתי בודק אותיות.
אמנם לא עובד בC#, אבל הקוד נראה בערך ככה.

קוד: בחירת הכל

if &#40;line&#91;i&#93; >= &#40;int&#41;'a' && line&#91;i&#93; <=&#40;int&#41;'z'

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

[haimbl]

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

קוד: בחירת הכל

  int indexLocation = 0;
            var words = new string[100];
            string input = string.Empty;

            while (!string.IsNullOrWhiteSpace((input = Console.ReadLine())))
            {
                string[] inputWords = input.Split(' ', ',', '.', '$');
               
                foreach (var inputWord in inputWords)
                {
                    words[indexLocation] = inputWord;
                    indexLocation++;
                }
            }

[bg1987]

לא חבל לפתור לא את התרגיל?
חוץ מזה, haimbl אתה פותר את זה הרבה יותר בקלות עם REGEX.

[shayze]

בכל מקרה עושה רושם שיש באג בקוד בתנאי העצירה (מקרה קצה)

[haimbl]

בכל מקרה עושה רושם שיש באג בקוד בתנאי העצירה (מקרה קצה)

...

איזה מקרה קצה?