变量不返回实际值

本文关键字:返回 变量 | 更新日期: 2023-09-27 18:18:47

我想正确返回一些变量(数组)

kazkas.Ads[n]; (n = how many ads are)
kazkas.Ads[n].id;
kazkas.Ads[n].Days[m].Stats.Clicks;  // every day have his own clicks
kazkas.Ads[n].Days[m].Stats.Impresons; // every day have his own impresions

,并在其他类中使用这些变量

 public static void GetAdsStats(string Ticket, DateTime start, DateTime end, int CamId)
    {
        var client = new CampaignStatsServiceClient();
        var id = new CampaignIdFilter();
        id.CampaignId = CamId;
        var statsdata = new GetAdStatsData();
        var kazkas = new Campaign();

        kazkas = client.GetAdStats(Ticket, new GetAdStatsData
        {
            IdFilter = id,
            StartDate = start,
            EndDate = end
        });
        long AllClicks = 0;
        long AllImpresions = 0;
        int reklamos = kazkas.Ads.Length;
        long[] statistikaClikai = new long[reklamos];
        long[] statistikaImpresions = new long[reklamos];
        for (int i = 0; i < reklamos; i++)
        {
            int dienos = kazkas.Ads[i].Days.Length;
            for (int lop = 0; lop < dienos; lop++)
            {
                AllClicks = AllClicks + kazkas.Ads[i].Days[lop].Stats.Clicks;
                AllImpresions = AllImpresions + kazkas.Ads[i].Days[lop].Stats.Impressions;
            }
            statistikaClikai[i] = AllClicks;
            statistikaImpresions[i] = AllImpresions;
        }
    }

我知道void类型不能返回任何东西,但这就是我如何知道我的方法工作(从调试)。就像你看到的,我试着用for循环来做。这里我有9个广告,每个广告有一天。

就像我说的,我想返回每个Ads id[in array],以及每个days.stats. impressions和days.stats.click

我该怎么做呢?关于如何从方法返回更多的变量/数组到其他类,我使用webservices,所以我不能使用数据库或类似的东西。

变量不返回实际值

从问题的否定票数可以看出,您需要设计返回值,然后针对它进行编码。

你的查询几乎做到了(现在):

kazkas.Ads[n]; (n = how many ads are)
kazkas.Ads[n].id;
kazkas.Ads[n].Days[m].Stats.Clicks;  // every day have his own clicks
kazkas.Ads[n].Days[m].Stats.Impressions; // every day have his own impressions

您现有的代码显示,这应该扩展为包括:

kazkas.Ads[n].Total.Clicks;
kazkas.Ads[n].Total.Impressions;

现在你可以开始设计了。首先,你需要一个只包含点击和印象的统计类:

public class Stat
{
   public long Impressions { get; set; }
   public long Clicks { get; set; }
}

这里的优化可能是使用结构体,但我不会深入讨论。

正如你目前所定义的那样,每天只有一个Stats属性:

public class DayStat
{
   public Stat Stats { get; set; }
}

最后我们可以定义顶层AdStat:

public class AdStat
{
   public int id { get; set; }
   public DayStat Day[];
   public Stat Total { get; set; }
}

等等……这里还有进一步的问题,例如确保创建数组并且Stat实例永远不会是null(这就是为什么制作这些类结构中的一些是一种选择)。但我真的是一个VB程序员,所以在我被抓到在so IDE中输入垃圾之前,我就停在这里:-)

创建包含所需成员的类或结构

public class Stat
{
   public int Id { get; set; }
   public long Clicks { get; set; }
   ...
}

将你的方法的签名从void GetAdsStats改为IEnumberable<Stat> GetAdsStats,或者返回一个统计数据的集合,或者使用yield关键字返回统计对象。

如果你不想让你的方法返回任何东西(返回类型void),不要使用以Get开头的名称。

的例子:

public static IEnumerable<Stat> GetAdsStats(...)
{
   ...
   var statList = new List<Stat>();
   for (int i = 0; i < reklamos; i++)
   {
        var stat = new Stat();
        statList.Add(stat);
        int dienos = kazkas.Ads[i].Days.Length;
        for (int lop = 0; lop < dienos; lop++)
        {
            AllClicks = AllClicks + kazkas.Ads[i].Days[lop].Stats.Clicks;
            AllImpresions = AllImpresions + kazkas.Ads[i].Days[lop].Stats.Impressions;
        }
        stat.Clicks = AllClicks;
        stat.Impression = AllImpresions;
    }
    return statList;
}

void更改为您想要返回的类型,例如Campaign,并返回适当的变量。在方法中定义的变量只存在于方法中,不能从其他方法或类访问。