Undefined offset: 25

Version imprimable

Voila mon problème :

Citation:

Notice: Undefined offset: 25 in C:\wamp64\www\joo\components\com\View\Books\tmpl\item.php on line 258

et voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
			<!-- Je recherche les video Youtube-->
            <?php
            $video_title = $basic['title']->value;
            $trouve3=true;
 
	            if ($video_title != '' && $basic['support']->value == "Vidéo") 
	            {
	            $search_youtube =file_get_contents("https://www.googleapis.com/youtube/v3/search?part=snippet&regionCode=fr&maxResults=25&q=".urlencode($video_title)."&key=AIzaSyAuz");
	            $search_youtubej =json_decode($search_youtube,true);
 
					for ($i = 0; $i <= 25; $i++)
					{	
						$youtubeTitle = $search_youtubej['items'][$i]['snippet']['title'];
						if(stristr($youtubeTitle,'bande annonce')||strstr($youtubeTitle,'bande-annonce'))
						{
							$trouve3=true;
							$youtubeID = $search_youtubej['items'][$i]['id']['videoId'];
							$youtubeDescription = $search_youtubej['items'][$i]['snippet']['description'];
							break;
						} else {
							$trouve3=false;
						}
					}
	            }

celui-ci fonctionne bien lorsqu'il trouve un résultat.
Mais s'il ne trouve rien j'ai un undefined offset. cela n'est pas gênant mais je souhaiterais le faire disparaitre.
Pouvez-vous m'aider ?
pour info voici la ligne 258:

Code:

$youtubeTitle = $search_youtubej['items'][$i]['snippet']['title'];

19/02/2019, 11h33
Invité
Bonjour,

1-

Code:

for ($i = 0; $i <= 25; $i++)

Ici, on a 26 valeurs (de 0 à 25). Or, tu en attends seulement 25 ("...maxResults=25").
Donc :

Code:

for ($i = 0; $i < 25; $i++)

Ceci devrait suffire à résoudre le problème.

2- Sinon, plus généralement, on teste d'abord les données (existence, validité...).
il suffit d'ajouter une condition dans la boucle for :
Code:

1 2 3 if( !empty($search_youtubej['items'][$i]) ) { ...
19/02/2019, 11h52
andry.aime

Bonjour,

Code:

for ($i = 0; $i < sizeof($search_youtubej['items']) ; $i++)

A+.

Merci jreaux62

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
					for ($i = 0; $i <= 25; $i++)
					{		            
						if( !empty($search_youtubej['items'][$i]) )
						{
						$youtubeTitle = $search_youtubej['items'][$i]['snippet']['title'];
							if(stristr($youtubeTitle,'bande annonce')||strstr($youtubeTitle,'bande-annonce'))
							{
								$trouve3=true;
								$youtubeID = $search_youtubej['items'][$i]['id']['videoId'];
								$youtubeDescription = $search_youtubej['items'][$i]['snippet']['description'];
								break;
							}
						} else {
							$trouve3=false;
						}
		            }

ceci a réglé mon problème.
Je m'obstinais avec un !isset alors que !empty était la solution.
:hola:

19/02/2019, 11h57
andry.aime

Ça fonctionne, mais si t'as moins de 25 éléments, la boucle continue sans erreur mais inutilement. Ensuite, si tu augmentes ton maxResult, tu dois aussi modifier la vérification dans sur la boucle. Tu dois virer le 25 codé en dur et utiliser la taille du tableau. Sinon il y a aussi la boucle foreach.

A+.
19/02/2019, 12h12
Slushgood

Merci andry.aime
Intéressant le foreach.
Pour le moment j'ai mis :

Code:

$search_youtube =file_get_contents("https://www.googleapis.com/youtube/v3/search?part=snippet&regionCode=fr&maxResults=24...

d'où : for ($i = 0; $i <= 25; $i++).
Mais c'est vrais qu'avec un maxResult ajustable le foreach s'impose.
A voir pour la suite :-)

andry.aime a raison ;)
(je n'avais pas regardé plus loin que le message d'erreur)

Une formulation plus correcte serait :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
<?php	// Je recherche les video Youtube
$video_title = $basic['title']->value;
$trouve3=true;
 
	if ($video_title != '' && $basic['support']->value == "Vidéo") 
	{
	$search_youtube =file_get_contents("https://www.googleapis.com/youtube/v3/search?part=snippet&regionCode=fr&maxResults=25&q=".urlencode($video_title)."&key=AIzaSyAuz");
	$search_youtubej =json_decode($search_youtube,true);
 
	  if( !empty($search_youtubej['items']) )
	  {
		foreach ($search_youtubej['items'] as $item)
		{	
			$youtubeTitle = $item['snippet']['title'];
			if(stristr($youtubeTitle,'bande annonce')||strstr($youtubeTitle,'bande-annonce'))
			{
				$trouve3=true;
				$youtubeID = $item['id']['videoId'];
				$youtubeDescription = $item['snippet']['description'];
				break;
			} else {
				$trouve3=false;
			}
		}
	  }
	}

On peut aussi aller plus loin dans les tests :
Remplacer (entre autres) :

Code:

$youtubeTitle = $item['snippet']['title'];

par :

Code:

$youtubeTitle = ( !empty($item['snippet']['title']) )? $item['snippet']['title'] : '';

19/02/2019, 14h30
Slushgood

merci jreaux62
sous cette forme-là je comprend mieux le principe.