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

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

28.07.2012, 18:20. Показов 1385. Ответов 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
Ответ Создать тему
Новые блоги и статьи
Отправка уведомления на почту при изменении наименования справочника
Maks 24.03.2026
Программная отправка письма электронной почты на примере изменения наименования справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи. . .
модель ЗдравоСохранения 5. Меньше увольнений- больше дохода!
anaschu 24.03.2026
Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .
Midnight Chicago Blues
kumehtar 24.03.2026
Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .
Контроль уникальности заводского номера - вариант №2
Maks 24.03.2026
В отличие от предыдущего варианта добавлено прерывание циклов, также добавлены новые переменные для сохранения контекста ошибки перед прерыванием цикла: Процедура ПередЗаписью(Отказ, РежимЗаписи,. . .
SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++
8Observer8 24.03.2026
Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip
Жизнь в неопределённости
kumehtar 23.03.2026
Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .
Модель здравоСохранения: работники работают быстрее после её введения.
anaschu 23.03.2026
geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .
Контроль уникальности заводского номера - вариант №1
Maks 23.03.2026
Алгоритм контроля уникальности заводского (или серийного) номера на примере документа выдачи шин для спецтехники с табличной частью в конфигурации КА2. Данные берутся из регистра сведений, по. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru