refactor(computed, effect): optimize dirty level handling

Author: johnsoncodehk

src/computed.ts

@@ -27,13 +27,12 @@ export class Computed<T = any> implements IComputed {
 	) { }
 
 	get(): T {
-		const dirtyLevel = this.dirtyLevel;
+		let dirtyLevel = this.dirtyLevel;
 		if (dirtyLevel === DirtyLevels.MaybeDirty) {
 			Subscriber.resolveMaybeDirty(this);
-			if (this.dirtyLevel === DirtyLevels.Dirty) {
-				this.update();
-			}
-		} else if (dirtyLevel === DirtyLevels.Dirty || dirtyLevel === DirtyLevels.Released) {
+			dirtyLevel = this.dirtyLevel;
+		}
+		if (dirtyLevel >= DirtyLevels.Dirty) {
 			this.update();
 		}
 		const activeTrackId = System.activeTrackId;

src/effect.ts

@@ -38,17 +38,16 @@ export class Effect<T = any> implements IEffect {
 	}
 
 	notify() {
-		const dirtyLevel = this.dirtyLevel;
-		if (dirtyLevel === DirtyLevels.SideEffectsOnly) {
-			this.dirtyLevel = DirtyLevels.None;
-			Subscriber.runInnerEffects(this.deps);
-		} else {
+		let dirtyLevel = this.dirtyLevel;
+		if (dirtyLevel > DirtyLevels.None) {
 			if (dirtyLevel === DirtyLevels.MaybeDirty) {
 				Subscriber.resolveMaybeDirty(this);
+				dirtyLevel = this.dirtyLevel;
 			}
-			if (this.dirtyLevel === DirtyLevels.Dirty) {
+			if (dirtyLevel === DirtyLevels.Dirty) {
 				this.run();
 			} else {
+				this.dirtyLevel = DirtyLevels.None;
 				Subscriber.runInnerEffects(this.deps);
 			}
 		}