nextcloud /
notes
Complex classes like NotesService often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use NotesService, and based on these observations, apply Extract Interface, too.
| 1 | <?php |
||
| 28 | class NotesService { |
||
| 29 | |||
| 30 | private $l10n; |
||
| 31 | private $root; |
||
| 32 | private $logger; |
||
| 33 | private $appName; |
||
| 34 | |||
| 35 | /** |
||
| 36 | * @param IRootFolder $root |
||
| 37 | * @param IL10N $l10n |
||
| 38 | * @param ILogger $logger |
||
| 39 | * @param String $appName |
||
| 40 | */ |
||
| 41 | public function __construct (IRootFolder $root, IL10N $l10n, ILogger $logger, $appName) { |
||
| 47 | |||
| 48 | |||
| 49 | /** |
||
| 50 | * @param string $userId |
||
| 51 | * @return array with all notes in the current directory |
||
| 52 | */ |
||
| 53 | public function getAll ($userId){ |
||
| 74 | |||
| 75 | |||
| 76 | /** |
||
| 77 | * Used to get a single note by id |
||
| 78 | * @param int $id the id of the note to get |
||
| 79 | * @param string $userId |
||
| 80 | * @throws NoteDoesNotExistException if note does not exist |
||
| 81 | * @return Note |
||
| 82 | */ |
||
| 83 | public function get ($id, $userId) { |
||
| 87 | |||
| 88 | private function getTags ($id) { |
||
| 112 | /** |
||
| 113 | * Creates a note and returns the empty note |
||
| 114 | * @param string $userId |
||
| 115 | * @see update for setting note content |
||
| 116 | * @return Note the newly created note |
||
| 117 | */ |
||
| 118 | public function create ($userId) { |
||
| 130 | |||
| 131 | |||
| 132 | /** |
||
| 133 | * Updates a note. Be sure to check the returned note since the title is |
||
| 134 | * dynamically generated and filename conflicts are resolved |
||
| 135 | * @param int $id the id of the note used to update |
||
| 136 | * @param string $content the content which will be written into the note |
||
| 137 | * the title is generated from the first line of the content |
||
| 138 | * @param int $mtime time of the note modification (optional) |
||
| 139 | * @throws NoteDoesNotExistException if note does not exist |
||
| 140 | * @return \OCA\Notes\Db\Note the updated note |
||
| 141 | */ |
||
| 142 | public function update ($id, $content, $userId, $category=null, $mtime=0) { |
||
| 191 | |||
| 192 | |||
| 193 | /** |
||
| 194 | * Set or unset a note as favorite. |
||
| 195 | * @param int $id the id of the note used to update |
||
| 196 | * @param boolean $favorite whether the note should be a favorite or not |
||
| 197 | * @throws NoteDoesNotExistException if note does not exist |
||
| 198 | * @return boolean the new favorite state of the note |
||
| 199 | */ |
||
| 200 | public function favorite ($id, $favorite, $userId){ |
||
| 215 | |||
| 216 | |||
| 217 | /** |
||
| 218 | * Deletes a note |
||
| 219 | * @param int $id the id of the note which should be deleted |
||
| 220 | * @param string $userId |
||
| 221 | * @throws NoteDoesNotExistException if note does not |
||
| 222 | * exist |
||
| 223 | */ |
||
| 224 | public function delete ($id, $userId) { |
||
| 229 | |||
| 230 | // removes characters that are illegal in a file or folder name on some operating systems |
||
| 231 | private function sanitisePath($str) { |
||
| 251 | |||
| 252 | private function getSafeTitleFromContent($content) { |
||
| 276 | |||
| 277 | /** |
||
| 278 | * @param Folder $folder |
||
| 279 | * @param int $id |
||
| 280 | * @throws NoteDoesNotExistException |
||
| 281 | * @return \OCP\Files\File |
||
| 282 | */ |
||
| 283 | private function getFileById ($folder, $id) { |
||
| 291 | |||
| 292 | /** |
||
| 293 | * @param string $userId the user id |
||
| 294 | * @return boolean true if folder is accessible, or Exception otherwise |
||
| 295 | */ |
||
| 296 | public function checkNotesFolder($userId) { |
||
| 300 | |||
| 301 | /** |
||
| 302 | * @param string $userId the user id |
||
| 303 | * @return Folder |
||
| 304 | */ |
||
| 305 | private function getFolderForUser ($userId) { |
||
| 314 | |||
| 315 | |||
| 316 | /** |
||
| 317 | * Finds a folder and creates it if non-existent |
||
| 318 | * @param string $path path to the folder |
||
| 319 | * @return Folder |
||
| 320 | */ |
||
| 321 | private function getOrCreateFolder($path) { |
||
| 329 | |||
| 330 | |||
| 331 | /** |
||
| 332 | * get path of file and the title.txt and check if they are the same |
||
| 333 | * file. If not the title needs to be renamed |
||
| 334 | * |
||
| 335 | * @param Folder $folder a folder to the notes directory |
||
| 336 | * @param string $title the filename which should be used |
||
| 337 | * @param string $extension the extension which should be used |
||
| 338 | * @param int $id the id of the note for which the title should be generated |
||
| 339 | * used to see if the file itself has the title and not a different file for |
||
| 340 | * checking for filename collisions |
||
| 341 | * @return string the resolved filename to prevent overwriting different |
||
| 342 | * files with the same title |
||
| 343 | */ |
||
| 344 | private function generateFileName (Folder $folder, $title, $extension, $id) { |
||
| 364 | |||
| 365 | /** |
||
| 366 | * gather note files in given directory and all subdirectories |
||
| 367 | * @param Folder $folder |
||
| 368 | * @return array |
||
| 369 | */ |
||
| 370 | private function gatherNoteFiles ($folder) { |
||
| 384 | |||
| 385 | |||
| 386 | /** |
||
| 387 | * test if file is a note |
||
| 388 | * |
||
| 389 | * @param \OCP\Files\File $file |
||
| 390 | * @return bool |
||
| 391 | */ |
||
| 392 | private function isNote($file) { |
||
| 403 | |||
| 404 | } |
||
| 405 |
Loading history...
Scrutinizer analyzes your
composer.json/composer.lockfile if available to determine the classes, and functions that are defined by your dependencies.It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis.