[javascript] rozpoznanie, walka Opcje

[asidro]

[JAVASCRIPT] pobierz, plaintext 
 
var health2 = 100; ///poziom zycia 2playera///
 
 
function infap() ///funkcja karty infantry///
{
if(health2 <= 100 && health2 >=5)
{
health2 -= 5;
document.getElementById("zdrowie2").innerHTML = health2;
}
else if(health2 <= 5)
{
health2 = 0;
document.getElementById("zdrowie2").innerHTML = health2;
}
}
 
 
[JAVASCRIPT] pobierz, plaintext 

[JAVASCRIPT] pobierz, plaintext 
 
function fight()
{
if ($('div#kartawalka1 img.infa1').length>0) //* rozpoznanie karty playera po przeniesieniu*//
{ 
$('.change').removeAttr("disabled");
 
 
$('div#kartawalka1').empty();
infap(); //*wywolanie funkcji*//
}
else if($('div#kartawalka1 img.tank1').length>0)
{
$('.change').removeAttr("disabled");
odblokuse();
 
$('div#kartawalka1').empty();
tankp();
}
else if($('div#kartawalka1 img.bomber1').length>0)
{
$('.change').removeAttr("disabled");
odblokuse();
 
 
$('div#kartawalka1').empty();
bomberp();	
}
 
[...]
else if ($('div#kartawalka2 img.infa5').length>0) //*rozpoznanie karty przeciwnika*//
{ 
 
slot5();
$('div#kartawalka2').empty();
 
$('.drawopp').removeAttr("disabled");
 
 
}
else if ($('div#kartawalka2 img.tank5').length>0)
{ 
 
slot5();
$('div#kartawalka2').empty();
 
$('.drawopp').removeAttr("disabled");
 
 
}
 
[JAVASCRIPT] pobierz, plaintext 

No i teraz nie wiem jak zadeklarować zmienne itd. żeby to wszystko logicznie współgrało. Jeżeli zrobie warunki dla kart gracza to wszystko działa ok i health2(przeciwnik) się zmienia ale gdy wszystko razem połącze to nie uzyskuje zamierzonego efektu na tej samej zasadzie. Mam nadzieje ze mnie zrozumiecie, jesli nie to wyjasnie bardziej szczegółowo.

[kamil4u]

Mam nadzieje ze mnie zrozumiecie, jesli nie to wyjasnie bardziej szczegółowo.

To ja jednak poproszę o te wyjaśnienia (IMG:style_emoticons/default/smile.gif)

Tu taka lista porad (IMG:style_emoticons/default/winksmiley.jpg) :
- uważaj na wyszukiwanie elementów - robisz to w sposób bardzo czasochłonny np:

Kod

($('div#kartawalka2 img.infa5')

Musi to działać bardzo powoli(oczywiście przy takim kodzie tego nie zauważysz, ale później przy większych projektach szybkość będzie mogła mieć znaczenie) - zastanów się nad strukturą HTML i jakbyś mógł zrobić, żeby odwoływać się do elementów lepiej - szybciej(bardziej bezpośrednio) (IMG:style_emoticons/default/smile.gif)
- zdecyduj się czy korzystasz z jQuery czy z czystego JS - ja polecam Ci czysty JS, choć na pewno będę w mniejszości (IMG:style_emoticons/default/tongue.gif)
- nie powtarzaj kodu: funkcja: infap -> document.getElementById("zdrowie2").innerHTML = health2;
- zapoznaj się ze switch-em