Google Sheets

@kurganoffiv · Регистрация: 08.02.2017

Студворк — интернет-сервис помощи студентам

Всем привет.
Не очень сильно разбираюсь в программировании, в связи с чем есть проблема: скрипт, представленный ниже, очень долго работает (1 - 10 минут). Данным скриптом выбираю данные из 30 книг. Подскажите, пожалуйста, как ускорить его работу!

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
function getInformation() {
   var lDefaultSettings = "Default Settings";
   var lResult = "Отчет";
   var arrayWorkSheets = new Array();
   var arrayResult = new Array();
   var arrayResultFinal = new Array();
 
  var cityNameResult_c = 1;
  var minWorkSheet_c = 18;
  var minWorkSheet_r = 18;
  var maxWorkSheet_c = 19;
  var maxWorkSheet_r = 18;
  
  var totalSummResult_c = 2;
  var clientSumResult_c = 9;
  var sumServiceResult_c = 11;
  var minResult_c = 4;
  var maxResult_c = 6;
 
try{
  var lr = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(lResult);
  var monthResult = lr.getRange(1,1).getValues();
  var yearResult = lr.getRange(2,1).getValues();
  var lrResult = lr.getLastRow();
  var arrayResult = lr.getRange("A3:A"+lrResult).getValues();
  
  
  var workSheetName = monthResult + " " + yearResult;
  var totalSumWorkSheet=0;
  var clientSumWorkSheet=0;
  var serviceSumWorkSheet=0;
  var lrDefaultSettings = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(lDefaultSettings).getLastRow();
  
  for (var rr = 0; rr < lrResult-2; rr++){
    var cityName = arrayResult[rr];    
     
    for (var rds = 3; rds < lrDefaultSettings+1; rds++){
      var ld = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(lDefaultSettings);
      //var vDefaultSettingsName = ld.getRange(rds, 1).getDisplayValue();
      var vDefaultSettingsName = ld.getRange(rds, 1).getValue();
      if (cityName==vDefaultSettingsName){
        var vDefaultSettingsId = ld.getRange(rds, 2).getValues();
        try{
          var openTable = SpreadsheetApp.openById(vDefaultSettingsId).getSheetByName(workSheetName);
          }
         catch(err){
           Browser.msgBox('Вероятно у меня нет доступа к листу '+cityName+'. Обновление данных остановлено');
           return;
         }
        if (openTable==null){
          Browser.msgBox('Лист с именем '+workSheetName+' в книге '+cityName+' отсутсвует. Обновление данных остановлено');
          return;
        }
        var minWorkSheet = openTable.getRange(minWorkSheet_r,minWorkSheet_c).getDisplayValues();
        var maxWorkSheet = openTable.getRange(maxWorkSheet_r,maxWorkSheet_c).getDisplayValues();
        var lrWorkSheet = openTable.getLastRow();
        var totalSumWorkSheet = 0;
        var clientSumWorkSheet = 0;
        var arrayWorkSheets = openTable.getRange(2,1,lrWorkSheet,6).getValues();
        
        for (var rws = 0; rws < lrWorkSheet-2; rws++){ 
          var sumWorkSheet = arrayWorkSheets[rws][5];
          var clientWorkSheet = arrayWorkSheets[rws][2];
          var sumServiceWorkSheet = arrayWorkSheets[rws][4];;
          var totalSumWorkSheet = totalSumWorkSheet + sumWorkSheet;
          var clientSumWorkSheet = clientSumWorkSheet + clientWorkSheet;
          var serviceSumWorkSheet = serviceSumWorkSheet + sumServiceWorkSheet;
          }
        
        arrayResultFinal.push([[cityName],[totalSumWorkSheet],[minWorkSheet],[maxWorkSheet],[clientSumWorkSheet],[serviceSumWorkSheet]]);
        var totalSumWorkSheet=0;
        var clientSumWorkSheet=0;
        var minWorkSheet=0;
        var maxWorkSheet=0;
        var serviceSumWorkSheet=0;
      }
      
    }
    
    }
     for (var i = 0; i < arrayResultFinal.length; i++){
       var a = arrayResultFinal[i][1];
       var b = arrayResultFinal[i][2];
       var c = arrayResultFinal[i][3];
       var d = arrayResultFinal[i][4];
       var e = arrayResultFinal[i][5];
       lr.getRange(i + 3, 2).setValue(a);
       lr.getRange(i + 3, 3).setValue(b);
       lr.getRange(i + 3, 5).setValue(c);
       lr.getRange(i + 3, 8).setValue(d);
       lr.getRange(i + 3,11).setValue(e);
     }
    
   Browser.msgBox('Обновление данных завершено');
}
catch(err){
  Browser.msgBox('Если ты это видишь, то сломалось что то очень серьезно!');
  return;
}
 
}

@Уф · 27.02.2017, 17:42

у тебя внутри циклов для обращении к ячейке каждый раз идет обращение к листу

JavaScript
1
2
for (var rds = 3; rds < lrDefaultSettings+1; rds++){
var ld = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(lDefaultSettings);

может надо повыносить такие строки за пределы циклов в начало?

@kurganoffiv · 27.02.2017, 17:50 **[ТС]**

Уф, в данном цикле я получаю ссылку на книгу, из которой буду брать данные. Далее я открываю данную книгу.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
for (var rds = 3; rds < lrDefaultSettings+1; rds++){
var ld = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(lDefaultSettings);
//var vDefaultSettingsName = ld.getRange(rds, 1).getDisplayValue();
var vDefaultSettingsName = ld.getRange(rds, 1).getValue();
if (cityName==vDefaultSettingsName){
var vDefaultSettingsId = ld.getRange(rds, 2).getValues();
try{
var openTable = SpreadsheetApp.openById(vDefaultSettingsId).getSheetByName(workSheetName);
}
catch(err){
Browser.msgBox('Вероятно у меня нет доступа к листу '+cityName+'. Обновление данных остановлено');
return;
}

Да, потенциал оптимизации есть: засунуть значения в массив и брать их оттуда, но маловероятно, что это кардинально изменил ситуацию. А Google Sheets по практике работаю быстрее?

@Уф · 27.02.2017, 22:49

три вложенных в друг друга цикла ясно что не будут быстро работать, еще у тебя привычка везде писать слово var даже если переменная уже объявлена, в строке 29 и 71 например по 5 раз одно и тоже, зачем?

т.е. выносить переменную из цикла ты не хочешь? для чего вообще просишь совета тогда?

@kurganoffiv · 28.02.2017, 11:49 **[ТС]**

Уф, это не привычка, это просто от незнания

Спасибо за совет, подумаю как избежать циклов.

@Уф · 28.02.2017, 15:50

могу попробовать помучить код, но надо живой пример, скинь таблицу какую-нибудь, или ссылку на неё, может че подкручу.

Новые блоги и статьи Все статьи Все блоги /
Мастер-класс по микросервисам на Node.js Reangularity 21.06.2025 Node. js стал одной из самых популярных платформ для микросервисной архитектуры не случайно. Его неблокирующая однопоточная модель и событийно-ориентированный подход делают его идеальным для. . .	Управление Arduino из WPF приложения Wired 21.06.2025 Зачем вообще связывать Arduino с WPF-приложением? Казалось бы, у Arduino есть собственная среда разработки, своя экосистема, свои способы управления. Однако при создании серьезных проектов. . .	Звёздная пыль kumehtar 20.06.2025 Я просто это себе представляю: как создавался этот мир. Как энергия слипалась в маленькие частички. Как они собирались в первые звёзды, как во вселенной впервые появился Свет. Как эти звёзды. . .	Создание нейросети с PyTorch AI_Generated 19.06.2025 Ключевое преимущество PyTorch — его питоновская натура. В отличие от TensorFlow, который изначально был построен как статический вычислительный граф, PyTorch предлагает динамический подход. Это. . .	JWT аутентификация в ASP.NET Core UnmanagedCoder 18.06.2025 Разрабатывая веб-приложения, я постоянно сталкиваюсь с дилеммой: как обеспечить надежную аутентификацию пользователей без ущерба для производительности и масштабируемости? Классические подходы на. . .
Краткий курс по С# aaLeXAA 18.06.2025 Здесь вы найдете все необходимые функции чтоб написать програму на C# Задание 1: КЛАСС FORM 1 public partial class Form1 : Form { Spisok listin = new Spisok(); . . .	50 самых полезных примеров кода Python для частых задач py-thonny 17.06.2025 Эффективность работы разработчика часто измеряется не количеством написаных строк, а скоростью решения задач. Готовые сниппеты значительно ускоряют разработку, помогают избежать типичных ошибок и. . .	C# и продвинутые приемы работы с БД stackOverflow 17.06.2025 Каждый . NET разработчик рано или поздно сталкивается с ситуацией, когда привычные методы работы с базами данных превращаются в источник бессонных ночей. Я сам неоднократно попадал в такие ситуации,. . .	Angular: Вопросы и ответы на собеседовании Reangularity 15.06.2025 Готовишься к техническому интервью по Angular? Я собрал самые распространенные вопросы, с которыми сталкиваются разработчики на собеседованиях в этом году. От базовых концепций до продвинутых. . .	Архитектура Onion в ASP.NET Core MVC stackOverflow 15.06.2025 Что такое эта "луковая" архитектура? Термин предложил Джеффри Палермо (Jeffrey Palermo) в 2008 году, и с тех пор подход только набирал обороты. Суть проста - представьте себе лук с его. . .

@Уф 890 / 725 / 447 Регистрация: 13.07.2015 Сообщений: 2,277
	27.02.2017, 22:49
	три вложенных в друг друга цикла ясно что не будут быстро работать, еще у тебя привычка везде писать слово var даже если переменная уже объявлена, в строке 29 и 71 например по 5 раз одно и тоже, зачем? т.е. выносить переменную из цикла ты не хочешь? для чего вообще просишь совета тогда? 0

@kurganoffiv 0 / 0 / 0 Регистрация: 08.02.2017 Сообщений: 9
	28.02.2017, 11:49 [ТС]
	Уф, это не привычка, это просто от незнания Спасибо за совет, подумаю как избежать циклов. 0

@Уф 890 / 725 / 447 Регистрация: 13.07.2015 Сообщений: 2,277
	28.02.2017, 15:50
	могу попробовать помучить код, но надо живой пример, скинь таблицу какую-нибудь, или ссылку на неё, может че подкручу. 0

Опции темы