Error With Table: Cannot read property 'toString' of undefined

Answers

• httpwilfred

  November 2017 edited November 2017

  Ended up with a solution. Went back to the last commit and rewrote code. Here's the result and it works fine.

  var characterLibrary = new p5.Table();
  
  var input = [" ", "!", "\"", "#", "$", "%", "&", "\'", "(", ")", "*", "+", ",", "-", ".", "/", "0", "1", "2", "3", "4", "5", "6", "7", "8", "9", ":", ";", "<", "=", ">", "?", "@", "A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z", "[", "\\", "]", "^", "_", "`", "a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z", "{", "|", "}", "~"];
  
  var asciiIndex;
  
  function setup() {
      createCanvas(200, 200);
      console.log("program started");               
  
      characterLibrary.addColumn("index");
      characterLibrary.addColumn("character");
  }
  
  function draw() {
      console.log("started generateSpots");
  
      for(var i = 0; i < input.length; i++) {
          asciiIndex = i+32;
          characterLibrary.addRow();
          characterLibrary.setNum(i, "index", asciiIndex);
          characterLibrary.setString(i, "character", input[i]);
      }
  
      console.log(millis());
      saveTable(characterLibrary, "characterLibrary.csv");
      noLoop();
  }
  

  Still not entirely sure what I was doing wrong, think it was something to do with the vars I was using. If someone wants to tell me that would be great. It would be good to know so I can avoid it in the future.
• blindfish

  November 2017 Answer ✓

  You're making things complicated for yourself by starting your loop variable at 32. This looks like the cause of the error:

  characterLibrary.setString(rowIndex, "character", input[i]);

  ...where the i referenced in input[i] starts at 32 - which will take you out of your input array bounds.

  This works for me:

  var characterLibrary;
  var input;
  
  function setup() {
      createCanvas(1, 1);
      input = [" ", "!", "\"", "#", "$", "%", "&", "\'", "(", ")", "*", "+", ",", "-", ".", "/", "0", "1", "2", "3", "4", "5", "6", "7", "8", "9", ":", ";", "<", "=", ">", "?", "@", "A", "B", "C", "D", "E", "F", "G", "H", "I", "J", "K", "L", "M", "N", "O", "P", "Q", "R", "S", "T", "U", "V", "W", "X", "Y", "Z", "[", "\\", "]", "^", "_", "`", "a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z", "{", "|", "}", "~"];
  
      characterLibrary = new p5.Table();
      characterLibrary.addColumn("index");
      characterLibrary.addColumn("character");
  
      for(var i = 0; i < input.length; i++) {
          characterLibrary.addRow();
          // if you want index to start at 32 
          // it's far simpler to just add 32 here...
          characterLibrary.setNum(i, "index", i+32);
          characterLibrary.setString(i, "character", input[i]);
      }
  
      // saveTable(characterLibrary, "characterLibrary.csv");
      // if you log the table you can inspect it directly in the browser console ;)
      console.log(characterLibrary);
  }
  

  Note you can also declare multiple variables inside a for loop. i and j are just conventions, but you can give them meaningful names...

  for(var i=0, charIndex=32; i<input.length; i++, charIndex++) {
    // use i for array index and charIndex where needed
  }
  
• httpwilfred

  November 2017

  That makes sense. Thanks for the help I appreciate it! I was really making it harder than it had to be.
• koogs

  November 2017

  Why do you have an array of strings(!) of characters? What's up with fromCharCode()?

  https://www.w3schools.com/jsref/jsref_fromCharCode.asp
• httpwilfred

  November 2017

  I'm new to javascript. I didn't know that existed. Thanks for the heads up. That will make it a lot easier.