עזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...

[נ נח]

@chv אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

עוד שאלות?

יש לי שאלה - למה שמת שני if במקום if else?
יותר קצר

function changebackground() {
    let backelement = $( 'body' ).css( 'backgroundImage' ).replace(/.*\s?url\([\'\"]?/, '').replace(/[\'\"]?\).*/, '')
    if (backelement == 'https://domain.com/images/sample.jpg') {
        $( 'body' ).css( 'backgroundImage', 'url(https://domain.com/images/secondPicture.jpg)' )
    }
    else {
        $( 'body' ).css( 'backgroundImage', 'url(https://domain.com/images/sample.jpg)' )
    }
}

[chv]

@נ-נח אין סיבה. אפשר גם if else..

[איציק דייטש]

@chv קודם כל...

תודה רבה!!

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

[chv]

@איציק-דייטש אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

אני גם לא מרוצה מהאחרון שנתת לי

אבל הוא לא Jquery

אני לא רואה איך מפעילים את הפונקציה ע"י הכפתור

כמו שעשית - על ידי listener
או פשוט בתכונה באלמנט של הכפתור - "()onclick="changebackground

[איציק דייטש]

@chv מה זה???

(/.*\s?url\([\'\"]?/, '').replace(/[\'\"]?\).*/, '')

אני לא אומר שזה jquery, אני רק אומר שאני לא מכיר את זה מהלימודים

[chv]

@איציק-דייטש אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

מה זה???

כמו שכתבתי - זה רגקס regex - קיצור של regular expression.
זו דרך לנתח, לשנות ולסדר טקסט על ידי תוכנה על פי כללים טקסטואליים.
https://גוגל-בפורומים.tk/#gsc.tab=0&gsc.q=רגקס

אני לא מכיר את זה מהלימודים

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

[מוטי אורן]

@chv למה שהוא לא יעשה משהו כזה? (זה אותו רעיון כמו ש @נ-נח הביא, אבל עם פיתרון לבעיה שהזכרת)

var backgroundImg = document.getElementsByClassName('welcome')[0];
var button = document.getElementById('change_bg');
var whichImage = 1;
button.addEventListener('click', changeBackground);
function changeBackground() {
  if (whichImage === 1) {
    backgroundImg.style.backgroundImage = 'url(התמונה השניה)';
    whichImage = 2;
  } else {
    backgroundImg.style.backgroundImage = 'url(התמונה הראשונה)';
    whichImage = 1
  }
}

[chv]

@מוטי-אורן צודק.

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

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

[מוטי אורן]

@chv אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

אנחנו כנראה כבר יודעים את זה

בדיוק!!

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

[איציק דייטש]

@chv @מוטי-אורן וואו, אני נדהם כל פעם מהצורה שבה אתם מסבירים, תודה רבה!!!!

@chv אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

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

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

@מוטי-אורן אמר בעזרה | יצירת כפתור שתחליף את הרקע כל פעם לרקע האחר...:

@איציק-דייטש שים לב להסבר: אם וכאשר היית כותב את הסקריפ...

תודה רבה!!

---

אוקיי, תוך כדי כתיבת התגובה, בדקתי את מה ש @מוטי-אורן הביא ... ו...

כל הכבוד!!

תודה רבה!!