split関数が機能しない理由があります

Question

salesDataを分割しようとしていますが、各オブジェクトの後にコンマでsplit()が必要です。

これは宿題です。オブジェクトマップに保存して計算しています。 今のところ、splitの機能が失敗することはありません。

私は、最も収益性の高い部門が何であるかを最終結果として示すために以下のコードを提供しました。

var salesData = [ 
{department : 'hardware', sales : 4500, day : 'Monday'}, 
{department : 'outdoor', sales : 1500, day : 'Monday'}, 
{department : 'carpentry', sales : 5500, day : 'Monday'}, 
{department : 'hardware', sales : 7500, day : 'Tuesday'}, 
{department : 'outdoor', sales : 2505, day : 'Tuesday'}, 
{department : 'carpentry', sales : 1540, day : 'Tuesday'}, 
{department : 'hardware', sales : 1500, day : 'Wednesday'}, 
{department : 'outdoor', sales : 8507, day : 'Wednesday'}, 
{department : 'carpentry', sales : 8009, day : 'Wednesday'}, 
{department : 'hardware', sales : 12000, day : 'Thursday'}, 
{department : 'outdoor', sales : 18007, day : 'Thursday'}, 
{department : 'carpentry', sales : 6109, day : 'Thursday'}, 
{department : 'hardware', sales : 7005, day : 'Friday'}, 
{department : 'outdoor', sales : 12006, day : 'Friday'}, 
{department : 'carpentry', sales : 16109, day : 'Friday'}, 
]; 

var departmentMap = {}; 

function mostProfitableDepartment() { 
var departmentSplit = salesData.split(','); 

    for (var i = 0; i < departmentSplit.length; i++) { 
    var currentDep = departmentSplit[i] 
    if (departmentMap[currentDep] === undefined) { 
    departmentMap[currentDep] = 0; 
    } 
    departmentMap[currentDep] = departmentMap[currentDep]++ 
} 
console.log(departmentMap); 
}mostProfitableDepartment(salesData); 

Answer 1

あなたは、配列のためのあなたの反復構造を使用することができArray.prototype.reduce()

var salesData = [ 
 
{department : 'hardware', sales : 4500, day : 'Monday'}, 
 
{department : 'outdoor', sales : 1500, day : 'Monday'}, 
 
{department : 'carpentry', sales : 5500, day : 'Monday'}, 
 
{department : 'hardware', sales : 7500, day : 'Tuesday'}, 
 
{department : 'outdoor', sales : 2505, day : 'Tuesday'}, 
 
{department : 'carpentry', sales : 1540, day : 'Tuesday'}, 
 
{department : 'hardware', sales : 1500, day : 'Wednesday'}, 
 
{department : 'outdoor', sales : 8507, day : 'Wednesday'}, 
 
{department : 'carpentry', sales : 8009, day : 'Wednesday'}, 
 
{department : 'hardware', sales : 12000, day : 'Thursday'}, 
 
{department : 'outdoor', sales : 18007, day : 'Thursday'}, 
 
{department : 'carpentry', sales : 6109, day : 'Thursday'}, 
 
{department : 'hardware', sales : 7005, day : 'Friday'}, 
 
{department : 'outdoor', sales : 12006, day : 'Friday'}, 
 
{department : 'carpentry', sales : 16109, day : 'Friday'}, 
 
], 
 

 
     deptMap = salesData.reduce((p,c) => p[c.department] ? (p[c.department].push({sales: c.sales, 
 
     \t                    day: c.day}),p) 
 
     \t             : (p[c.department] = [{sales: c.sales, 
 
     \t                   day: c.day}],p) ,{}); 
 
console.log(deptMap);

Answer 2

を利用することにより、次のようにあなたがするかもしれない、あなたが既に持っています。次に、部門のプロパティを実際の部門とし、売上をカウントします。

var salesData = [{ department: 'hardware', sales: 4500, day: 'Monday' }, { department: 'outdoor', sales: 1500, day: 'Monday' }, { department: 'carpentry', sales: 5500, day: 'Monday' }, { department: 'hardware', sales: 7500, day: 'Tuesday' }, { department: 'outdoor', sales: 2505, day: 'Tuesday' }, { department: 'carpentry', sales: 1540, day: 'Tuesday' }, { department: 'hardware', sales: 1500, day: 'Wednesday' }, { department: 'outdoor', sales: 8507, day: 'Wednesday' }, { department: 'carpentry', sales: 8009, day: 'Wednesday' }, { department: 'hardware', sales: 12000, day: 'Thursday' }, { department: 'outdoor', sales: 18007, day: 'Thursday' }, { department: 'carpentry', sales: 6109, day: 'Thursday' }, { department: 'hardware', sales: 7005, day: 'Friday' }, { department: 'outdoor', sales: 12006, day: 'Friday' }, { department: 'carpentry', sales: 16109, day: 'Friday' }], 
 
    departmentMap = {}; 
 

 
function mostProfitableDepartment() { 
 
    var currentDep, i; 
 
    for (i = 0; i < salesData.length; i++) { 
 
     currentDep = salesData[i].department; 
 
     if (!departmentMap[currentDep]) { 
 
      departmentMap[currentDep] = 0; 
 
     } 
 
     departmentMap[currentDep] += salesData[i].sales; 
 
    } 
 
} 
 

 
mostProfitableDepartment(salesData); 
 

 
console.log(departmentMap);

Array#forEachと、より簡潔なスタイルで、上記のようにSAM。

var salesData = [{ department: 'hardware', sales: 4500, day: 'Monday' }, { department: 'outdoor', sales: 1500, day: 'Monday' }, { department: 'carpentry', sales: 5500, day: 'Monday' }, { department: 'hardware', sales: 7500, day: 'Tuesday' }, { department: 'outdoor', sales: 2505, day: 'Tuesday' }, { department: 'carpentry', sales: 1540, day: 'Tuesday' }, { department: 'hardware', sales: 1500, day: 'Wednesday' }, { department: 'outdoor', sales: 8507, day: 'Wednesday' }, { department: 'carpentry', sales: 8009, day: 'Wednesday' }, { department: 'hardware', sales: 12000, day: 'Thursday' }, { department: 'outdoor', sales: 18007, day: 'Thursday' }, { department: 'carpentry', sales: 6109, day: 'Thursday' }, { department: 'hardware', sales: 7005, day: 'Friday' }, { department: 'outdoor', sales: 12006, day: 'Friday' }, { department: 'carpentry', sales: 16109, day: 'Friday' }], 
 
    departmentMap = {}; 
 

 
function mostProfitableDepartment() { 
 
    salesData.forEach(function (item) { 
 
     departmentMap[item.department] = (departmentMap[item.department] || 0) + item.sales; 
 
    }); 
 
} 
 

 
mostProfitableDepartment(salesData); 
 

 
console.log(departmentMap);

Answer 3

私はあなたがJSONとしてSALESDATAを検討し、その値