Поместить его в метод правильно (TypeScript Newbie)

Question

Поместить его в метод правильно (TypeScript Newbie)

0

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

var item = new RoadMapExampleItem();
                                        item.calculatedCompleted = m.calculatedCompleted;
                                        item.estimatedCompleted = m.estimatedCompleted;
                                        item.epicId = e.id;
                                        item.epicText = e.name;
                                        item.initiativeId = i.id;
                                        item.initiativeText = i.name;
                                        item.mileStoneId = m.id;
                                        item.mileStoneText = m.name;
                                        item.epicProgress = e.progress;
                                        item.mileStoneProgress = m.progress;
                                        item.initiativeProgress = i.progress;
                                        item.versionId = v.id;
                                        item.versionText = v.versionText;
                                        result.push(item);
                                    });

это весь код

private buildRoadMapExampleItems(versions: Version[]): RoadMapExampleItem[] {
        var result = new Array<RoadMapExampleItem>();

        if (versions != null)
            versions.forEach(v => {
                if (v.mileStones != null)
                    v.mileStones.forEach(m => {
                        if (m.initiatives != null && m.initiatives.length > 0) {
                            m.initiatives.forEach(i => {
                                if (i.epics != null && i.epics.length > 0) {
                                    i.epics.forEach(e => {
                                        var item = new RoadMapExampleItem();
                                        item.calculatedCompleted = m.calculatedCompleted;
                                        item.estimatedCompleted = m.estimatedCompleted;
                                        item.epicId = e.id;
                                        item.epicText = e.name;
                                        item.initiativeId = i.id;
                                        item.initiativeText = i.name;
                                        item.mileStoneId = m.id;
                                        item.mileStoneText = m.name;
                                        item.epicProgress = e.progress;
                                        item.mileStoneProgress = m.progress;
                                        item.initiativeProgress = i.progress;
                                        item.versionId = v.id;
                                        item.versionText = v.versionText;
                                        result.push(item);
                                    });
                                }
                                else {
                                    var item = new RoadMapExampleItem();
                                    item.calculatedCompleted = m.calculatedCompleted;
                                    item.estimatedCompleted = m.estimatedCompleted;
                                    item.initiativeId = i.id;
                                    item.initiativeText = i.name;
                                    item.mileStoneId = m.id;
                                    item.mileStoneText = m.name;
                                    item.versionId = v.id;
                                    item.versionText = v.versionText;
                                    item.mileStoneProgress = m.progress;
                                    item.initiativeProgress = i.progress;
                                    result.push(item);
                                }
                            });
                        }

C.B. 29 июнь 2016, в 10:31

Источник

Теги:

javascript

angularjs

2 ответа

0

RoadMapExampleItem

class RoadMapExampleItem {
   constructor(public calculatedCompleted:number, public estimatedCompleted: number) //repeat all fields
}

Применение

 var item = new RoadMapExampleItem(m.calculatedCompleted, m.estimatedCompleted);

Piyush.kapoor 29 июнь 2016, в 06:16

Ещё вопросы

Dylan Meeus · Accepted Answer · 2016-06-29T06-36-00.000Z

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

 i.epics.forEach()

вы могли бы написать что-то вроде

 i.epics.forEach(e => result.push(this.createItem(e)));

И создайте второй метод в этом блоке метода, например

private createItem(e) : RoadMapExampleItem
{
             var item : RoadMapExampleItem = new RoadMapExampleItem();
                                    item.calculatedCompleted = m.calculatedCompleted;
                                    item.estimatedCompleted = m.estimatedCompleted;
                                    item.epicId = e.id;
                                    item.epicText = e.name;
                                    item.initiativeId = i.id;
                                    item.initiativeText = i.name;
                                    item.mileStoneId = m.id;
                                    item.mileStoneText = m.name;
                                    item.epicProgress = e.progress;
                                    item.mileStoneProgress = m.progress;
                                    item.initiativeProgress = i.progress;
                                    item.versionId = v.id;
                                    item.versionText = v.versionText;
 return item;
 }

Это будет способ отделить код.

Но если я могу дать немного больше информации, а не просто сказать, как отделить именно то, что вы просили. Возможно, это немного грязно, чтобы иметь "item.epicId = e.id, item.epicText = e.name,..". Это может быть хорошим местом, чтобы взглянуть на шаблон Builder. (Если бы вы только начали программировать, я бы не стал беспокоиться об этом, шаблоны проектирования - это то, что вы узнаете позже.

Кроме того, может быть разумно заметить, что это машинопись, поэтому вы можете использовать "типы". Когда вы создаете "элемент", вы можете использовать item: RoadMapExampleIem. И, наконец, вместо var вы можете использовать let в будущем.

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

    if (versions != null)
        versions.forEach(v => {
            if (v.mileStones != null)
                v.mileStones.forEach(m => {
                    if (m.initiatives != null && m.initiatives.length > 0) {
                        m.initiatives.forEach(i => {
                            if (i.epics != null && i.epics.length > 0) {

Но хорошо, что вы уже подумываете о методах! ;-)

Я любезно благодарю вас. Я работаю над этим прямо сейчас.
@CecilBoye Добро пожаловать. Если это решило вашу проблему, вы можете сообщить другим, приняв этот ответ в качестве решения ;-)
Я на полпути, моя единственная проблема сейчас в том, что он не может найти е в качестве параметра. он говорит [ts] Параметр 'e' неявно имеет тип 'any'. (параметр) e: любой. Должен ли я что-то импортировать?
Есть ли способ, которым вы могли бы поделиться текущим кодом у вас где-нибудь? (Pastebin или что-то), чтобы я мог посмотреть. Это не было бы импортом для этого.
Неважно, это сработало. Мне нужно было импортировать только пару вещей. Но ваш код работал на 95% для меня, так что спасибо.