Чудо-скрипт для отключения неотключаемых подписей

Elina74

В смысле, код для удаления части текста в посте, если в нем содержится три подряд знака "-".
Покритикуйте, что ли, если есть чего критиковать.

 

function func1 {

   var a = document.getElementsByTagName('font');   

   for(i=0; i<a.length; i++){

       if(a[i].className == 'post'){

            var out = "";

            var p = a[i].innerHTML;            

            var flag = 0;            

            for(j=0; j<p.length-3; j++){

               if(p.charAt(j)=='-' && p.charAt(j+1)=='-' && p.charAt(j+2)=='-'){

                   flag = 1;

                   break;

               }

               else{

                   out = out + p.substr(j,1);

               }

            }

            if(flag){

                a[i].innerHTML = out + "<br/><br/>";

            }

            

       }

   }

}

window.onload = func1;

freezer

будет медленно работать... а если подпись отбивать подчеркиванием или картинкой?

olga1969

Проверяй лучше

 p.charAt(j)=='-'

по отдельности, и в случае равенства, проверяй дополнительно

 p.charAt(j+1)=='-' && p.charAt(j+2)=='-')

.
А то, грубо говоря, у тебя каждый символ по три раза проверяется.

olga1969

Хотя...
если результатом

  p.charAt(j)=='-'

будет false, то следующие два выражения в if-statement могут и не вычисляться...

garikus



function func1 {

    var a = document.getElementsByTagName('font');   

    for (i = 0; i < a.length; i++){

        if (a[i].className == 'post') {

           var p = a[i].innerHTML;

           var j = 0;

           while j < p.length - 2) && !p.charAt[j] == '-') && (p.charAt[j + 1] == '-') && (p.charAt[j + 2] == '-' {

                j++;

            }

            if (j != p.length - 2) {

                a[i].innerHTML = p.substr(0, j) + "<br/><br/>";

            }

        }

    }

}

Elina74

В предыдущем коде не учитывалось, что три знака минуса в конце длинной строки "----------------" это не то же самое, что просто три минуса. Исправлено. Теперь не съедаеются части полезных обсуждений:



function func1 {

   var a = document.getElementsByTagName('font');   

   for(i=0; i<a.length; i++){

       if(a[i].className == 'post'){

            var out = "";

            var p = a[i].innerHTML;            

            var flag = 0;            

            var minus_len=0;

            for(j=0; j<p.length-1; j++){

               if(p.charAt(j) == '-'){

                   minus_len++;

                   if(p.charAt(j+1) != '-' && minus_len==3){

                       flag = 1;

                       break;

                   }

                   if(p.charAt(j+1) != '-' && minus_len != 3){

                       for(k=0; k<minus_len; k++){

                           out = out + '-';

                       }

                       minus_len = 0;

                   }

               }

               else{

                   out = out + p.charAt(j,1);

               }

            }

            if(flag){

                a[i].innerHTML = out + "<br/><br/>";

            }

            

       }

   }

}

window.onload = func1;

garikus



function func1 {

	var a = document.getElementsByTagName('font');   

	for (i = 0; i < a.length; i++) {

		if (a[i].className == 'post') {

			var p = a[i].innerHTML;

			var j = 0;

			do {

				while j < p.length) && !(p.charAt[j] == '-' { j++; }

				var k = j;

				while j < p.length) && (p.charAt[j] == '-' { j++; }

				if (j == k) { j++; }

			} while !j >= p.length) || (j - k == 3;

			if (j - k == 3) {

				a[i].innerHTML = p.substr(0, j) + "<br/><br/>";

			}

		}

	}

}

window.onload = func1;

Оставить комментарий