כתבתי בשפת C++ פונקציה SPLIT, בטוח רובכם מכירים את זה משפות תכנות אחרות (זה נקרא גם EXPLODE) , הפונקציה שלי מתוכננת ככה שתוכל להפריד מחרוזות לתת מחרוזות לפי סימן מסוים שהמשתמש קובע, למשל:
סימן ההפרדה שלי הוא |
והמשתמש מזין את הטקסט: hi|bye|bla
התכנית דואגת להפריד את המחרוזת הגדולה הזאת לשלוש מחרוזות קטנות: hi , bye , bla.

בקיצור , נקלעתי לבעיה, ככל הנראה בעיה בהקצאת הזיכרון, אם תואילו לבדוק את הקוד הבא אני אודה לכם בלב ונפש.
https://2007-uploaded.fresh.co.il/2...04/74719017.cpp

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

תשתמש במשתנה אחר..

דורון

אוף עדיין יש בעיה

עדיין יש בעיה.
סתכל:
https://2007-uploaded.fresh.co.il/2...05/44854046.cpp

שים לב:

קוד:

 
 
for (int i = 0; i<strlen(String); i++)
	{
		if(String[i] == Sep)
		{
			if(index == 0) 
			{
				for(y=0; y < i; y++)
					sString[0] += String[y];	
			}
			else
			{
				for(y=pos; y <= (i-pos); y++)
sString[index] += String[i]; 

			}
		    pos=i;
			index++;
		}
	}
 
 

מה בעצם קורה? את pos אתה מעדכן כל פעם בלולאה אחרי התנאים. אתה צריך לעדכן אותו
רק בתו הראשון שאחרי המפריד.. כרגע, הספירה שלך מתחילה נניח מ10 ונגמרת ב1.
כי i-pos=11-1=1. ולכן הוא לא נכנס לשם בכלל.

דורון

עדיין זה לא מצליח..

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

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

נסה עכשיו לתקוף את הבעיה, ותעלה לכאן קוד במידה ואתה ממשיך להתקל בבעיות.

דורון

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

תודה. הסתדרתי.

https://2007-uploaded.fresh.co.il/2...05/82002685.cpp

דרך הקומפיילר שלי(VISUAL C++ 6) זה עובד מעולה, אבל דרך קובץ הEXE זה לא עובד, וגם דרך הקומפיילר DEV C++ זה לא עובד.

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

אני מתבסס על הקוד הקודם שהעלת, ועל התגובה הקודמת שלי:

קוד:

 pos=0;
index=0;
for (int i = 0; i<strlen(String); i++)
{
 if(String[i] == Sep)
		{
  for(y=pos; y < i; y++)
  {
		  sString[index] += String[y]; 
  }
  pos=i;
  index++;
 }
}
 

דורון

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

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

והאם ניסית להשתמש בקוד שנתתי לך כאן?

דורון

כן, וזה עדיין לא עובד!

דורון

https://2007-uploaded.fresh.co.il/2...05/55747351.cpp

הנה

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

קוד:

#include<iostream>

#include<string>

usingnamespace std;

int Getseps(string String, char Sep);

string *Split(string String, char Sep);

 

int _tmain(int argc, _TCHAR* argv[])

{

string *String = new string[5];

char *x = newchar[20];

strcpy(x , "Hi/Bye/Good Bye/farewell");

String = Split(x, '/');



for (int i=0; i<= 3; i++)

cout << String[i] << endl;

cin >> String[0];

return 0;

}

 

int Getseps(string str,char Sep)

{

int index =0;

int i;

for (i = 0; i<str.length(); i++)

{

if(str[i] == Sep)

{

index++;

}

}

return index;

}

string *Split(string str, char Sep)

{

int index;

int y;

int pos;

int i;

string *sString= new string[Getseps(str,Sep)+1];

pos=0;

index=0;

for ( i = 0; i<str.length(); i++)

{

if(str[i] == Sep)

{

for(y=pos; y < i; y++)

{

sString[index] += str[y]; 

}

pos=i;

index++;

}

}

return sString;

}

ציטוט:

במקור נכתב על ידי AmiHod1 https://2007-uploaded.fresh.co.il/2...05/55747351.cpp הנה

דורון

לא הבנתי מה שינית שם ולמה..

דורון

אבל למה, מה זה משנה?ומדוע זה גרם לשגיאה?
ומה זה int _tmain(int argc, _TCHAR* argv[])?

ההבדל היחיד הוא שהיות ואתה גם כך משתמש ב<string>, ביטלתי את כל ההתייחסויות שלך לchar* ואח"כ לstrlen, והחלפתי אותם בשימוש באוביקט string. הקוד עכשיו עובד - אבל לא מבצע את מה שהוא אמור
לבצע בגלל כשל לוגי שיש לך באלגוריתם.

שים לב לקאונטרים שנשמרים בלולאה.. ומה קורה עבור המקטע האחרון במחרוזת.

דורון

אבל למה STRING כן עובד וCHAR * לא עובד?

יכול להיות שאת char* לא הקצאת כמו שצריך, או שהתוכנית "עפה".
אם אתה עובד ב-++C, תקפיד להשתמש ב-string. הרבה יותר פשוט.

????????????

רכז בתגובה הבאה אלי את כל השאלות שנשארו לך, אני אגיב עם תשובות ואנעל את האשכול.

דורון