reorder get fileobj

Author: hshpy

drivers/alias/driver.go

@@ -66,14 +66,15 @@ func (d *Alias) Drop(ctx context.Context) error {
 	return nil
 }
 
+func (d *Alias) GetRoot(ctx context.Context) (model.Obj, error) {
+	return &model.Object{
+		Name:     "Root",
+		IsFolder: true,
+		Path:     "/",
+	}, nil
+}
+
 func (d *Alias) Get(ctx context.Context, path string) (model.Obj, error) {
-	if utils.PathEqual(path, "/") {
-		return &model.Object{
-			Name:     "Root",
-			IsFolder: true,
-			Path:     "/",
-		}, nil
-	}
 	root, sub := d.getRootAndPath(path)
 	dsts, ok := d.pathMap[root]
 	if !ok {

drivers/crypt/driver.go

@@ -186,14 +186,15 @@ func (d *Crypt) List(ctx context.Context, dir model.Obj, args model.ListArgs) ([
 	return result, nil
 }
 
+func (d *Crypt) GetRoot(ctx context.Context) (model.Obj, error) {
+	return &model.Object{
+		Name:     "Root",
+		IsFolder: true,
+		Path:     "/",
+	}, nil
+}
+
 func (d *Crypt) Get(ctx context.Context, path string) (model.Obj, error) {
-	if utils.PathEqual(path, "/") {
-		return &model.Object{
-			Name:     "Root",
-			IsFolder: true,
-			Path:     "/",
-		}, nil
-	}
 	remoteFullPath := ""
 	var remoteObj model.Obj
 	var err, err2 error

drivers/netease_music/driver.go

@@ -42,14 +42,14 @@ func (d *NeteaseMusic) Drop(ctx context.Context) error {
 	return nil
 }
 
-func (d *NeteaseMusic) Get(ctx context.Context, path string) (model.Obj, error) {
-	if path == "/" {
-		return &model.Object{
-			IsFolder: true,
-			Path:     path,
-		}, nil
-	}
+func (d *NeteaseMusic) GetRoot(ctx context.Context) (model.Obj, error) {
+	return &model.Object{
+		IsFolder: true,
+		Path:     "/",
+	}, nil
+}
 
+func (d *NeteaseMusic) Get(ctx context.Context, path string) (model.Obj, error) {
 	fragments := strings.Split(path, "/")
 	if len(fragments) > 1 {
 		fileName := fragments[1]

drivers/strm/driver.go

@@ -89,14 +89,15 @@ func (d *Strm) Drop(ctx context.Context) error {
 	return nil
 }
 
+func (d *Strm) GetRoot(ctx context.Context) (model.Obj, error) {
+	return &model.Object{
+		Name:     "Root",
+		IsFolder: true,
+		Path:     "/",
+	}, nil
+}
+
 func (d *Strm) Get(ctx context.Context, path string) (model.Obj, error) {
-	if utils.PathEqual(path, "/") {
-		return &model.Object{
-			Name:     "Root",
-			IsFolder: true,
-			Path:     "/",
-		}, nil
-	}
 	root, sub := d.getRootAndPath(path)
 	dsts, ok := d.pathMap[root]
 	if !ok {

internal/op/fs.go

@@ -177,14 +177,6 @@ func Get(ctx context.Context, storage driver.Driver, path string) (model.Obj, er
 	path = utils.FixAndCleanPath(path)
 	log.Debugf("op.Get %s", path)
 
-	// get the obj directly without list so that we can reduce the io
-	if g, ok := storage.(driver.Getter); ok {
-		obj, err := g.Get(ctx, path)
-		if err == nil {
-			return model.WrapObjName(obj), nil
-		}
-	}
-
 	// is root folder
 	if utils.PathEqual(path, "/") {
 		var rootObj model.Obj
@@ -227,7 +219,24 @@ func Get(ctx context.Context, storage driver.Driver, path string) (model.Obj, er
 
 	// not root folder
 	dir, name := stdpath.Split(path)
-	files, err := List(ctx, storage, dir, model.ListArgs{})
+	key := Key(storage, dir)
+	if files, ok := listCache.Get(key); ok {
+		log.Debugf("use cache when list %s", dir)
+		for _, f := range files {
+			if f.GetName() == name {
+				return f, nil
+			}
+		}
+	}
+	// get the obj directly without list so that we can reduce the io
+	if g, ok := storage.(driver.Getter); ok {
+		obj, err := g.Get(ctx, path)
+		if err == nil {
+			return model.WrapObjName(obj), nil
+		}
+	}
+
+	files, err := List(ctx, storage, dir, model.ListArgs{Refresh: true})
 	if err != nil {
 		return nil, errors.WithMessage(err, "failed get parent list")
 	}