dousu5608 2014-12-03 22:06
浏览 41
已采纳

golang:如何在循环中填充多结构图?

I have log files of customer interactions with an API. I want to parse those logs and feed the results into a map of structs so that I can organize the data into helpful information. For example, I would like to respond to the following query: "show me the total number of requests per user per day".

I have created what seems like an adequate structure to hold the data. However, when I try to run the program I get the error: invalid operation: dates[fields[1]] (type *Dates does not support indexing) [process exited with non-zero status].

http://play.golang.org/p/8u3jX26ktt

package main

import (
    "fmt"
    "strings"
)

type Stats struct {
    totalNumberOfRequests int
}
type Customer struct {
    listOfCustomers map[string]Stats // map[customerid]Stats
}
type Dates struct {
    listOfDates map[string]Customer // map[date]Customer
}

var requestLog = []string{
    "2011-10-05, 1234, apiquery",
    "2011-10-06, 1234, apiquery",
    "2011-10-06, 5678, apiquery",
    "2011-10-09, 1234, apiquery",
    "2011-10-12, 1234, apiquery",
    "2011-10-13, 1234, apiquery",
}

func main() {
    dates := new(Dates)
    for _, entry := range requestLog {
        fmt.Println("entry:", entry)
        fields := strings.Split(entry, "'")
        dates.listOfDates[fields[0]].listOfCustomers[fields[1]].totalNumberOfRequests++
    }
}

Is there a better structure to use? Or is there a way to make this structure work for this particular purpose?

  • 写回答

1条回答 默认 最新

  • doufan6544 2014-12-04 12:44
    关注

    If I understood your expectations about the output here's a solution. However I don't like that "Customer is a map with id and Stat.. I think it should be a simpler struct with two Fields (cid string and stat Stats). Also the Dates structure does not allow for multiple customers on the same date, so I've changed to map single date to list of users.

    I've also added more "tests scenarios" to cover the cases for a customer accessing the resource multiple times on the same date.

    You don't seem to use the "apiquery" your example, so the code below does not match against it.

    Regarding the change to pointers in the structs - see this issue (as noted in the comments to your question)

    package main
    
    import (
        "fmt"
        "strings"
    )
    
    type Stats struct {
        totalNumberOfRequests int
    }
    type Customer struct {
        customerWithStat map[string]*Stats // a customer with it's corresponding stats
    }
    
    type Dates struct {
        listOfDates map[string][]*Customer // map[date]list of customers (for each date)
    }
    
    var requestLog = []string{
        "2011-10-05, 1234, apiquery",
        "2011-10-06, 5678, apiquery",
        "2011-10-06, 1234, apiquery",
        "2011-10-06, 1234, apiquery",
        "2011-10-06, 5678, apiquery",
        "2011-10-06, 1234, apiquery",
        "2011-10-09, 1234, apiquery",
        "2011-10-12, 1234, apiquery",
        "2011-10-13, 1234, apiquery",
        "2011-10-13, 1234, apiquery",
        "2011-10-06, 1234, apiquery",
    }
    
    func main() {
        listOfDates := make(map[string][]*Customer)
        dates := Dates{listOfDates}
        for _, entry := range requestLog {
            fields := strings.Split(entry, ",")
            curDateStr := strings.TrimSpace(fields[0])
            curCustIdStr := strings.TrimSpace(fields[1])
    
            if customersAtDate, dateExists := dates.listOfDates[curDateStr]; dateExists {
                // Date already exist
                for _, curCustomer := range customersAtDate {
                    if curStat, customerExists := curCustomer.customerWithStat[curCustIdStr]; customerExists {
                        // The user has already accessed this resource - just increment
                        curStat.totalNumberOfRequests++
                    } else {
                        // New user - set access to 1
                        curCustomer.customerWithStat[curCustIdStr] = &Stats{1}
                    }
                }
            } else {
                // New Date
    
                // Init the Statistic for the new customer
                newCustomerData := make(map[string]*Stats)
                newCustomerData[curCustIdStr] = &Stats{1}
    
                // Create the customer itself
                newCustomer := &Customer{newCustomerData}
    
                // add to the current day list
                dates.listOfDates[curDateStr] = append(dates.listOfDates[curDateStr], newCustomer)
    
            }
        }
    
        // Print result
        for date, customers := range dates.listOfDates {
            fmt.Println("Date: ", date)
            for _, customer := range customers {
                for cid, stat := range customer.customerWithStat {
                    fmt.Println("  Customer: ", cid)
                    fmt.Println("  # Requests: ", *stat)
                }
            }
        }
    }
    

    This will output:

    Date:  2011-10-05
      Customer:  1234
      # Requests:  {1}
    Date:  2011-10-06
      Customer:  5678
      # Requests:  {2}
      Customer:  1234
      # Requests:  {4}
    Date:  2011-10-09
      Customer:  1234
      # Requests:  {1}
    Date:  2011-10-12
      Customer:  1234
      # Requests:  {1}
    Date:  2011-10-13
      Customer:  1234
      # Requests:  {2}
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 identifier of an instance of 类 was altered from xx to xx错误
  • ¥100 反编译微信小游戏求指导
  • ¥15 docker模式webrtc-streamer 无法播放公网rtsp
  • ¥15 学不会递归,理解不了汉诺塔参数变化
  • ¥30 软件自定义无线电该怎样使用
  • ¥15 R语言mediation包做中介分析,直接效应和间接效应都很小,为什么?
  • ¥15 Jenkins+k8s部署slave节点offline
  • ¥15 如何实现从tello无人机上获取实时传输的视频流,然后将获取的视频通过yolov5进行检测
  • ¥15 WPF使用Canvas绘制矢量图问题
  • ¥15 用三极管设计一个单管共射放大电路