Discussion :

[Slushgood]

Voila mon problème :

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

et voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Invité]

Bonjour,


1-

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
if( !empty($search_youtubej['items'][$i]) )
{
...

[andry.aime]

Bonjour,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

A+.

[Slushgood]

Merci jreaux62

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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+.

[Slushgood]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$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 :-)

[Invité]

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

Une formulation plus correcte serait :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

par :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Slushgood]

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