Skip to main content

Reduce redundant code by optimizing of appendRow() App script

I'm working on a project where I shall send data from my app to a google spreadsheet in 2 sheets through app script.

I set up a paraphrase to secure the connection. Hkey

I have a lot of data to send like computer1,brand1,cpu1,ram1,computer2,brand2,cpu2,ram2 until 20.

my first version of the code comprise a lot of data like below:

function doGet(e) {
  var ss = SpreadsheetApp.getActive();
  var sheet1 = ss.getSheetByName("Sheet1");
  var Sheet2 = ss.getSheetByName('Sheet2');

  var Region = e.parameter.Region ; 
  var Place = e.parameter.Place ;
  var Staff = e.parameter.Staff ;

  var computer1 = e.parameter.computer1;
  var brand1 = e.parameter.brand1;
  var cpu1 = e.parameter.cpu1;
  var ram1 = e.parameter.ram1;

  var computer2 = e.parameter.computer2;
  var brand2 = e.parameter.brand2;
  var cpu2 = e.parameter.cpu2;
  var ram2 = e.parameter.ram2;

  var Hkey = e.parameter.Hkey;

if (Hkey == 'AB12xyz') {
  sheet1.appendRow(Region, Place, Staff, computer, brand, cpu, ram);
  Sheet2.appendRow(Region, Place, Staff, computer, brand, cpu, ram);
  }
}

Then I optimised the code to avoid repeated tasks. where if the computer is true, then append computer1,brand1,cpu1,ram1 and so on...until computer20,brand20,cpu20,ram20

Optimised code :

function doGet(e) {
  var ss = SpreadsheetApp.getActive();
  var sheet1 = ss.getSheetByName("Sheet1");
  var Sheet2 = ss.getSheetByName('Sheet2');

  var ID = "FS" + new Date("GMT+4", "ddMM");
  
  var Region = e.parameter.Region ; 
  var Place = e.parameter.Place ;
  var Staff = e.parameter.Staff ;
  var Shop = e.parameter.Shop;
  var Location = e.parameter.Location ;
  var Date_Checked = e.parameter.Date_Checked ;
  var Time_Checked = e.parameter.Time_Checked ;
  var Link = "check your inbox" ;
  var Remarks = e.parameter.Remarks ;

  var Hkey = e.parameter.Hkey;

  if (Hkey == 'AB12xyz') {
    for (var i = 1; i <= 20; i++) {
      var computer = e.parameter['computer' + i];
      var brand = e.parameter['brand' + i];
      var cpu = e.parameter['cpu' + i];
      var ram = e.parameter['ram' + i];

      if (computer) {
        var row = [ID,Region,Place,Staff,Shop,Location, Date_Checked,Time_Checked,Link,Remarks];
        sheet1.appendRow(row, computer, brand, cpu, ram);
        Sheet2.appendRow(row, computer, brand, cpu, ram);
       
      }
    }

The issue is that nothing is appended on the 2 sheets of the spreadsheet. When checking the script deployment overview, I got only failures.

Via Active questions tagged javascript - Stack Overflow https://ift.tt/SywV0cm

Comments

Popular posts from this blog

How to show number of registered users in Laravel based on usertype?

i'm trying to display data from the database in the admin dashboard i used this: <?php use Illuminate\Support\Facades\DB; $users = DB::table('users')->count(); echo $users; ?> and i have successfully get the correct data from the database but what if i want to display a specific data for example in this user table there is "usertype" that specify if the user is normal user or admin i want to user the same code above but to display a specific usertype i tried this: <?php use Illuminate\Support\Facades\DB; $users = DB::table('users')->count()->WHERE usertype =admin; echo $users; ?> but it didn't work, what am i doing wrong? source https://stackoverflow.com/questions/68199726/how-to-show-number-of-registered-users-in-laravel-based-on-usertype

Why is my reports service not connecting?

I am trying to pull some data from a Postgres database using Node.js and node-postures but I can't figure out why my service isn't connecting. my routes/index.js file: const express = require('express'); const router = express.Router(); const ordersCountController = require('../controllers/ordersCountController'); const ordersController = require('../controllers/ordersController'); const weeklyReportsController = require('../controllers/weeklyReportsController'); router.get('/orders_count', ordersCountController); router.get('/orders', ordersController); router.get('/weekly_reports', weeklyReportsController); module.exports = router; My controllers/weeklyReportsController.js file: const weeklyReportsService = require('../services/weeklyReportsService'); const weeklyReportsController = async (req, res) => { try { const data = await weeklyReportsService; res.json({data}) console...

How to split a rinex file if I need 24 hours data

Trying to divide rinex file using the command gfzrnx but getting this error. While doing that getting this error msg 'gfzrnx' is not recognized as an internal or external command Trying to split rinex file using the command gfzrnx. also install'gfzrnx'. my doubt is I need to run this program in 'gfzrnx' or in 'cmdprompt'. I am expecting a rinex file with 24 hrs or 1 day data.I Have 48 hrs data in RINEX format. Please help me to solve this issue. source https://stackoverflow.com/questions/75385367/how-to-split-a-rinex-file-if-i-need-24-hours-data