Как можно укоротить код?

Question

[ostrooovskyy]

Здравствуйте, друзья разработчики. Хотел бы укоротить данный код.

public async getRegDate(
        id: number,
        config?: Omit<AxiosRequestConfig, "method" | "url">
    ): Promise<Date> {
        const data = (
            await axios({
                url: `https://vk.com/foaf.php?id=${id}`,
                method: "GET",
                ...config,
            })
        ).data as string;
        const firstIndex = data.indexOf("<ya:modified dc:date=") + 23;
        const secondIndex = data.indexOf('"/>', firstIndex);
        return new Date(data.substring(firstIndex, secondIndex));
    }
}

Заранее всем спасибо!

Comments

[Константин Б.]

Зачем?) ну вот так на 2 строки меньше получится

const url = `https://vk.com/foaf.php?id=${id}`;
const { data } = await axios({ url, method: "GET",  ...config });

[ostrooovskyy]

Константин Б., Возможно так и будет лучше.

[Lynn «Кофеман»]

Оффтопик: надёжнее сначала спред-оператор, а потом урл и метод, чтобы точно не прилетело что-то не то из конфига

[Aetae]

Как вариант - не теребить строки, а работать с докуметом:

public async getRegDate(
        id: number,
        config?: Omit<AxiosRequestConfig, 'method' | 'url' | 'responseType'>
    ): Promise<Date> {
        var {data} = await axios.get(`https://vk.com/foaf.php?id=${id}`, {
            ...config,
            responseType: 'document'
        });
        return new Date(data.querySelector('ya\\:modified').getAttribute('dc:date'));
    }

[ostrooovskyy]

Aetae, попробую так, тоже спасибо за такое решение.

[WbICHA]

Lynn «Кофеман», там же тайпскрипт, так что если специально не ломать, всё ок будет.

[Lynn «Кофеман»]

WbICHA, Omit же не запрещает поле, а просто убирает его из интерфейса. А typescript не запрещает лишние поля, он их просто игнорирует.

Т.е. литерал с url написать не выйдет, а вот прилететь через какой-нибудь spread может любой треш

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

[WbICHA]

Lynn «Кофеман», вообще да, не подумал, согласен.

Answer 1

[Кирилл Романов]

const { data } = await axios.get<string>(`https://vk.com/foaf.php?id=${id}`, config);