0

Monotouch 要素に基づいて要素を作成しています。これで何が起こるかというと、各セルに件名、ドメイン名、および Fav-Icon を含むリスト (Twitter タイムラインのような) が作成されます。問題は次のとおりです。リストの最初と最後のお気に入りアイコンが 2 回表示されるため、それらが重なって表示されます。

以下はコードです。誰が私が犯した間違いを見ることができますか? /ロバート

public override void Draw (RectangleF rect)
{
    const int padright = 12;//21;
    var ctx = UIGraphics.GetCurrentContext ();
    bool highlighted = (Superview.Superview as UITableViewCell).Highlighted;

                var bounds = Bounds;
                var midx = bounds.Width/2;
                if (highlighted){
                    UIColor.FromRGB (4, 0x79, 0xef).SetColor ();
                    ctx.FillRect (bounds);
                    //Images.MenuShadow.Draw (bounds, CGBlendMode.Normal, 0.5f);
                    //textColor = UIColor.White;
                } else {
                    UIColor.White.SetColor ();
                    ctx.FillRect (bounds);
                    ctx.DrawLinearGradient (bottomGradient, new PointF (midx, bounds.Height-17), new PointF (midx, bounds.Height), 0);
                    ctx.DrawLinearGradient (topGradient, new PointF (midx, 1), new PointF (midx, 3), 0);
                }

        float boxWidth;
        float boxHeight = Bounds.Height;
        boxWidth = 0;
        //SizeF ssize;
        //string label;
        //label = Date;
        //ssize = StringSize (label, SubjectFont);
        //float dateSize = ssize.Width + padright + 5;
        //DrawString (label, new RectangleF (Bounds.Width-dateSize-5, 5, dateSize, 14), DateFont, UILineBreakMode.Clip, UITextAlignment.Right);

        const int offset = 46; //33;
        float bw = Bounds.Width-offset;
        UIColor.Black.SetColor ();

        //Extracting the domain name that I get.
        string domainName = new Uri(Sender).DnsSafeHost;
        if(domainName.StartsWith("www"))
        {
            domainName = domainName.Substring(4, domainName.Length-4);
        }

        //Checking to see if there is any subdomain.
        //If so, I start at the first dot to get the domainname "clean".
        int countDots = domainName.Split('.').Length - 1;
        if(countDots > 1)
        {
        Console.WriteLine("Dots: " + countDots);
        int i = domainName.IndexOf('.') + 1;
        Console.WriteLine("Position: " + i);
            domainName = domainName.Substring(i).Trim();
            Console.WriteLine("Domain: " + domainName);
        }

        DrawString (Subject, new RectangleF (offset, 6, bw-boxWidth-5, 24), SubjectFont, UILineBreakMode.TailTruncation, UITextAlignment.Left);
        DrawString (domainName, new PointF (offset, 28), bw/*-dateSize*/, SenderFont, UILineBreakMode.TailTruncation);

        //Getting the fav icon from the domainname and posting it to g.etfv.co.
        string favUrl = "http://www."+domainName;
        Console.WriteLine("Fav icon: " + favUrl);
        Uri imageBackground = null;
        imageBackground = new Uri ("http://g.etfv.co/"+ favUrl);
        var avatar = ImageLoader.DefaultRequestImage (imageBackground, this);
        var imageRect = new RectangleF(15f, 10f, 16f, 16f); 
        using (var myImage = new UIImageView(imageRect))
        {
            if(avatar == null)
            {
                //A local avatar if something goes wrong
                string localAvatar = "Images/avatar.png";
                myImage.Image = UIImage.FromFile(localAvatar);
            }
            else
            {myImage.Image = avatar;}
            AddSubview(myImage);
        }
        avatar = null;
    }
4

1 に答える 1

0

誰が私が犯した間違いを見ることができますか?

パート1...

「したがって、それらは互いに重なっている」というあなたの説明は完全には明確ではありません。

しかし、この方法の背後に問題が隠されているのではないでしょうか?

        var avatar = ImageLoader.DefaultRequestImage (imageBackground, this);

これがアバターを非同期的に要求する場合、非同期メソッドが返されたときに、セルが別の場所で再利用されていないことを確認する必要があります。

TweetStation では、これはvoid IImageUpdated.UpdatedImage (long onId)コールバックの次の行で行われます。

    // Discard notifications that might have been queued for an old cell
    if (tweet == null || (tweet.UserId != onId && tweet.RetweeterId != onId))
        return;

コールバックに同様の行がありますか?

誰が私が犯した間違いを見ることができますか?

パート2...

1 つのメソッドを長すぎて読めないという間違いを犯したと思います。

「メソッドの抽出」などのリファクタリング ツールを使用して、意味のある小さな関数に分解することを検討してください。見る

これはすべて通常のリファクタリングの一部です。コードを 1 つの関数に記述し、それが長くなり扱いにくくなるため、小さな部分にリファクタリングします...

...そして多くの人が、私が書いたメソッドは小さすぎると言うでしょう...

...だから、何をするにしても、あなたとあなたのコードに適したものを見つけるだけです:)

于 2013-02-11T18:19:47.503 に答える