היי!

כחלק מתכנית גדולה יותר..
אני צריכה פונקציה בעלת ההצהרה:

קוד PHP:

 int Count_Letter (char* Str, char letter) 


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

מישהו יכול לעזור לי? קטע קוד יהיה נחמד

אם האות צריכה להיות מ A עד Z אז היא חייבת להיות גדולה.
לכן אני מניח שהתכוונת לאות מA עד Z או מ a עד z

1. חשבי את ההפרש בין a ל A ואחסני אותו במשתנה toUpper
2. אם האות גדולה מ Z חסרי ממנה את toUpper והציבי חזרה במשתנה letter
3. אם האות בletter קטנה מ A סיימי בשגיאה => הערך מחוץ לטווח
4. אם האות בletter גדולה מ Z סיימי בשגיאה => הערך מחוץ לטווח
5. הגדירי משתנה count=0
6. עבור כל אות במחרוזת
6.1 אם האות גדולה מZ חסרי ממנה את toUpper והציבי במשתנה temp
6.2 אם האות במשתנה temp שווה לאות בletter הגדילי את count ב 1
7. החזירי את הערך count


*עריכה*
אני מניח שהמחרוזות מקודדות בASCII
http://www.ascii.cl/

_________________________________________________

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

מנהל פורום מדע בדיוני ופנטסיה ופורום מדע טכנולוגיה וטבע בפרש

קוד PHP:

 #include <stdio.h>

int Count_Letter (char* Str, char letter);
int main ()
{
    char string[10],letter;
    printf ("Enter a string and a letter\n");
    scanf ("%s %c",string,&letter);
    printf ("\nCount is %d\n",Count_Letter (string, letter));
    fflush(stdin);
    scanf ("%c",&letter);
}

int Count_Letter (char* Str, char letter)
{
    if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
    {
        int i=0, count=0;
        while (Str[i] != '\0')
        {
            if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z'))
            {
                if (Str[i] == letter)
                    count++;
                else
                {
                    if (letter <= 'Z')
                    {
                        if (Str[i] == letter + 32)
                            count++;
                    }
                    else                    
                        if (letter >= 'a')
                        {
                            if (Str[i] == letter - 32)
                                count++;
                        }
                }
            }
            else
                return 0;
            i++;
        }
        return count;
    }
    else
        return 0;
} 


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

עריכה:
הערה - המספר "32" הוא ההפרש בין A ל- a.
אם את רוצה להראות שהבנת מה המשמעות של המספר הזה (שאינו שרירותי), תוכלי לכתוב במקום "32":
'a' - 'A'
ובמקום 32-
'A' - 'a'

בברכה, מתן.
www.MatanNarkiss.com

וואו, תודה על התגובה המהירה !

כמה שאלות מהירות:

קוד PHP:

 scanf ("%s %c",string,&letter); 


לא אמור להיות & ליד string ?
מה זה fflush(stdin); ?

משהו נראה לי קצת מוזר כאן..

קוד PHP:

 int main () {
    char string[10],letter;
    printf ("Enter a string and a letter\n");
    scanf ("%s %c",string,&letter);
    printf ("\nCount is %d\n",Count_Letter (string, letter));
    fflush(stdin);
    scanf ("%c",&letter);
} 


בסוף יש סוגריים מסולסלות { שסוגרות את הסוגריים של int main () {. הסוגריים של int main לא אמורות להסגר רק בסוף הקוד?

וגם

קוד PHP:

 int Count_Letter (char* Str, char letter); 


ניתן להכריז ככה על משתנה מחוץ ל int main ? אם זאת הפונקציה, אז לא אמורים להיות סוגריים מסולסות אחרי ההכרזה שלה?

שוב תודה רבה!

אני אשב על הקוד מאוחר יותר ואחזור אם יהיו לי שאלות נוספות

כמה תשובות זריזות.
1. כשסורקים לתוך מערך (מחרוזת היא מקרה פרטי של מערך) אין חובה להשתמש ב &. ניתן להשתמש ב & בסריקה למחרוזת, זו לא שגיאה, אבל כיוון שמערך הוא בעצמו מצביע - אין צורך לציין את הכתובות של המערך (הערך עצמו הוא הכתובת לתא הראשון).
נשמע מסובך?

2. מה זה fflush... קורה שבמהלך הרצת תוכנית, יש קלטים שנשמרים בזיכרון ויוצאים ברגעים לא מתוכננים.
כאשר אנחנו מקלידים במקלדת קלט מסויים ומכניסים אותו לתוכנית שלנו, הוא נקלט במשתנים המתאימים, אך לפעמים חלקים ממנו נשארים בזיכרון ו'יוצאים' בחזרה לתוכנית ברגעים לא מתוכננים. זה יכול בפועל לגרום לקבלת קלטים לא רצויים.
זה לא משהו שאפשר לצפות. יש לי קוד שכתבתי שעובד מצויין 3-4 פעמים ואז בפעם הבאה שאני מזין קלט (באותה הלולאה שעבדה מצויין 4 פעמים קודם לכן) הכל מתחרבש... פתאום יש לי גלישה מגודל מערך, או איזו לולאה אינסופית...
הפתרון הוא לפני הסריקה לבצע ניקוי של הזיכרון הזה ע"י פונקציה fflush כפי שרשמתי אותה מעלה.
חפשי את המחרוזת (fflush(stdin בגוגל, אני בטוח שתמצאי עוד מידע עליה.

3-4. שימי לב שהסוגריים שציטטת שייכות לפונקציה int main.
הסוגריים שבסוף הקוד (השורה האחרונה ביותר) הם הסוגריים שסוגרים את הפונקציה int Count_Letter.
אין שום קשר בין הפונצקיות, מלבד העובדה שאני משתמש בשניה בתוך הראשונה.
לפי השאלה הרביעית שלך, אני מבין שאת רגילה לכתוב את הפונצקיות מעל ה main ולקנח בסוף הקוד שלך עם הפונקציה main.
כאשר את מממשת את הפונקציה מעל ה main, את לא צריכה להצהיר עליה כיוון שהיא מוצהרת מטעם מימושה. אבל, אם את רוצה לממש את הפונקציה מתחת ל main, את צריכה פעם אחת להצהיר עליה מעל ה main (כדי שהיא תדע שקיימת פונקציה כזו בכלל), ואז מתחת ל main את מממשת מה שאת רוצה.

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

אם יש שאלות, אשמח לענות.

בברכה, מתן.
www.MatanNarkiss.com

לגבי ה- fflush עשיתי חיפוש ומצאתי סיבות מדוע לא להשתמש בזה (וגם בפונקציות דומות כמו scanf וכל המשפחה...)

ראה כאן: http://www.linuxforums.org/forum/li...n-function.html

הם מתחילים לחפור לגבי פונקציות אחרות שצריך להשתמש בהן...

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

בברכה, מתן.
www.MatanNarkiss.com

הפונקציה הזאת היא בעצם חלק מתכנית יותר גדולה שמופיעה כאן:
http://webcourse.cs.technion.ac.il/...hw/WCFiles/Q1.c
(אם אני מעתיקה את הקוד לפורום התגובה די מתחרבשת..


כאשר כאן:

קוד PHP:

 /*Q1 Part A: Count_Letter*/
int Count_Letter (char* Str, char letter)
{
    /*Add the relevant changes*/
    /**************************/
 
 
} 


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

קוד PHP:

 if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
    {
        int i=0, count=0;
        while (Str[i] != '\0')
        {
            if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z'))
            {
                if (Str[i] == letter)
                    count++;
                else
                {
                    if (letter <= 'Z')
                    {
                        if (Str[i] == letter + 32)
                            count++;
                    }
                    else                    
                        if (letter >= 'a')
                        {
                            if (Str[i] == letter - 32)
                                count++;
                        }
                }
            }
            else
                return 0;
            i++;
        }
        return count;
    }
    else
        return 0; 


נכון?

קודם כל - זה לא יפה להעתיק
הרעיון הוא שתביני מה נעשה כדי שתדעי לעשות בעצמך

ולעניינינו - כן, את פשוט מעתיקה את התוכן של הפונקציה לתוך האיזור שכתוב "add relavent changes".

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

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

זהירות - ספויילר!

קוד PHP:

 #include <stdio.h>
bla bla bla 


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

בברכה, מתן.
www.MatanNarkiss.com

ציטוט:

במקור נכתב על ידי Narxx קודם כל - זה לא יפה להעתיק הרעיון הוא שתביני מה נעשה כדי שתדעי לעשות בעצמך ולעניינינו - כן, את פשוט מעתיקה את התוכן של הפונקציה לתוך האיזור שכתוב "add relavent changes". עכשיו שאני רואה את התרגיל שלך, את צריכה לשנות את אחד מתנאי היציאה שיחזיר 1- אם התו לא חוקי (כמו בדוגמא עם "!"). שימי לב לזה, אוהבים להוריד ציון על בדיקות כאלה. את יכולה להעתיק את כל הקוד גם אם התגובה קצת תתחרבש... במקרה הכי גרוע, תעטפי את הקוד בתוך תגיד ספויילטר: זהירות - ספויילר! קוד PHP:  #include <stdio.h> bla bla bla  כך שזה לא יחרבש את הדף עצמו... פשוט נוח יותר לעקוב אחרי השגיאות שלך כשאת מצרפת את כל הקוד, ואפשר לראות בדיוק מה עשית.

אוקיי, שים לב מה ניסיתי

קוד PHP:

 #include <stdio.h>

/*Q1 Part A: Count_Letter*/
int Count_Letter (char* Str, char letter)
{
    if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
    {
        int i=0, count=0;
        while (Str[i] != '\0')
        {
            if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z'))
            {
                if (Str[i] == letter)
                    count++;
                else
                {
                    if (letter <= 'Z')
                    {
                        if (Str[i] == letter + 32)
                            count++;
                    }
                    else                    
                        if (letter >= 'a')
                        {
                            if (Str[i] == letter - 32)
                                count++;
                        }
                }
            }
            else
                return 0;
            i++;
        }
        return count;
    }
    else
        return 0;

}
 
 

/******MAIN******/
void main()
{
    int i;
    /*Q1A*/
    printf("%d\n",Count_Letter("Jim loves pizza",'z'));/*2*/
    printf("%d\n",Count_Letter("Jim loves pizza",'Z')); /*2*/
    printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'y')); /*0*/
    printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'!')); /*-1*/
} 


והפלט הוא משום מה רק אפסים, סימן שמשהו מתפקשש באיזשהו מקום..
ולשאלות קטנות:
מה המשמעות של Str[i] != '\0' ? סוף מחרוזת?

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

הנה קוד שנראה לי קצת יותר יעיל

קוד PHP:

 int Count_Letter (char* Str, char letter)
{
    char opposite;
    
    if(letter >= 'A' && letter <= 'Z') opposite = letter + 32;
    else if(letter >= 'a' && letter <= 'z') opposite = letter - 32;
    else return -1;

    int i=0, count=0;
    
    while (Str[i] != '\0')
    {
        if(Str[i] == letter || Str[i] == opposite) count++;
        i++;
    }
    
    return count;
} 


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

ציטוט:

במקור נכתב על ידי Dragonion הבעיה היא הבדיקה אם האות של המחרוזת נמצאת בתוך הטווח. ברגע שהאות הנבדקת שווה למשהו שלא בטווח (במקרה הזה - רווח, ששווה ל 32), הפונקצייה מחזירה 0. פשוט תמחקי את התנאי הזה כי הוא במילא לא מועיל וזה יפעל. הנה קוד שנראה לי קצת יותר יעיל קוד PHP:  int Count_Letter (char* Str, char letter) {     char opposite;       if(letter >= 'A' && letter <= 'Z') opposite = letter + 32;     else if(letter >= 'a' && letter <= 'z') opposite = letter - 32;     else return -1;       int i=0, count=0;       while (Str[i] != '\0')     {         if(Str[i] == letter || Str[i] == opposite) count++;         i++;     }       return count; }  ו /0 אכן מסמל סוף של מחרוזת. בעיקרון לכל המחרוזת, לא משנה איפה, יהיה את התו הזה בסוף (ואם לא יהיה אותו, את מן הסתם תדעי על זה).

עובד מצויין תודה!

שימי לב שאת צריכה להחזיר 1- כאשר את בודקת אות שלא בטווח.
למחוק את ה return 0 יעשה רק חלק מהעבודה... |רמז רמז|

בברכה, מתן.
www.MatanNarkiss.com

ציטוט:

במקור נכתב על ידי Narxx שימי לב שאת צריכה להחזיר 1- כאשר את בודקת אות שלא בטווח. למחוק את ה return 0 יעשה רק חלק מהעבודה... |רמז רמז|

המממ..
ואם אני רוצה שאם האות לא בטווח שהפונקציה תחזיר פלט שאומר ERROR או משהו בסגנון, לכתוב לולאת if-else נוספת בסוף ומותר לרשום return Error ?

את לא יכולה להחזיר ERROR*, כיוון שהפונקציה מוגדרת כמי שמחזירה int (מספר שלם בלבד).
אבל, את יכולה להתייחס ל int הזה במיין כפי שאת רוצה.
למשל, אם קיבלת שגיאה (זאת אומרת, הפונקציה החזירה 1-), תדפיסי ERROR ב main:

קוד PHP:

 int main ()
{
    int result = Count_Letter ("bla bla bla", b);
   if (result == (-1)) // also can write: if (result < 0)
       printf ("Input error!\n");
   else
       printf ("%d",result);
} 


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

* מאוחר יותר בטח תלמדו לעבוד עם enum ואז תראי שכן אפשר להחזיר "ERROR" אבל זה למעשה יהיה int בתחפושת... אבל חבל סתם להקדים את המאוחר

בברכה, מתן.
www.MatanNarkiss.com

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

קוד PHP:

 enum {
STRING_ERROR = -1,
SINGLE_CHAR_ERROR = -2,
} 


ואז הקוד יהפוך להרבה יותר קריא:

קוד PHP:

 if (result == STRING_ERROR) // also can write: if (result < 0)
       printf ("Input error!\n");
   else
       printf ("%d",result);
} 


הראל

נכון... רק נחכה שהיא תלמד להשתמש ב enum או define ואז היא תורשה להשתמש בזה בתרגיל שלה

בברכה, מתן.
www.MatanNarkiss.com

ציטוט:

במקור נכתב על ידי Narxx נכון... רק נחכה שהיא תלמד להשתמש ב enum או define ואז היא תורשה להשתמש בזה בתרגיל שלה

אין לי מושג מה זה enum אבל ב define אני כן יכולה להשתמש
השאלה היא מה זה enum ואיך בדיוק הוא מקביל ל define?

למשל בקטע קוד הזה:

קוד PHP:

 if(letter >= 'A' && letter <= 'Z') opposite = letter + 32;
    else if(letter >= 'a' && letter <= 'z') opposite = letter - 32;
    else return -1; 


אני יכולה לעשות define בצורה הבאה על מנת שאקבל הודעת שגיאה?

קוד PHP:

 #DEFINE ERROR -1  
  
    if(letter >= 'A' && letter <= 'Z') opposite = letter + 32;
    else if(letter >= 'a' && letter <= 'z') opposite = letter - 32;
    else return -1; 


או שאני עושה פה סלט גדול .. ?

לא סלט בכלל... את יכולה לעשות בדיוק מה שכתבת:

קוד PHP:

 #define ERROR -1 


ואז להשתמש במילה ERROR במקום המספר 1- בצורה הבאה:

קוד PHP:

 return ERROR; 


למה זה טוב? פשוט נוח להחזיר מילים במקום להחזיר מספרים (מי זוכר ש 1- זו הודעת שגיאה? יותר קל לכתוב להחזיר "שגיאה").
מבחינת ה main - החזרת int ולא מילה... זאת אומרת שמבחינת התוכנית שלך זה בדיוק אותו הדבר, פשוט השתמשת במילה במקום מספר רק בשביל הנוחות בתכנות.
enum עובד באותו האופן של define.

זהירות - ספויילר!

קוד PHP:

 #include <stdio.h>
int Count_Letter (char* Str, char letter);
int main ()
{
    char letter;
    printf("%d\n",Count_Letter("Jim loves pizza",'z'));/*2*/
    printf("%d\n",Count_Letter("Jim loves pizza",'Z')); /*2*/
    printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'y')); /*0*/
    printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'!')); /*-1*/
    fflush(stdin);
    scanf ("%c",&letter);
}

int Count_Letter (char* Str, char letter)
{
    #define ERROR -1
    if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
    {
        int i=0, count=0;
        while (Str[i] != '\0')
        {
            if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z') || (Str[i] == ' '))
            {
                if (Str[i] == letter)
                    count++;
                else
                {
                    if (letter <= 'Z')
                    {
                        if (Str[i] == letter + 32)
                            count++;
                    }
                    else                    
                        if (letter >= 'a')
                        {
                            if (Str[i] == letter - 32)
                                count++;
                        }
                }
            }
            else
                return ERROR;
            i++;
        }
        return count;
    }
    else
        return ERROR;
} 


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

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

בברכה, מתן.
www.MatanNarkiss.com

ציטוט:

במקור נכתב על ידי Narxx לא סלט בכלל... את יכולה לעשות בדיוק מה שכתבת: קוד PHP:  #define ERROR -1  ואז להשתמש במילה ERROR במקום המספר 1- בצורה הבאה: קוד PHP:  return ERROR;  למה זה טוב? פשוט נוח להחזיר מילים במקום להחזיר מספרים (מי זוכר ש 1- זו הודעת שגיאה? יותר קל לכתוב להחזיר "שגיאה"). מבחינת ה main - החזרת int ולא מילה... זאת אומרת שמבחינת התוכנית שלך זה בדיוק אותו הדבר, פשוט השתמשת במילה במקום מספר רק בשביל הנוחות בתכנות. enum עובד באותו האופן של define. זהירות - ספויילר! קוד PHP:  #include <stdio.h> int Count_Letter (char* Str, char letter); int main () {     char letter;     printf("%d\n",Count_Letter("Jim loves pizza",'z'));/*2*/     printf("%d\n",Count_Letter("Jim loves pizza",'Z')); /*2*/     printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'y')); /*0*/     printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'!')); /*-1*/     fflush(stdin);     scanf ("%c",&letter); }   int Count_Letter (char* Str, char letter) {     #define ERROR -1     if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))     {         int i=0, count=0;         while (Str[i] != '\0')         {             if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z') || (Str[i] == ' '))             {                 if (Str[i] == letter)                     count++;                 else                 {                     if (letter <= 'Z')                     {                         if (Str[i] == letter + 32)                             count++;                     }                     else                                             if (letter >= 'a')                         {                             if (Str[i] == letter - 32)                                 count++;                         }                 }             }             else                 return ERROR;             i++;         }         return count;     }     else         return ERROR; }  התוצאה כפי שרואים על המסך היא בדויק מה שכתוב שאת צריכה לקבל בתרגיל. שימי לב שאפילו שהחזרתי ERROR, מבחינת התוכנית - החזרתי 1-. בעריכה: שימי לב שבבדיקת קלט שלי, אני מניח שלמחרוזת אסור להיות סימנים. מהתרגיל נובע כי מותר למחרוזת להיות סימנים ורק לתו הבודד שבודקים אסור להיות סימנים. מה שצריך לשנות בתוכנית שלי, אם כך, זה את תנאי הבדיקה שיכיל גם סימנים (או רק סימני קריאה בשביל לתת לדוגמאות הספציפיות האלה לרוץ).

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

קוד PHP:

 if ((Str[i] >= 'a' && Str[i] <= 'z') || (Str[i] >= 'A' && Str[i] <= 'Z') || (Str[i] == ' ') || (Str[i] == '!')) 


ואם למשל במצב של שגיאה, הייתי רוצה להחזיר מהפונקציה שסופרת את מספר האותיות חזרה ל main לא מספר אלא שורת טקסט כלשהי, מבחינה תחבירית מותר לעשות את זה?
נגיד
DEFINE ERROR wrong input commrad ?

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

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

קוד PHP:

 #include <stdio.h>
#define ERROR "bla bla"

int main ()
{
    printf("%s", ERROR);
    char x;
    scanf ("%c",&x);
} 


אני אקבל על המסך את הכיתוב bla bla (תקמפלי ותריצי...).
באותו האופן, אני יכול להחזיר את הערך הזה מפונקציה:

קוד PHP:

 #include <stdio.h>
#define ERROR "bla bla"

char* func1 ();

int main ()
{
    printf("%s", func1());
    char x;
    scanf ("%c",&x);
}

char* func1 ()
{
    return ERROR;
} 


התוצאה תהיה (באופן מפתיע) זהה לחלוטין... יציג על המסך bla bla.

שימי לב רק שבקוד שלך, הפונקציה מחזירה int כי ככה הגדירו לך.
את לא יכולה להחזיר מחרוזת אם הפונקציה מוגדרת להחזיר int.
כך שבמקרה שלך, ה main ידפיס בכל מקרה 1- אם התבצעה שגיאה - אבל זה בסדר... ככה הם רצו שתעשי בתרגיל.

בברכה, מתן.
www.MatanNarkiss.com

אוקיי, הבנתי!
אז אני פשוט אשמיט את השורת if עם הבדיקה עם המחרוזת

ציטוט:

במקור נכתב על ידי Narxx את לא יכולה להחזיר ERROR*, כיוון שהפונקציה מוגדרת כמי שמחזירה int (מספר שלם בלבד). אבל, את יכולה להתייחס ל int הזה במיין כפי שאת רוצה. למשל, אם קיבלת שגיאה (זאת אומרת, הפונקציה החזירה 1-), תדפיסי ERROR ב main: קוד PHP:  int main () {     int result = Count_Letter ("bla bla bla", b); if (result == (-1)) // also can write: if (result < 0)      printf ("Input error!\n"); else      printf ("%d",result); }  שימי לב שבשיטה זו, את יכולה לדעת לפי ערך ההחזרה, לא רק כמה פעמים האות מופיעה במחרוזת, אלא גם לסמן שגיאות (לא חייבים להשתמש רק ב 1- .. את יכולה לסמן אינסוף שגיאות ע"י שימוש במספרים שליליים במקרה הזה... לדוגמא, 1- מציין שגיאה במחרוזת (הוכנסו תווים שאינם אותיות) ו 2- מציין שגיאה בתו הבודד (אינו אות אלא סימן או מספר). * מאוחר יותר בטח תלמדו לעבוד עם enum ואז תראי שכן אפשר להחזיר "ERROR" אבל זה למעשה יהיה int בתחפושת... אבל חבל סתם להקדים את המאוחר

אז זהו, שאסור לנו לגעת בפונקצית main. הכל צריך להתבצע בתור הפונקציה count_letter עצמה.
(הנחיית התרגיל)

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

בברכה, מתן.
www.MatanNarkiss.com

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

משחקים זה כיף!
תשלחי לי כשתסיימי

בברכה, מתן.
www.MatanNarkiss.com

חח זה יקח קצת זמן זה יותר מסובך מהשאלה הזאת, והשאלה הזאת היא רק פונקציה אחת מתוך 3 שצריך לכתוב

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

בהצלחה!

בברכה, מתן.
www.MatanNarkiss.com

היי :-) חזרתי עם שאלות לגבי פונקציה נוספת:

השאלה:


[התמונה הבאה מגיעה מקישור שלא מתחיל ב https ולכן לא הוטמעה בדף כדי לשמור על https תקין: http://img140.imageshack.us/img140/7757/28899585iu3.jpg]


הקוד:
קוד PHP:




<DIV dir=ltr align=left>#include <stdio.h>



int Count_Letter (char* Str, char letter)
{
#define ERROR -1
if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
{
int i=0, count=0;
while (Str[i] != '\0')
{

if (Str[i] == letter)
{
count++;
}
else
{
if (letter <= 'Z')
{
if (Str[i] == letter + 32)
{
count++;
}
}
else
{
if (letter >= 'a')
{
if (Str[i] == letter - 32)
{
count++;
}
}
}
}
i++;
} // end while
return count;
} // end first IF
else
{
return ERROR;
}
}
/* function 1
returns -1 as error. */

char Popular_Letter (char* Str)
{

int i, countAll[26], mostUsed[2] = {0, -1};

// Count letters
for(i=0; i<26; i++){
countAll[i] = Count_Letter (Str, 'A'+i);
if( mostUsed[0] < countAll[i] ){
mostUsed[0] = countAll[i];
mostUsed[1] = i;
}
}

// This only accrues if only non alphabetical characters exist in the string
if( mostUsed[1] == -1 )
mostUsed[1] = '!'-'A';

return mostUsed[1]+'A';
}





char Count_Word (char* Str, int n)
{
int i=0;
int space=0;
char *temp=0;
char Letter;
while ( /*(Str[i] != '\0') &&*/ (space <= n) )
{

if (Str[i] == ' ')
{
space++;
}

if (space == n-1){
temp = Str+i+1;
}

if (space == n ){
Str[i]='\0';
Letter = Popular_Letter (temp); }

i++;

}

return Letter;

}








/******MAIN******/
void main()
{
int i;
/*Q1A*/
printf("%d\n",Count_Letter("Jim loves pizza",'z'));/*2*/
printf("%d\n",Count_Letter("Jim loves pizza",'Z')); /*2*/
printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'y')); /*0*/
printf("%d\n",Count_Letter("I can't believe it's not butter!!!",'!')); /*-1*/

/*Q1B*/
printf("%c\n",Popular_Letter("Jim loves pizzzza")); /*z*/
printf("%c\n",Popular_Letter("Jim loves pizza")); /*i*/


/*Q1C*/
printf("%c\n",Count_Word("aaa abab ccccc",1)); /* 'A' */
printf("%c\n",Count_Word("aaa abab ccccc",2)); /* 'A' */
printf("%c\n",Count_Word("aaa abab

אני רואה את התרגיל, אבל אני לא רואה מה השאלה שלך...
מלבד זה שאת רוצה שנעשה לך את שיעורי הבית... יש משהו שאת רוצה שנסביר לך, שנבהיר לך, שנדגים לך, שנראה לך, שנבהיר לך, שנלמד אותך...?

בברכה, מתן.
www.MatanNarkiss.com

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

סעיפים א' ו-ב' עובדים, הבעיה היא בסעיף ג' (צבוע בכחול שונה) ובפרט בחלק הצבוע באדום בסעיף ג'. ההודעת שגיאה לא ממש ברורה, אני משום מה כבר לא מקבלת אותה על המחשב שלי גם כן.. אבל בתאוריה מה שסעיף ג' אמור לעשות זה לספור את מספר הרווחים וכשהוא מגיע לרווח שלפני המילה המתאימה (נגיד בדוגמא שלמעלה n=5 אז כשהוא מגיע לרווח הרביעי) הוא 'מסמן' בפוינטר את האות הראשונה של אותה מילה וממשיך לרוץ עד הרווח שאחרי המילה, אותה הוא מחליף בבאקסלאש 0 כדי שזה יסומן בתור הסוף. ככה המילה 'מסומנת', ועליה מופעלת הפונקציה השנייה שסופרת את האות הכי פופולארית באותה מילה ומחזירה למשתנה Letter את אותה אות פופולארית וזו מודפסת.
אבל.. לא עובד :\

#include <stdio.h>

int Count_Letter (char* Str, char letter)
{
#define ERROR -1
if ((letter >= 'A' && letter <= 'Z') || (letter >= 'a' && letter <= 'z'))
{
int i=0, count=0;
while (Str[i] != '')
{

if (Str[i] == letter)
{
count++;
}
else
{
if (letter <= 'Z')
{
if (Str[i] == letter + 32)
{
count++;
}
}
else
{
if (letter >= 'a')
{
if (Str[i] == letter - 32)
{
count++;
}
}
}
}
i++;
} // end while
return count;
} // end first IF
else
{
return ERROR;
}
}

/* function 1

returns -1 as error. */

char Popular_Letter (char* Str)
{

int i, countAll[26], mostUsed[2] = {0, -1};

// Count letters
for(i=0; i<26; i++){
countAll[i] = Count_Letter (Str, 'A'+i);
if( mostUsed[0] < countAll[i] ){
mostUsed[0] = countAll[i];
mostUsed[1] = i;
}
}

// This only accrues if only non alphabetical characters exist in the string
if( mostUsed[1] == -1 )
mostUsed[1] = '!'-'A';

return mostUsed[1]+'A';
}

char Count_Word (char* Str, int n)

{

int i=0;

int space=0;
char *temp=0;
char Letter;

while ( /*(Str[i] != '') &&*/ (space <= n) )

{

if (Str[i] == ' ')
{
space++;
}

if (space == n-1){
temp = Str+i+1;
}

if (space == n ){
Str[i]='';
Letter = Popular_Letter (temp); }

i++;

}

return Letter;

}

/******MAIN******/
void main()
{
int i;

/*Q1A*/

printf("%dn",Count_Letter("Jim loves pizza",'z'));/*2*/
printf("%dn",Count_Letter("Jim loves pizza",'Z')); /*2*/
printf("%dn",Count_Letter("I can't believe it's not butter!!!",'y')); /*0*/
printf("%dn",Count_Letter("I can't believe it's not butter!!!",'!')); /*-1*/

/*Q1B*/
printf("%cn",Popular_Letter("Jim loves pizzzza")); /*z*/
printf("%cn",Popular_Letter("Jim loves pizza")); /*i*/

<FONT color=#ff0000> /*Q1C*/
printf("%c\n",Count_Word("aaa abab ccccc",1)); /* 'A' */
printf("%c\n",Count_Word("aaa abab ccccc",2)); /* 'A' */
printf("%c\n",Count_Word("aaa abab

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

קוד PHP:

 char Count_Word (char* Str, int n)
{
    int i=0,j;
    int space=0;
    char word[80];
    while (space < n-1 )
    {
        if (Str[i] == ' ')
            space++;
        i++;
    }
    for (j=0; (Str[i] != '\0') && (Str[i] != ' '); i++,j++)
    {
        word[j] = Str[i];
    }
    word[j] = '\0';
    return Popular_Letter (word);
} 


מה שהיא עושה זה למעשה בונה מחרוזת בגודל 80, ולתוך המחרוזת הזו היא מעתיקה את המילה ה nית (כפי שנשלח ב main) ושולחת את המחרוזת הזו לפונקציה popular letter, שעובדת כמו שצריך (לטענתך).
בהתחלה ניסיתי להשתמש במחרוזת שקיבלת (Str) עצמה, אבל מסתבר שאסור לי לשנות אותה (שגיאת זמן-ריצה). כנראה שהיא const או משהו מטעם יצירתה כפי שהיא ב main.
למה גודל 80? סתם שיהיה מספיק גדול להכיל מילים ארוכות. יכלתי גם להשתמש במילה בגודל 20 וזה עדיין היה עובד כי אין לך מילים כל כך ארוכות בתרגיל.

בברכה, מתן.
www.MatanNarkiss.com

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

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

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

בברכה, מתן.
www.MatanNarkiss.com

הכל מובן, תודה רבה