глобальные переменные объекта

[cyklop77]

помогите пожалуйста разобраться с одним важным для меня моментом

 

допустим, у меня есть объект с набором свойств и методов. при инициализации объекта метод method1 должен вывести в консоль свойство this.prop1. я могу поступить двумя способами:

 

1.

var Game = function() {

  this.prop1 = 1111;

  this.prop2 = 2222;


  this.method1(); 

};


Game.prototype = {

  method1: function() {   

    console.log(this.prop1);        

  }

};

2.

var Game = function() {

  this.prop1 = 1111;

  this.prop2 = 2222;


  this.method1(this.prop1); 

};


Game.prototype = {

  method1: function(prop1) {   

    console.log(prop1);        

  }

};

как видите, первый способ допускает использование 'глобальных'(в пределах объекта) переменных, второй - нет

 

какой из них использовать будет правильнее? и главное почему? 

 

я не могу определиться, у каждого способа есть свои плюсы и минусы

[wwt]

Зависит от задач имхо

[Николя223]

Ну ты же делаешь API и не расчитываешь, что твое API будет взаимодействовать с чем то другим, ведь так? тогда логично предположить, что уместно Первое, но и второе тоже правильно... 
Я использую первый способ (Правда не в JS))) Когда работаю с большими объемными массивами. очень удобно хранить все под рукой.
Но естественно и второй способ использую часто. Только возникает вопрос, зачем ты так объявляешь переменные, если не будешь ими пользоваться ))) всё зависит от задач =)  

[cyklop77]

да как же не буду пользоваться? ещё как буду)

[cyklop77]

Я использую первый способ (Правда не в JS))) Когда работаю с большими объемными массивами. очень удобно хранить все под рукой.

Но естественно и второй способ использую часто.   

 

а как ты решаешь какой из этих способов использовать в определённой ситуации?

[Николя223]

Смотри пример
 

class myClass {   static $arr = array();   function __construct(){     global $db;     $sql = $db->query("SELECT * FROM table");     foreach( $sql->rows AS $row){        $id = $row['id'];        self::$arr[$id] = $row;     }   }  public  function any_method(){     $myarr = self::$arr;     print_r($myarr);   }} 

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

Да, конечно, я мог и передать эту переменную в функцию, но мне все равно данный массив ещё не раз потребуется, поэтому я сохраняю его так. 
И что получается, а получается, что метод any_method? будет заточен только на эту одну переменную.  (но можно же и сделать дополнительную переменную под неё, но считаю что это не очень красиво, всё дублировать). 
Если у меня такие задачи, то я с вероятностью 98% сделаю так. мне так удобнее. Но если же, у меня данная функция будет работать с независимыми переменными,  а не только с одной глобальной, то естественно лучше передавать параметром