2015-08-16 53 views
0

所以,我有这个函数可以获取JSON对象,但我想使它更简单,所以我创建了一个函数来获取JSON对象的值。为什么它不起作用?JavaScript将字符串转换为json对象

var itemData = { 
 
    weapon: function() { 
 
     return { 
 
      1: { 
 
       'name': 'Dagger', 
 
        'extra_skill': 'none', 
 
        'cost': 500, 
 
        'attack': 5 
 
      }, 
 
      2: { 
 
       'name': 'Pickaxe', 
 
        'extra_skill': 'mining', 
 
        'cost': 25, 
 
        'attack': 5 
 
      } 
 
     } 
 
    }, 
 
    getWeapon: function (value, x) { 
 
     var obj = JSON.parse(value); 
 
     return itemData.weapon()[x].obj 
 
    } 
 
} 
 

 
// outputs: Dagger 
 
console.log(itemData.weapon()[1].name) 
 

 
// Get the name of weapon 1 
 
// however, it outputs: Uncaught SyntaxError: Unexpected token a 
 
console.log('Getting weapon... ' + itemData.getWeapon('name', 1))

我在做什么错?

+5

JSON是一个**文本注释**。如果您正在编写JavaScript代码,那么除非您要处理字符串,否则不会处理JSON。在你的问题中根本没有JSON;因此,不需要'JSON.parse'(事实上,这是问题的一部分)。 –

回答

5

实际上,您根本不需要JSON解析就可以正常工作,因为没有任何地方需要解析JSON字符串。

这里是一个工作示例:

var itemData = { 
 
    weapon: function() { 
 
     return [ 
 
      { 
 
       'name': 'Dagger', 
 
        'extra_skill': 'none', 
 
        'cost': 500, 
 
        'attack': 5 
 
      }, 
 
      { 
 
       'name': 'Pickaxe', 
 
        'extra_skill': 'mining', 
 
        'cost': 25, 
 
        'attack': 5 
 
      } 
 
     ]; 
 
    }, 
 
    getWeapon: function (value, x) { 
 
     return itemData.weapon()[x][value]; 
 
    } 
 
} 
 

 
// outputs: Dagger 
 
console.log(itemData.weapon()[0].name) 
 

 
// outputs: Getting weapon... Pickaxe 
 
console.log('Getting weapon... ' + itemData.getWeapon('name', 1))

+0

我喜欢这个答案在getWeapon中使用'value'的方式,因为'getWeapon'的意图似乎总是返回'name'。 –

+1

我明白我现在做错了什么。谢谢。 –

+0

对于每个类都有一个责任的类/函数是很好的,所以我建议O.P.澄清'getWeapon'应该返回的具体内容。如果计划总是返回'name',那么'name'应该在'getWeapon'中进行硬编码。这可以避免在传入'value'时无意中造成拼写错误。 –

相关问题