Форум программистов, компьютерный форум, киберфорум
JavaScript
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.71/7: Рейтинг темы: голосов - 7, средняя оценка - 4.71
 Аватар для Влад.pas
12 / 12 / 4
Регистрация: 30.08.2011
Сообщений: 62

Функция работает некорректно

28.07.2012, 18:20. Показов 1362. Ответов 3
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Добрый день! Сделал функцию, которая должна удалить определённый символ в определённом тексте. Принцип работы: функция конкатенирует всё, что перед и после выбранного символа и записывает результат в переменную. Значение переменной возвращается. Символ следует указывать в виде его индекса в строковом массиве.
Вот код:
JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
function delChar(x,text)
{
 
 if (x > text.length - 1) // убедимся, что индекс указанного символа не превышает границ массива
{
}
else // если всё ок, то делаем своё дело
{
 var tlength = text.length - 1; // эта переменная содержит индекс последнего символа
 for(var i = 0; i < x; i++) // конкатенируем всё перед нашем символом и записываем в result
{
  var result = result + text[i];
}
 for(var last = x + 1; last <= tlength; last++) // всё после символа записываем туда же
{
 result = result + text[last];
}
 return result; // возвращаем результат 
}
}
Проблема: функция возвращает не только результат, но перед ним ставит undefined. К примеру удалим первый символ в слове Welcome! Получится underfinedelcome!

Вызываю функцию так: delChar(0, 'Welcome!');

В чём ошибка???

Добавлено через 13 минут
Проблема решена: первоначально переменной result не было задано значение -> отсюда undefined перед результатом) В код нужно добавить var result = ""; в начало тела первого цикла.

Добавлено через 16 минут
Вот, вроде, конечный вид с поправками:
JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
function delChar(x,text)
{
 if (x > text.length - 1 | x < 0)
{
}
else
{
 var tlength = text.length - 1;
 var result = "";
if (x > 0)
{
 for(var i = 0; i < x; i++)
{
 result = result + text[i];
}
 for(var last = x + 1; last <= tlength; last++)
{
 result = result + text[last];
}
 return result;
}
else
{
 for(var last = x + 1; last <= tlength; last++)
{ 
 result = result + text[last];
}
 return result;
}
}
}
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
28.07.2012, 18:20
Ответы с готовыми решениями:

Некорректно работает карусель
Всем доброго времени суток! Проблема заключается в том что на сайте http://***.ru некорректно работает карусель &quot;Популярные...

Не работает функция
Не работает функция, в чем может быть проблема? &lt;html&gt; &lt;head&gt;&gt; &lt;title&gt;H@H@H@&lt;/title&gt; &lt;script type=&quot;text/javascript&quot;...

Не работает функция
Здравствуйте, помогите разобратся почему функция не срабатывает, в Js разбираюсь плохо, но очень надо : Работаю массивом в json...

3
 Аватар для Vovan-VE
13210 / 6599 / 1041
Регистрация: 10.01.2008
Сообщений: 15,069
28.07.2012, 18:43
JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
String.prototype.delChar = function(x) {
    if (x >= 0 && x < this.length){
        var s = '';
        if (x >= 1)
            s = this.substr(0, x);
 
        if (x < this.length - 1)
            s += this.substr(x + 1);
 
        return s;
    };
};
 
alert('foobar'.delChar(3));
1
супермизантроп
Эксперт JS
3941 / 2979 / 692
Регистрация: 18.04.2012
Сообщений: 8,629
28.07.2012, 19:02
Покритикую маленько.

1. Функция ваша по вашей же логике должна чего-то возвращать.
А в случае неисполнения самого первого условия никакого возвращаемого значения не будет.
Надо хотя бы "пустоту" вернуть, а вы не возвращаете НИЧЕГО. Появится у вас на выходе undefined.

2. Избранный вами способ обращения к конкретному символу исходной строки (конструкция text [i]) не работает в браузере MSIE - там вы ВСЕГДА будете получать строку из сплошных undefined'ов.
Для "выдёргивания" N-ного символа строки существует кроссбраузерный метод строка.charAt (N),
т.е. вместо text [i] и text [last] надо прописать text.charAt (i) и text.charAt (last).

3. Конструкцию result = result + text[last];,
во-первых, проще записать как result += text[last];
во-вторых, лучше этим способом конкатенации вообще не пользоваться - он очень ресурсозатратен.

4. Ну и, наконец, у вас и так ИЗНАЧАЛЬНО имеется строка (если можно так сказать - уже "отконкатенированная").
Вы разбиваете строку на отдельные символы, которые затем снова соединяете...
Зачем? Это нерационально.
-----

Короче:
1) в Javascript'e имеется метод строка. substring (начало подстроки, конец подстроки),
который из любой строки выделит любую подстроку по индексам её начала и конца.
2) есть также разновидность этого метода строка. substr (начало подстроки), который вернёт обрезанную с начала подстроку.

Используя два этих метода, можно существенно упростить вашу функцию:
JavaScript
1
2
3
4
5
6
function delChar (x, text)
{
if (!x) return text;
if (x >= text.length) return '';
return (text.substring (0, x) + text.substr (x + 1));
}
1
 Аватар для Влад.pas
12 / 12 / 4
Регистрация: 30.08.2011
Сообщений: 62
29.07.2012, 08:11  [ТС]
kalabuni, спасибо за критику, она очень помогла. 1 и 3 замечание - моя рассеянность. 2 - очень полезное - люблю кроссбраузерные программы. Спасибо за код в конце. Всё исправлю.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
29.07.2012, 08:11
Помогаю со студенческими работами здесь

Не работает функция
Пожалуйста, помогите! В чем ошибка? &lt;div id='cre' style=&quot;display='block'&quot;&gt;123&lt;/div&gt; &lt;script&gt; function Show () { if...

Не работает функция
window.onload = function () { &quot;use strict&quot; createResultTable (&quot;resultTable&quot;); } function createResultTable ( id ) { ...

Не работает функция
Здравствуйте. Подскажите пожалуйста почему вот это на локалке работает, а когда это же вставляю на сервак, то получаю ошибку ...

Не работает функция на JavaScript
Я еще новичок и учу яваскрипт по видеокурсу Михаила Русакова. В очередном уроке нужно составить функцию, которая выводит на экран подряд 3...

Функция settimeout не работает
&lt;html&gt; &lt;body&gt; &lt;body bgcolor='#000000'&gt; &lt;form name='a123'&gt; &lt;font color='99ccff'&gt; MY TEXT &lt;/font color='99ccff'&gt; &lt;script...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
4
Ответ Создать тему
Новые блоги и статьи
моя боль
iceja 24.01.2026
Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .
Модель сукцессии микоризы
anaschu 24.01.2026
Решили писать научную статью с неким РОманом
http://iceja.net/ математические сервисы
iceja 20.01.2026
Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .
http://iceja.net/ сервер решения полиномов
iceja 18.01.2026
Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .
Расчёт переходных процессов в цепи постоянного тока
igorrr37 16.01.2026
/ * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .
Восстановить юзерскрипты Greasemonkey из бэкапа браузера
damix 15.01.2026
Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений.
anaschu 11.01.2026
https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/
WordPad для Windows 11
Jel 10.01.2026
WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru