Кто поможет допилить скрипт для Google Spreadsheets?

Question

[T-ed]

Есть такой скрипт:

function onOpen() {
  SpreadsheetApp.getUi().createMenu("Extra")
    .addItem("Обновить данные", "updatePoints")
    .addToUi();
}

function updatePoints() {
    var sh = SpreadsheetApp.getActiveSheet();
    var datesColumn = sh.getRange( "A1:A" ).getValues();
    var lastRow = datesColumn.filter( String ).length;
    var oldDate = datesColumn[ lastRow - 1 ][ 0 ];
    var oldPoints = parseFloat( sh.getRange( "B" + lastRow ).getValues() );
    //обновляем валютки на сегодня
    sh.getRange( 'e2' ).setValue( '=ROUND(IMPORTXML("http://www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value");2)' );
    //сохраняем текущую дату-время
    var currentDate = Utilities.formatDate( new Date(), "GMT+3", "dd.MM HH:mm" );
    sh.getRange( 'e1' ).setValue( currentDate );
    //забираем текущие цифры
    var currentPoints = parseFloat( sh.getRange( 'e2' ).getValues() );
    if ( currentPoints !== oldPoints ) {
        //сохраняем новые поинты и дату-время обновления
        lastRow++;
        sh.getRange( "B" + lastRow ).setValue( currentPoints );
        sh.getRange( "A" + lastRow ).setValue( currentDate );
        Logger.log( currentPoints + " <> " + oldPoints )
            //MailApp.sendEmail("", "[Alert] ["+currentDate+"] points raised to " + currentPoints, "Old points " + oldPoints
                              //+"\nNew points "+currentPoints+"\n\nUpdated:"+currentDate,{noReply:true});
    } else {
        Logger.log( currentPoints + " = " + oldPoints )
    }
}

По задумке должен забирать и сохранять в Гугл Таблицу курс определенной валюты к рублю, в примере USD. Проблема в том, что курс получается 560,408. Пробую округлить с помощью ROUND до 56, 04 но ничего не выходит, я полный чайник в этом. Где ошибка, подскажите, пожалуйста.

Answer 1

[Алексей Копендаков]

Хм, так тут просто округление идет до 56.00 и все. Для округления до двух знаков после запятой, надо так и писать:
sh.getRange( 'e2' ).setValue( '=ROUND(IMPORTXML("www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value");2)' );
У меня так получилось округлить....

Comments

[T-ed]

Так вы же написали то же самое, что у меня в скрипте. Результат 560,408

[Алексей Копендаков]

T-ed, Не, не, Для начала, ваша строка и моя -

sh.getRange( 'e2' ).setValue( '=ROUND(IMPORTXML("http://www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value"))' );
sh.getRange( 'e2' ).setValue( '=ROUND(IMPORTXML("http://www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value");2)' );

Отличие - ";2" - округлять до 2х знаков после запятой, у Вас в функции ROUND - округляет вообще до целого. Так что я даже не понимаю, откуда у вас может взяться 560,408

[T-ed]

Извиняюсь, не понимаю, как я упорол этот косяк. У меня изначально эта строка выглядела так же, как у вас, хз как я потерял ";2" при копировании. Сейчас поправил вопрос. Проблема остается, на выходе получаем 560,408, когда должно быть 56, 04. ХЗ почему

[Алексей Копендаков]

Вся шутка в том, что у меня получается именно 56,04
А откинуть round? что записывает в ячейку? 56,0408 ? или что-то другое? Просто получить 560,408 можно только, если умножить на 10. Попробуйте именно скопировать и вставить мою строку.

[T-ed]

Я так и делаю, в смысле копирую и вставляю. Что за мистика творится. Давайте попробуем так, вот еще раз весь мой скрипт, который есть сейчас

function onOpen() {
  SpreadsheetApp.getUi().createMenu("Extra")
    .addItem("Обновить данные", "updatePoints")
    .addToUi();
}

function updatePoints() {
    var sh = SpreadsheetApp.getActiveSheet();
    var datesColumn = sh.getRange( "A1:A" ).getValues();
    var lastRow = datesColumn.filter( String ).length;
    var oldDate = datesColumn[ lastRow - 1 ][ 0 ];
    var oldPoints = parseFloat( sh.getRange( "B" + lastRow ).getValues() );
    //обновляем валютки на сегодня
  sh.getRange( 'e2' ).setValue( '=ROUND(IMPORTXML("http://www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value");2)' );
    //сохраняем текущую дату-время
    var currentDate = Utilities.formatDate( new Date(), "GMT+3", "dd.MM HH:mm" );
    sh.getRange( 'e1' ).setValue( currentDate );
    //забираем текущие цифры
    var currentPoints = parseFloat( sh.getRange( 'e2' ).getValues() );
    if ( currentPoints !== oldPoints ) {
        //сохраняем новые поинты и дату-время обновления
        lastRow++;
        sh.getRange( "B" + lastRow ).setValue( currentPoints );
        sh.getRange( "A" + lastRow ).setValue( currentDate );
        Logger.log( currentPoints + " <> " + oldPoints )
            //MailApp.sendEmail("[e-mail]", "[Alert] ["+currentDate+"] points raised to " + currentPoints, "Old points " + oldPoints
                              //+"\nNew points "+currentPoints+"\n\nUpdated:"+currentDate,{noReply:true});
    } else {
        Logger.log( currentPoints + " = " + oldPoints )
    }
}

У меня по нему в ячейке "Текущий курс" получается 560,408. А у вас сколько получается?

[T-ed]

Без ROUND получается тоже 560,408. Похоже, что у меня ROUND не работает

[Алексей Копендаков]

в ответ мне добавилась строчка
A B
03.02 17:37 56,04

И в E1 - 56,04 при наведении - =ROUND(IMPORTXML("www.cbr.ru/scripts/XML_daily.asp"; "//ValCurs/Valute[CharCode=""USD""]/Value");2)

Это я один в один создал новую таблицу в гугле, добавил скрипт, в таблицу закинул 3 строки - в колонку А - 3 даты, от текущей, до позавчера, и в колонку B - числа от фанаря.

[Алексей Копендаков]

T-ed, Если без Round в ячейке E2 получается 560,408 - то это не ROUND виноват, а парсер, а если тупо зайти на www.cbr.ru/scripts/XML_daily.asp - сколько валюта показывается?

[T-ed]

Когда открываю в браузере и нахожу в списке доллар, показывается 56,0408

Как такое может быть? Оба выполняем идентичный скрипт в Google Spreadsheet, но у вас все ок, у меня какая то неведомая хрень. Пребываю в легком шоке

[Алексей Копендаков]

T-ed, Могу предложить только следующее - создать новую таблицу, и скопировать туда код скрипта.
Это раз.
И второе - Что показывается в ячейке E2 ?

[T-ed]

Да, до меня тоже дошло, что нужно опробовать пересоздать файл и тут как раз вы то же самое посоветовали. Я этот файл копировал у другого человека и потом сам вносил правки в код, чтобы подогнать под свои нужды. Я Хз как так вышло, но без видимых причин появилась вот такая ошибка. Сейчас все ок, вам огромное спасибо за помощь!
Столько времени убил из-за какой-то ерунды)