היי

יש לי קוד עם הרבה לופים, on enter frame -ים וכו'

הבעיה היא שכאשר הקוד עובד הוא מעט מאוד את ניגון הסרט (יותר נכון משחק)

כיצד אפשר לסדר (לשפר) את הביצועים?

תודה מראש

לצמצם את כמות ה onEnterFrame.
זה חלק שלוקח הרבה משאבים והרבה עושים יותר ממש שצריך.

בעייקרון כמה שפחות פקודות יותר טוב

ואם יש לך רקורסיה עדיף לנסות למצוא פיתרון אחר

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

רקורסיה? מה זה ?


ואם אין לי אפשרות לצמצם את הOEN, אין עוד דרך ?


תודה שענית כזה מהר [=

אם אתה לא יודע מה זה רקורסיה אז כנראה שאין לך.

ותמיד יש דרך לצמצם את מספר ה onEnterFrame לאחד.

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

oen = on enter frame

כן, הצלחתי לחשוב על רעיון איך לצמצם

תודה על העזרה

אז חשבתי שהיה לי פתרון, אבל בסוף לא

אז ככה:
יש לי לופים שמשכפלים
לכל שכפול יש OEN משלו
זה בטוח מה שמעת את המשחק, למישהו יש רעיון איך לחבר אותם?

תודה מראש

כן אני יודע שזאת הודעה שלישית ברצף, אבל אתם תיסלחו לי פעם אחת לא?

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

כלומר לירייה יש אאפ משלה, אז שמתי אותיה בתוך האאפ הראשי, בתוך הלופ

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

אותו דבר קורה לי עם כל הדברים על המסך

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

תודה מראש

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

אני בדרך כלל שם את המשתנים על האוביקט לדוגמא
fire0.speed
fire1.speed
וככה אין עירבוב בין המשתנים

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

היי

כן ככה גם אני עושה
הנה אני ייתן קצת קוד

קוד:

 _root.onMouseDown = function() {
bc++;
bulletNum++;
laser.duplicateMovieClip("laser"+bc, d++);
_root["laser"+bc]._x = _root.man._x + _root.man._width / 2 ;
_root["laser"+bc]._y = _root.man._y 
shot.start();
} 

זו היא הפונקציה להכנת היריות, לא בon enter frame ולא בloop

קוד:

 _root.onEnterFrame = function() {
for(i=0; i<allEnemys; i++) {
_root["laser"+bc].laserSpeed = 20;
_root["laser"+bc]._x += _root["laser"+bc].laserSpeed;

וזה בתוך on enter frame
בתוך loop שהתנאי של הלופ מתעדכן בתוך ה on enter frame

מדוע זה לא עובד?
תודה מראש

א. למה יש את המשתנה bulletNum?
ב. למה בעומק אתה שם משתנה שייבדוק את העומק הבא, ישנה פונקציה שמורה - getNextHighestDepth(), כנראה פה הבעיה כי לא נראה לי גם שאפשר גם להעלות וגם לקלוט אותו כמו שאתה עשית.
ג. למה אתה מציב את מהירות היריות ל 20 ואחר כך בודק מהי, קצת הגיון :| גם אין טעם להשים את המשתנה על כל אובייקט יריה אם הוא אותו הדבר, פשוט תעשה משתנה על הבמה הראשית וכולם יישתמשו בו.
ד. הייתי ממליץ לך לשם נוחות לעשות משתנה-מוביקליפ למוביקליפ _root["laser"+bc]._ כל פעם ואז יהיה לך יותר נוח במקום לכתוב כל פעם את כל השם, למשל:

קוד:

 l:MovieClip = _root["laser"+bc]._

ואם תרצה נגיד לבדוק את מיקום היריה תעשה פשוט l._x.

היי

לא היה לי מחשב כמה ימים \=

1- bulletNum זה כמה יריות יש, אם יריייה נעלמת אני מוריד ממנו אחד
זה היריות של האוייב ושל השחקן

2- d אצלי זה getNextHighestDepth()
לא יודע, לי יותר נוח להשתמש בזה
אני שיניתי את זה עכשיו לgetNext...

3- כן זה טעות, זה לא בסקריפט המקורי, לא איך זה נכנס לשם

4- אני יעשה את זה..

תודה

שיניתי את הדברים ועדיין אין שינוי \=
מה עוד יכולה להיות הבעיה?

הנה הקוד שעשיתי:

קוד:

 _root.onMouseDown = function() {
 bc++;
 laser.duplicateMovieClip("laser"+bc, getNextHighestDepth());
 _root["laser"+bc]._x = _root.man._x + _root.man._width / 2 ;
 _root["laser"+bc]._y = _root.man._y 
 myLaser = _root["laser"+bc];
 shot.start();
}
 

והלולאה

קוד:

 for(i=0; i<allEnemys; i++) {
//bullet!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!  
  myLaser._x += laserSpeed;
  if(myLaser._x - myLaser._width > SW) {
   myLaser.removeMovieClip();
  }
 

זה בתוך on enter frame
למה כשאני יורה
הירייה עוצרת כאשר אני יורה שוב?

קוד:

 

_root.onMouseDown = function() {
bc++;
laser.duplicateMovieClip("laser"+bc, laser.getNextHighestDepth());
_root["laser"+bc]._x = _root.man._x + _root.man._width / 2 ;
_root["laser"+bc]._y = _root.man._y 
myLaser = _root["laser"+bc];
shot.start();
}

יצאתי טמבל \=

עדיין יש לי בעיות, הפעם אין לי מושג למה

טוב להגיד את האמת
שום דבר עכשיו לא עובד \=

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

בכל מקרה אני חושב שאני יתחיל מהתחלה
יהיה הרבה יותר פשוט

קוד:

_root.onMouseDown = function() {
 bc++;
 laser.duplicateMovieClip("laser"+bc, laser.getNextHighestDepth());
 _root["laser"+bc]._x = _root.man._x + _root.man._width / 2 ;
 _root["laser"+bc]._y = _root.man._y 
 shot.start();
 myLaser = _root["laser"+bc];
}
  

ככה אני מכין את הירייה

קוד:

 for(i=0; i<bc; i++) {
  myLaser._x += laserSpeed;
  if(myLaser._x - myLaser._width > SW) {
   myLaser.removeMovieClip();
   bc -= 1;
  } 

וזה בתוך on enter frame
עכשיו כשאר אני יורה, הירייה הראשונה מתפקדת כמו שצריך
אבל היריות האחרי כבר נהרסות

ועוד שאלה קטנה, אם יש לי לופ בתוך ה on enter frame בתוך הלופ, זה בסדר? כי עכשיו כשאני מנסה לעשות לופם בon enter frame ליריות, יש לי בתוך היריות לופ לפגיעה באוייבים והוא לא עובד

מקווה שהבנתם
תודה

למה אי אפשר לערוך בפורום הזה!?

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

ניסיתי הרבה דברים, אף אחד מהם לא עבד
רעיונות?

נסה במקום laser.getNextHighestDepth() לכתוב את ה bc.
נרהא לי שההודעה הקודמת הייתה טעות שלי :|

היי

נראה לי אתה היחיד שעוזר בפורום הזה (טוב נו גם בן אור)

ניסיתי כבר bc
האם יש דרך לשכלל את bc?
נגיד bc*10?
משו בסגנון

אני בדרך כלל משתמש ב getNextHighestDepth בגלל זה אני שואל...

בכל מקרה כאשר אני עושה סתם bc הוא מעלים לי אחרי הירייה הראשונה את החללית שלי
יש כזה דבר getNextHighestDepth + bc?

תודה לעוזרים

נ.ב. הייתי מנסה פשוט אין לי מחשב \=

נגיע לפתרון מהר יותר בטח זה סתם משהו קטן ולא קשור :|
אני בהתחלה חשבתי שאתה משכפל לתוך laser אז אמרתי לך שתכתוב laser.getNextHighestDepth() אבל אז ראיתי שזה duplicate אז אתה משכפל את laser ל _root אז אין סיבה שזה לא יעבוד.
נסה אולי להדגיש את _root.getNextHighestDepth().