zlodiak

как думаете, если вместо такого:

var volume = bla bla,
    result;

if(APP.helper.isNumCheck(volume)) { 
  result = volume;
} else {
  result = undefined;
};  

this.model.set({shippOptionsVolume: result});  

писать такое:

var volume = bla bla;

if(APP.helper.isNumCheck(volume)) { 
  this.model.set({shippOptionsVolume: volume});
} else {
  this.model.set({shippOptionsVolume: undefined});
}; 

,то коллеги обзывать индусом не начнут?

какой вариант вы использовали бы?

помогите пожалуйста решить проблему

вот заготовка для игры крестики-нолики на backbone:

Edit fiddle - JSFiddle [jsfiddle.net]

zlodiak/ttt_1 [github.com]

пока она умеет после клика по любой клетке выводить в консоль координаты клетки. каждой клетке соответствует своя модель. для вывода в консоль координаты берутся из атрибутов элемента 'data-*'

проблема в том, что эти же самые координаты хранятся в модели каждой клетки. таким образом получается дублирование данных(в модели и в атрибутах элемента)

хотелось бы чтобы после каждого клика данные брались из модели, а не из атрибутов элемента

если кто-нибудь пользуется backbone + jasmine, то подскажите пожалуйста как можно протестировать такое простенькое представление:

APP.DepartCityView = Backbone.View.extend({   

  id: 'departCityWidget',

  template: _.template($('#departCityTpl').html()),

  render: function () {  
    this.$el.html(this.template());
    return this;
  } 

});

проблема в том, что я не понимаю что именно здесь можно протестировать. вот моя попытка:

describe("DepartCityView:", function() {
  beforeEach(function () {
    this.view = new APP.DepartCityView();
  });  

  it ('should init successful', function () {
    expect(this.view).toBeDefined();
  }); 

  it ("produces the correct HTML", function() {
    this.view.render();

    var id = this.view.$el.attr('id');
    expect(id).toEqual('departCityWidget');
  }); 
});

но даже здесь второй тест явно бесполезный. а хотелось бы протестировать в этом представлении вообще всё, что тестируется

вот шаблон:

<script type="text/template" id="departCityTpl">
  <div class="panel panel-default">
    <div class="panel-body row">
      <div class="col-xs-12">
        <h2>Город отправления</h2>
      </div>

      <div class="widget_content form-group col-xs-12">
        <input type="text" class="form-control" id="fldDepartCity">
        <div id="errMsg_departCity" class="help-block"></div>
      </div>
    </div>
  </div>
</script>

можете смеяться, но у меня такой параноидальный вопрос. допустим есть метод:

  errMsgManage: function(msgArr, elem) { 
    var msgCont = this.$el.find(elem);

    msgCont.empty();

    for(key in msgArr) { msgCont.append(msgArr[key]) };
  } 

здесь один элемент ищется более одного раза. поэтому логика говорит, что его нужно сохранить в переменную msgCont и затем использовать хоть 2 раза, хоть 200 раз

с другой стороны, код будет более удобно читаться, если в этом случае не придерживаться описанного выше правила:

errMsgManage: function(msgArr, elem) { 
  this.$el.find(elem).empty();
  for(key in msgArr) { this.$el.find(elem).append(msgArr[key]) };
} 

ясно, что от поиска в dom одного лишнего элемента процессор не расплавится

как бы вы записали этот метод?

ребята, скажите как дальше жить?

я, следуя совету автора популярной книжки "совершенный код", даю имена переменным очень конкретные. например такие:

this.cityOfDestinationView = new APP.CityOfDestinationView({
  model: this.cityOfDestinationModel
});

но в результате у меня получается какой-то адский, неудобночитаемый код. вот небольшой пример:
Edit fiddle - JSFiddle [jsfiddle.net]
zlodiak/shippingCalculator1 [github.com]

как думаете, это нормально? как бы вы посоветовали давать имена переменным?

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

[jsfiddle.net]
zlodiak/ticTacToe_2 [github.com]

на code review столько много букв не примут, а обзор хотелось бы услышать..

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

на js сделал игру крестики нолики.
fiddle: Edit fiddle - JSFiddle [jsfiddle.net]
github: zlodiak/ticTacToe [github.com]

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

а надобы чтобы после каждого клика выводилось только одно сообщение

чуваки, оцените следующую инкарнацию этой замечательной игры:
Afrika tanks [prozaik.16mb.com]
zlodiak/afrika_tanks_2 [github.com]

скажите пожалуйста что особенно плохо в коде, что можно доработать. в общем обзор кода прошу

чуваки, а вот так лучше стало?
только что-то у меня не получилось константу DIRECTION совсем наверх вынести и теперь каждый экземпляр имеет свою константу. так допустимо делать?

Tank = function(id){ 
  this.x_coord = helper.randomIntFromZero(481);
  this.y_coord = helper.randomIntFromZero(481);
  this.id = id; 
  this.arrow; 
  this.direction;
  this.DIRECTION = ['up', 'right', 'bottom', 'left'];

  this.Create();
}

Tank.prototype.Create = function(){
  this.direction = this.DIRECTION[helper.randomIntFromZero(4)];

  var tank = $('<div class="tank" id="' + this.id + '"></div>').css({
    left: this.x_coord + 'px',
    top: this.y_coord + 'px'
  });

  $('#board').append(tank);
}

Tank.prototype.checkArrowDirection = function(){
  switch (this.direction) {
   case 'up':
      this.arrow = '▲';
      break
   case 'right':
      this.arrow = '►';
      break
   case 'bottom':
      this.arrow = '▼';
      break
   case 'left':
      this.arrow = '◄';
      break         
   default:
      console.log('error arrow direction');
      break
  }
}

Tank.prototype.checkBorderCollision = function(){
  switch (this.direction) {
   case 'up':
      this.y_coord -= 10;
      if(this.y_coord <= 0) this.y_coord = 0; 
      break
   case 'right':
      this.x_coord += 10;
      if(this.x_coord >= 480) this.x_coord = 480;
      break
   case 'bottom':
      this.y_coord += 10;
      if(this.y_coord >= 480) this.y_coord = 480;
      break
   case 'left':
      this.x_coord -= 10;
      if(this.x_coord <= 0) this.x_coord = 0;
      break         
   default:
      console.log('error direction definition');
      break
  }
}

Tank.prototype.checkChangeDirection = function(){
  if(helper.randomIntFromZero(100) > 75){
    this.direction = this.DIRECTION[helper.randomIntFromZero(4)];
  }
}

Tank.prototype.Offset = function(){
  $('#' + this.id).css({
    left: this.x_coord + 'px',
    top: this.y_coord + 'px'
  }).html(this.arrow);
}

Tank.prototype.Move = function(){
  this.checkChangeDirection();
  this.checkBorderCollision();
  this.checkArrowDirection();
  this.Offset();
}

ребята, помогтие пожалуйста исправить конструктор для танков Tank, если это вообще возможно

песочница с полным неработающим скриптом здесь: yeaaNj [codepen.io]
проблема появилась после того как я внёс изменения в этот конструктор:

    
Tank = function(id){ 
  var self = this, 
      DIRECTION = ['up', 'right', 'bottom', 'left'];

  this.x_coord = helper.randomIntFromZero(481);
  this.y_coord = helper.randomIntFromZero(481);
  this.id = id; 
  this.arrow; 
  this.direction;
}

Tank.prototype.Create = function(){
  self.direction = DIRECTION[helper.randomIntFromZero(4)];

  var tank = $('<div class="tank" id="' + self.id + '"></div>').css({
    left: self.x_coord + 'px',
    top: self.y_coord + 'px'
  });

  $('#board').append(tank);
}

Tank.prototype.checkArrowDirection = function(){
  switch (self.direction) {
   case 'up':
      self.arrow = '▲';
      break
   case 'right':
      self.arrow = '►';
      break
   case 'bottom':
      self.arrow = '▼';
      break
   case 'left':
      self.arrow = '◄';
      break         
   default:
      console.log('error arrow direction');
      break
  }
}

Tank.prototype.checkBorderCollision = function(){
  switch (self.direction) {
   case 'up':
      self.y_coord -= 10;
      if(self.y_coord <= 0) self.y_coord = 0; 
      break
   case 'right':
      self.x_coord += 10;
      if(self.x_coord >= 480) self.x_coord = 480;
      break
   case 'bottom':
      self.y_coord += 10;
      if(self.y_coord >= 480) self.y_coord = 480;
      break
   case 'left':
      self.x_coord -= 10;
      if(self.x_coord <= 0) self.x_coord = 0;
      break         
   default:
      console.log('error direction definition');
      break
  }
}

Tank.prototype.checkChangeDirection = function(){
  if(helper.randomIntFromZero(100) > 75){
    self.direction = DIRECTION[helper.randomIntFromZero(4)];
  }
}

Tank.prototype.Offset = function(){
  $('#' + self.id).css({
    left: self.x_coord + 'px',
    top: self.y_coord + 'px'
  }).html(self.arrow);
}

Tank.prototype.Move = function(){
  this.checkChangeDirection();
  this.checkBorderCollision();
  this.checkArrowDirection();
  this.Offset();
}

ps:
как этот скрипт в результате должен работать можно посмотреть здесь: ZQpzXQ [codepen.io]
но здесь не используются прототипы и поэтому код неудобно читать

выбор выводить метры или сантиметры нужно производить либо до вызова метода контроллера (в роутере) либо в шаблоне вьюхи

@master, то есть в шаблон добавить логику?..

вот спасибо, пинки очень полезные

ещё меня терзают смутные сомнения по поводу метода output() в контроллере. как думаете, если вместо него написать два метода таких:

output_height_in_m(),

output_height_in_cm()

, то это не нарушит фен шуй MVC? или правильнее оставить как есть?

ребята, не посылайте только. со мной в последнее время случился паттерн головного мозга и теперь я пишу простенькие никому не нужные примеры. вот один из них. дайте пожалуйста обзор кода, всё ли зесь верно с точки зрения MVC? что можно улучшить и исправить?

class RunnerModel(object):
    def __init__(self, name, height):
        ''' default height in mm '''
        self.name = name
        self.height = height


class RunnerView(object):
    def __init__(self, model):
        self.model = model

    def output_height_in_m(self):
        print('runner name: ' + str(self.model.name))
        print('runner height(m):' + str(self.model.height/1000))

    def output_height_in_cm(self):
        print('runner name: ' + str(self.model.name))
        print('runner height(cm):' + str(self.model.height/10))        


class RunnerController(object):
    def __init__(self, model, measure):
        self.model = model
        self.view = RunnerView(self.model)
        self.measure = measure

    def output(self):
        if self.measure == 'm':
            self.view.output_height_in_m()
        elif self.measure == 'cm':
            self.view.output_height_in_cm()


if __name__ == "__main__":
    runner_1  = RunnerModel(name='sergey', height=1820)
    controller_1 = RunnerController(runner_1, 'cm')
    controller_1.output()
    controller_1 = RunnerController(runner_1, 'm')
    controller_1.output()    

после запуска скрипта в консоль выводится следующее:100:

(python3_env)kalinin@kalinin ~/python3/python3_files/mvc_1 $ python index.py
runner name: sergey
runner height(cm):182.0
runner name: sergey
runner height(m):1.82