1
0
mirror of https://github.com/danog/psalm.git synced 2024-11-27 12:55:26 +01:00
psalm/docs/running_psalm/issues/PossiblyNullArgument.md

55 lines
1.2 KiB
Markdown
Raw Normal View History

2020-03-19 17:32:49 +01:00
# PossiblyNullArgument
Emitted when calling a function with a value thats possibly null when the function does not expect it
```php
2022-06-21 22:45:45 +02:00
<?php
function foo(string $s): void {}
2020-03-19 17:32:49 +01:00
foo(rand(0, 1) ? "hello" : null);
```
## Common Problem Cases
### Using a Function Call inside `if`
```php
2022-11-10 16:59:04 +01:00
<?php
if (is_string($cat->getName()) {
foo($cat->getName());
}
```
This fails since it's not guaranteed that subsequent calls to `$cat->getName()` always give the same result.
#### Possible Solutions
* Use a variable:
```php
2022-11-10 16:59:04 +01:00
<?php
$catName = $cat->getName();
if (is_string($catName) {
foo($catName);
}
unset($catName);
```
2022-06-21 20:56:20 +02:00
* Add [`@psalm-mutation-free`](../../annotating_code/supported_annotations.md#psalm-mutation-free) to the declaration of the function
### Calling Another Function After `if`
```php
2022-11-10 16:59:04 +01:00
<?php
if (is_string($cat->getName()) {
changeCat();
foo($cat->getName());
}
```
This fails since psalm cannot know if `changeCat()` does actually modify `$cat`.
#### Possible Solutions
2022-06-21 21:55:32 +02:00
* Add [`@psalm-mutation-free`](../../annotating_code/supported_annotations.md#psalm-mutation-free) to the declaration of the other function (here: `changeCat()`) too
* Use a variable: See above