Home/ Questions/Q 6958185
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-27T15:06:26+00:00

In this subclass of UIWebView, I am looking to set up a method to

  • 0

In this subclass of UIWebView, I am looking to set up a method to create an embedded youtube video. However, no matter how I edit this code (which I found most of online), it is always giving me a warning at the line:

self = [[UIWebView alloc] initWithFrame:frame];

Im not sure whether this is because of Xcode 4.2 or iOS 5 or the fact that I am using self. Is there actually something wrong? If so, how might I fix it?

Entire method code:

- (videosView *)initWithStringAsURL:(NSString *)urlString frame:(CGRect)frame;
{
    if (self = [super init]) 
    {
        // Create webview with requested frame size
        self = [[UIWebView alloc] initWithFrame:frame];

        // HTML to embed YouTube video
        NSString *youTubeVideoHTML = @"<html><head>\
        <body style=\"margin:0\">\
        <embed id=\"yt\" src=\"%@\" type=\"application/x-shockwave-flash\" \
        width=\"%0.0f\" height=\"%0.0f\"></embed>\
        </body></html>"; 

        // Populate HTML with the URL and requested frame size
        NSString *html = [NSString stringWithFormat:youTubeVideoHTML, urlString, frame.size.width, frame.size.height];



        // Load the html into the webview
        [self loadHTMLString:html baseURL:nil];
        [self setUserInteractionEnabled:YES];

    }
    return self;  
}
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You already have the memory allocated (pointed to by this), so don’t alloc more:

    self = [super initWithFrame:frame];

    And, looking a bit more, you shouldn’t init super twice. Move that statement into the if, instead of the plain super init:

    if (self = [super initWithFrame:frame]) {...}
    • 0