变量不返回实际值

时间:2013-06-18 09:58:21

标签: c# asp.net

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

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个广告,每个广告都有一天。

就像我说我想要返回每个广告ID [在数组中],并且每天都返回.stats.impresions和days.stats.click

我该怎么做?如何将更多的变量/数组从方法返回到其他类,我正在使用webservises,所以我不能使用类似的数据库。

3 个答案:

答案 0 :(得分:1)

使用您需要的成员创建一个类或结构

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;
}

答案 1 :(得分:1)

从问题的下划线可以看出,您需要设计返回值,然后对其进行编码。

你的查询差不多了(现在):

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;

所以现在你已经准备好了。首先,您需要一个仅包含CLicks和Impressions的Stat Class:

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中输入废话之前停在这里: - )

答案 2 :(得分:0)

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