[JavaScript]Zmiana zawartości Opcje

[sher]

[XML] pobierz, plaintext 
function up(id) {
	if (document.getElementById('up'+id).style.color == "black" &&  document.getElementById('down'+id).style.color == "black"){
        document.getElementById('up'+id).style.color = "green";
	document.getElementById('up'+id).style.BorderColor = "green";
	var ids = document.getElementById('rate'+id);
	parseInt(ids);
	return ids.innerHTML = ids+1;
	}
}
[XML] pobierz, plaintext 

Jak w elemencie jest cyfra 0, i powinno zmienić się na 1, to zamiast tego pojawia się "[object HTMLSpanElement]1". Jak pozbyć się tego tekstu przed jedynką?

Ten post edytował sher 3.04.2015, 13:54:33

[trueblue]

parseInt zwraca jakąś wartość, a Ty jej nigdzie nie przypisujesz, ale całe szczęście, że tego nie robisz, bo próbujesz zamienić na integer obiekt html.
Jeśli w spanie jest czysty tekst, to pobierz go za pomocą:

[HTML] pobierz, plaintext 
var ids = document.getElementById('rate'+id).innerHTML;
[HTML] pobierz, plaintext 

[sher]

Hmm, teraz nic się nie zmienia. Cały czas pozostaje 0.

[trueblue]

Pokaż kod po zmianach.

[sher]

[XML] pobierz, plaintext 
function up(id) {
	if (document.getElementById('up'+id).style.color == "black" &&  document.getElementById('down'+id).style.color == "black"){
    document.getElementById('up'+id).style.color = "green";
	document.getElementById('up'+id).style.BorderColor = "green";
	var ids = document.getElementById('rate'+id).innerHTML;
	parseInt(ids);
	return ids.innerHTML = ids+1;
	}
}
[XML] pobierz, plaintext 

albo

[XML] pobierz, plaintext 
function up(id) {
	if (document.getElementById('up'+id).style.color == "black" &&  document.getElementById('down'+id).style.color == "black"){
    document.getElementById('up'+id).style.color = "green";
	document.getElementById('up'+id).style.BorderColor = "green";
	var ids = document.getElementById('rate'+id).innerHTML;
	parseInt(ids);
	return ids = ids+1;
	}
}
[XML] pobierz, plaintext 

próbowałem tak i tak.

Edit: jestem upośledzony, zaraz to ogarnę, poczekajcie z odpisywaniem
Edit2: okej udało się,

[XML] pobierz, plaintext 
function down(id) {
	if (document.getElementById('up'+id).style.color == "black" &&  document.getElementById('down'+id).style.color == "black"){
    document.getElementById('down'+id).style.color = "red";
	document.getElementById('down'+id).style.BorderColor = "red";
	var ids = document.getElementById('rate'+id).innerHTML;
	var idx = document.getElementById('rate'+id);
	parseInt(ids);
	return idx.innerHTML = ids-1;
	}
}
[XML] pobierz, plaintext 

Jeszcze jedno pytanie, jak zrobić, żeby zamiast "01" wyświetlało się samo "1"?

Ten post edytował sher 3.04.2015, 14:31:41

[trueblue]

parseInt zwraca wartość, Ty jej nigdzie nie przypisujesz.
Nie wiem co Twoja funkcja ma ostatecznie robić, ale jak chcesz zwrócić wartość, to zrób jak w przykładzie 2. bez przypisywania wartości do zmiennej ids, jeśli chcesz przypisać wartość do zawartości elementu, to zrób jak w przykładzie 1., ale bez return. Tam również musisz pamiętać, że pod ids jest zawartość elementu, a nie on sam.

[IProSoft]

Chyba brakuje Ci tego

[JAVASCRIPT] pobierz, plaintext 
var ids = parseInt( document.getElementById('rate'+id).innerHTML );
document.getElementById('rate'+id).innerHTML = ids + 1;
[JAVASCRIPT] pobierz, plaintext 

[markuz]

[JAVASCRIPT] pobierz, plaintext 
var up = function up(id) {
	var styleUp = document.getElementById('up' + id).style;
	var styleDown = document.getElementById('down' + id).style;
 
	if(styleUp.color == 'black' && styleDown.color == 'black') {
		styleUp.color = 'green';
		styleUp.BorderColor  = 'green';
		var id = parseInt(document.getElementById('rate' + id).innerHTML);
 
		return (id + 1);
	}
 
	return 0;
};
[JAVASCRIPT] pobierz, plaintext 

Jeżeli odnosisz się do jakiejś właściwości więcej niż 1 raz warto ją umieścić w zmiennej (wtedy kod jest wydajniejszy).

[sher]

Dobra, ogarnąłem wszystko. Myślę, że za bardzo się pośpieszyłem z tematem, prawdopodobnie po jakimś czasie i chwili szukania sam bym to zrobił, ale tak czy siak, dzięki wszystkim za pomoc
Temat do zamknięcia.

Ten post edytował sher 3.04.2015, 14:45:36