Conditions | 1 |
Paths | 1 |
Total Lines | 303 |
Code Lines | 295 |
Lines | 0 |
Ratio | 0 % |
Changes | 1 | ||
Bugs | 0 | Features | 1 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
82 | public function dataNewFunction() { |
||
83 | return array( |
||
84 | array('array_fill_keys', '5.1', array(12, 13), '5.3'), |
||
85 | array('error_get_last', '5.1', array(14), '5.3'), |
||
86 | array('image_type_to_extension', '5.1', array(15), '5.2'), |
||
87 | array('memory_get_peak_usage', '5.1', array(16), '5.2'), |
||
88 | array('sys_get_temp_dir', '5.1', array(17), '5.2'), |
||
89 | array('timezone_abbreviations_list', '5.1', array(18), '5.2'), |
||
90 | array('timezone_identifiers_list', '5.1', array(19), '5.2'), |
||
91 | array('timezone_name_from_abbr', '5.1', array(20), '5.2'), |
||
92 | array('stream_socket_shutdown', '5.1', array(21), '5.2'), |
||
93 | array('imagegrabscreen', '5.1', array(22), '5.2'), |
||
94 | array('imagegrabwindow', '5.1', array(23), '5.2'), |
||
95 | array('libxml_disable_entity_loader', '5.1', array(24), '5.2'), |
||
96 | array('mb_stripos', '5.1', array(25), '5.2'), |
||
97 | array('mb_stristr', '5.1', array(26), '5.2'), |
||
98 | array('mb_strrchr', '5.1', array(27), '5.2'), |
||
99 | array('mb_strrichr', '5.1', array(28), '5.2'), |
||
100 | array('mb_strripos', '5.1', array(29), '5.2'), |
||
101 | array('ming_setSWFCompression', '5.1', array(30), '5.2'), |
||
102 | array('openssl_csr_get_public_key', '5.1', array(31), '5.2'), |
||
103 | array('openssl_csr_get_subject', '5.1', array(32), '5.2'), |
||
104 | array('openssl_pkey_get_details', '5.1', array(33), '5.2'), |
||
105 | array('spl_object_hash', '5.1', array(34), '5.2'), |
||
106 | array('iterator_apply', '5.1', array(35), '5.2'), |
||
107 | array('preg_last_error', '5.1', array(36), '5.2'), |
||
108 | array('pg_field_table', '5.1', array(37), '5.2'), |
||
109 | array('posix_initgroups', '5.1', array(38), '5.2'), |
||
110 | array('gmp_nextprime', '5.1', array(39), '5.2'), |
||
111 | array('xmlwriter_full_end_element', '5.1', array(40), '5.2'), |
||
112 | array('xmlwriter_write_raw', '5.1', array(41), '5.2'), |
||
113 | array('xmlwriter_start_dtd_entity', '5.1', array(42), '5.2'), |
||
114 | array('xmlwriter_end_dtd_entity', '5.1', array(43), '5.2'), |
||
115 | array('xmlwriter_write_dtd_entity', '5.1', array(44), '5.2'), |
||
116 | array('filter_has_var', '5.1', array(45), '5.2'), |
||
117 | array('filter_id', '5.1', array(46), '5.2'), |
||
118 | array('filter_input_array', '5.1', array(47), '5.2'), |
||
119 | array('filter_input', '5.1', array(48), '5.2'), |
||
120 | array('filter_list', '5.1', array(49), '5.2'), |
||
121 | array('filter_var_array', '5.1', array(50), '5.2'), |
||
122 | array('filter_var', '5.1', array(51), '5.2'), |
||
123 | array('json_decode', '5.1', array(52), '5.2'), |
||
124 | array('json_encode', '5.1', array(53), '5.2'), |
||
125 | array('zip_close', '5.1', array(54), '5.2'), |
||
126 | array('zip_entry_close', '5.1', array(55), '5.2'), |
||
127 | array('zip_entry_compressedsize', '5.1', array(56), '5.2'), |
||
128 | array('zip_entry_compressionmethod', '5.1', array(57), '5.2'), |
||
129 | array('zip_entry_filesize', '5.1', array(58), '5.2'), |
||
130 | array('zip_entry_name', '5.1', array(59), '5.2'), |
||
131 | array('zip_entry_open', '5.1', array(60), '5.2'), |
||
132 | array('zip_entry_read', '5.1', array(61), '5.2'), |
||
133 | array('zip_open', '5.1', array(62), '5.2'), |
||
134 | array('zip_read', '5.1', array(63), '5.2'), |
||
135 | |||
136 | array('array_replace', '5.2', array(65), '5.3'), |
||
137 | array('array_replace_recursive', '5.2', array(66), '5.3'), |
||
138 | array('class_alias', '5.2', array(67), '5.3'), |
||
139 | array('forward_static_call', '5.2', array(68), '5.3'), |
||
140 | array('forward_static_call_array', '5.2', array(69), '5.3'), |
||
141 | array('gc_collect_cycles', '5.2', array(70), '5.3'), |
||
142 | array('gc_disable', '5.2', array(71), '5.3'), |
||
143 | array('gc_enable', '5.2', array(72), '5.3'), |
||
144 | array('gc_enabled', '5.2', array(73), '5.3'), |
||
145 | array('get_called_class', '5.2', array(74), '5.3'), |
||
146 | array('gethostname', '5.2', array(75), '5.3'), |
||
147 | array('header_remove', '5.2', array(76), '5.3'), |
||
148 | array('lcfirst', '5.2', array(77), '5.3'), |
||
149 | array('parse_ini_string', '5.2', array(78), '5.3'), |
||
150 | array('quoted_printable_encode', '5.2', array(79), '5.3'), |
||
151 | array('str_getcsv', '5.2', array(80), '5.3'), |
||
152 | array('stream_context_set_default', '5.2', array(81), '5.3'), |
||
153 | array('stream_supports_lock', '5.2', array(82), '5.3'), |
||
154 | array('stream_context_get_params', '5.2', array(83), '5.3'), |
||
155 | array('date_add', '5.2', array(84), '5.3'), |
||
156 | array('date_create_from_format', '5.2', array(85), '5.3'), |
||
157 | array('date_diff', '5.2', array(86), '5.3'), |
||
158 | array('date_get_last_errors', '5.2', array(87), '5.3'), |
||
159 | array('date_parse_from_format', '5.2', array(88), '5.3'), |
||
160 | array('date_sub', '5.2', array(89), '5.3'), |
||
161 | array('timezone_version_get', '5.2', array(90), '5.3'), |
||
162 | array('gmp_testbit', '5.2', array(91), '5.3'), |
||
163 | array('hash_copy', '5.2', array(92), '5.3'), |
||
164 | array('imap_gc', '5.2', array(93), '5.3'), |
||
165 | array('imap_utf8_to_mutf7', '5.2', array(94), '5.3'), |
||
166 | array('imap_mutf7_to_utf8', '5.2', array(95), '5.3'), |
||
167 | array('json_last_error', '5.2', array(96), '5.3'), |
||
168 | array('mysqli_fetch_all', '5.2', array(97), '5.3'), |
||
169 | array('mysqli_get_connection_status', '5.2', array(98), '5.3'), |
||
170 | array('mysqli_poll', '5.2', array(99), '5.3'), |
||
171 | array('mysqli_read_async_query', '5.2', array(100), '5.3'), |
||
172 | array('openssl_random_pseudo_bytes', '5.2', array(101), '5.3'), |
||
173 | array('pcntl_signal_dispatch', '5.2', array(102), '5.3'), |
||
174 | array('pcntl_sigprocmask', '5.2', array(103), '5.3'), |
||
175 | array('pcntl_sigtimedwait', '5.2', array(104), '5.3'), |
||
176 | array('pcntl_sigwaitinfo', '5.2', array(105), '5.3'), |
||
177 | array('preg_filter', '5.2', array(106), '5.3'), |
||
178 | array('msg_queue_exists', '5.2', array(107), '5.3'), |
||
179 | array('shm_has_vars', '5.2', array(108), '5.3'), |
||
180 | array('acosh', '5.2', array(109), '5.3'), |
||
181 | array('asinh', '5.2', array(110), '5.3'), |
||
182 | array('atanh', '5.2', array(111), '5.3'), |
||
183 | array('expm1', '5.2', array(112), '5.3'), |
||
184 | array('log1p', '5.2', array(113), '5.3'), |
||
185 | array('enchant_broker_describe', '5.2', array(114), '5.3'), |
||
186 | array('enchant_broker_dict_exists', '5.2', array(115), '5.3'), |
||
187 | array('enchant_broker_free_dict', '5.2', array(116), '5.3'), |
||
188 | array('enchant_broker_free', '5.2', array(117), '5.3'), |
||
189 | array('enchant_broker_get_error', '5.2', array(118), '5.3'), |
||
190 | array('enchant_broker_init', '5.2', array(119), '5.3'), |
||
191 | array('enchant_broker_list_dicts', '5.2', array(120), '5.3'), |
||
192 | array('enchant_broker_request_dict', '5.2', array(121), '5.3'), |
||
193 | array('enchant_broker_request_pwl_dict', '5.2', array(122), '5.3'), |
||
194 | array('enchant_broker_set_ordering', '5.2', array(123), '5.3'), |
||
195 | array('enchant_dict_add_to_personal', '5.2', array(124), '5.3'), |
||
196 | array('enchant_dict_add_to_session', '5.2', array(125), '5.3'), |
||
197 | array('enchant_dict_check', '5.2', array(126), '5.3'), |
||
198 | array('enchant_dict_describe', '5.2', array(127), '5.3'), |
||
199 | array('enchant_dict_get_error', '5.2', array(128), '5.3'), |
||
200 | array('enchant_dict_is_in_session', '5.2', array(129), '5.3'), |
||
201 | array('enchant_dict_quick_check', '5.2', array(130), '5.3'), |
||
202 | array('enchant_dict_store_replacement', '5.2', array(131), '5.3'), |
||
203 | array('enchant_dict_suggest', '5.2', array(132), '5.3'), |
||
204 | array('finfo_buffer', '5.2', array(133), '5.3'), |
||
205 | array('finfo_close', '5.2', array(134), '5.3'), |
||
206 | array('finfo_file', '5.2', array(135), '5.3'), |
||
207 | array('finfo_open', '5.2', array(136), '5.3'), |
||
208 | array('finfo_set_flags', '5.2', array(137), '5.3'), |
||
209 | array('intl_error_name', '5.2', array(138), '5.3'), |
||
210 | array('intl_get_error_code', '5.2', array(139), '5.3'), |
||
211 | array('intl_get_error_message', '5.2', array(140), '5.3'), |
||
212 | array('intl_is_failure', '5.2', array(141), '5.3'), |
||
213 | array('mysqli_get_cache_stats', '5.2', array(142), '5.3'), |
||
214 | |||
215 | array('hex2bin', '5.3', array(144), '5.4'), |
||
216 | array('http_response_code', '5.3', array(145), '5.4'), |
||
217 | array('get_declared_traits', '5.3', array(146), '5.4'), |
||
218 | array('getimagesizefromstring', '5.3', array(147), '5.4'), |
||
219 | array('stream_set_chunk_size', '5.3', array(148), '5.4'), |
||
220 | array('socket_import_stream', '5.3', array(149), '5.4'), |
||
221 | array('trait_exists', '5.3', array(150), '5.4'), |
||
222 | array('header_register_callback', '5.3', array(151), '5.4'), |
||
223 | array('class_uses', '5.3', array(152), '5.4'), |
||
224 | array('session_status', '5.3', array(153), '5.4'), |
||
225 | array('session_register_shutdown', '5.3', array(154), '5.4'), |
||
226 | array('mysqli_error_list', '5.3', array(155), '5.4'), |
||
227 | array('mysqli_stmt_error_list', '5.3', array(156), '5.4'), |
||
228 | array('libxml_set_external_entity_loader', '5.3', array(157), '5.4'), |
||
229 | array('ldap_control_paged_result', '5.3', array(158), '5.4'), |
||
230 | array('ldap_control_paged_result_response', '5.3', array(159), '5.4'), |
||
231 | array('transliteral_create', '5.3', array(160), '5.4'), |
||
232 | array('transliteral_create_from_rules', '5.3', array(161), '5.4'), |
||
233 | array('transliteral_create_inverse', '5.3', array(162), '5.4'), |
||
234 | array('transliteral_get_error_code', '5.3', array(163), '5.4'), |
||
235 | array('transliteral_get_error_message', '5.3', array(164), '5.4'), |
||
236 | array('transliteral_list_ids', '5.3', array(165), '5.4'), |
||
237 | array('transliteral_transliterate', '5.3', array(166), '5.4'), |
||
238 | array('zlib_decode', '5.3', array(167), '5.4'), |
||
239 | array('zlib_encode', '5.3', array(168), '5.4'), |
||
240 | |||
241 | array('array_column', '5.4', array(170), '5.5'), |
||
242 | array('boolval', '5.4', array(171), '5.5'), |
||
243 | array('json_last_error_msg', '5.4', array(172), '5.5'), |
||
244 | array('password_get_info', '5.4', array(173), '5.5'), |
||
245 | array('password_hash', '5.4', array(174), '5.5'), |
||
246 | array('password_needs_rehash', '5.4', array(175), '5.5'), |
||
247 | array('password_verify', '5.4', array(176), '5.5'), |
||
248 | array('hash_pbkdf2', '5.4', array(177), '5.5'), |
||
249 | array('openssl_pbkdf2', '5.4', array(178), '5.5'), |
||
250 | array('curl_escape', '5.4', array(179), '5.5'), |
||
251 | array('curl_file_create', '5.4', array(180), '5.5'), |
||
252 | array('curl_multi_setopt', '5.4', array(181), '5.5'), |
||
253 | array('curl_multi_strerror', '5.4', array(182), '5.5'), |
||
254 | array('curl_pause', '5.4', array(183), '5.5'), |
||
255 | array('curl_reset', '5.4', array(184), '5.5'), |
||
256 | array('curl_share_close', '5.4', array(185), '5.5'), |
||
257 | array('curl_share_init', '5.4', array(186), '5.5'), |
||
258 | array('curl_share_setopt', '5.4', array(187), '5.5'), |
||
259 | array('curl_strerror', '5.4', array(188), '5.5'), |
||
260 | array('curl_unescape', '5.4', array(189), '5.5'), |
||
261 | array('imageaffinematrixconcat', '5.4', array(190), '5.5'), |
||
262 | array('imageaffinematrixget', '5.4', array(191), '5.5'), |
||
263 | array('imagecrop', '5.4', array(192), '5.5'), |
||
264 | array('imagecropauto', '5.4', array(193), '5.5'), |
||
265 | array('imageflip', '5.4', array(194), '5.5'), |
||
266 | array('imagepalettetotruecolor', '5.4', array(195), '5.5'), |
||
267 | array('imagescale', '5.4', array(196), '5.5'), |
||
268 | array('mysqli_begin_transaction', '5.4', array(197), '5.5'), |
||
269 | array('mysqli_release_savepoint', '5.4', array(198), '5.5'), |
||
270 | array('mysqli_savepoint', '5.4', array(199), '5.5'), |
||
271 | array('pg_escape_literal', '5.4', array(200), '5.5'), |
||
272 | array('pg_escape_identifier', '5.4', array(201), '5.5'), |
||
273 | array('socket_sendmsg', '5.4', array(202), '5.5'), |
||
274 | array('socket_recvmsg', '5.4', array(203), '5.5'), |
||
275 | array('socket_cmsg_space', '5.4', array(204), '5.5'), |
||
276 | array('cli_get_process_title', '5.4', array(205), '5.5'), |
||
277 | array('cli_set_process_title', '5.4', array(206), '5.5'), |
||
278 | array('datefmt_format_object', '5.4', array(207), '5.5'), |
||
279 | array('datefmt_get_calendar_object', '5.4', array(208), '5.5'), |
||
280 | array('datefmt_get_timezone', '5.4', array(209), '5.5'), |
||
281 | array('datefmt_set_timezone', '5.4', array(210), '5.5'), |
||
282 | array('datefmt_get_calendar_object', '5.4', array(211), '5.5'), |
||
283 | array('intlcal_create_instance', '5.4', array(212), '5.5'), |
||
284 | array('intlcal_get_keyword_values_for_locale', '5.4', array(213), '5.5'), |
||
285 | array('intlcal_get_now', '5.4', array(214), '5.5'), |
||
286 | array('intlcal_get_available_locales', '5.4', array(215), '5.5'), |
||
287 | array('intlcal_get', '5.4', array(216), '5.5'), |
||
288 | array('intlcal_get_time', '5.4', array(217), '5.5'), |
||
289 | array('intlcal_set_time', '5.4', array(218), '5.5'), |
||
290 | array('intlcal_add', '5.4', array(219), '5.5'), |
||
291 | array('intlcal_set_time_zone', '5.4', array(220), '5.5'), |
||
292 | array('intlcal_after', '5.4', array(221), '5.5'), |
||
293 | array('intlcal_before', '5.4', array(222), '5.5'), |
||
294 | array('intlcal_set', '5.4', array(223), '5.5'), |
||
295 | array('intlcal_roll', '5.4', array(224), '5.5'), |
||
296 | array('intlcal_clear', '5.4', array(225), '5.5'), |
||
297 | array('intlcal_field_difference', '5.4', array(226), '5.5'), |
||
298 | array('intlcal_get_actual_maximum', '5.4', array(227), '5.5'), |
||
299 | array('intlcal_get_actual_minumum', '5.4', array(228), '5.5'), |
||
300 | array('intlcal_get_day_of_week_type', '5.4', array(229), '5.5'), |
||
301 | array('intlcal_get_first_day_of_week', '5.4', array(230), '5.5'), |
||
302 | array('intlcal_get_greatest_minimum', '5.4', array(231), '5.5'), |
||
303 | array('intlcal_get_least_maximum', '5.4', array(232), '5.5'), |
||
304 | array('intlcal_get_locale', '5.4', array(233), '5.5'), |
||
305 | array('intlcal_get_maximum', '5.4', array(234), '5.5'), |
||
306 | array('intlcal_get_minimal_days_in_first_week', '5.4', array(235), '5.5'), |
||
307 | array('intlcal_get_minimum', '5.4', array(236), '5.5'), |
||
308 | array('intlcal_get_time_zone', '5.4', array(237), '5.5'), |
||
309 | array('intlcal_get_type', '5.4', array(238), '5.5'), |
||
310 | array('intlcal_get_weekend_transition', '5.4', array(239), '5.5'), |
||
311 | array('intlcal_in_daylight_time', '5.4', array(240), '5.5'), |
||
312 | array('intlcal_is_equivalent_to', '5.4', array(241), '5.5'), |
||
313 | array('intlcal_is_lenient', '5.4', array(242), '5.5'), |
||
314 | array('intlcal_equals', '5.4', array(243), '5.5'), |
||
315 | array('intlcal_get_repeated_wall_time_option', '5.4', array(244), '5.5'), |
||
316 | array('intlcal_get_skipped_wall_time_option', '5.4', array(245), '5.5'), |
||
317 | array('intlcal_set_repeated_wall_time_option', '5.4', array(246), '5.5'), |
||
318 | array('intlcal_set_skipped_wall_time_option', '5.4', array(247), '5.5'), |
||
319 | array('intlcal_from_date_time', '5.4', array(248), '5.5'), |
||
320 | array('intlcal_to_date_time', '5.4', array(249), '5.5'), |
||
321 | array('intlcal_get_error_code', '5.4', array(250), '5.5'), |
||
322 | array('intlcal_get_error_message', '5.4', array(251), '5.5'), |
||
323 | array('intlgregcal_create_instance', '5.4', array(252), '5.5'), |
||
324 | array('intlgregcal_set_gregorian_change', '5.4', array(253), '5.5'), |
||
325 | array('intlgregcal_get_gregorian_change', '5.4', array(254), '5.5'), |
||
326 | array('intlgregcal_is_leap_year', '5.4', array(255), '5.5'), |
||
327 | array('intlz_create_time_zone', '5.4', array(256), '5.5'), |
||
328 | array('intlz_create_default', '5.4', array(257), '5.5'), |
||
329 | array('intlz_get_id', '5.4', array(258), '5.5'), |
||
330 | array('intlz_get_gmt', '5.4', array(259), '5.5'), |
||
331 | array('intlz_get_unknown', '5.4', array(260), '5.5'), |
||
332 | array('intlz_create_enumeration', '5.4', array(261), '5.5'), |
||
333 | array('intlz_count_equivalent_ids', '5.4', array(262), '5.5'), |
||
334 | array('intlz_create_time_zone_id_enumeration', '5.4', array(263), '5.5'), |
||
335 | array('intlz_get_canonical_id', '5.4', array(264), '5.5'), |
||
336 | array('intlz_get_region', '5.4', array(265), '5.5'), |
||
337 | array('intlz_get_tz_data_version', '5.4', array(266), '5.5'), |
||
338 | array('intlz_get_equivalent_id', '5.4', array(267), '5.5'), |
||
339 | array('intlz_use_daylight_time', '5.4', array(268), '5.5'), |
||
340 | array('intlz_get_offset', '5.4', array(269), '5.5'), |
||
341 | array('intlz_get_raw_offset', '5.4', array(270), '5.5'), |
||
342 | array('intlz_has_same_rules', '5.4', array(271), '5.5'), |
||
343 | array('intlz_get_display_name', '5.4', array(272), '5.5'), |
||
344 | array('intlz_get_dst_savings', '5.4', array(273), '5.5'), |
||
345 | array('intlz_from_date_time_zone', '5.4', array(274), '5.5'), |
||
346 | array('intlz_to_date_time_zone', '5.4', array(275), '5.5'), |
||
347 | array('intlz_get_error_code', '5.4', array(276), '5.5'), |
||
348 | array('intlz_get_error_message', '5.4', array(277), '5.5'), |
||
349 | |||
350 | array('gmp_root', '5.5', array(279), '5.6'), |
||
351 | array('gmp_rootrem', '5.5', array(280), '5.6'), |
||
352 | array('hash_equals', '5.5', array(281), '5.6'), |
||
353 | array('ldap_escape', '5.5', array(282), '5.6'), |
||
354 | array('ldap_modify_batch', '5.4.25', array(283), '5.6', '5.4'), |
||
355 | array('mysqli_get_links_stats', '5.5', array(284), '5.6'), |
||
356 | array('openssl_get_cert_locations', '5.5', array(285), '5.6'), |
||
357 | array('openssl_x509_fingerprint', '5.5', array(286), '5.6'), |
||
358 | array('openssl_spki_new', '5.5', array(287), '5.6'), |
||
359 | array('openssl_spki_verify', '5.5', array(288), '5.6'), |
||
360 | array('openssl_spki_export_challenge', '5.5', array(289), '5.6'), |
||
361 | array('openssl_spki_export', '5.5', array(290), '5.6'), |
||
362 | array('pg_connect_poll', '5.5', array(291), '5.6'), |
||
363 | array('pg_consume_input', '5.5', array(292), '5.6'), |
||
364 | array('pg_flush', '5.5', array(293), '5.6'), |
||
365 | array('pg_socket', '5.5', array(294), '5.6'), |
||
366 | array('session_abort', '5.5', array(295), '5.6'), |
||
367 | array('session_reset', '5.5', array(296), '5.6'), |
||
368 | |||
369 | array('random_bytes', '5.6', array(298), '7.0'), |
||
370 | array('random_int', '5.6', array(299), '7.0'), |
||
371 | array('error_clear_last', '5.6', array(300), '7.0'), |
||
372 | array('gmp_random_seed', '5.6', array(301), '7.0'), |
||
373 | array('intdiv', '5.6', array(302), '7.0'), |
||
374 | array('preg_replace_callback_array', '5.6', array(303), '7.0'), |
||
375 | array('gc_mem_caches', '5.6', array(304), '7.0'), |
||
376 | array('get_resources', '5.6', array(305), '7.0'), |
||
377 | array('posix_setrlimit', '5.6', array(306), '7.0'), |
||
378 | array('inflate_add', '5.6', array(307), '7.0'), |
||
379 | array('deflate_add', '5.6', array(308), '7.0'), |
||
380 | array('inflate_init', '5.6', array(309), '7.0'), |
||
381 | array('deflate_init', '5.6', array(310), '7.0'), |
||
382 | |||
383 | ); |
||
384 | } |
||
385 | |||
387 |
You can fix this by adding a namespace to your class:
When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.