performance (5)

Author: jcheron

Ajax/bootstrap/html/HtmlTabs.php

@@ -40,7 +40,8 @@ protected function addTab_($tab, $index=null) {
 	}
 
 	public function setActive($index){
-		for ($i=0;$i<sizeof($this->tabs);$i++){
+		$size=\sizeof($this->tabs);
+		for ($i=0;$i<$size;$i++){
 			$this->tabs[$i]->setActive($i==$index);
 		}
 	}
@@ -159,7 +160,7 @@ public function fadeEffect() {
 			}
 		}
 	}
-	
+
 	public function on($event, $jsCode,$stopPropagation=false,$preventDefault=false){
 		foreach ($this->tabs as $tab){
 			$tab->on($event,$jsCode,$stopPropagation,$preventDefault);

Ajax/semantic/html/collections/HtmlGrid.php

@@ -286,13 +286,14 @@ protected function createItem($value) {
 	 */
 	public function setValues($values, $force=true) {
 		$count=$this->count();
+		$valuesSize=\sizeof($values);
 		if ($this->_createCols === false || $force === true) {
-			for($i=$count; $i < \sizeof($values); $i++) {
+			for($i=$count; $i < $valuesSize; $i++) {
 				$colSize=\sizeof($values[$i]);
 				$this->addItem(new HtmlGridRow($this->identifier . "-row-" . ($this->count() + 1), $colSize, $this->_colSizing, $this->_implicitRows));
 			}
 		}
-		$count=\min(array ($this->count(),\sizeof($values) ));
+		$count=\min(array ($this->count(),$valuesSize ));
 		for($i=0; $i < $count; $i++) {
 			$this->content[$i]->setValues($values[$i], $this->_createCols === false);
 		}

Ajax/semantic/html/content/HtmlGridRow.php

@@ -101,12 +101,13 @@ public function setColor($color){
 
 	public function setValues($values,$force=false){
 		$count=$this->count();
+		$valuesSize=\sizeof($values);
 		if($force===true){
-			for($i=$count;$i<\sizeof($values);$i++){
+			for($i=$count;$i<$valuesSize;$i++){
 				$this->addItem(new HtmlGridCol($this->identifier."-col-".($this->count()+1),null));
 			}
 		}
-		$count=\min(array($this->count(),\sizeof($values)));
+		$count=\min(array($this->count(),$valuesSize));
 		for($i=0;$i<$count;$i++){
 			$this->content[$i]->setValue($values[$i]);
 		}

Ajax/semantic/html/modules/HtmlTab.php

@@ -79,7 +79,8 @@ public function setTabContent($index,$content){
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function setTabsContent($contents){
-		for($i=0;$i<\sizeof($contents);$i++){
+		$size=\sizeof($contents);
+		for($i=0;$i<$size;$i++){
 			$this->setTabContent($i, $contents[$i]);
 		}
 		return $this;
@@ -204,14 +205,16 @@ public function getTab($index){
 	 * @return \Ajax\semantic\html\modules\HtmlTab
 	 */
 	public function setMenu($menu){
-		for($i=0;$i<\sizeof($this->content);$i++){
+		$contentSize=\sizeof($this->content);
+		for($i=0;$i<$contentSize;$i++){
 			if($menu->getItem($i)!==NULL){
 				if(isset($this->content[$i])){
 					$menu->getItem($i)->addToProperty("data-tab",$this->content[$i]->getProperty("data-tab"));
 				}
 			}
 		}
-		for($i=0;$i<$menu->count();$i++){
+		$menuSize=$menu->count();
+		for($i=0;$i<$menuSize;$i++){
 			$menu->getItem($i)->removeProperty("href");
 			if(isset($this->content[$i])===false){
 				$this->content[$i]=$this->createSegment($i, "New content", $menu->getItem($i)->getIdentifier());