2 回答
TA贡献1811条经验 获得超6个赞
我也有Java背景,但年龄较大。;-)
一些建议:
选择或,但您不应同时使用两者。在下面的示例中,我使用
undefined
null
undefined
;在模块中,静态成员可以用简单的变量替换;
优先于多个可选参数;
interface
您的IIFE()是无用的;
(() => { … }();
当推理就足够了时,不要声明类型;
首选基元类型 ( 而不是 )。
string
String
下面是一个示例:
const NAMES = ['Spring', 'Summer', 'Fall'];
const OFFSET = 2002;
const numberMap = new Map<number, Semester>();
const labelMap = new Map<string, Semester>();
export interface SemesterOptions {
semesterNumber?: number
label?: string
}
export class Semester {
private semesterNumber: number;
private label: string;
constructor({ semesterNumber, label }: SemesterOptions) {
if (semesterNumber === undefined) {
if (label === undefined) {
throw new Error('invalid options');
}
const parsed = parseLabelToSemesterOptions(label);
semesterNumber = parsed.semesterNumber;
label = parsed.label;
} else if (label === undefined) {
if (semesterNumber < 1) {
throw new Error('Semester number cannot be less than 1');
}
label = NAMES[semesterNumber % NAMES.length] + ' ' + (semesterNumber / 3 + OFFSET);
}
this.semesterNumber = semesterNumber;
this.label = label;
}
static ofNumber(semesterNumber: number): Semester {
let semester = numberMap.get(semesterNumber);
if (!semester) {
semester = new Semester({ semesterNumber });
numberMap.set(semester.semesterNumber, semester);
if (!labelMap.has(semester.label)) {
labelMap.set(semester.label, semester);
}
}
return semester;
}
static ofLabel(label: string): Semester {
let semester = labelMap.get(label);
if (!semester) {
semester = new Semester({ label });
labelMap.set(semester.label, semester);
if (!numberMap.has(semester.semesterNumber)) {
numberMap.set(semester.semesterNumber, semester);
}
}
return semester;
}
}
function parseLabelToSemesterOptions(labelToParse: string): Required<SemesterOptions> {
const tokens = labelToParse.split('\\ ');
if (tokens.length !== 2) {
throw new Error('Semester label has incorrect number of tokens');
}
const label = tokens[0];
const year = parseInt(tokens[1], 10);
if (year < OFFSET) {
throw new Error('Year cannot be earlier than ' + OFFSET);
}
const nameIndex = NAMES.indexOf(label);
if (nameIndex === -1) {
throw new Error(`Name of the semester must be one of ${NAMES.join(', ')}`);
}
const semesterNumber = (year - OFFSET) * 3 + nameIndex;
return {
semesterNumber,
label
}
}
对于 :使用获取并测试结果是否为(或错误)。有一个带有测试的快捷方式:getOrDefaultundefinedfalsy
const val = myMap.get(myKey) || myDefautValue;
...但在你的情况下,声明更合适。if
对于 :,首先使用测试密钥是否已存在,然后在必要时使用 set。putIfAbsent
TA贡献1802条经验 获得超4个赞
关于构造函数
你猜对了,没有像你在Java中习惯的那样有多个构造函数(又名构造函数重载)。TypeScript 中只有一个构造函数,并且需要有可选参数。
在您的情况下,学期编号和标签可以是未定义的或数字/字符串,无需进行类型检查。
检查“传入的内容”的方法不需要在一个if语句中进行如此多的检查。在这里看看下面的答案:https://stackoverflow.com/a/44017547/8745384
关于地图
“putIfAb”的等价物是一个简单的“设置(键,值)”
Semester.semesterMap.set(semester.semesterNumber, semester);
对于获取或默认设置,您可以编写
const semester: Semester = Semester.semesterMap.get(semesterNumber) || new Semester(semesterNumber);
它的作用与我之前对if语句的描述相同。检查空值或未定义或空(如果是这种情况)执行新学期(...)。
希望我能帮忙。我最近也从 Java 切换到了类型脚本,所以也许有一种比我描述的更好的方法。
添加回答
举报